Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <pedro@codesourcery.com>
To: Jan Kratochvil <jan.kratochvil@redhat.com>,
	Oleg Nesterov <oleg@redhat.com>
Cc: gdb-patches@sourceware.org, Tom Tromey <tromey@redhat.com>
Subject: Re: FYI: fix 2 tests when glibc debuginfo is installed
Date: Fri, 14 Oct 2011 21:42:00 -0000	[thread overview]
Message-ID: <201110142241.46746.pedro@codesourcery.com> (raw)
In-Reply-To: <20111014212510.GA20587@host1.jankratochvil.net>

On Friday 14 October 2011 22:25:10, Jan Kratochvil wrote:
> On Fri, 14 Oct 2011 23:19:09 +0200, Pedro Alves wrote:
> > On Friday 14 October 2011 20:37:05, Jan Kratochvil wrote:
> > > thanks; although these testcases are broken anyway, they should be updated for
> > > Linux kernels 3.1.x which always keep inferior stopped if it was stopped
> > > during PTRACE_ATTACH; probably to XFAIL older kernels.
> > 
> > Urgh.  Even if you SIGCONT the process before PTRACE_DETACH?
> 
> Yes.  But I do not think it is problem, one can SIGCONT it safely after
> PTRACE_DETACH.  Just it may be (T)-stopped for a moment but why not.

So,

no debugger:

$ kill -SIGSTOP PID
*stop*
$ kill -SIGCONT PID
*continue*

with debugger:

$ kill -SIGSTOP PID
*stop*
                           $ strace/gdb -p PID
                           ...
$ kill -SIGCONT PID
                           ^C/detach
*stop*

Oleg, do you have a pointer to a discussion or description of the
change?  Why doesn't at least the SIGCONT cancel the stop on detach?

> The opposite - the upstream 2.6.x kernel state - was IMO worse, it was resumed
> and one could not safely keep it stopped.
> 
> 
> > Why was the behavior changed incompatibly if we're having the all new
> > PTRACE_SEIZE?  Do you have a lkml reference to the change?
> 
> I do not have anything specific, AFAIK it just happened as part of all the
> changes incl. PTRACE_SEIZE.  I have updated the behavior into two new *-3x.c
> tests with Oleg Nesterov ~review/check/approval in:
> 	http://sourceware.org/systemtap/wiki/utrace/tests

-- 
Pedro Alves


  reply	other threads:[~2011-10-14 21:42 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-14 16:05 Tom Tromey
2011-10-14 19:37 ` Jan Kratochvil
2011-10-14 21:19   ` Pedro Alves
2011-10-14 21:25     ` Jan Kratochvil
2011-10-14 21:42       ` Pedro Alves [this message]
2011-10-15 14:52         ` Oleg Nesterov
2011-10-25 17:03           ` Pedro Alves
2011-10-25 17:38             ` Oleg Nesterov
2011-10-25 18:16               ` Pedro Alves
2011-10-25 18:38                 ` Oleg Nesterov
2011-10-25 19:50                   ` Pedro Alves
2011-10-16 16:41       ` Jan Kratochvil

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=201110142241.46746.pedro@codesourcery.com \
    --to=pedro@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=oleg@redhat.com \
    --cc=tromey@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