From: Daniel Jacobowitz <drow@false.org>
To: "Maciej W. Rozycki" <macro@mips.com>
Cc: gdb-patches@sourceware.org, "Maciej W. Rozycki" <macro@linux-mips.org>
Subject: Re: mi*-watch.exp: Test both hardware and software watchpoints
Date: Tue, 11 Sep 2007 16:19:00 -0000 [thread overview]
Message-ID: <20070911161922.GA16571@caradoc.them.org> (raw)
In-Reply-To: <Pine.LNX.4.61.0709111404330.32532@perivale.mips.com>
On Tue, Sep 11, 2007 at 04:42:05PM +0100, Maciej W. Rozycki wrote:
> The reason for MIPS is (and I suppose for i386 likewise) as soon as the
> frame pointer is destroyed in the callee, the watchpoint vanishes. There
> is this in_function_epilogue_p() hook that may be used to single-step out
> of the function epilogue which is currently not implemented.
It isn't generally sufficient, either. It's implemented on PowerPC
but the test still fails. The frame pointer is corrupted one
instruction before the epilogue is detected, and I couldn't come up
with a reasonable way of making it work.
> 2007-09-11 Maciej W. Rozycki <macro@mips.com>
>
> * gdb.mi/mi-watch.exp (test_watchpoint_all): New function.
> Move all the tests here and run them twice, once using software
> watchpoints and once using hardware watchpoints.
> * gdb.mi/mi2-watch.exp (test_watchpoint_all): Likewise.
>
> OK to apply?
In principle, I think so, but give it a day or two to see if anyone
objects. How about xfailing the out of scope test for software
watchpoints?
In practice, there's one problem. You're going to create lots of
tests with duplicated test names. The usual way to fix this is to
pass a prefix around and apply it everywhere. The clever way to
fix this, inspired by Jan's recent change to sepdebug.exp, is to
modify the global pf_prefix.
If you do that please leave the default prefix present though; save
the value at the beginning of the test (i.e. after default_gdb_init)
and append to it.
--
Daniel Jacobowitz
CodeSourcery
next prev parent reply other threads:[~2007-09-11 16:19 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-11 15:42 Maciej W. Rozycki
2007-09-11 16:19 ` Daniel Jacobowitz [this message]
2007-09-14 14:13 ` Maciej W. Rozycki
2007-09-14 14:24 ` Daniel Jacobowitz
2007-09-14 15:28 ` Maciej W. Rozycki
2007-09-14 15:40 ` Daniel Jacobowitz
2007-09-14 16:25 ` Maciej W. Rozycki
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=20070911161922.GA16571@caradoc.them.org \
--to=drow@false.org \
--cc=gdb-patches@sourceware.org \
--cc=macro@linux-mips.org \
--cc=macro@mips.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