PERFORCE change 68512 for review

David Xu davidxu at FreeBSD.org
Fri Jan 7 16:39:28 PST 2005


http://perforce.freebsd.org/chv.cgi?CH=68512

Change 68512 by davidxu at davidxu_tiger on 2005/01/08 00:39:10

	remove stale comments.

Affected files ...

.. //depot/projects/davidxu_thread/src/lib/libthread/thread/thr_mutex.c#14 edit

Differences ...

==== //depot/projects/davidxu_thread/src/lib/libthread/thread/thr_mutex.c#14 (text+ko) ====

@@ -309,11 +309,6 @@
 		ret = umtx_trylock(&(*mutex)->m_lock, curthread->tid);
 		if (ret == 0) {
 			(*mutex)->m_owner = curthread;
-			/*
-			 * XXX there should be a separated list for
-			 * owned mutex, separated it from priority
-			 * mutex list
-			 */
 			/* Add to the list of owned mutexes: */
 			MUTEX_ASSERT_NOT_OWNED(*mutex);
 			TAILQ_INSERT_TAIL(&curthread->mutexq,
@@ -482,11 +477,6 @@
 		ret = umtx_trylock(&(*m)->m_lock, curthread->tid);
 		if (ret == 0) {
 			(*m)->m_owner = curthread;
-			/*
-			 * XXX there should be a separated list for
-			 * owned mutex, separated it from priority
-			 * mutex list
-			 */
 			/* Add to the list of owned mutexes: */
 			MUTEX_ASSERT_NOT_OWNED(*m);
 			TAILQ_INSERT_TAIL(&curthread->mutexq,
@@ -980,11 +970,6 @@
 				 */
 				(*m)->m_count = 0;
 				(*m)->m_owner = NULL;
-				/*
-				 * XXX there should be a separated list
-				 * for owned mutex, separated it from
-				 * priority mutex list
-				 */
 				/* Remove the mutex from the threads queue. */
 				MUTEX_ASSERT_IS_OWNED(*m);
 				TAILQ_REMOVE(&curthread->mutexq, (*m), m_qe);


More information about the p4-projects mailing list