From: "Breazeal, Don" <donb@codesourcery.com>
To: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: RE: [PATCH 0/4] Exec events in gdbserver on Linux
Date: Mon, 28 Apr 2014 17:18:00 -0000 [thread overview]
Message-ID: <DA279C53C4A5884A907135DFCD7A059A0E1DBDAB@NA-MBX-02.mgc.mentorg.com> (raw)
In-Reply-To: <DA279C53C4A5884A907135DFCD7A059A0E1DAD34@NA-MBX-02.mgc.mentorg.com>
Ping^2
thanks
--Don
> -----Original Message-----
> From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] On Behalf Of Breazeal, Don
> Sent: Wednesday, April 16, 2014 9:45 AM
> To: gdb-patches@sourceware.org
> Subject: RE: [PATCH 0/4] Exec events in gdbserver on Linux
>
> Ping for maintainer comments on this patch series:
> https://sourceware.org/ml/gdb-patches/2014-04/msg00037.html
> https://sourceware.org/ml/gdb-patches/2014-04/msg00040.html
> https://sourceware.org/ml/gdb-patches/2014-04/msg00072.html
>
> The documentation piece was previously approved:
> https://sourceware.org/ml/gdb-patches/2014-04/msg00071.html
>
> Thanks,
> --Don
>
>
> > -----Original Message-----
> > From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> > owner@sourceware.org] On Behalf Of Breazeal, Don
> > Sent: Wednesday, April 02, 2014 3:24 PM
> > To: gdb-patches@sourceware.org
> > Subject: [PATCH 0/4] Exec events in gdbserver on Linux
> >
> > This patch implements support for exec events in gdbserver on
> GNU/Linux.
> > This is a step towards the "follow fork/exec" item in the local/remote
> > debugging feature parity project:
> > (https://sourceware.org/gdb/wiki/LocalRemoteFeatureParity). Luis had
> > started work on this last year, and handed it off to me at the end of
> > January. (BTW, thanks to Luis for his advice on this, although any
> > problems with this patch are entirely mine.)
> >
> > Follow-exec-mode and rerun behave as expected in multiprocess mode
> > (target extended-remote), where follow-exec-mode maintains the
> > specified inferiors and rerun runs the last executable file to be
> exec'd.
> > Catchpoints for exec are not implemented in this patch series, since
> > this will be easier to do once fork and vfork events are also
> supported.
> >
> > - Patch 1/4 implements support for the extended ptrace event
> > PTRACE_EVENT_EXIT, which is a prerequisite for the exec event support.
> > - Patch 2/4 implements the exec event support.
> > - Patch 3/4 adds documentation for the new RSP "Stop Reply" message.
> > - Patch 4/4 extends the tests gdb.threads/non-ldr-exc-[1-4].exp to
> > test in non-stop mode as well as in all-stop mode.
> >
> > There are a couple of significant aspects to this patch. First, it
> > uses the ptrace extension PTRACE_EVENT_EXIT to detect thread exit, in
> > particular the exit of a thread group leader when a non-leader calls
> > exec. Use of this event was necessary due to a race condition in the
> > two-thread case. It is only used internally; exit events detected
> > this way are not exposed to the user, and exit processing was changed
> > as little as possible.
> >
> > Second, it sends a signal to each lwp after an exec event in order to
> > identify and clean up the lwp entry for the exec'ing thread. This
> > happens in the normal course of things for all-stop mode, but in non-
> > stop mode gdbserver uses SIGSTOP to stop, then resume all of the non-
> > leader threads to accomplish this.
> >
> > More detailed explanations of these issues is included in the patch
> > 2/4 email.
> >
> > My intent is to follow up with implementations of remote fork/vfork
> > events and associated catchpoints.
> >
> > --Don
> >
> > gdb/common/linux-ptrace.c | 45 ++++-
> > gdb/doc/gdb.texinfo | 6 +
> > gdb/gdbserver/linux-low.c | 274
> > +++++++++++++++++++++++++---
> > gdb/gdbserver/linux-low.h | 5 +
> > gdb/gdbserver/remote-utils.c | 28 ++-
> > gdb/remote.c | 27 ++-
> > gdb/testsuite/gdb.threads/non-ldr-exc-1.exp | 15 +-
> > gdb/testsuite/gdb.threads/non-ldr-exc-2.exp | 15 +-
> > gdb/testsuite/gdb.threads/non-ldr-exc-3.exp | 15 +-
> > gdb/testsuite/gdb.threads/non-ldr-exc-4.exp | 15 +-
> > 10 files changed, 395 insertions(+), 50 deletions(-)
next prev parent reply other threads:[~2014-04-28 17:18 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-02 22:24 Breazeal, Don
2014-04-16 16:44 ` Breazeal, Don
2014-04-28 17:18 ` Breazeal, Don [this message]
2014-05-06 15:42 ` Breazeal, Don
2014-05-07 14:08 ` Pedro Alves
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=DA279C53C4A5884A907135DFCD7A059A0E1DBDAB@NA-MBX-02.mgc.mentorg.com \
--to=donb@codesourcery.com \
--cc=gdb-patches@sourceware.org \
/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