diff options
| author | Alexander Motin <[email protected]> | 2024-11-19 10:00:16 -0500 |
|---|---|---|
| committer | GitHub <[email protected]> | 2024-11-19 07:00:16 -0800 |
| commit | 534688948c395619af328c60ba3b863bfcf2ef20 (patch) | |
| tree | acb8b81d492f5be89b2632591732430ce1f3f759 /cmd | |
| parent | ffe211279556f34a1a555173bc0fe8ff53244be2 (diff) | |
Remove hash_elements_max accounting from DBUF and ARC
Those values require global atomics to get current hash_elements
values in few of the hottest code paths, while in all the years I
never cared about it. If somebody wants, it should be easy to
get it by periodic sampling, since neither ARC header nor DBUF
counts change so fast that it would be difficult to catch.
For now I've left hash_elements_max kstat for ARC, since it was
used/reported by arc_summary and it would break older versions,
but now it just reports the current value.
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Alexander Motin <[email protected]>
Sponsored by: iXsystems, Inc.
Closes #16759
Diffstat (limited to 'cmd')
| -rwxr-xr-x | cmd/arc_summary | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/cmd/arc_summary b/cmd/arc_summary index c24d400fa..72381d266 100755 --- a/cmd/arc_summary +++ b/cmd/arc_summary @@ -662,10 +662,7 @@ def section_arc(kstats_dict): print() print('ARC hash breakdown:') - prt_i1('Elements max:', f_hits(arc_stats['hash_elements_max'])) - prt_i2('Elements current:', - f_perc(arc_stats['hash_elements'], arc_stats['hash_elements_max']), - f_hits(arc_stats['hash_elements'])) + prt_i1('Elements:', f_hits(arc_stats['hash_elements'])) prt_i1('Collisions:', f_hits(arc_stats['hash_collisions'])) prt_i1('Chain max:', f_hits(arc_stats['hash_chain_max'])) |