Re: [PATCH 2/4] 'catch syscall' feature -- Architecture-dependent part

[Tom Tromey <tromey@redhat.com>]

>>>>> "Sérgio" == Sérgio Durigan Júnior <sergiodj@linux.vnet.ibm.com> writes:

Sérgio> 	* i386-linux-tdep.c: Define the syscall's XML struct to hold
Sérgio> 	information about syscalls.
Sérgio> 	(init_sysinfo): New.

There is some code duplication between this and ppc-linux-tdep.c.
It seems to me that much of the Linux code will be shared by all
ports.  So, I think the duplicated stuff should go in linux-nat.c.

Sérgio> 	(i386_linux_get_syscall_number): New.
Sérgio> 	(i386_linux_syscall_name_from_number): New.
Sérgio> 	(i386_linux_syscall_number_from_name): New.
Sérgio> 	(i386_linux_get_syscalls_names): New.

In the current patch, these could all be static.  Aside from
i386_linux_get_syscall_number, perhaps they ought to be shared in
linux-nat.c though.  The same applies for the equivalent PPC functions.

Sérgio> +/* Return the current system call's number present in the
Sérgio> +   r0 register.  When the function fails, it returns -1.  */
Sérgio> +LONGEST
Sérgio> +ppc_linux_get_syscall_number (struct gdbarch *gdbarch,
Sérgio> +                              ptid_t ptid)
Sérgio> +{
[...]
Sérgio> +  buf = (gdb_byte *) xmalloc (tdep->wordsize * sizeof (gdb_byte));

I suspect you need a cleanup here, rather than an explicit xfree.  I
am not sure.  If regcache_cooked_read can call error, then you need
one.

Tom