Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 8 Jan 2005 00:39:27 GMT
From:      David Xu <davidxu@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 68512 for review
Message-ID:  <200501080039.j080dRS1078731@repoman.freebsd.org>

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

Change 68512 by davidxu@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);



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