This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA] 2nd send: Solve the watchpoint failures on second run.
- From: Pierre Muller <muller at ics dot u-strasbg dot fr>
- To: Michael Snyder <msnyder at redhat dot com>
- Cc: gdb-patches at sources dot redhat dot com, jimb at redhat dot com
- Date: Fri, 30 Aug 2002 09:09:24 +0200
- Subject: Re: [RFA] 2nd send: Solve the watchpoint failures on second run.
- References: <4.2.0.58.20020715162013.02270ed8@ics.u-strasbg.fr>
At 00:13 30/08/2002 , Michael Snyder a écrit:
>Pierre Muller wrote:
> >
> > I got no answer to that RFA,
> > http://sources.redhat.com/ml/gdb-patches/2002-06/msg00394.html
> > thus I send it again.
> >
> > This bug is related to bug 515 of database.
> > The following note:
> > >(This patch, together with the previous RFA that I sent today,
> > >fixes the problem for both statically and dynamically linked
> > >programs).
> > Can be modified into:
> > The following patch together with the patch commited
> > on july 4. 2002
> > http://sources.redhat.com/ml/gdb-cvs/2002-07/msg00036.html
> > fiwes bug 515 both for statically and dynamically linked programs.
> >
> > If you use my previous patch and compile the test program
> > of the 515 bug with --static option,
> > you will still miss the change of x value on the second run.
> > This comes from the fact that the val field of the watchpoint is
> > not reset correctly when the program is restarted.
> >
> > I first tried to fix this problem by using evaluate_expression
> > in the breakpoint_init_inferior function. But, while this fixes the problem for
> > the win32 native debugger, it leads to problems for linux,
> > beacuse the program is not yet loaded into memory
> > when this function is called: instead, the bash used for launch the debuggee
> > seems to be in memory...
> >
> > So the only way I found to fix that error was to reset val field to NULL
> > so that the val field is computed again
> > correctly at the first insert_breakpoint call.
>
>Looks OK to me, Pierre -- except please move the '{' from
>the end of the line.
>
>Thanks
>Michael
OK I commited the following:
2002-08-30 Pierre Muller <muller@ics.u-strasbg.fr>
* breakpoint.c (breakpoint_init_inferior): Reset the val field of
watchpoints to NULL.
(insert_breakpoints): set val field of watchpoints if NULL.
$ cvs diff -u -p breakpoint.c
Index: breakpoint.c
===================================================================
RCS file: /cvs/src/src/gdb/breakpoint.c,v
retrieving revision 1.84
diff -u -p -r1.84 breakpoint.c
--- breakpoint.c 27 Aug 2002 19:06:35 -0000 1.84
+++ breakpoint.c 30 Aug 2002 07:12:26 -0000
@@ -738,7 +738,19 @@ insert_breakpoints (void)
if (b->enable_state == bp_permanent)
/* Permanent breakpoints cannot be inserted or removed. */
continue;
- else if (b->type != bp_watchpoint
+ if ((b->type == bp_watchpoint
+ || b->type == bp_hardware_watchpoint
+ || b->type == bp_read_watchpoint
+ || b->type == bp_access_watchpoint) && (!b->val))
+ {
+ struct value *val;
+ val = evaluate_expression (b->exp);
+ release_value (val);
+ if (VALUE_LAZY (val))
+ value_fetch_lazy (val);
+ b->val = val;
+ }
+ if (b->type != bp_watchpoint
&& b->type != bp_hardware_watchpoint
&& b->type != bp_read_watchpoint
&& b->type != bp_access_watchpoint
@@ -1566,6 +1578,14 @@ breakpoint_init_inferior (enum inf_conte
/* Likewise for watchpoints on local expressions. */
if (b->exp_valid_block != NULL)
delete_breakpoint (b);
+ if (context == inf_starting)
+ {
+ /* Reset val field to force reread of starting value
+ in insert_breakpoints. */
+ if (b->val)
+ value_free (b->val);
+ b->val = NULL;
+ }
break;
default:
/* Likewise for exception catchpoints in dynamic-linked
Pierre Muller
Institut Charles Sadron
6,rue Boussingault
F 67083 STRASBOURG CEDEX (France)
mailto:muller@ics.u-strasbg.fr
Phone : (33)-3-88-41-40-07 Fax : (33)-3-88-41-40-99