From: "Maciej W. Rozycki" <macro@mips.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: gdb-patches@sourceware.org, "Maciej W. Rozycki" <macro@linux-mips.org>
Subject: Re: mips-tdep.c: Fix sw watchpoint-out-of-scope events
Date: Wed, 19 Sep 2007 12:31:00 -0000 [thread overview]
Message-ID: <Pine.LNX.4.61.0709191219020.27377@perivale.mips.com> (raw)
In-Reply-To: <20070918155400.GA1128@caradoc.them.org>
On Tue, 18 Sep 2007, Daniel Jacobowitz wrote:
> This patch is fine. I wish it wasn't necessary, but for now it is.
Well, actually then, I have thought, rather than trying to fix the
unfixable, why don't we mimic the behaviour of hardware watchpoints? As
they do not single-step (the whole point of them, at least for the write
watchpoints) a scope software breakpoint is placed at the return target of
the scope covered. We could do the same for software watchpoints --
insert a scope software breakpoint, single-step till the watchpoints goes
out of scope and then resume at the full speed till the scope breakpoint
is hit. Was there any particular reason for not doing so in the first
place?
I'll go forward with the patch regardless as it would become obsolete
automatically if we accepted the hw watchpoint model for sw ones too as
in_function_epilogue_p() would become irrelevant then as far as I can
tell.
> Hmm, do you think we should accept that line as a pass then? The same
> will happen on other architectures too.
Well, it depends on the policy we have (if we have any in this area) --
the patch fixes non-PIC o32 and presumably new ABIs (I'll try to see
whether I have a way to test it), but does not quite so for PIC o32
(beacuse of the load to $gp after the return). There may be other
architectures with a similar problem, but if MIPS o32 was to be the only
one affected, then a PASS would be reasonable, as stated originally.
Though I have no idea how to take care of the load to $gp at the moment --
perhaps some logic from the scope breakpoint calculation could be reused.
Maciej
next prev parent reply other threads:[~2007-09-19 12:31 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-11 16:24 Maciej W. Rozycki
2007-09-18 15:54 ` Daniel Jacobowitz
2007-09-19 12:31 ` Maciej W. Rozycki [this message]
2007-09-19 12:38 ` Daniel Jacobowitz
2007-09-19 13:05 ` Maciej W. Rozycki
2007-09-19 13:16 ` Daniel Jacobowitz
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=Pine.LNX.4.61.0709191219020.27377@perivale.mips.com \
--to=macro@mips.com \
--cc=drow@false.org \
--cc=gdb-patches@sourceware.org \
--cc=macro@linux-mips.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