diff options
author | Adam D. Moss <[email protected]> | 2020-10-20 11:39:52 -0700 |
---|---|---|
committer | GitHub <[email protected]> | 2020-10-20 11:39:52 -0700 |
commit | 666aa69f32ff2558ef9e9a27bc4cb5559e21d795 (patch) | |
tree | 8a4b4647c3a84ef1aae50a0e4d0aacc5d77d3432 /tests/runfiles/common.run | |
parent | 241c62bdd7fbc0683d54c40b4a184cf889ebd774 (diff) |
Non-l2arc pool reads shouldn't be l2arc misses
The current l2_misses accounting behavior treats all reads to pools
without a configured l2arc as an l2arc miss, IFF there is at least
one other pool on the system which does have an l2arc configured.
This makes it extremely hard to tune for an improved l2arc hit/miss
ratio because this ratio will be modulated by reads from pools which
do not (and should not) have l2arc devices; its upper limit will
depend on the ratio of reads from l2arc'd pools and non-l2arc'd pools.
This PR prevents ARC reads affecting l2arc stats (n.b. l2_misses is
the only relevant one) where the target spa doesn't have an l2arc.
Includes new test - l2arc_l2miss_pos.ksh
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: George Amanakis <[email protected]>
Signed-off-by: Adam Moss <[email protected]>
Closes #10921
Diffstat (limited to 'tests/runfiles/common.run')
-rw-r--r-- | tests/runfiles/common.run | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/runfiles/common.run b/tests/runfiles/common.run index d935f5b04..ef4858cc7 100644 --- a/tests/runfiles/common.run +++ b/tests/runfiles/common.run @@ -899,7 +899,7 @@ post = tags = ['functional', 'log_spacemap'] [tests/functional/l2arc] -tests = ['l2arc_arcstats_pos', 'l2arc_mfuonly_pos', +tests = ['l2arc_arcstats_pos', 'l2arc_mfuonly_pos', 'l2arc_l2miss_pos', 'persist_l2arc_001_pos', 'persist_l2arc_002_pos', 'persist_l2arc_003_neg', 'persist_l2arc_004_pos', 'persist_l2arc_005_pos', 'persist_l2arc_006_pos', 'persist_l2arc_007_pos', 'persist_l2arc_008_pos'] |