CVSROOT:        /cvs
Module name:    src
Changes by:     k...@cvs.openbsd.org    2020/08/23 03:35:32

Modified files:
        sys/sys        : sysctl.h 
        sys/kern       : kern_sysctl.c vfs_subr.c vfs_syscalls.c 

Log message:
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

Reply via email to