[v1] Fix bug in nios2 and m68k semihosting

[Qemu-devel] [PATCH 0/2] Fix bug in nios2 and m68k semihosting

Sandra Loosemore posted 2 patches 4 years, 8 months ago

Diff against v2
Download series mbox

Test docker-clang@ubuntu passed

Test FreeBSD passed

Test asan failed

Test docker-mingw@fedora passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190821142151.19995-1-sandra@codesourcery.com

Maintainers: Chris Wulff <crwulff@gmail.com>, Marek Vasut <marex@denx.de>, Laurent Vivier <laurent@vivier.eu>

There is a newer version of this series

target/m68k/m68k-semi.c   | 4 ++--
target/nios2/nios2-semi.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] Fix bug in nios2 and m68k semihosting

Posted by Sandra Loosemore 4 years, 8 months ago

I noticed recently that the exit semihosting call on nios2 was
ignoring its parameter and always returning status 0 instead.  It
turns out the handler was retrieving the value of the wrong register.
Since the nios2 semihosting implementation was basically
cut-and-pasted from that for m68k, I checked m68k also and it had the
same bug.  This set of patches fixes both of them.

Sandra Loosemore (2):
  target/nios2: Fix bug in semihosted exit handling
  target/m68k: Fix bug in semihosted exit handling

 target/m68k/m68k-semi.c   | 4 ++--
 target/nios2/nios2-semi.c | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.8.1