On Wed, 12 Apr 2023 at 12:55, Pavel Skripkin <paskrip...@gmail.com> wrote:
>
> Currently doing 'reset' command in sandbox with tracing enabled causes
> SIGSEV
>
> ```
> Hit any key to stop autoboot:  0
> =>
> =>
> => reset
> resetting ...
> Segmentation fault (core dumped)
>
> ```
>
> Tracing callback uses RAM buffer for storing tracing reports, but
> state_uninit() function unmaps whole RAM, which causes SIGSEV on umapped
> memory inside tracing subsystem.
>
> Fix it by disabling tracing before unmapping memory
>
> Signed-off-by: Pavel Skripkin <paskrip...@gmail.com>
> ---
>  arch/sandbox/cpu/state.c | 4 ++++
>  1 file changed, 4 insertions(+)

Reviewed-by: Simon Glass <s...@chromium.org>

Reply via email to