[PATCH, v2] Share ptrace options discovery/linux native code between GDB and gdbserver

Tom Tromey tromey@redhat.com
Tue Aug 20 15:38:00 GMT 2013


>>>>> "Luis" == Luis Machado <lgustavo@codesourcery.com> writes:

Luis> This is the second iteration of the patch to do some sharing of ptrace
Luis> and native linux stuff between GDB and gdbserver.

Luis> I've included fixes based on reviews, so hopefully this is in better
Luis> shape now.

Thanks.  I like it.  I have a few nits though.

Luis>           * common/linux-ptrace.c: Include nat/linux-waitpid.h.

I suppose at some point this file ought to be moved.

Luis>           * config.in (PTRACE_TYPE_ARG4): Undefine.

Usually for config.in I just write:

	* config.in: Rebuild.

... since presumably you just used autoheader.

Luis>           * configure.ac (AC_CACHE_CHECK): Add void * to the list of
Luis>           ptrace's 4th argument's types.
Luis>           Check the type of PTRACE_TYPE_ARG4.

AC_CACHE_CHECK isn't the right function name.  Perhaps no name is best
here.

Also, doesn't this check need to be done in gdbserver as well?

Tom



More information about the Gdb-patches mailing list