On 8/21/19 4:21 PM, Sandra Loosemore wrote: > This patch fixes a bug that caused semihosted exit to always return > status 0; it was incorrectly using the value of register R_ARG0 (which > contains the HOSTED_EXIT request number) instead of register R_ARG1. >
Fixes: 413a99a92c1 > Signed-off-by: Sandra Loosemore <san...@codesourcery.com> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > target/nios2/nios2-semi.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/target/nios2/nios2-semi.c b/target/nios2/nios2-semi.c > index d7a80dd..06c0861 100644 > --- a/target/nios2/nios2-semi.c > +++ b/target/nios2/nios2-semi.c > @@ -215,8 +215,8 @@ void do_nios2_semihosting(CPUNios2State *env) > args = env->regs[R_ARG1]; > switch (nr) { > case HOSTED_EXIT: > - gdb_exit(env, env->regs[R_ARG0]); > - exit(env->regs[R_ARG0]); > + gdb_exit(env, env->regs[R_ARG1]); > + exit(env->regs[R_ARG1]); > case HOSTED_OPEN: > GET_ARG(0); > GET_ARG(1); >