From: Elena Zannoni <ezannoni@redhat.com>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: Andrew Cagney <ac131313@redhat.com>,
Elena Zannoni <ezannoni@redhat.com>,
gdb-patches@sources.redhat.com
Subject: Re: [RFC] PTRACE_ATTACH problem on new Linux kernels
Date: Tue, 18 Feb 2003 03:32:00 -0000 [thread overview]
Message-ID: <15953.43573.646843.825027@localhost.redhat.com> (raw)
In-Reply-To: <20030218031431.GA31807@nevyn.them.org>
Daniel Jacobowitz writes:
> On Mon, Feb 17, 2003 at 09:24:01PM -0500, Andrew Cagney wrote:
> > Solution 0 is to discard the STOP in infrun.c as part of the stop
> > analyzis.
>
> I like this; but I can't think how to do it without some global state
> bit saying just-attached-expecting-SIGSTOP.
>
We kind of have it already, it is stop_soon_quietly, I am not sure it
is specific enough though.
> >
> > > A first solution could be that upon continuing, gdb never sends a
> > > SIGSTOP through the ptrace call. I.e. the stop_signal in
> > > ptrace(PTRACE_CONT, pid, stop_signal) could be changed to
> > > TARGET_SIGNAL_0 if it is TARGET_SIGNAL_STOP (such a call is in
> > > proceed(), and we already do some signal munging there).
> > >
> > > Another solution is to throw away the TARGET_SIGNAL_STOP that is saved
> > > in stop_signal when we do an attach. This would be in
> > > attach_command(), in infcmd.c. This way it would not come into play at
> > > all at the next continue.
> >
> > This will make the desperatly needed objective of trying to eliminate
> > the global stop_signal variable just that bit more difficult.
> >
> > If the already nasty hacks in HP/PA and solib code is ignored, the
> > only places stop_signal is modified is in infrun.c.
> >
> > > Yet another solution is that we 'hide' the TARGET_SIGNAL_STOP in
> > > child_resume(), in i386-linux-nat.c but this would not be applicable
> > > to the other linux arches.
> >
> > Or discard the signal in resume()?
> >
> > Regardless, remembering that GDB is just one client of the kernel, the
> > kernel group should probably also restore the behavour that is
> > conistent with solaris and (most likely) every other ptrace
> > implementation.
>
> I'm not sure what Solaris does - don't we use procfs instead of ptrace
> there anyway? Do we still get a SIGSTOP at attach?
Unless I was hallucinating gdb was getting a SIGSTOP and passing that
down to resume, later. I'll look again.
>
> But Roland made a very convincing case for this new behavior; for
> programs like strace which just pass all signals through, this prevents
> SIGSTOPs being silently cancelled, which is a definite plus.
>
> --
> Daniel Jacobowitz
> MontaVista Software Debian GNU/Linux Developer
next prev parent reply other threads:[~2003-02-18 3:32 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-02-18 0:53 Elena Zannoni
2003-02-18 2:39 ` Andrew Cagney
2003-02-18 3:14 ` Daniel Jacobowitz
2003-02-18 3:32 ` Elena Zannoni [this message]
2003-02-19 17:19 ` Andrew Cagney
2003-02-19 17:28 ` Daniel Jacobowitz
2003-02-18 3:28 ` Elena Zannoni
2003-02-18 20:06 ` Andrew Cagney
2003-02-18 20:43 ` Elena Zannoni
2003-02-19 17:16 ` Andrew Cagney
2003-02-19 17:40 ` Kevin Buettner
2003-04-08 18:53 ` Elena Zannoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=15953.43573.646843.825027@localhost.redhat.com \
--to=ezannoni@redhat.com \
--cc=ac131313@redhat.com \
--cc=drow@mvista.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox