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;