diff options
| author | Matthew Ahrens <[email protected]> | 2018-03-21 15:37:32 -0700 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2018-03-21 15:37:32 -0700 |
| commit | 2fd92c3d6c7fba9a28ce4779ff6623dc911e7b78 (patch) | |
| tree | 9eaa716acdd15e77999f15d0d72a60cfb2b228d1 /module/zfs/arc.c | |
| parent | 8d9e7c8fbe6e131fac64c16c0714e5120d012daa (diff) | |
enable zfs_dbgmsg() by default, without dprintf()
zfs_dbgmsg() should record a message by default. As a general
principal, these messages shouldn't be too verbose. Furthermore, the
amount of memory used is limited to 4MB (by default).
dprintf() should only record a message if this is a debug build, and
ZFS_DEBUG_DPRINTF is set in zfs_flags. This flag is not set by default
(even on debug builds). These messages are extremely verbose, and
sometimes nontrivial to compute.
SET_ERROR() should only record a message if ZFS_DEBUG_SET_ERROR is set
in zfs_flags. This flag is not set by default (even on debug builds).
This brings our behavior in line with illumos. Note that the message
format is unchanged (including file, line, and function, even though
these are not recorded on illumos).
Reviewed-by: Giuseppe Di Natale <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Prakash Surya <[email protected]>
Signed-off-by: Matthew Ahrens <[email protected]>
Closes #7314
Diffstat (limited to 'module/zfs/arc.c')
| -rw-r--r-- | module/zfs/arc.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/module/zfs/arc.c b/module/zfs/arc.c index c6ff6171c..9c0c5513d 100644 --- a/module/zfs/arc.c +++ b/module/zfs/arc.c @@ -7138,6 +7138,7 @@ arc_tempreserve_space(uint64_t reserve, uint64_t txg) if (reserve + arc_tempreserve + anon_size > arc_c / 2 && anon_size > arc_c / 4) { +#ifdef ZFS_DEBUG uint64_t meta_esize = refcount_count(&arc_anon->arcs_esize[ARC_BUFC_METADATA]); uint64_t data_esize = @@ -7146,6 +7147,7 @@ arc_tempreserve_space(uint64_t reserve, uint64_t txg) "anon_data=%lluK tempreserve=%lluK arc_c=%lluK\n", arc_tempreserve >> 10, meta_esize >> 10, data_esize >> 10, reserve >> 10, arc_c >> 10); +#endif DMU_TX_STAT_BUMP(dmu_tx_dirty_throttle); return (SET_ERROR(ERESTART)); } |