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

[Sérgio Durigan Júnior <sergiodj at linux dot vnet dot ibm dot com>]

On Mon, 2009-01-26 at 16:46 -0200, Sérgio Durigan Júnior wrote:
> Hi Ulrich,
> 
> On Mon, 2009-01-26 at 19:36 +0100, uweigand@pctc1.boeblingen.de.ibm.com
> wrote:
> > Segio Durigan Junior wrote:
> > 
> > > +static void
> > > +linux_enable_tracesysgood (ptid_t ptid)
> > > +{
> > > +  int pid = ptid_get_lwp (ptid);
> > > +
> > > +  if (pid == 0)
> > > +    pid = ptid_get_pid (ptid);
> > > +
> > > +  if (linux_supports_tracesysgood (pid) == 0)
> > > +    return;
> > > +
> > > +  current_ptrace_options |= PTRACE_O_TRACESYSGOOD;
> > > +  linux_passed_by_entrypoint_flag = 1;
> > > +
> > > +  ptrace (PTRACE_SETOPTIONS, pid, 0, current_ptrace_options);
> > > +}
> > 
> > I don't see any place where this function is ever called ...
> > Could you explain how this is supposed to work?  Or am I
> > overlooking something here?
> 
> If you take a closer look, you'll see that it's called in:
> 
> - linux_child_post_attach
> - linux_child_post_startup_inferior

Wow, that made me see another mistake... I'm really sorry, it seems I
have mixed the arch-dep and indep patches (specifically the linux-nat.c
file).

I don't think it's a real trouble, but if you want I can resubmit the patches.

Sorry about that. Regards,

-- 
Sérgio Durigan Júnior
Linux on Power Toolchain - Software Engineer
Linux Technology Center - LTC
IBM Brazil