On Tue, Apr 25, 2023 at 04:43:19PM +0300, Pavel Skripkin wrote: > Hi Simon, > > Simon Glass <s...@chromium.org> says: > > 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> > > Thank you for review! > > +Tom, I guess? > > @Tom, could you, please, pick following patch? > > > https://lists.denx.de/pipermail/u-boot/2023-April/515192.html
Simon takes sandbox patches himself and I've assigned this to him in patchwork, thanks for submitting the patch. -- Tom
signature.asc
Description: PGP signature