From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23313 invoked by alias); 4 Jun 2009 03:16:48 -0000 Received: (qmail 23304 invoked by uid 22791); 4 Jun 2009 03:16:47 -0000 X-SWARE-Spam-Status: No, hits=-2.5 required=5.0 tests=AWL,BAYES_00,SPF_PASS X-Spam-Check-By: sourceware.org Received: from fencepost.gnu.org (HELO fencepost.gnu.org) (140.186.70.10) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 04 Jun 2009 03:16:42 +0000 Received: from eliz by fencepost.gnu.org with local (Exim 4.67) (envelope-from ) id 1MC3RX-0003Di-Hj; Wed, 03 Jun 2009 23:16:39 -0400 From: Eli Zaretskii To: "Pierre Muller" CC: gdb-patches@sourceware.org In-reply-to: <000601c9e4a3$b2f2f980$18d8ec80$@u-strasbg.fr> (muller@ics.u-strasbg.fr) Subject: Re: [RFA] Fix a windows bug if two watchpoints are used Reply-to: Eli Zaretskii References: <000301c9e49e$c479eba0$4d6dc2e0$@u-strasbg.fr> <000601c9e4a3$b2f2f980$18d8ec80$@u-strasbg.fr> Message-Id: Date: Thu, 04 Jun 2009 03:16:00 -0000 X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2009-06/txt/msg00058.txt.bz2 > From: "Pierre Muller" > Cc: > Date: Thu, 4 Jun 2009 01:33:27 +0200 > Content-Language: en-us > > > Shouldn't we instead fix the logic of i386_stopped_data_address, to > > get out of the loop on the first watchpoint that is found to be hit? > > The function does not support more than one watchpoint anyway, so why > > continue checking the bits in dr[6] after we've found one set already? > > > > Would such a change fix your problem without the other complications? > > It would hide the problem. Why hide, and what problem are we talking about? The situation you describe has no rational explanation, and looks more like a Windows bug than anything else: you in effect show a contradiction between two debug registers that should tell a coherent story, but don't. Fixing such problems without a good understanding of their exact reasons is always a bit phenomenological. My phenomenology is based on the premise that the OS uses the debug registers in the order we scan the bits in dr[6], so the first one we find set has better chances to be consistent with what really happened than anything else. > But what happens if you have different watchpoints > on the same address (say one 'watch' and one 'awatch')? > Are you sure your suggestion would not affect > such cases? It will work even in those cases, yes. We only support multiple watchpoints that break simultaneously if they watch the same address, anyway (there's only one address that i386_stopped_data_address returns). The i386 debug register support code will use a single debug register for watching such an address, no matter how many watchpoints the user sets and of what kind. We do this sharing of debug registers entirely in GDB (see i386_insert_aligned_watchpoint and the dr_ref_count[] array it uses); the OS is never told to use more than one debug register for every address we watch, even if we watch it with several watchpoints. The callers of i386_stopped_data_address take the address it returns, and check all the watchpoints that watch this address to see which one(s) of them triggered and which did not. That code is in breakpoint.c, AFAIR. > > Btw, I don't understand this part of i386_stopped_data_address: > > > > if (maint_show_dr && addr == 0) > > i386_show_dr ("stopped_data_addr", 0, 0, hw_write); > > > > Isn't that backwards? why display the address if it is zero? > I think this is because if addr is non-zero, you already > have a call to i386_show_dr before with "watchpoint hit". > This call is simply to state that stopped_data_address > was called but didn't find a hit. > But the correct condition should be (rc == 0) instead > of (addr == 0) as setting a watchpoint at (CORE_ADDR) 0 > should also work on target where this address is not protected. Yes, I agree.