This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] testsuite: asm-source.exp: use 'sys_exit' on powerpc.inc


> From: Edjunior Barbosa Machado <emachado@linux.vnet.ibm.com>
> Date: Tue, 23 Nov 2010 16:45:39 -0200
> 
> With this patch, ppc32 uses 'sys_exit' sequence on gdbasm_exit0 macro.

Isn't this somewhat OS-dependent?  It'll work on OpenBSD since the
exit system call is indeed #1 there, but there is no guarantee that's
true on all OSes isn't it?

And doesn't this pass random garbage as the argument to the exit system call?

> gdb/testsuite/
> 2010-11-23  Edjunior Machado  <emachado@br.ibm.com>
> 
>          * powerpc.inc: Use 'sys_exit' on gdbasm_exit0 macro.
> 
> diff --git a/gdb/testsuite/gdb.asm/powerpc.inc b/gdb/testsuite/gdb.asm/powerpc.inc
> index a0797bf..ab700c2 100644
> --- a/gdb/testsuite/gdb.asm/powerpc.inc
> +++ b/gdb/testsuite/gdb.asm/powerpc.inc
> @@ -29,9 +29,9 @@
>  
>  	comment "exit (0)"
>  	.macro gdbasm_exit0
> -	comment "Don't know how to exit, but this will certainly halt..."
> -	li	0, 0
> -	lwz 	0, 0(0)
> +	comment "sys_exit"
> +	li	0, 1
> +	sc
>  	.endm
>  
>  	comment "crt0 startup"
> -- 
> 1.7.1
> 
> 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]