Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: gdb-patches@sourceware.org
Subject: Re: RFC: Mark outer frames
Date: Thu, 03 Sep 2009 22:52:00 -0000	[thread overview]
Message-ID: <20090903225219.GA1266@adacore.com> (raw)
In-Reply-To: <20090828213237.GA9175@caradoc.them.org>

> This patch was originally for an entirely different problem, namely,
> the dreaded "Value is not active" error.  What happens with that is
> that we connect to some target, and we're in the middle of its startup
> code before there's a stack frame.  No unwinder thinks we have a way
> out of this frame, so nothing returns a valid this_id.  So the frame
> id is null_frame_id - which is overloaded to mean 'unknown id' and
> 'no frame at all'.  The value checks are using it one way and the
> unwinder is using it the other.

I just started looking at PR/9786:
    info frame: generates an error when remote debugging

Basically, on x86:

    (gdb) target rem :4444
    Remote debugging using :4444
    0xb7f6d840 in _start () from /lib/ld-linux.so.2
    (gdb) info frame
    Stack level 0, frame at 0x0:
     eip = 0xb7f6d840 in _start; saved eip 
    /[...]/findvar.c:304: internal-error: value_of_register_lazy: Assertion `frame_id_p (get_frame_id (frame))' failed.

This is the same issue as above, I believe, and your patch fixes
the issue too. After applying it, I get:

    (gdb) info frame
    Stack level 0, frame at 0x0:
     eip = 0xb7f6d840 in _start; saved eip 0xb7f6d840
     Outermost frame: unwinder did not report frame ID
     Arglist at unknown address.
     Locals at unknown address, Previous frame's sp in esp

So, should we apply you patch? (we don't have time for much else
except trying to see if we could plug that particular hole with
a temporary, localized, patch...

-- 
Joel


  parent reply	other threads:[~2009-09-03 22:52 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-28 22:16 Daniel Jacobowitz
2009-08-29  7:24 ` Eli Zaretskii
2009-08-31 22:54 ` Joel Brobecker
2009-09-01 22:41   ` Doug Evans
2009-09-02 17:03 ` Joel Brobecker
2009-09-02 17:33   ` Daniel Jacobowitz
2009-09-03 22:52 ` Joel Brobecker [this message]
2009-09-10  2:29   ` [gdb-7.0] " Joel Brobecker
2009-09-10 19:12     ` Daniel Jacobowitz
2009-09-10 22:41       ` Joel Brobecker
2009-09-12 22:13         ` [gdb-7.0/doco] " Joel Brobecker
2009-09-12 22:14           ` Joel Brobecker
2009-09-13  3:11             ` Eli Zaretskii
2009-09-13 16:29               ` Joel Brobecker

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=20090903225219.GA1266@adacore.com \
    --to=brobecker@adacore.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