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>