From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4518 invoked by alias); 28 Nov 2001 22:31:19 -0000 Mailing-List: contact gdb-patches-help@sourceware.cygnus.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 4462 invoked from network); 28 Nov 2001 22:31:15 -0000 Received: from unknown (HELO factorix.sdv.fr) (194.206.196.2) by hostedprojects.ges.redhat.com with SMTP; 28 Nov 2001 22:31:15 -0000 Received: from ordimaison (ip-76-201.evc.net [212.95.76.201]) by factorix.sdv.fr (8.11.4/8.11.4) with SMTP id fASLwBS01189 for ; Wed, 28 Nov 2001 22:58:11 +0100 Message-Id: <3.0.6.32.20011128234055.00b33480@ics.u-strasbg.fr> X-Sender: muller@ics.u-strasbg.fr X-Mailer: QUALCOMM Windows Eudora Light Version 3.0.6 (32) Date: Sat, 17 Nov 2001 22:30:00 -0000 To: gdb-patches@sources.redhat.com From: muller@cerbere.u-strasbg.fr Subject: Re: [RFC/RFA] Add hardware watchpoint support for cygwin target. In-Reply-To: <20011128193011.GA6502@redhat.com> References: <8011-Wed28Nov2001201312+0200-eliz@is.elta.co.il> <4.2.0.58.20011128183252.00acd198@ics.u-strasbg.fr> <8011-Wed28Nov2001201312+0200-eliz@is.elta.co.il> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" X-SW-Source: 2001-11/txt/msg00330.txt.bz2 At 14:30 28/11/01 -0500, Christopher Faylor wrote: >On Wed, Nov 28, 2001 at 08:13:12PM +0200, Eli Zaretskii wrote: >>> Date: Wed, 28 Nov 2001 18:44:44 +0100 >>> From: Pierre Muller >>> >>> But te are some annoying things, >>> the most annoying is that an exception seems to be generated >>> on read access of the watched area even if you only set a normal >>> watchpoint (which should use a write-only debug feature). >> >>So you are saying that watch, rwatch, and awatch all yield the same >>behavior? >> >>Are you sure that you pass the watchpoint information correctly to >>the OS? For example, is the format of DR7 as the OS wants it >>identical to what GDB uses? The layout of bits in dr_control_mirror >>follows Intel documentation, but the OS might want those bits in a >>different format (that's what the corresponding DPMI call does, for >>example). I don't have Windows docs, so I cannot check this. >> >>> > /* Get the value of the DR6 debug status register from the inferior. >>> > Here we just return the value stored in D_REGS, as we've got it >>> > from the last go32_wait call. */ >> >>I believe you didn't really mean ``go32_wait'' here ;-) > >I'd like some clarification on this before I can accept the patch. It >seems like the described behavior would be annoying indeed. It would >be nice to fix this. As tested and explained in my previous mail, the unwanted messages are generated by DLL loading events, and not by wrong debug register generated stops... The behavior of the i386 debug register seems to be "normal" (i.e. as the intel docs or Ralph Brwon interrupt list specifies them). The problem is only that the messages should probably be removed by disaling output while loading new DLLs. Does this answer your worries? From mboxrd@z Thu Jan 1 00:00:00 1970 From: muller@cerbere.u-strasbg.fr To: gdb-patches@sources.redhat.com Subject: Re: [RFC/RFA] Add hardware watchpoint support for cygwin target. Date: Wed, 28 Nov 2001 14:31:00 -0000 Message-ID: <3.0.6.32.20011128234055.00b33480@ics.u-strasbg.fr> References: <8011-Wed28Nov2001201312+0200-eliz@is.elta.co.il> <4.2.0.58.20011128183252.00acd198@ics.u-strasbg.fr> <20011128193011.GA6502@redhat.com> X-SW-Source: 2001-11/msg00545.html Message-ID: <20011128143100.QZcjZ82rH7Mws5WRu3PA76QFlvPqJbNjTIBgs7mu9v0@z> At 14:30 28/11/01 -0500, Christopher Faylor wrote: >On Wed, Nov 28, 2001 at 08:13:12PM +0200, Eli Zaretskii wrote: >>> Date: Wed, 28 Nov 2001 18:44:44 +0100 >>> From: Pierre Muller >>> >>> But te are some annoying things, >>> the most annoying is that an exception seems to be generated >>> on read access of the watched area even if you only set a normal >>> watchpoint (which should use a write-only debug feature). >> >>So you are saying that watch, rwatch, and awatch all yield the same >>behavior? >> >>Are you sure that you pass the watchpoint information correctly to >>the OS? For example, is the format of DR7 as the OS wants it >>identical to what GDB uses? The layout of bits in dr_control_mirror >>follows Intel documentation, but the OS might want those bits in a >>different format (that's what the corresponding DPMI call does, for >>example). I don't have Windows docs, so I cannot check this. >> >>> > /* Get the value of the DR6 debug status register from the inferior. >>> > Here we just return the value stored in D_REGS, as we've got it >>> > from the last go32_wait call. */ >> >>I believe you didn't really mean ``go32_wait'' here ;-) > >I'd like some clarification on this before I can accept the patch. It >seems like the described behavior would be annoying indeed. It would >be nice to fix this. As tested and explained in my previous mail, the unwanted messages are generated by DLL loading events, and not by wrong debug register generated stops... The behavior of the i386 debug register seems to be "normal" (i.e. as the intel docs or Ralph Brwon interrupt list specifies them). The problem is only that the messages should probably be removed by disaling output while loading new DLLs. Does this answer your worries?