From: PAUL GILLIAM <pgilliam@us.ibm.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: gdb@sources.redhat.com
Subject: Re: Instrcutions that must not be stepped.
Date: Fri, 16 Jun 2006 04:36:00 -0000 [thread overview]
Message-ID: <1150415676.3346.30.camel@dufur.beaverton.ibm.com> (raw)
In-Reply-To: <20060608022654.GA31271@nevyn.them.org>
On Wed, 2006-06-07 at 22:26 -0400, Daniel Jacobowitz wrote:
> On Wed, Jun 07, 2006 at 05:19:59PM -0700, PAUL GILLIAM wrote:
> > Does there currently exist an arch. independent way to detect
> > instruction sequences that must not be single stepped? Failing that, is
> > there some hook I can use to implement this for just the PowerPC?
>
> Nope. You'd have to add one.
The closer I look at this, the more it looks like a special case of
software single step. The difference is that it may not have to be for
*every* single step: just those that are trying to step "atomic"
sequences.
I propose changing the meaning of SOFTWARE_SINGLE_STEP_P () from "This
arch has no hardware to do single step and must use software." to "There
may be circumstances where this arch will have to do single stepping
with out hardware support." And make SOFTWARE_SINGLE_STEP return 1 if a
software single step was needed and 0 if it was not. This would require
a minor change for those arches currently using SOFTWARE_SINGLE_STEP and
a little tweeking in "infrun.c".
The only difference between doing a software single step as it is now
and doing an "atomic single step" is how the decision of where to place
temporary breakpoints is made.
> Reading the instruction before stepping is going to slow down single
> stepping. Is there some other way we can handle this?
I can't think of any way. I did some timing runs, with and without
reading the next instruction from the target and in my native ppc64
case, it wasn't too bad. Any way, it would only be a hit on arches that
needed it. And if it was real bad, we could let the user decide.
-=# Paul #=-
next prev parent reply other threads:[~2006-06-16 0:59 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-08 2:27 PAUL GILLIAM
2006-06-08 3:50 ` Daniel Jacobowitz
2006-06-16 4:36 ` PAUL GILLIAM [this message]
2006-06-17 12:26 ` PAUL GILLIAM
2006-06-18 4:57 ` Mark Kettenis
2006-06-20 20:13 ` PAUL GILLIAM
2006-06-20 22:53 ` Paul Koning
2006-06-20 23:34 ` PAUL GILLIAM
2006-06-21 1:17 ` Daniel Jacobowitz
2006-06-09 14:12 John Yates
2006-06-09 14:28 ` Daniel Jacobowitz
2006-06-10 0:33 ` PAUL GILLIAM
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=1150415676.3346.30.camel@dufur.beaverton.ibm.com \
--to=pgilliam@us.ibm.com \
--cc=drow@false.org \
--cc=gdb@sources.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