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 0/1] Threaded Watchpoints


On Mon, Sep 10, 2007 at 12:34:14PM -0300, Luis Machado wrote:
> > I've regression tested i386, amd64, and ia64.  I tested S/390 by hand
> > and it works, but the extra logic in watchthreads.exp for that
> > platform hasn't been tested (no DejaGNU or expect on my test system).
> 
> I couldn't build the code for ppc due to a call to the "save_siginfo"
> function. Why is exactly that function's been moved to "linux-nat.c"? It
> seems that it's being called only in that file, and with a PPC-specific
> ptrace request "PTRACE_GETSIGINFO", which is not declared anywhere
> except for "ppc-linux-nat.c".
> 
> Could you please clarify this specific point?

PTRACE_GETSIGINFO is not specific to PowerPC.  It's
architecture-independent (although somewhat recent); IA-64 uses it
for watchpoints just like PowerPC does.  If your system headers are
not new enough to define it, the definition from ppc-linux-nat.c
should probably move to linux-nat.c too.

If that doesn't help, could you show me the error message?  Thanks.

-- 
Daniel Jacobowitz
CodeSourcery


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