Module Name:    src
Committed By:   riastradh
Date:           Fri Jul  7 18:02:52 UTC 2023

Modified Files:
        src/sys/kern: kern_lock.c

Log Message:
Revert unintentional changes to kern_lock.c in previous commit.


To generate a diff of this commit:
cvs rdiff -u -r1.185 -r1.186 src/sys/kern/kern_lock.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/kern/kern_lock.c
diff -u src/sys/kern/kern_lock.c:1.185 src/sys/kern/kern_lock.c:1.186
--- src/sys/kern/kern_lock.c:1.185	Fri Jul  7 17:05:13 2023
+++ src/sys/kern/kern_lock.c	Fri Jul  7 18:02:52 2023
@@ -1,4 +1,4 @@
-/*	$NetBSD: kern_lock.c,v 1.185 2023/07/07 17:05:13 riastradh Exp $	*/
+/*	$NetBSD: kern_lock.c,v 1.186 2023/07/07 18:02:52 riastradh Exp $	*/
 
 /*-
  * Copyright (c) 2002, 2006, 2007, 2008, 2009, 2020 The NetBSD Foundation, Inc.
@@ -31,10 +31,9 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_lock.c,v 1.185 2023/07/07 17:05:13 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_lock.c,v 1.186 2023/07/07 18:02:52 riastradh Exp $");
 
 #ifdef _KERNEL_OPT
-#include "opt_ddb.h"
 #include "opt_lockdebug.h"
 #endif
 
@@ -54,10 +53,6 @@ __KERNEL_RCSID(0, "$NetBSD: kern_lock.c,
 #include <sys/ksyms.h>
 #endif
 
-#ifdef DDB
-#include <ddb/ddb.h>
-#endif
-
 #include <machine/lock.h>
 
 #include <dev/lockstat.h>
@@ -121,17 +116,17 @@ panic:	panic("%s: %s caller=%p", __func_
  * so that they show up in profiles.
  */
 
-#ifdef LOCKDEBUG
 #define	_KERNEL_LOCK_ABORT(msg)						\
     LOCKDEBUG_ABORT(__func__, __LINE__, kernel_lock, &_kernel_lock_ops, msg)
+
+#ifdef LOCKDEBUG
 #define	_KERNEL_LOCK_ASSERT(cond)					\
 do {									\
 	if (!(cond))							\
 		_KERNEL_LOCK_ABORT("assertion failed: " #cond);		\
 } while (/* CONSTCOND */ 0)
 #else
-#define	_KERNEL_LOCK_ABORT(cond)	__nothing
-#define	_KERNEL_LOCK_ASSERT(cond)	__nothing
+#define	_KERNEL_LOCK_ASSERT(cond)	/* nothing */
 #endif
 
 static void	_kernel_lock_dump(const volatile void *, lockop_printer_t);
@@ -142,6 +137,10 @@ lockops_t _kernel_lock_ops = {
 	.lo_dump = _kernel_lock_dump,
 };
 
+#ifdef LOCKDEBUG
+
+#include <ddb/ddb.h>
+
 static void
 kernel_lock_trace_ipi(void *cookie)
 {
@@ -149,11 +148,11 @@ kernel_lock_trace_ipi(void *cookie)
 	printf("%s[%d %s]: hogging kernel lock\n", cpu_name(curcpu()),
 	    curlwp->l_lid,
 	    curlwp->l_name ? curlwp->l_name : curproc->p_comm);
-#ifdef DDB
 	db_stacktrace();
-#endif
 }
 
+#endif
+
 /*
  * Initialize the kernel lock.
  */
@@ -196,9 +195,11 @@ _kernel_lock(int nlocks)
 	LOCKSTAT_TIMER(spintime);
 	LOCKSTAT_FLAG(lsflag);
 	struct lwp *owant;
+#ifdef LOCKDEBUG
 	static struct cpu_info *kernel_lock_holder;
 	u_int spins = 0;
 	u_int starttime = getticks();
+#endif
 	int s;
 	struct lwp *l = curlwp;
 

Reply via email to