-
-
Notifications
You must be signed in to change notification settings - Fork 49
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
This change adds new patches from Proxmox for the ZFS project.
- Loading branch information
1 parent
c68cdfe
commit b8de274
Showing
4 changed files
with
461 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
134 changes: 134 additions & 0 deletions
134
debian/patches/zfs/0010-arcstat-Fix-integer-division-with-python3.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,134 @@ | ||
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 | ||
From: Valmiky Arquissandas <kayvlim@gmail.com> | ||
Date: Fri, 8 Oct 2021 16:32:27 +0100 | ||
Subject: [PATCH] arcstat: Fix integer division with python3 | ||
|
||
The arcstat script requests compatibility with python2 and python3, but | ||
PEP 238 modified the / operator and results in erroneous output when | ||
run under python3. | ||
|
||
This commit replaces instances of / with //, yielding the expected | ||
result in both versions of Python. | ||
|
||
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> | ||
Reviewed-by: John Kennedy <john.kennedy@delphix.com> | ||
Reviewed-by: Ryan Moeller <ryan@ixsystems.com> | ||
Signed-off-by: Valmiky Arquissandas <foss@kayvlim.com> | ||
Closes #12603 | ||
(cherry picked from commit 2d02bba23d83ae8fede8d281edc255f01ccd28e9) | ||
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com> | ||
--- | ||
cmd/arcstat/arcstat.in | 66 +++++++++++++++++++++--------------------- | ||
1 file changed, 33 insertions(+), 33 deletions(-) | ||
|
||
diff --git a/cmd/arcstat/arcstat.in b/cmd/arcstat/arcstat.in | ||
index 9e7c52a6c..cd9a803a2 100755 | ||
--- a/cmd/arcstat/arcstat.in | ||
+++ b/cmd/arcstat/arcstat.in | ||
@@ -441,73 +441,73 @@ def calculate(): | ||
|
||
v = dict() | ||
v["time"] = time.strftime("%H:%M:%S", time.localtime()) | ||
- v["hits"] = d["hits"] / sint | ||
- v["miss"] = d["misses"] / sint | ||
+ v["hits"] = d["hits"] // sint | ||
+ v["miss"] = d["misses"] // sint | ||
v["read"] = v["hits"] + v["miss"] | ||
- v["hit%"] = 100 * v["hits"] / v["read"] if v["read"] > 0 else 0 | ||
+ v["hit%"] = 100 * v["hits"] // v["read"] if v["read"] > 0 else 0 | ||
v["miss%"] = 100 - v["hit%"] if v["read"] > 0 else 0 | ||
|
||
- v["dhit"] = (d["demand_data_hits"] + d["demand_metadata_hits"]) / sint | ||
- v["dmis"] = (d["demand_data_misses"] + d["demand_metadata_misses"]) / sint | ||
+ v["dhit"] = (d["demand_data_hits"] + d["demand_metadata_hits"]) // sint | ||
+ v["dmis"] = (d["demand_data_misses"] + d["demand_metadata_misses"]) // sint | ||
|
||
v["dread"] = v["dhit"] + v["dmis"] | ||
- v["dh%"] = 100 * v["dhit"] / v["dread"] if v["dread"] > 0 else 0 | ||
+ v["dh%"] = 100 * v["dhit"] // v["dread"] if v["dread"] > 0 else 0 | ||
v["dm%"] = 100 - v["dh%"] if v["dread"] > 0 else 0 | ||
|
||
- v["phit"] = (d["prefetch_data_hits"] + d["prefetch_metadata_hits"]) / sint | ||
+ v["phit"] = (d["prefetch_data_hits"] + d["prefetch_metadata_hits"]) // sint | ||
v["pmis"] = (d["prefetch_data_misses"] + | ||
- d["prefetch_metadata_misses"]) / sint | ||
+ d["prefetch_metadata_misses"]) // sint | ||
|
||
v["pread"] = v["phit"] + v["pmis"] | ||
- v["ph%"] = 100 * v["phit"] / v["pread"] if v["pread"] > 0 else 0 | ||
+ v["ph%"] = 100 * v["phit"] // v["pread"] if v["pread"] > 0 else 0 | ||
v["pm%"] = 100 - v["ph%"] if v["pread"] > 0 else 0 | ||
|
||
v["mhit"] = (d["prefetch_metadata_hits"] + | ||
- d["demand_metadata_hits"]) / sint | ||
+ d["demand_metadata_hits"]) // sint | ||
v["mmis"] = (d["prefetch_metadata_misses"] + | ||
- d["demand_metadata_misses"]) / sint | ||
+ d["demand_metadata_misses"]) // sint | ||
|
||
v["mread"] = v["mhit"] + v["mmis"] | ||
- v["mh%"] = 100 * v["mhit"] / v["mread"] if v["mread"] > 0 else 0 | ||
+ v["mh%"] = 100 * v["mhit"] // v["mread"] if v["mread"] > 0 else 0 | ||
v["mm%"] = 100 - v["mh%"] if v["mread"] > 0 else 0 | ||
|
||
v["arcsz"] = cur["size"] | ||
v["size"] = cur["size"] | ||
v["c"] = cur["c"] | ||
- v["mfu"] = d["mfu_hits"] / sint | ||
- v["mru"] = d["mru_hits"] / sint | ||
- v["mrug"] = d["mru_ghost_hits"] / sint | ||
- v["mfug"] = d["mfu_ghost_hits"] / sint | ||
- v["eskip"] = d["evict_skip"] / sint | ||
- v["el2skip"] = d["evict_l2_skip"] / sint | ||
- v["el2cach"] = d["evict_l2_cached"] / sint | ||
- v["el2el"] = d["evict_l2_eligible"] / sint | ||
- v["el2mfu"] = d["evict_l2_eligible_mfu"] / sint | ||
- v["el2mru"] = d["evict_l2_eligible_mru"] / sint | ||
- v["el2inel"] = d["evict_l2_ineligible"] / sint | ||
- v["mtxmis"] = d["mutex_miss"] / sint | ||
+ v["mfu"] = d["mfu_hits"] // sint | ||
+ v["mru"] = d["mru_hits"] // sint | ||
+ v["mrug"] = d["mru_ghost_hits"] // sint | ||
+ v["mfug"] = d["mfu_ghost_hits"] // sint | ||
+ v["eskip"] = d["evict_skip"] // sint | ||
+ v["el2skip"] = d["evict_l2_skip"] // sint | ||
+ v["el2cach"] = d["evict_l2_cached"] // sint | ||
+ v["el2el"] = d["evict_l2_eligible"] // sint | ||
+ v["el2mfu"] = d["evict_l2_eligible_mfu"] // sint | ||
+ v["el2mru"] = d["evict_l2_eligible_mru"] // sint | ||
+ v["el2inel"] = d["evict_l2_ineligible"] // sint | ||
+ v["mtxmis"] = d["mutex_miss"] // sint | ||
|
||
if l2exist: | ||
- v["l2hits"] = d["l2_hits"] / sint | ||
- v["l2miss"] = d["l2_misses"] / sint | ||
+ v["l2hits"] = d["l2_hits"] // sint | ||
+ v["l2miss"] = d["l2_misses"] // sint | ||
v["l2read"] = v["l2hits"] + v["l2miss"] | ||
- v["l2hit%"] = 100 * v["l2hits"] / v["l2read"] if v["l2read"] > 0 else 0 | ||
+ v["l2hit%"] = 100 * v["l2hits"] // v["l2read"] if v["l2read"] > 0 else 0 | ||
|
||
v["l2miss%"] = 100 - v["l2hit%"] if v["l2read"] > 0 else 0 | ||
v["l2asize"] = cur["l2_asize"] | ||
v["l2size"] = cur["l2_size"] | ||
- v["l2bytes"] = d["l2_read_bytes"] / sint | ||
+ v["l2bytes"] = d["l2_read_bytes"] // sint | ||
|
||
v["l2pref"] = cur["l2_prefetch_asize"] | ||
v["l2mfu"] = cur["l2_mfu_asize"] | ||
v["l2mru"] = cur["l2_mru_asize"] | ||
v["l2data"] = cur["l2_bufc_data_asize"] | ||
v["l2meta"] = cur["l2_bufc_metadata_asize"] | ||
- v["l2pref%"] = 100 * v["l2pref"] / v["l2asize"] | ||
- v["l2mfu%"] = 100 * v["l2mfu"] / v["l2asize"] | ||
- v["l2mru%"] = 100 * v["l2mru"] / v["l2asize"] | ||
- v["l2data%"] = 100 * v["l2data"] / v["l2asize"] | ||
- v["l2meta%"] = 100 * v["l2meta"] / v["l2asize"] | ||
+ v["l2pref%"] = 100 * v["l2pref"] // v["l2asize"] | ||
+ v["l2mfu%"] = 100 * v["l2mfu"] // v["l2asize"] | ||
+ v["l2mru%"] = 100 * v["l2mru"] // v["l2asize"] | ||
+ v["l2data%"] = 100 * v["l2data"] // v["l2asize"] | ||
+ v["l2meta%"] = 100 * v["l2meta"] // v["l2asize"] | ||
|
||
v["grow"] = 0 if cur["arc_no_grow"] else 1 | ||
v["need"] = cur["arc_need_free"] |
112 changes: 112 additions & 0 deletions
112
debian/patches/zfs/0011-arc-stat-summary-guard-access-to-l2arc-MFU-MRU-stats.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,112 @@ | ||
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 | ||
From: Thomas Lamprecht <t.lamprecht@proxmox.com> | ||
Date: Wed, 10 Nov 2021 09:29:47 +0100 | ||
Subject: [PATCH] arc stat/summary: guard access to l2arc MFU/MRU stats | ||
|
||
commit 085321621e79a75bea41c2b6511da6ebfbf2ba0a added printing MFU | ||
and MRU stats for 2.1 user space tools, but those keys are not | ||
available in the 2.0 module. That means it may break the arcstat and | ||
arc_summary tools after upgrade to 2.1 (user space), before a reboot | ||
to the new 2.1 ZFS kernel-module happened, due to python raising a | ||
KeyError on the dict access then. | ||
|
||
Move those two keys to a .get accessor with `0` as fallback, as it | ||
should be better to show some possible wrong data for new stat-keys | ||
than throwing an exception. | ||
|
||
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com> | ||
|
||
also move l2_mfu_asize l2_mru_asize l2_prefetch_asize | ||
l2_bufc_data_asize l2_bufc_metadata_asize to .get accessor | ||
(these are only present with a cache device in the pool) | ||
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com> | ||
--- | ||
cmd/arc_summary/arc_summary3 | 28 ++++++++++++++-------------- | ||
cmd/arcstat/arcstat.in | 14 +++++++------- | ||
2 files changed, 21 insertions(+), 21 deletions(-) | ||
|
||
diff --git a/cmd/arc_summary/arc_summary3 b/cmd/arc_summary/arc_summary3 | ||
index 7b28012ed..fe6a6d9e2 100755 | ||
--- a/cmd/arc_summary/arc_summary3 | ||
+++ b/cmd/arc_summary/arc_summary3 | ||
@@ -617,13 +617,13 @@ def section_arc(kstats_dict): | ||
prt_i1('L2 cached evictions:', f_bytes(arc_stats['evict_l2_cached'])) | ||
prt_i1('L2 eligible evictions:', f_bytes(arc_stats['evict_l2_eligible'])) | ||
prt_i2('L2 eligible MFU evictions:', | ||
- f_perc(arc_stats['evict_l2_eligible_mfu'], | ||
+ f_perc(arc_stats.get('evict_l2_eligible_mfu', 0), # 2.0 module compat | ||
arc_stats['evict_l2_eligible']), | ||
- f_bytes(arc_stats['evict_l2_eligible_mfu'])) | ||
+ f_bytes(arc_stats.get('evict_l2_eligible_mfu', 0))) | ||
prt_i2('L2 eligible MRU evictions:', | ||
- f_perc(arc_stats['evict_l2_eligible_mru'], | ||
+ f_perc(arc_stats.get('evict_l2_eligible_mru', 0), # 2.0 module compat | ||
arc_stats['evict_l2_eligible']), | ||
- f_bytes(arc_stats['evict_l2_eligible_mru'])) | ||
+ f_bytes(arc_stats.get('evict_l2_eligible_mru', 0))) | ||
prt_i1('L2 ineligible evictions:', | ||
f_bytes(arc_stats['evict_l2_ineligible'])) | ||
print() | ||
@@ -765,20 +765,20 @@ def section_l2arc(kstats_dict): | ||
f_perc(arc_stats['l2_hdr_size'], arc_stats['l2_size']), | ||
f_bytes(arc_stats['l2_hdr_size'])) | ||
prt_i2('MFU allocated size:', | ||
- f_perc(arc_stats['l2_mfu_asize'], arc_stats['l2_asize']), | ||
- f_bytes(arc_stats['l2_mfu_asize'])) | ||
+ f_perc(arc_stats.get('l2_mfu_asize', 0), arc_stats['l2_asize']), | ||
+ f_bytes(arc_stats.get('l2_mfu_asize', 0))) # 2.0 module compat | ||
prt_i2('MRU allocated size:', | ||
- f_perc(arc_stats['l2_mru_asize'], arc_stats['l2_asize']), | ||
- f_bytes(arc_stats['l2_mru_asize'])) | ||
+ f_perc(arc_stats.get('l2_mru_asize', 0), arc_stats['l2_asize']), | ||
+ f_bytes(arc_stats.get('l2_mru_asize', 0))) # 2.0 module compat | ||
prt_i2('Prefetch allocated size:', | ||
- f_perc(arc_stats['l2_prefetch_asize'], arc_stats['l2_asize']), | ||
- f_bytes(arc_stats['l2_prefetch_asize'])) | ||
+ f_perc(arc_stats.get('l2_prefetch_asize', 0), arc_stats['l2_asize']), | ||
+ f_bytes(arc_stats.get('l2_prefetch_asize',0))) # 2.0 module compat | ||
prt_i2('Data (buffer content) allocated size:', | ||
- f_perc(arc_stats['l2_bufc_data_asize'], arc_stats['l2_asize']), | ||
- f_bytes(arc_stats['l2_bufc_data_asize'])) | ||
+ f_perc(arc_stats.get('l2_bufc_data_asize', 0), arc_stats['l2_asize']), | ||
+ f_bytes(arc_stats.get('l2_bufc_data_asize', 0))) # 2.0 module compat | ||
prt_i2('Metadata (buffer content) allocated size:', | ||
- f_perc(arc_stats['l2_bufc_metadata_asize'], arc_stats['l2_asize']), | ||
- f_bytes(arc_stats['l2_bufc_metadata_asize'])) | ||
+ f_perc(arc_stats.get('l2_bufc_metadata_asize', 0), arc_stats['l2_asize']), | ||
+ f_bytes(arc_stats.get('l2_bufc_metadata_asize', 0))) # 2.0 module compat | ||
|
||
print() | ||
prt_1('L2ARC breakdown:', f_hits(l2_access_total)) | ||
diff --git a/cmd/arcstat/arcstat.in b/cmd/arcstat/arcstat.in | ||
index cd9a803a2..ea45dc602 100755 | ||
--- a/cmd/arcstat/arcstat.in | ||
+++ b/cmd/arcstat/arcstat.in | ||
@@ -482,8 +482,8 @@ def calculate(): | ||
v["el2skip"] = d["evict_l2_skip"] // sint | ||
v["el2cach"] = d["evict_l2_cached"] // sint | ||
v["el2el"] = d["evict_l2_eligible"] // sint | ||
- v["el2mfu"] = d["evict_l2_eligible_mfu"] // sint | ||
- v["el2mru"] = d["evict_l2_eligible_mru"] // sint | ||
+ v["el2mfu"] = d.get("evict_l2_eligible_mfu", 0) // sint | ||
+ v["el2mru"] = d.get("evict_l2_eligible_mru", 0) // sint | ||
v["el2inel"] = d["evict_l2_ineligible"] // sint | ||
v["mtxmis"] = d["mutex_miss"] // sint | ||
|
||
@@ -498,11 +498,11 @@ def calculate(): | ||
v["l2size"] = cur["l2_size"] | ||
v["l2bytes"] = d["l2_read_bytes"] // sint | ||
|
||
- v["l2pref"] = cur["l2_prefetch_asize"] | ||
- v["l2mfu"] = cur["l2_mfu_asize"] | ||
- v["l2mru"] = cur["l2_mru_asize"] | ||
- v["l2data"] = cur["l2_bufc_data_asize"] | ||
- v["l2meta"] = cur["l2_bufc_metadata_asize"] | ||
+ v["l2pref"] = cur.get("l2_prefetch_asize", 0) | ||
+ v["l2mfu"] = cur.get("l2_mfu_asize", 0) | ||
+ v["l2mru"] = cur.get("l2_mru_asize", 0) | ||
+ v["l2data"] = cur.get("l2_bufc_data_asize", 0) | ||
+ v["l2meta"] = cur.get("l2_bufc_metadata_asize", 0) | ||
v["l2pref%"] = 100 * v["l2pref"] // v["l2asize"] | ||
v["l2mfu%"] = 100 * v["l2mfu"] // v["l2asize"] | ||
v["l2mru%"] = 100 * v["l2mru"] // v["l2asize"] |
Oops, something went wrong.