Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 20 Mar 2018 15:51:05 +0000 (UTC)
From:      Mark Johnston <markj@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r331245 - in head/sys: kern sys
Message-ID:  <201803201551.w2KFp5AD098664@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: markj
Date: Tue Mar 20 15:51:05 2018
New Revision: 331245
URL: https://svnweb.freebsd.org/changeset/base/331245

Log:
  Drop KTR_CONTENTION.
  
  It is incomplete, has not been adopted in the other locking primitives,
  and we have other means of measuring lock contention (lock_profiling,
  lockstat, KTR_LOCK). Drop it to slightly de-clutter the mutex code and
  free up a precious KTR class index.
  
  Reviewed by:	jhb, mjg
  MFC after:	1 week
  Differential Revision:	https://reviews.freebsd.org/D14771

Modified:
  head/sys/kern/kern_mutex.c
  head/sys/sys/ktr_class.h

Modified: head/sys/kern/kern_mutex.c
==============================================================================
--- head/sys/kern/kern_mutex.c	Tue Mar 20 15:44:17 2018	(r331244)
+++ head/sys/kern/kern_mutex.c	Tue Mar 20 15:51:05 2018	(r331245)
@@ -473,9 +473,6 @@ __mtx_lock_sleep(volatile uintptr_t *c, uintptr_t v)
 	struct turnstile *ts;
 	uintptr_t tid;
 	struct thread *owner;
-#ifdef KTR
-	int cont_logged = 0;
-#endif
 #ifdef LOCK_PROFILING
 	int contested = 0;
 	uint64_t waittime = 0;
@@ -629,17 +626,6 @@ retry_turnstile:
 		 */
 		mtx_assert(m, MA_NOTOWNED);
 
-#ifdef KTR
-		if (!cont_logged) {
-			CTR6(KTR_CONTENTION,
-			    "contention: %p at %s:%d wants %s, taken by %s:%d",
-			    (void *)tid, file, line, m->lock_object.lo_name,
-			    WITNESS_FILE(&m->lock_object),
-			    WITNESS_LINE(&m->lock_object));
-			cont_logged = 1;
-		}
-#endif
-
 		/*
 		 * Block on the turnstile.
 		 */
@@ -657,13 +643,6 @@ retry_turnstile:
 #endif
 		v = MTX_READ_VALUE(m);
 	}
-#ifdef KTR
-	if (cont_logged) {
-		CTR4(KTR_CONTENTION,
-		    "contention end: %s acquired by %p at %s:%d",
-		    m->lock_object.lo_name, (void *)tid, file, line);
-	}
-#endif
 #if defined(KDTRACE_HOOKS) || defined(LOCK_PROFILING)
 	if (__predict_true(!doing_lockprof))
 		return;

Modified: head/sys/sys/ktr_class.h
==============================================================================
--- head/sys/sys/ktr_class.h	Tue Mar 20 15:44:17 2018	(r331244)
+++ head/sys/sys/ktr_class.h	Tue Mar 20 15:51:05 2018	(r331245)
@@ -65,7 +65,7 @@
 #define	KTR_VM		0x00100000		/* The virtual memory system */
 #define	KTR_INET	0x00200000		/* IPv4 stack */
 #define	KTR_RUNQ	0x00400000		/* Run queue */
-#define	KTR_CONTENTION	0x00800000		/* Lock contention */
+#define	KTR_SPARE5	0x00800000
 #define	KTR_UMA		0x01000000		/* UMA slab allocator */
 #define	KTR_CALLOUT	0x02000000		/* Callouts and timeouts */
 #define	KTR_GEOM	0x04000000		/* GEOM I/O events */



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