Cleanup ZFS debug infrastructure
authorBrian Behlendorf <behlendorf1@llnl.gov>
Fri, 20 Jan 2012 23:02:57 +0000 (15:02 -0800)
committerBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 2 Feb 2012 19:24:30 +0000 (11:24 -0800)
commitd7e398ce1a3e6f9c705af43955a684685a798c32
tree06ff1d5c4431f8194d5b50c1824532b816902dc0
parent0c5dde492ff4fadcc9c43477c81b3d3824ac41db
Cleanup ZFS debug infrastructure

Historically the internal zfs debug infrastructure has been
scattered throughout the code.  Since we expect to start making
more use of this code this patch performs some cleanup.

* Consolidate the zfs debug infrastructure in the zfs_debug.[ch]
  files.  This includes moving the zfs_flags and zfs_recover
  variables, plus moving the zfs_panic_recover() function.

* Remove the existing unused functionality in zfs_debug.c and
  replace it with code which correctly utilized the spl logging
  infrastructure.

* Remove the __dprintf() function from zfs_ioctl.c.  This is
  dead code, the dprintf() functionality in the kernel relies
  on the spl log support.

* Remove dprintf() from hdr_recl().  This wasn't particularly
  useful and was missing the required format specifier anyway.

* Subsequent patches should unify the dprintf() and zfs_dbgmsg()
  functions.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
include/sys/zfs_context.h
include/sys/zfs_debug.h
lib/libzpool/kernel.c
module/zfs/arc.c
module/zfs/spa_misc.c
module/zfs/zfs_debug.c
module/zfs/zfs_ioctl.c