This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Fw: [ppc-linux-nat]: set access flag for h/w watchpoint even if it is only read or write (fwd)
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Wu Zhou <woodzltc at cn dot ibm dot com>, gdb-patches at sourceware dot org
- Date: Wed, 12 Jul 2006 21:15:48 +0300
- Subject: Re: Fw: [ppc-linux-nat]: set access flag for h/w watchpoint even if it is only read or write (fwd)
- References: <OF40F802D1.E1DA610C-ON482571A8.004F44D3-482571A8.004F3DA7@cn.ibm.com> <20060711110910.6298fq7oys0gwogw@imap.linux.ibm.com> <20060712173125.GC24622@nevyn.them.org>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Wed, 12 Jul 2006 13:31:25 -0400
> From: Daniel Jacobowitz <drow@false.org>
> Cc: eliz@gnu.org, gdb-patches@sourceware.org
>
> First, there's the question of targets which can't set true "read"
> watchpoints, but can set "access" watchpoints. Right now the x86
> simply accepts a request to insert a read watchpoint, and generates an
> access watchpoint instead. How about if we changed it to refuse to
> insert read watchpoints, changed breakpoint.c to attempt an access
> watchpoint if inserting a read watchpoint fails, and then store in the
> breakpoint which type was inserted?
Sorry, I don't understand your plan. How is it different from what
GDB does now (silently replace read with access watchpoint)? And what
type will we store when the user says "rwatch"?
> Second, there's the question of which sort of watchpoint we've hit. If
> the target could tell us, say, as a return value from
> target_stopped_data_address, then we could only check read watchpoints
> when a read watchpoint triggers. But I think this is a smaller issue;
> at worst we might report that both a write and read watchpoint had
> triggered when really only a write watchpoint had.
I think this is bad: if a user sets two different watchpoints at the
same address, she wants each watchpoint to fire when its specific
conditions are met.
My philosophy here is that watchpoints are a kind of silver bullet:
you use them when you have almost no idea who could touch the data in
question. So watchpoints should perform as close to the spec as
possible, because the user is already in a very confused state;
confusing her even more with imprecise watch reports would be a bad
mantra, IMO.
> And there's corner
> cases, like instructions which both read and write an address; they
> should trigger both read and write watchpoints but I doubt any platform
> gives us enough information to figure out that that's happened.
In those cases, it is okay to show both watchpoints, or even just the
write one.