Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Mark Kettenis <mark.kettenis@xs4all.nl>
Cc: pedro_alves@portugalmail.pt, gdb-patches@sourceware.org
Subject: Re: [PATCH] i386_skip_prologue.
Date: Sat, 30 Dec 2006 21:31:00 -0000	[thread overview]
Message-ID: <20061230213132.GA27379@nevyn.them.org> (raw)
In-Reply-To: <200612302120.kBULKqOJ031833@brahms.sibelius.xs4all.nl>

On Sat, Dec 30, 2006 at 10:20:52PM +0100, Mark Kettenis wrote:
> I'm sorry.  I tend to suffer from packet loss at my side.

No problem - that's what I'm here for :-)

> > Is there a general philosophy we could adopt that would apply to most
> > or all targets?
> > 
> > Here's a proposal to get us started: when skipping the prologue to
> > place a breakpoint or finish single stepping (skip_prologue), try to
> > skip to the end of the first sal.  But some targets may optionally run
> > their prologue analyzer and make sure it doesn't see anything it
> > objects to - like jumps.  That would mean the i386 prologue analyzer
> > would need to know about this new call to __main.  When scanning the
> > prologue to build frame unwind information, ignore sals entirely.
> > Scan until we hit the saved PC or until we believe we understand
> > the entire frame.
> 
> The last time I tried using sals on i386, I simply encountered too
> many cases where the line number information couldn't be trusted and
> putting a breakpoint on a function that was defenitely called never
> hit.

Yeah.  That's definitely the biggest risk.

I think the first step, for Pedro's specific problem, should be
to recognize the call to __main as special and skippable.  We can
play with sals later.

-- 
Daniel Jacobowitz
CodeSourcery


      reply	other threads:[~2006-12-30 21:31 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <455EE79E.6000109@portugalmail.pt>
     [not found] ` <uveldatp9.fsf@gnu.org>
     [not found]   ` <455EF845.40902@portugalmail.pt>
     [not found]     ` <455F2754.5060703@portugalmail.pt>
     [not found]       ` <20061118163738.GA14800@nevyn.them.org>
2006-12-09 20:32         ` Pedro Alves
2006-12-30 20:48           ` Daniel Jacobowitz
2006-12-30 21:21             ` Mark Kettenis
2006-12-30 21:31               ` Daniel Jacobowitz [this message]

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=20061230213132.GA27379@nevyn.them.org \
    --to=drow@false.org \
    --cc=gdb-patches@sourceware.org \
    --cc=mark.kettenis@xs4all.nl \
    --cc=pedro_alves@portugalmail.pt \
    /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