Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Ulrich Weigand" <uweigand@de.ibm.com>
To: dje@google.com (Doug Evans)
Cc: gdb-patches@sourceware.org, pedro@codesourcery.com
Subject: Re: [RFA] dummy frame handling cleanup, plus inferior fun call signal   handling improvement
Date: Wed, 14 Jan 2009 15:07:00 -0000	[thread overview]
Message-ID: <200901141506.n0EF6JPS014241@d12av02.megacenter.de.ibm.com> (raw)
In-Reply-To: <e394668d0901070836o74740eb2t734313849bc3448d@mail.gmail.com> from "Doug Evans" at Jan 07, 2009 08:36:32 AM

Doug Evans wrote:

> I also meant to ask another question.
> What's the intended handling of unwindonsignal if the program stops in
> another thread?
> Should the inferior function call in the original thread be unwound?
> GDB's current behavior is busted as it pops the dummy frame on the wrong thread.
> This patch doesn't unwind the dummy frame (so at least it doesn't
> clobber the wrong thread), but I can change it to unwind the dummy
> frame of the original thread if you like.

I think the behaviour implemented by your patch is reasonable.

Ideally, you'd probably want to delay handling events in other
threads until execution of the inferior call has completed.
That may be nontrivial to implement, however ...

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com


  reply	other threads:[~2009-01-14 15:07 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-07  6:52 Doug Evans
2009-01-07 16:36 ` Doug Evans
2009-01-14 15:07   ` Ulrich Weigand [this message]
2009-01-07 17:02 ` Pedro Alves
2009-01-14 15:07 ` Ulrich Weigand
2009-01-19  7:24   ` Doug Evans
2009-01-19 14:40     ` Ulrich Weigand
  -- strict thread matches above, loose matches on Subject: below --
2008-11-18 21:01 Doug Evans
2008-11-19 14:07 ` Doug Evans
2008-11-20 15:02 ` Doug Evans
2008-11-20 15:06   ` Doug Evans
2008-12-01 20:52     ` Doug Evans
2008-12-01 21:22       ` Pedro Alves
2008-12-02  1:20         ` Doug Evans
2008-12-03  6:04           ` Doug Evans
2008-12-04 15:32             ` Ulrich Weigand
2008-12-04 15:54               ` Pedro Alves
2008-12-04 22:32               ` Doug Evans
2008-12-04 22:42                 ` Pedro Alves
2008-12-05  0:18                   ` Ulrich Weigand
2008-12-05  0:37                     ` Pedro Alves
2008-12-05  0:30                 ` Ulrich Weigand
2008-11-26 19:17 ` Doug Evans

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=200901141506.n0EF6JPS014241@d12av02.megacenter.de.ibm.com \
    --to=uweigand@de.ibm.com \
    --cc=dje@google.com \
    --cc=gdb-patches@sourceware.org \
    --cc=pedro@codesourcery.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