Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 10 Jul 2004 14:18:54 GMT
From:      David Xu <davidxu@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 56961 for review
Message-ID:  <200407101418.i6AEIs9c027827@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=56961

Change 56961 by davidxu@davidxu_alona on 2004/07/10 14:18:38

	Add a comment.

Affected files ...

.. //depot/projects/davidxu_ksedbg/src/lib/libpthread/thread/thr_priority_queue.c#3 edit

Differences ...

==== //depot/projects/davidxu_ksedbg/src/lib/libpthread/thread/thr_priority_queue.c#3 (text+ko) ====

@@ -271,6 +271,16 @@
 			/* Mark the list as not being in the queue: */
 			pql->pl_queued = 0;
 		} else {
+			/*
+			 * note there may be a suspension event during this
+			 * test, If TMDF_DONOTRUNUSER is set after we tested it,
+			 * we will run the thread, this seems be a problem,
+			 * fortunatly, when we are being debugged, all context
+			 * switch will be done by kse_switchin, that is a syscall,
+			 * kse_switchin will check the flag again, the thread
+			 * will be returned via upcall, so next time, UTS won't
+			 * run the thread.
+			 */ 
 			while (pthread != NULL && !THR_IN_CRITICAL(pthread) &&
 			       (pthread->tcb->tcb_tmbx.tm_dflags &
 			         TMDF_DONOTRUNUSER)) {



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200407101418.i6AEIs9c027827>