This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fw: [ppc-linux-nat]: set access flag for h/w watchpoint even if it is only read or write (fwd)


> 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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]