diff options
| author | kn <kn@cvs.openbsd.org> | 2020-08-23 09:35:33 +0000 |
|---|---|---|
| committer | kn <kn@cvs.openbsd.org> | 2020-08-23 09:35:33 +0000 |
| commit | 6f30adc27aeee1bf4f4bcaafab1f71a3f33f00ba (patch) | |
| tree | 4c31ef55316bcd01cfafa96b77a428ac95687b90 /sys/kern/vfs_syscalls.c | |
| parent | a6dfbb3a3c130690b6d6caae90eadeda30c64039 (diff) | |
Remove unused debug_syncprt, improve debug sysctl handling
"syncprt" is unused since kern/vfs_syscalls.c r1.147 from 2008.
Adding new debug sysctls is a bit opaque and looking at kern/kern_sysctl.c
the only visible difference between used and stub ctldebug structs in the
debugvars[] array is their extern keyword, indicating that it is defined
elsewhere.
sys/sysctl.h declares all debugN members as extern upfront, but these
declarations are not needed.
Remove the unused debug sysctl, rename the only remaining one to something
meaningful and remove forward declarations from /sys/sysctl.h; this way,
adding new debug sysctls is a matter of adding extern and coming up with a
name, which is nicer to read on its own and better to grep for.
OK mpi
Diffstat (limited to 'sys/kern/vfs_syscalls.c')
| -rw-r--r-- | sys/kern/vfs_syscalls.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/sys/kern/vfs_syscalls.c b/sys/kern/vfs_syscalls.c index 4f22f24a152..20aaff11d81 100644 --- a/sys/kern/vfs_syscalls.c +++ b/sys/kern/vfs_syscalls.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_syscalls.c,v 1.346 2020/08/22 11:47:23 kn Exp $ */ +/* $OpenBSD: vfs_syscalls.c,v 1.347 2020/08/23 09:35:32 kn Exp $ */ /* $NetBSD: vfs_syscalls.c,v 1.71 1996/04/23 10:29:02 mycroft Exp $ */ /* @@ -523,11 +523,6 @@ dounmount_leaf(struct mount *mp, int flags, struct proc *p) /* * Sync each mounted filesystem. */ -#ifdef DEBUG_SYSCTL -int syncprt = 0; -struct ctldebug debug0 = { "syncprt", &syncprt }; -#endif - int sys_sync(struct proc *p, void *v, register_t *retval) { |