lin-lwp.c fix

lin-lwp.c fix

[Mark Kettenis]

This patch fixes a problem in lin-lwp.c:lin_lwp_resume().  I think
this fixes the problem Kevin reported earlier this week (but I'd
appreciate it if you could check if that's really true Kevin), and
supersedes the patch given in my reply to his bug report.

Checked in.

Mark


Index: ChangeLog
from  Mark Kettenis  <kettenis@gnu.org>

	* lin-lwp.c (lin_lwp_resume): Don't mark LWP as not stopped until
	we're absolutely sure we're going to resume it.

Index: lin-lwp.c
===================================================================
RCS file: /cvs/src/src/gdb/lin-lwp.c,v
retrieving revision 1.6
diff -u -p -r1.6 lin-lwp.c
--- lin-lwp.c 2001/03/01 01:39:21 1.6
+++ lin-lwp.c 2001/03/30 19:42:05
@@ -439,10 +439,6 @@ lin_lwp_resume (int pid, int step, enum 
     {
       pid = GET_LWP (lp->pid);
 
-      /* Mark LWP as not stopped to prevent it from being continued by
-	 resume_callback.  */
-      lp->stopped = 0;
-
       /* Remember if we're stepping.  */
       lp->step = step;
 
@@ -455,6 +451,10 @@ lin_lwp_resume (int pid, int step, enum 
 	  gdb_assert (signo == TARGET_SIGNAL_0);
 	  return;
 	}
+
+      /* Mark LWP as not stopped to prevent it from being continued by
+	 resume_callback.  */
+      lp->stopped = 0;
     }
 
   if (resume_all)

Re: lin-lwp.c fix

[Kevin Buettner]

On Mar 30,  9:46pm, Mark Kettenis wrote:

> This patch fixes a problem in lin-lwp.c:lin_lwp_resume().  I think
> this fixes the problem Kevin reported earlier this week (but I'd
> appreciate it if you could check if that's really true Kevin),

I've just tested your fix and am happy to report that I am no longer
able to reproduce the problem.

> and
> supersedes the patch given in my reply to his bug report.

I tested your earlier patch too, but your new one seems to work better.
(I was able to occassionally get gdb to go into an infinite loop with
your previous patch.  I'm not able to do that anymore.)

Thanks,

Kevin