Compare commits
34 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| e295f30e6a | |||
| 96c807af63 | |||
| 88fd6e053b | |||
| 4f818e9880 | |||
| 310afb0d19 | |||
| 0f9a07b53e | |||
| aa99285dda | |||
| 7e3b7d81a1 | |||
| 28de0abfa9 | |||
| a20ffcd44f | |||
| 1382616c40 | |||
| 1621cb1079 | |||
| 508220ed2c | |||
| 6da885c3b1 | |||
| 2840fef531 | |||
| 1f8dab1515 | |||
| 2c95b92384 | |||
| 9e8946d4b9 | |||
| 8c6520d1fc | |||
| aa26132525 | |||
| 13c7e925aa | |||
| a80c5e3597 | |||
| 149fd91bb2 | |||
| 362d3432be | |||
| f5ed5be89a | |||
| 5891aaec34 | |||
| 63e591d8a9 | |||
| d855afe7be | |||
| 34d701d1ac | |||
| 40fe66e33e | |||
| 1b7710c13c | |||
| 2f5fca8a1a | |||
| 8ba2c83746 | |||
| dff6b68bf5 |
Vendored
+28
-6
@@ -1,14 +1,36 @@
|
|||||||
zfs-linux (2.1.15-pve1) bullseye; urgency=medium
|
zfs-linux (2.2.0-pve3) bookworm; urgency=medium
|
||||||
|
|
||||||
* update ZFS to 2.1.15
|
* pick bug-fixes staged for 2.2.1:
|
||||||
|
- add a tunable to disable BRT support and disable it by default
|
||||||
|
- fix block cloning between unencrypted and encrypted datasets
|
||||||
|
- disable block cloning by default
|
||||||
|
|
||||||
-- Proxmox Support Team <support@proxmox.com> Fri, 29 Mar 2024 15:18:31 +0100
|
-- Proxmox Support Team <support@proxmox.com> Fri, 17 Nov 2023 17:32:58 +0100
|
||||||
|
|
||||||
zfs-linux (2.1.14-pve1) bullseye; urgency=medium
|
zfs-linux (2.2.0-pve2) bookworm; urgency=medium
|
||||||
|
|
||||||
* update ZFS to 2.1.14
|
* avoid error from zfs-mount when /etc/exports.d does not exist (yet)
|
||||||
|
|
||||||
-- Proxmox Support Team <support@proxmox.com> Fri, 01 Dec 2023 13:47:16 +0100
|
* ensure vdev_stat struct layout compat between 2.1 and 2.2, avoiding
|
||||||
|
false-positive detection of the non-allocating feature from 2.2 when the
|
||||||
|
kernel still used the 2.1 module.
|
||||||
|
|
||||||
|
-- Proxmox Support Team <support@proxmox.com> Sun, 12 Nov 2023 16:02:02 +0100
|
||||||
|
|
||||||
|
zfs-linux (2.2.0-pve1) bookworm; urgency=medium
|
||||||
|
|
||||||
|
* update ZFS to 2.2.0
|
||||||
|
|
||||||
|
* zfsutils-linux:
|
||||||
|
- install new systemd units to trim a pool periodically
|
||||||
|
- ship new `zilstat` binary
|
||||||
|
- and new man pages for zfs lock, zfs unlock and vdev properties
|
||||||
|
- remove man pages for zfs jail and zfs unjail, those are for FreeBSD only
|
||||||
|
and the respective commands where never exposed for Linux
|
||||||
|
|
||||||
|
* fix #5014: re-enable blk-mq optimization
|
||||||
|
|
||||||
|
-- Proxmox Support Team <support@proxmox.com> Sun, 15 Oct 2023 12:09:24 +0200
|
||||||
|
|
||||||
zfs-linux (2.1.13-pve1) bookworm; urgency=medium
|
zfs-linux (2.1.13-pve1) bookworm; urgency=medium
|
||||||
|
|
||||||
|
|||||||
Vendored
+2
-15
@@ -151,6 +151,7 @@ Section: contrib/doc
|
|||||||
Architecture: all
|
Architecture: all
|
||||||
Depends:
|
Depends:
|
||||||
${sphinxdoc:Depends},
|
${sphinxdoc:Depends},
|
||||||
|
${sphinxdoc:Built-Using},
|
||||||
${misc:Depends}
|
${misc:Depends}
|
||||||
Recommends:
|
Recommends:
|
||||||
python3-pyzfs
|
python3-pyzfs
|
||||||
@@ -202,7 +203,7 @@ Description: OpenZFS root filesystem capabilities for Linux - dracut
|
|||||||
Package: zfsutils-linux
|
Package: zfsutils-linux
|
||||||
Section: contrib/admin
|
Section: contrib/admin
|
||||||
Architecture: linux-any
|
Architecture: linux-any
|
||||||
Depends: python3, ${misc:Depends}, ${python3:Depends}, ${shlibs:Depends}
|
Depends: python3, ${misc:Depends}, ${shlibs:Depends}
|
||||||
Recommends: zfs-zed
|
Recommends: zfs-zed
|
||||||
Suggests: nfs-kernel-server,
|
Suggests: nfs-kernel-server,
|
||||||
samba-common-bin (>= 3.0.23),
|
samba-common-bin (>= 3.0.23),
|
||||||
@@ -285,17 +286,3 @@ Description: Solaris Porting Layer user-space utilities for Linux (dummy)
|
|||||||
to Linux primitives.
|
to Linux primitives.
|
||||||
.
|
.
|
||||||
This is a transitional dummy package. It can safely be removed.
|
This is a transitional dummy package. It can safely be removed.
|
||||||
|
|
||||||
Package: zfs-dbg
|
|
||||||
Section: contrib/metapackages
|
|
||||||
Architecture: all
|
|
||||||
Suggests: libnvpair3linux-dbgsym,
|
|
||||||
libpam-zfs-dbgsym,
|
|
||||||
libuutil3linux-dbgsym,
|
|
||||||
libzfs4linux-dbgsym,
|
|
||||||
libzfsbootenv1linux-dbgsym,
|
|
||||||
libzpool5linux-dbgsym,
|
|
||||||
zfs-test-dbgsym,
|
|
||||||
zfsutils-linux-dbgsym,
|
|
||||||
zfs-zed-dbgsym,
|
|
||||||
Description: Transitional package. It can be safely removed.
|
|
||||||
|
|||||||
Vendored
+13
-41
@@ -37,25 +37,26 @@ Copyright: 2011, 2013, Nexenta Systems, Inc.
|
|||||||
2007, 2009, Sun Microsystems, Inc.
|
2007, 2009, Sun Microsystems, Inc.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/arc_summary/*
|
Files: cmd/arc_summary
|
||||||
Copyright:
|
Copyright:
|
||||||
2010, 2011, Jason J. Hellenthal <jhell@DataIX.net>
|
2010, 2011, Jason J. Hellenthal <jhell@DataIX.net>
|
||||||
2010, Martin Matuska <mm@FreeBSD.org>
|
2010, Martin Matuska <mm@FreeBSD.org>
|
||||||
2008, Ben Rockwood <benr@cuddletech.com>
|
2008, Ben Rockwood <benr@cuddletech.com>
|
||||||
|
2017, Scot W. Stevenson <scot.stevenson@gmail.com>
|
||||||
License: BSD-2-clause
|
License: BSD-2-clause
|
||||||
|
|
||||||
Files: cmd/arcstat/*
|
Files: cmd/arcstat.in
|
||||||
Source: http://github.com/mharsch/arcstat
|
Source: http://github.com/mharsch/arcstat
|
||||||
Copyright:
|
Copyright:
|
||||||
2007, Oracle and/or its affiliates.
|
2007, Oracle and/or its affiliates.
|
||||||
2010-2015, Mike Harsch
|
2010-2015, Mike Harsch
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/dbufstat/*
|
Files: cmd/dbufstat.in
|
||||||
Copyright: 2013, Lawrence Livermore National Security, LLC
|
Copyright: 2013, Lawrence Livermore National Security, LLC
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/mount_zfs/*
|
Files: cmd/mount_zfs.c
|
||||||
Copyright: 2011, Lawrence Livermore National Security, LLC
|
Copyright: 2011, Lawrence Livermore National Security, LLC
|
||||||
2005, 2010, Oracle and/or its affiliates.
|
2005, 2010, Oracle and/or its affiliates.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
@@ -64,7 +65,7 @@ Files: cmd/raidz_test/*
|
|||||||
Copyright: 2016 Gvozden Nešković.
|
Copyright: 2016 Gvozden Nešković.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/vdev_id/*
|
Files: udev/vdev_id
|
||||||
Copyright: 2011, 2013, Nexenta Systems, Inc.
|
Copyright: 2011, 2013, Nexenta Systems, Inc.
|
||||||
2007, 2009, Sun Microsystems, Inc.
|
2007, 2009, Sun Microsystems, Inc.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
@@ -106,7 +107,7 @@ Copyright:
|
|||||||
2018 Datto Inc.
|
2018 Datto Inc.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/zhack/*
|
Files: cmd/zhack.c
|
||||||
Copyright: 2013, Steven Hartland.
|
Copyright: 2013, Steven Hartland.
|
||||||
2011, 2012, 2014, Delphix.
|
2011, 2012, 2014, Delphix.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
@@ -139,7 +140,7 @@ Copyright:
|
|||||||
2010, Sun Microsystems, Inc.
|
2010, Sun Microsystems, Inc.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/ztest/*
|
Files: cmd/ztest.c
|
||||||
Copyright:
|
Copyright:
|
||||||
2005, 2010, Oracle and/or its affiliates.
|
2005, 2010, Oracle and/or its affiliates.
|
||||||
2011, 2018 by Delphix.
|
2011, 2018 by Delphix.
|
||||||
@@ -150,7 +151,7 @@ Copyright:
|
|||||||
2017, Intel Corporation.
|
2017, Intel Corporation.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: cmd/zvol_id/*
|
Files: udev/zvol_id.c
|
||||||
Copyright: 2011, Fajar A. Nugraha.
|
Copyright: 2011, Fajar A. Nugraha.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
@@ -158,27 +159,6 @@ Files: config/*
|
|||||||
Copyright: 1996-2012, Free Software Foundation, Inc.
|
Copyright: 1996-2012, Free Software Foundation, Inc.
|
||||||
License: GPL-2+ with autoconf exception
|
License: GPL-2+ with autoconf exception
|
||||||
|
|
||||||
Files: config/ltoptions.m4
|
|
||||||
config/lt~obsolete.m4
|
|
||||||
config/ltversion.m4
|
|
||||||
config/libtool.m4
|
|
||||||
config/ltsugar.m4
|
|
||||||
Copyright: 1996-2012, Free Software Foundation, Inc.
|
|
||||||
License: PERMISSIVE
|
|
||||||
This file is free software; the Free Software Foundation gives
|
|
||||||
unlimited permission to copy and/or distribute it, with or without
|
|
||||||
modifications, as long as this notice is preserved.
|
|
||||||
|
|
||||||
Files: config/install-sh
|
|
||||||
Copyright: 1994, X Consortium
|
|
||||||
License: Expat
|
|
||||||
|
|
||||||
Files: configure
|
|
||||||
Copyright: 1992-1996, 1998-2010, Free Software
|
|
||||||
License: PERMISSIVE2
|
|
||||||
This configure script is free software; the Free Software Foundation
|
|
||||||
gives unlimited permission to copy, distribute and modify it.
|
|
||||||
|
|
||||||
Files: contrib/bash_completion.d/*
|
Files: contrib/bash_completion.d/*
|
||||||
Copyright: 2010, 2013, Aneurin Price <aneurin.price@gmail.com>
|
Copyright: 2010, 2013, Aneurin Price <aneurin.price@gmail.com>
|
||||||
License: Expat
|
License: Expat
|
||||||
@@ -203,11 +183,6 @@ Copyright:
|
|||||||
2018-2020, Mo Zhou <lumin@debian.org>
|
2018-2020, Mo Zhou <lumin@debian.org>
|
||||||
License: GPL-2+
|
License: GPL-2+
|
||||||
|
|
||||||
Files: debian/po/*
|
|
||||||
Copyright:
|
|
||||||
2013, The Debian po file translators.
|
|
||||||
License: CDDL-1.0
|
|
||||||
|
|
||||||
Files: etc/init.d/zfs-*.in
|
Files: etc/init.d/zfs-*.in
|
||||||
Copyright:
|
Copyright:
|
||||||
2016, Carlo Landmeter <clandmeter@gmail.com>
|
2016, Carlo Landmeter <clandmeter@gmail.com>
|
||||||
@@ -421,10 +396,6 @@ Copyright: 2007, Sun Microsystems, Inc.
|
|||||||
2013, Delphix
|
2013, Delphix
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: man/man8/zstreamdump.8
|
|
||||||
Copyright: 2009, Sun Microsystems, Inc.
|
|
||||||
License: CDDL-1.0
|
|
||||||
|
|
||||||
Files: module/*
|
Files: module/*
|
||||||
Copyright: 2011-2014, Delphix.
|
Copyright: 2011-2014, Delphix.
|
||||||
2007, 2009, 2010, Sun Microsystems, Inc.
|
2007, 2009, 2010, Sun Microsystems, Inc.
|
||||||
@@ -577,7 +548,6 @@ Files: module/zfs/dmu_zfetch.c
|
|||||||
module/zfs/rrwlock.c
|
module/zfs/rrwlock.c
|
||||||
module/zfs/space_map.c
|
module/zfs/space_map.c
|
||||||
module/zfs/space_reftree.c
|
module/zfs/space_reftree.c
|
||||||
module/zfs/vdev_cache.c
|
|
||||||
module/zfs/vdev_mirror.c
|
module/zfs/vdev_mirror.c
|
||||||
module/zfs/vdev_missing.c
|
module/zfs/vdev_missing.c
|
||||||
module/zfs/vdev_queue.c
|
module/zfs/vdev_queue.c
|
||||||
@@ -645,14 +615,16 @@ Copyright: 2013, Steven Hartland.
|
|||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: module/zfs/gzip.c
|
Files: module/zfs/gzip.c
|
||||||
module/zfs/sha256.c
|
|
||||||
module/zfs/spa_boot.c
|
|
||||||
module/zfs/unique.c
|
module/zfs/unique.c
|
||||||
module/zfs/zfs_byteswap.c
|
module/zfs/zfs_byteswap.c
|
||||||
module/zfs/zle.c
|
module/zfs/zle.c
|
||||||
Copyright: 2005-2010, Sun Microsystems, Inc.
|
Copyright: 2005-2010, Sun Microsystems, Inc.
|
||||||
License: CDDL-1.0
|
License: CDDL-1.0
|
||||||
|
|
||||||
|
Files: module/icp/algs/sha2/*
|
||||||
|
Copyright: 2022, Tino Reichardt <milky-zfs@mcmilk.de>
|
||||||
|
License: CDDL-1.0
|
||||||
|
|
||||||
Files: module/zfs/lz4.c
|
Files: module/zfs/lz4.c
|
||||||
Copyright: 2011-2013, Yann Collet
|
Copyright: 2011-2013, Yann Collet
|
||||||
License: BSD-2-clause
|
License: BSD-2-clause
|
||||||
|
|||||||
+3
-3
@@ -13,13 +13,13 @@ Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
|||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/etc/systemd/system/zfs-zed.service.in b/etc/systemd/system/zfs-zed.service.in
|
diff --git a/etc/systemd/system/zfs-zed.service.in b/etc/systemd/system/zfs-zed.service.in
|
||||||
index be80025a4..20ce8e632 100644
|
index be2fc6734..7606604ec 100644
|
||||||
--- a/etc/systemd/system/zfs-zed.service.in
|
--- a/etc/systemd/system/zfs-zed.service.in
|
||||||
+++ b/etc/systemd/system/zfs-zed.service.in
|
+++ b/etc/systemd/system/zfs-zed.service.in
|
||||||
@@ -4,7 +4,7 @@ Documentation=man:zed(8)
|
@@ -5,7 +5,7 @@ ConditionPathIsDirectory=/sys/module/zfs
|
||||||
ConditionPathIsDirectory=/sys/module/zfs
|
|
||||||
|
|
||||||
[Service]
|
[Service]
|
||||||
|
EnvironmentFile=-@initconfdir@/zfs
|
||||||
-ExecStart=@sbindir@/zed -F
|
-ExecStart=@sbindir@/zed -F
|
||||||
+ExecStart=/usr/sbin/zed -F
|
+ExecStart=/usr/sbin/zed -F
|
||||||
Restart=always
|
Restart=always
|
||||||
|
|||||||
@@ -14,13 +14,13 @@ Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
|||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/etc/systemd/system/zfs-import-scan.service.in b/etc/systemd/system/zfs-import-scan.service.in
|
diff --git a/etc/systemd/system/zfs-import-scan.service.in b/etc/systemd/system/zfs-import-scan.service.in
|
||||||
index 598ef501b..e4f3a70c1 100644
|
index c5dd45d87..1c792edf0 100644
|
||||||
--- a/etc/systemd/system/zfs-import-scan.service.in
|
--- a/etc/systemd/system/zfs-import-scan.service.in
|
||||||
+++ b/etc/systemd/system/zfs-import-scan.service.in
|
+++ b/etc/systemd/system/zfs-import-scan.service.in
|
||||||
@@ -13,7 +13,7 @@ ConditionPathIsDirectory=/sys/module/zfs
|
@@ -14,7 +14,7 @@ ConditionPathIsDirectory=/sys/module/zfs
|
||||||
[Service]
|
|
||||||
Type=oneshot
|
Type=oneshot
|
||||||
RemainAfterExit=yes
|
RemainAfterExit=yes
|
||||||
|
EnvironmentFile=-@initconfdir@/zfs
|
||||||
-ExecStart=@sbindir@/zpool import -aN -o cachefile=none $ZPOOL_IMPORT_OPTS
|
-ExecStart=@sbindir@/zpool import -aN -o cachefile=none $ZPOOL_IMPORT_OPTS
|
||||||
+ExecStart=@sbindir@/zpool import -aN -d /dev/disk/by-id -o cachefile=none $ZPOOL_IMPORT_OPTS
|
+ExecStart=@sbindir@/zpool import -aN -d /dev/disk/by-id -o cachefile=none $ZPOOL_IMPORT_OPTS
|
||||||
|
|
||||||
|
|||||||
+2
-2
@@ -13,10 +13,10 @@ Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
|||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/cmd/zed/zed.d/zed.rc b/cmd/zed/zed.d/zed.rc
|
diff --git a/cmd/zed/zed.d/zed.rc b/cmd/zed/zed.d/zed.rc
|
||||||
index 1dfd43454..0180dd827 100644
|
index 78dc1afc7..41d5539ea 100644
|
||||||
--- a/cmd/zed/zed.d/zed.rc
|
--- a/cmd/zed/zed.d/zed.rc
|
||||||
+++ b/cmd/zed/zed.d/zed.rc
|
+++ b/cmd/zed/zed.d/zed.rc
|
||||||
@@ -42,7 +42,7 @@ ZED_EMAIL_ADDR="root"
|
@@ -41,7 +41,7 @@ ZED_EMAIL_ADDR="root"
|
||||||
##
|
##
|
||||||
# Minimum number of seconds between notifications for a similar event.
|
# Minimum number of seconds between notifications for a similar event.
|
||||||
#
|
#
|
||||||
|
|||||||
+25
-10
@@ -3,29 +3,44 @@ From: Antonio Russo <antonio.e.russo@gmail.com>
|
|||||||
Date: Fri, 20 Mar 2020 17:28:43 +0100
|
Date: Fri, 20 Mar 2020 17:28:43 +0100
|
||||||
Subject: [PATCH] dont symlink zed scripts
|
Subject: [PATCH] dont symlink zed scripts
|
||||||
|
|
||||||
(cherry picked and adapted from 5cee380324d74e640d5dd7a360faba3994c8007f [0])
|
Of the zedlet scripts shipped by upstream, a subset are enabled by
|
||||||
|
default, by creating symlinks in /etc/zfs/zed.d. These symlinks are
|
||||||
|
shipped in the zfs-zed package. dpkg, however, does not support
|
||||||
|
conffile handling of symlinks, and therefore any changes (removals) to
|
||||||
|
the symlinks are not preserved on package upgrade.
|
||||||
|
|
||||||
[0] https://salsa.debian.org/zfsonlinux-team/zfs.git
|
To address this policy violation, we:
|
||||||
|
|
||||||
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
1. During package build, create a list of enabled-by-default zedlets,
|
||||||
|
instead of creating symlinks.
|
||||||
|
2. On package removal, identify all enabled-by-default zedlets whose
|
||||||
|
symlinks do not exist (i.e., were removed by the user). This is done
|
||||||
|
by creating "whiteout" links to /dev/null in their place).
|
||||||
|
3. On package installation, create links to enabled-by-default zedlets
|
||||||
|
UNLESS there is already a file there (i.e., abort if there is a
|
||||||
|
whiteout link).
|
||||||
|
4. We also clean up broken symlinks to removed zedlets at package
|
||||||
|
postinst.
|
||||||
|
|
||||||
Description: track default symlinks, instead of symlinking
|
Description: track default symlinks, instead of symlinking
|
||||||
Forwarded: no need
|
Forwarded: no need
|
||||||
|
(cherry picked from https://salsa.debian.org/zfsonlinux-team/zfs/-/commit/5cee380324d7)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
---
|
---
|
||||||
cmd/zed/zed.d/Makefile.am | 2 +-
|
cmd/zed/zed.d/Makefile.am | 2 +-
|
||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/cmd/zed/zed.d/Makefile.am b/cmd/zed/zed.d/Makefile.am
|
diff --git a/cmd/zed/zed.d/Makefile.am b/cmd/zed/zed.d/Makefile.am
|
||||||
index 1905a9207..6dc06252a 100644
|
index 812558cf6..f802cf140 100644
|
||||||
--- a/cmd/zed/zed.d/Makefile.am
|
--- a/cmd/zed/zed.d/Makefile.am
|
||||||
+++ b/cmd/zed/zed.d/Makefile.am
|
+++ b/cmd/zed/zed.d/Makefile.am
|
||||||
@@ -51,7 +51,7 @@ install-data-hook:
|
@@ -48,7 +48,7 @@ zed-install-data-hook:
|
||||||
for f in $(zedconfdefaults); do \
|
set -x; for f in $(zedconfdefaults); do \
|
||||||
test -f "$(DESTDIR)$(zedconfdir)/$${f}" -o \
|
[ -f "$(DESTDIR)$(zedconfdir)/$${f}" ] ||\
|
||||||
-L "$(DESTDIR)$(zedconfdir)/$${f}" || \
|
[ -L "$(DESTDIR)$(zedconfdir)/$${f}" ] || \
|
||||||
- ln -s "$(zedexecdir)/$${f}" "$(DESTDIR)$(zedconfdir)"; \
|
- $(LN_S) "$(zedexecdir)/$${f}" "$(DESTDIR)$(zedconfdir)"; \
|
||||||
+ echo "$${f}" >> "$(DESTDIR)$(zedexecdir)/DEFAULT-ENABLED" ; \
|
+ echo "$${f}" >> "$(DESTDIR)$(zedexecdir)/DEFAULT-ENABLED" ; \
|
||||||
done
|
done
|
||||||
chmod 0600 "$(DESTDIR)$(zedconfdir)/zed.rc"
|
|
||||||
|
|
||||||
|
SHELLCHECKSCRIPTS += $(dist_zedconf_DATA) $(dist_zedexec_SCRIPTS) $(nodist_zedexec_SCRIPTS)
|
||||||
|
|||||||
@@ -13,16 +13,28 @@ can contain characters which will be escaped by systemd.
|
|||||||
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
---
|
---
|
||||||
etc/systemd/system/50-zfs.preset.in | 1 +
|
etc/Makefile.am | 1 +
|
||||||
etc/systemd/system/Makefile.am | 1 +
|
etc/systemd/system/50-zfs.preset | 1 +
|
||||||
etc/systemd/system/zfs-import@.service.in | 16 ++++++++++++++++
|
etc/systemd/system/zfs-import@.service.in | 16 ++++++++++++++++
|
||||||
3 files changed, 18 insertions(+)
|
3 files changed, 18 insertions(+)
|
||||||
create mode 100644 etc/systemd/system/zfs-import@.service.in
|
create mode 100644 etc/systemd/system/zfs-import@.service.in
|
||||||
|
|
||||||
diff --git a/etc/systemd/system/50-zfs.preset.in b/etc/systemd/system/50-zfs.preset.in
|
diff --git a/etc/Makefile.am b/etc/Makefile.am
|
||||||
|
index 7187762d3..de131dc87 100644
|
||||||
|
--- a/etc/Makefile.am
|
||||||
|
+++ b/etc/Makefile.am
|
||||||
|
@@ -54,6 +54,7 @@ dist_systemdpreset_DATA = \
|
||||||
|
systemdunit_DATA = \
|
||||||
|
%D%/systemd/system/zfs-import-cache.service \
|
||||||
|
%D%/systemd/system/zfs-import-scan.service \
|
||||||
|
+ %D%/systemd/system/zfs-import@.service \
|
||||||
|
%D%/systemd/system/zfs-import.target \
|
||||||
|
%D%/systemd/system/zfs-mount.service \
|
||||||
|
%D%/systemd/system/zfs-scrub-monthly@.timer \
|
||||||
|
diff --git a/etc/systemd/system/50-zfs.preset b/etc/systemd/system/50-zfs.preset
|
||||||
index e4056a92c..030611419 100644
|
index e4056a92c..030611419 100644
|
||||||
--- a/etc/systemd/system/50-zfs.preset.in
|
--- a/etc/systemd/system/50-zfs.preset
|
||||||
+++ b/etc/systemd/system/50-zfs.preset.in
|
+++ b/etc/systemd/system/50-zfs.preset
|
||||||
@@ -1,6 +1,7 @@
|
@@ -1,6 +1,7 @@
|
||||||
# ZFS is enabled by default
|
# ZFS is enabled by default
|
||||||
enable zfs-import-cache.service
|
enable zfs-import-cache.service
|
||||||
@@ -31,18 +43,6 @@ index e4056a92c..030611419 100644
|
|||||||
enable zfs-import.target
|
enable zfs-import.target
|
||||||
enable zfs-mount.service
|
enable zfs-mount.service
|
||||||
enable zfs-share.service
|
enable zfs-share.service
|
||||||
diff --git a/etc/systemd/system/Makefile.am b/etc/systemd/system/Makefile.am
|
|
||||||
index 35f833de5..af3ae597c 100644
|
|
||||||
--- a/etc/systemd/system/Makefile.am
|
|
||||||
+++ b/etc/systemd/system/Makefile.am
|
|
||||||
@@ -7,6 +7,7 @@ systemdunit_DATA = \
|
|
||||||
zfs-zed.service \
|
|
||||||
zfs-import-cache.service \
|
|
||||||
zfs-import-scan.service \
|
|
||||||
+ zfs-import@.service \
|
|
||||||
zfs-mount.service \
|
|
||||||
zfs-share.service \
|
|
||||||
zfs-volume-wait.service \
|
|
||||||
diff --git a/etc/systemd/system/zfs-import@.service.in b/etc/systemd/system/zfs-import@.service.in
|
diff --git a/etc/systemd/system/zfs-import@.service.in b/etc/systemd/system/zfs-import@.service.in
|
||||||
new file mode 100644
|
new file mode 100644
|
||||||
index 000000000..9b4ee9371
|
index 000000000..9b4ee9371
|
||||||
|
|||||||
@@ -15,36 +15,36 @@ Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
|||||||
rename man/{man1/arcstat.1 => man8/arcstat.8} (99%)
|
rename man/{man1/arcstat.1 => man8/arcstat.8} (99%)
|
||||||
|
|
||||||
diff --git a/man/Makefile.am b/man/Makefile.am
|
diff --git a/man/Makefile.am b/man/Makefile.am
|
||||||
index 64650c2b9..95a66a62f 100644
|
index 36c1aede1..94fd96e58 100644
|
||||||
--- a/man/Makefile.am
|
--- a/man/Makefile.am
|
||||||
+++ b/man/Makefile.am
|
+++ b/man/Makefile.am
|
||||||
@@ -8,7 +8,6 @@ dist_man_MANS = \
|
@@ -2,7 +2,6 @@ dist_noinst_man_MANS = \
|
||||||
man1/ztest.1 \
|
%D%/man1/cstyle.1
|
||||||
man1/raidz_test.1 \
|
|
||||||
man1/zvol_wait.1 \
|
dist_man_MANS = \
|
||||||
- man1/arcstat.1 \
|
- %D%/man1/arcstat.1 \
|
||||||
\
|
%D%/man1/raidz_test.1 \
|
||||||
man5/vdev_id.conf.5 \
|
%D%/man1/test-runner.1 \
|
||||||
\
|
%D%/man1/zhack.1 \
|
||||||
@@ -22,6 +21,7 @@ dist_man_MANS = \
|
@@ -22,6 +21,7 @@ dist_man_MANS = \
|
||||||
man7/zpoolconcepts.7 \
|
%D%/man7/zpoolconcepts.7 \
|
||||||
man7/zpoolprops.7 \
|
%D%/man7/zpoolprops.7 \
|
||||||
\
|
\
|
||||||
+ man8/arcstat.8 \
|
+ %D%/man8/arcstat.8 \
|
||||||
man8/fsck.zfs.8 \
|
%D%/man8/fsck.zfs.8 \
|
||||||
man8/mount.zfs.8 \
|
%D%/man8/mount.zfs.8 \
|
||||||
man8/vdev_id.8 \
|
%D%/man8/vdev_id.8 \
|
||||||
diff --git a/man/man1/arcstat.1 b/man/man8/arcstat.8
|
diff --git a/man/man1/arcstat.1 b/man/man8/arcstat.8
|
||||||
similarity index 99%
|
similarity index 99%
|
||||||
rename from man/man1/arcstat.1
|
rename from man/man1/arcstat.1
|
||||||
rename to man/man8/arcstat.8
|
rename to man/man8/arcstat.8
|
||||||
index a69cd8937..dfe9c971b 100644
|
index 82358fa68..a8fb55498 100644
|
||||||
--- a/man/man1/arcstat.1
|
--- a/man/man1/arcstat.1
|
||||||
+++ b/man/man8/arcstat.8
|
+++ b/man/man8/arcstat.8
|
||||||
@@ -13,7 +13,7 @@
|
@@ -13,7 +13,7 @@
|
||||||
.\" Copyright (c) 2020 by AJ Jordan. All rights reserved.
|
.\" Copyright (c) 2020 by AJ Jordan. All rights reserved.
|
||||||
.\"
|
.\"
|
||||||
.Dd May 26, 2021
|
.Dd December 23, 2022
|
||||||
-.Dt ARCSTAT 1
|
-.Dt ARCSTAT 1
|
||||||
+.Dt ARCSTAT 8
|
+.Dt ARCSTAT 8
|
||||||
.Os
|
.Os
|
||||||
|
|||||||
+14
-14
@@ -22,15 +22,15 @@ l2_bufc_data_asize l2_bufc_metadata_asize to .get accessor
|
|||||||
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
---
|
---
|
||||||
cmd/arc_summary/arc_summary3 | 28 ++++++++++++++--------------
|
cmd/arc_summary | 28 ++++++++++++++--------------
|
||||||
cmd/arcstat/arcstat.in | 14 +++++++-------
|
cmd/arcstat.in | 14 +++++++-------
|
||||||
2 files changed, 21 insertions(+), 21 deletions(-)
|
2 files changed, 21 insertions(+), 21 deletions(-)
|
||||||
|
|
||||||
diff --git a/cmd/arc_summary/arc_summary3 b/cmd/arc_summary/arc_summary3
|
diff --git a/cmd/arc_summary b/cmd/arc_summary
|
||||||
index 9d0c2d30d..fd2581ae2 100755
|
index 426e02070..9de198150 100755
|
||||||
--- a/cmd/arc_summary/arc_summary3
|
--- a/cmd/arc_summary
|
||||||
+++ b/cmd/arc_summary/arc_summary3
|
+++ b/cmd/arc_summary
|
||||||
@@ -609,13 +609,13 @@ def section_arc(kstats_dict):
|
@@ -655,13 +655,13 @@ def section_arc(kstats_dict):
|
||||||
prt_i1('L2 cached evictions:', f_bytes(arc_stats['evict_l2_cached']))
|
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_i1('L2 eligible evictions:', f_bytes(arc_stats['evict_l2_eligible']))
|
||||||
prt_i2('L2 eligible MFU evictions:',
|
prt_i2('L2 eligible MFU evictions:',
|
||||||
@@ -48,7 +48,7 @@ index 9d0c2d30d..fd2581ae2 100755
|
|||||||
prt_i1('L2 ineligible evictions:',
|
prt_i1('L2 ineligible evictions:',
|
||||||
f_bytes(arc_stats['evict_l2_ineligible']))
|
f_bytes(arc_stats['evict_l2_ineligible']))
|
||||||
print()
|
print()
|
||||||
@@ -757,20 +757,20 @@ def section_l2arc(kstats_dict):
|
@@ -851,20 +851,20 @@ def section_l2arc(kstats_dict):
|
||||||
f_perc(arc_stats['l2_hdr_size'], arc_stats['l2_size']),
|
f_perc(arc_stats['l2_hdr_size'], arc_stats['l2_size']),
|
||||||
f_bytes(arc_stats['l2_hdr_size']))
|
f_bytes(arc_stats['l2_hdr_size']))
|
||||||
prt_i2('MFU allocated size:',
|
prt_i2('MFU allocated size:',
|
||||||
@@ -79,11 +79,11 @@ index 9d0c2d30d..fd2581ae2 100755
|
|||||||
|
|
||||||
print()
|
print()
|
||||||
prt_1('L2ARC breakdown:', f_hits(l2_access_total))
|
prt_1('L2ARC breakdown:', f_hits(l2_access_total))
|
||||||
diff --git a/cmd/arcstat/arcstat.in b/cmd/arcstat/arcstat.in
|
diff --git a/cmd/arcstat.in b/cmd/arcstat.in
|
||||||
index d2b2e28d1..8004940b3 100755
|
index 8df1c62f7..833348d0e 100755
|
||||||
--- a/cmd/arcstat/arcstat.in
|
--- a/cmd/arcstat.in
|
||||||
+++ b/cmd/arcstat/arcstat.in
|
+++ b/cmd/arcstat.in
|
||||||
@@ -482,8 +482,8 @@ def calculate():
|
@@ -565,8 +565,8 @@ def calculate():
|
||||||
v["el2skip"] = d["evict_l2_skip"] // sint
|
v["el2skip"] = d["evict_l2_skip"] // sint
|
||||||
v["el2cach"] = d["evict_l2_cached"] // sint
|
v["el2cach"] = d["evict_l2_cached"] // sint
|
||||||
v["el2el"] = d["evict_l2_eligible"] // sint
|
v["el2el"] = d["evict_l2_eligible"] // sint
|
||||||
@@ -94,7 +94,7 @@ index d2b2e28d1..8004940b3 100755
|
|||||||
v["el2inel"] = d["evict_l2_ineligible"] // sint
|
v["el2inel"] = d["evict_l2_ineligible"] // sint
|
||||||
v["mtxmis"] = d["mutex_miss"] // sint
|
v["mtxmis"] = d["mutex_miss"] // sint
|
||||||
|
|
||||||
@@ -498,11 +498,11 @@ def calculate():
|
@@ -581,11 +581,11 @@ def calculate():
|
||||||
v["l2size"] = cur["l2_size"]
|
v["l2size"] = cur["l2_size"]
|
||||||
v["l2bytes"] = d["l2_read_bytes"] // sint
|
v["l2bytes"] = d["l2_read_bytes"] // sint
|
||||||
|
|
||||||
@@ -1,134 +0,0 @@
|
|||||||
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 0128fd817..d2b2e28d1 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"]
|
|
||||||
@@ -0,0 +1,99 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Date: Mon, 23 Oct 2023 14:45:06 -0700
|
||||||
|
Subject: [PATCH] zvol: Remove broken blk-mq optimization
|
||||||
|
|
||||||
|
This fix removes a dubious optimization in zfs_uiomove_bvec_rq()
|
||||||
|
that saved the iterator contents of a rq_for_each_segment(). This
|
||||||
|
optimization allowed restoring the "saved state" from a previous
|
||||||
|
rq_for_each_segment() call on the same uio so that you wouldn't
|
||||||
|
need to iterate though each bvec on every zfs_uiomove_bvec_rq() call.
|
||||||
|
However, if the kernel is manipulating the requests/bios/bvecs under
|
||||||
|
the covers between zfs_uiomove_bvec_rq() calls, then it could result
|
||||||
|
in corruption from using the "saved state". This optimization
|
||||||
|
results in an unbootable system after installing an OS on a zvol
|
||||||
|
with blk-mq enabled.
|
||||||
|
|
||||||
|
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
|
||||||
|
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Closes #15351
|
||||||
|
(cherry picked from commit 7c9b6fed16ed5034fd1cdfdaedfad93dc97b1557)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
---
|
||||||
|
include/os/linux/spl/sys/uio.h | 8 --------
|
||||||
|
module/os/linux/zfs/zfs_uio.c | 29 -----------------------------
|
||||||
|
2 files changed, 37 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/include/os/linux/spl/sys/uio.h b/include/os/linux/spl/sys/uio.h
|
||||||
|
index cce097e16..a4b600004 100644
|
||||||
|
--- a/include/os/linux/spl/sys/uio.h
|
||||||
|
+++ b/include/os/linux/spl/sys/uio.h
|
||||||
|
@@ -73,13 +73,6 @@ typedef struct zfs_uio {
|
||||||
|
size_t uio_skip;
|
||||||
|
|
||||||
|
struct request *rq;
|
||||||
|
-
|
||||||
|
- /*
|
||||||
|
- * Used for saving rq_for_each_segment() state between calls
|
||||||
|
- * to zfs_uiomove_bvec_rq().
|
||||||
|
- */
|
||||||
|
- struct req_iterator iter;
|
||||||
|
- struct bio_vec bv;
|
||||||
|
} zfs_uio_t;
|
||||||
|
|
||||||
|
|
||||||
|
@@ -138,7 +131,6 @@ zfs_uio_bvec_init(zfs_uio_t *uio, struct bio *bio, struct request *rq)
|
||||||
|
} else {
|
||||||
|
uio->uio_bvec = NULL;
|
||||||
|
uio->uio_iovcnt = 0;
|
||||||
|
- memset(&uio->iter, 0, sizeof (uio->iter));
|
||||||
|
}
|
||||||
|
|
||||||
|
uio->uio_loffset = io_offset(bio, rq);
|
||||||
|
diff --git a/module/os/linux/zfs/zfs_uio.c b/module/os/linux/zfs/zfs_uio.c
|
||||||
|
index 3efd4ab15..c2ed67c43 100644
|
||||||
|
--- a/module/os/linux/zfs/zfs_uio.c
|
||||||
|
+++ b/module/os/linux/zfs/zfs_uio.c
|
||||||
|
@@ -204,22 +204,6 @@ zfs_uiomove_bvec_rq(void *p, size_t n, zfs_uio_rw_t rw, zfs_uio_t *uio)
|
||||||
|
this_seg_start = orig_loffset;
|
||||||
|
|
||||||
|
rq_for_each_segment(bv, rq, iter) {
|
||||||
|
- if (uio->iter.bio) {
|
||||||
|
- /*
|
||||||
|
- * If uio->iter.bio is present, then we know we've saved
|
||||||
|
- * uio->iter from a previous call to this function, and
|
||||||
|
- * we can skip ahead in this rq_for_each_segment() loop
|
||||||
|
- * to where we last left off. That way, we don't need
|
||||||
|
- * to iterate over tons of segments we've already
|
||||||
|
- * processed - we can just restore the "saved state".
|
||||||
|
- */
|
||||||
|
- iter = uio->iter;
|
||||||
|
- bv = uio->bv;
|
||||||
|
- this_seg_start = uio->uio_loffset;
|
||||||
|
- memset(&uio->iter, 0, sizeof (uio->iter));
|
||||||
|
- continue;
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
/*
|
||||||
|
* Lookup what the logical offset of the last byte of this
|
||||||
|
* segment is.
|
||||||
|
@@ -260,19 +244,6 @@ zfs_uiomove_bvec_rq(void *p, size_t n, zfs_uio_rw_t rw, zfs_uio_t *uio)
|
||||||
|
copied = 1; /* We copied some data */
|
||||||
|
}
|
||||||
|
|
||||||
|
- if (n == 0) {
|
||||||
|
- /*
|
||||||
|
- * All done copying. Save our 'iter' value to the uio.
|
||||||
|
- * This allows us to "save our state" and skip ahead in
|
||||||
|
- * the rq_for_each_segment() loop the next time we call
|
||||||
|
- * call zfs_uiomove_bvec_rq() on this uio (which we
|
||||||
|
- * will be doing for any remaining data in the uio).
|
||||||
|
- */
|
||||||
|
- uio->iter = iter; /* make a copy of the struct data */
|
||||||
|
- uio->bv = bv;
|
||||||
|
- return (0);
|
||||||
|
- }
|
||||||
|
-
|
||||||
|
this_seg_start = this_seg_end + 1;
|
||||||
|
}
|
||||||
|
|
||||||
-87
@@ -1,87 +0,0 @@
|
|||||||
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Rich Ercolani <214141+rincebrain@users.noreply.github.com>
|
|
||||||
Date: Sat, 26 Aug 2023 14:25:46 -0400
|
|
||||||
Subject: [PATCH] Avoid save/restoring AMX registers to avoid a SPR erratum
|
|
||||||
|
|
||||||
Intel SPR erratum SPR4 says that if you trip into a vmexit while
|
|
||||||
doing FPU save/restore, your AMX register state might misbehave...
|
|
||||||
and by misbehave, I mean save all zeroes incorrectly, leading to
|
|
||||||
explosions if you restore it.
|
|
||||||
|
|
||||||
Since we're not using AMX for anything, the simple way to avoid
|
|
||||||
this is to just not save/restore those when we do anything, since
|
|
||||||
we're killing preemption of any sort across our save/restores.
|
|
||||||
|
|
||||||
If we ever decide to use AMX, it's not clear that we have any
|
|
||||||
way to mitigate this, on Linux...but I am not an expert.
|
|
||||||
|
|
||||||
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
|
|
||||||
Signed-off-by: Rich Ercolani <rincebrain@gmail.com>
|
|
||||||
Closes #14989
|
|
||||||
Closes #15168
|
|
||||||
(cherry picked from commit 277f2e587b085d1eb8aa48b4ac0768a9ef5745ab)
|
|
||||||
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
|
||||||
---
|
|
||||||
include/os/linux/kernel/linux/simd_x86.h | 19 ++++++++++++++-----
|
|
||||||
1 file changed, 14 insertions(+), 5 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/include/os/linux/kernel/linux/simd_x86.h b/include/os/linux/kernel/linux/simd_x86.h
|
|
||||||
index 660f0d42d..455167ac8 100644
|
|
||||||
--- a/include/os/linux/kernel/linux/simd_x86.h
|
|
||||||
+++ b/include/os/linux/kernel/linux/simd_x86.h
|
|
||||||
@@ -157,6 +157,15 @@
|
|
||||||
#endif
|
|
||||||
#endif
|
|
||||||
|
|
||||||
+#ifndef XFEATURE_MASK_XTILE
|
|
||||||
+/*
|
|
||||||
+ * For kernels where this doesn't exist yet, we still don't want to break
|
|
||||||
+ * by save/restoring this broken nonsense.
|
|
||||||
+ * See issue #14989 or Intel errata SPR4 for why
|
|
||||||
+ */
|
|
||||||
+#define XFEATURE_MASK_XTILE 0x60000
|
|
||||||
+#endif
|
|
||||||
+
|
|
||||||
#include <linux/mm.h>
|
|
||||||
#include <linux/slab.h>
|
|
||||||
|
|
||||||
@@ -319,18 +328,18 @@ kfpu_begin(void)
|
|
||||||
union fpregs_state *state = zfs_kfpu_fpregs[smp_processor_id()];
|
|
||||||
#if defined(HAVE_XSAVES)
|
|
||||||
if (static_cpu_has(X86_FEATURE_XSAVES)) {
|
|
||||||
- kfpu_do_xsave("xsaves", &state->xsave, ~0);
|
|
||||||
+ kfpu_do_xsave("xsaves", &state->xsave, ~XFEATURE_MASK_XTILE);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
#if defined(HAVE_XSAVEOPT)
|
|
||||||
if (static_cpu_has(X86_FEATURE_XSAVEOPT)) {
|
|
||||||
- kfpu_do_xsave("xsaveopt", &state->xsave, ~0);
|
|
||||||
+ kfpu_do_xsave("xsaveopt", &state->xsave, ~XFEATURE_MASK_XTILE);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
if (static_cpu_has(X86_FEATURE_XSAVE)) {
|
|
||||||
- kfpu_do_xsave("xsave", &state->xsave, ~0);
|
|
||||||
+ kfpu_do_xsave("xsave", &state->xsave, ~XFEATURE_MASK_XTILE);
|
|
||||||
} else if (static_cpu_has(X86_FEATURE_FXSR)) {
|
|
||||||
kfpu_save_fxsr(&state->fxsave);
|
|
||||||
} else {
|
|
||||||
@@ -415,12 +424,12 @@ kfpu_end(void)
|
|
||||||
union fpregs_state *state = zfs_kfpu_fpregs[smp_processor_id()];
|
|
||||||
#if defined(HAVE_XSAVES)
|
|
||||||
if (static_cpu_has(X86_FEATURE_XSAVES)) {
|
|
||||||
- kfpu_do_xrstor("xrstors", &state->xsave, ~0);
|
|
||||||
+ kfpu_do_xrstor("xrstors", &state->xsave, ~XFEATURE_MASK_XTILE);
|
|
||||||
goto out;
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
if (static_cpu_has(X86_FEATURE_XSAVE)) {
|
|
||||||
- kfpu_do_xrstor("xrstor", &state->xsave, ~0);
|
|
||||||
+ kfpu_do_xrstor("xrstor", &state->xsave, ~XFEATURE_MASK_XTILE);
|
|
||||||
} else if (static_cpu_has(X86_FEATURE_FXSR)) {
|
|
||||||
kfpu_restore_fxsr(&state->fxsave);
|
|
||||||
} else {
|
|
||||||
--
|
|
||||||
2.39.2
|
|
||||||
|
|
||||||
@@ -0,0 +1,123 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Date: Mon, 23 Oct 2023 14:39:59 -0700
|
||||||
|
Subject: [PATCH] Revert "zvol: Temporally disable blk-mq"
|
||||||
|
|
||||||
|
This reverts commit aefb6a2bd6c24597cde655e9ce69edd0a4c34357.
|
||||||
|
|
||||||
|
aefb6a2bd temporally disabled blk-mq until we could fix a fix for
|
||||||
|
|
||||||
|
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Closes #15439
|
||||||
|
(cherry picked from commit 05c4710e8958832afc2868102c9535a4f18115be)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
---
|
||||||
|
man/man4/zfs.4 | 57 ++++++++++++++++++++++++++++
|
||||||
|
module/os/linux/zfs/zvol_os.c | 12 ++++++
|
||||||
|
tests/zfs-tests/include/tunables.cfg | 2 +-
|
||||||
|
3 files changed, 70 insertions(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/man/man4/zfs.4 b/man/man4/zfs.4
|
||||||
|
index 71a3e67ee..cfadd79d8 100644
|
||||||
|
--- a/man/man4/zfs.4
|
||||||
|
+++ b/man/man4/zfs.4
|
||||||
|
@@ -2317,6 +2317,63 @@ If
|
||||||
|
.Sy zvol_threads
|
||||||
|
to the number of CPUs present or 32 (whichever is greater).
|
||||||
|
.
|
||||||
|
+.It Sy zvol_blk_mq_threads Ns = Ns Sy 0 Pq uint
|
||||||
|
+The number of threads per zvol to use for queuing IO requests.
|
||||||
|
+This parameter will only appear if your kernel supports
|
||||||
|
+.Li blk-mq
|
||||||
|
+and is only read and assigned to a zvol at zvol load time.
|
||||||
|
+If
|
||||||
|
+.Sy 0
|
||||||
|
+(the default) then internally set
|
||||||
|
+.Sy zvol_blk_mq_threads
|
||||||
|
+to the number of CPUs present.
|
||||||
|
+.
|
||||||
|
+.It Sy zvol_use_blk_mq Ns = Ns Sy 0 Ns | Ns 1 Pq uint
|
||||||
|
+Set to
|
||||||
|
+.Sy 1
|
||||||
|
+to use the
|
||||||
|
+.Li blk-mq
|
||||||
|
+API for zvols.
|
||||||
|
+Set to
|
||||||
|
+.Sy 0
|
||||||
|
+(the default) to use the legacy zvol APIs.
|
||||||
|
+This setting can give better or worse zvol performance depending on
|
||||||
|
+the workload.
|
||||||
|
+This parameter will only appear if your kernel supports
|
||||||
|
+.Li blk-mq
|
||||||
|
+and is only read and assigned to a zvol at zvol load time.
|
||||||
|
+.
|
||||||
|
+.It Sy zvol_blk_mq_blocks_per_thread Ns = Ns Sy 8 Pq uint
|
||||||
|
+If
|
||||||
|
+.Sy zvol_use_blk_mq
|
||||||
|
+is enabled, then process this number of
|
||||||
|
+.Sy volblocksize Ns -sized blocks per zvol thread.
|
||||||
|
+This tunable can be use to favor better performance for zvol reads (lower
|
||||||
|
+values) or writes (higher values).
|
||||||
|
+If set to
|
||||||
|
+.Sy 0 ,
|
||||||
|
+then the zvol layer will process the maximum number of blocks
|
||||||
|
+per thread that it can.
|
||||||
|
+This parameter will only appear if your kernel supports
|
||||||
|
+.Li blk-mq
|
||||||
|
+and is only applied at each zvol's load time.
|
||||||
|
+.
|
||||||
|
+.It Sy zvol_blk_mq_queue_depth Ns = Ns Sy 0 Pq uint
|
||||||
|
+The queue_depth value for the zvol
|
||||||
|
+.Li blk-mq
|
||||||
|
+interface.
|
||||||
|
+This parameter will only appear if your kernel supports
|
||||||
|
+.Li blk-mq
|
||||||
|
+and is only applied at each zvol's load time.
|
||||||
|
+If
|
||||||
|
+.Sy 0
|
||||||
|
+(the default) then use the kernel's default queue depth.
|
||||||
|
+Values are clamped to the kernel's
|
||||||
|
+.Dv BLKDEV_MIN_RQ
|
||||||
|
+and
|
||||||
|
+.Dv BLKDEV_MAX_RQ Ns / Ns Dv BLKDEV_DEFAULT_RQ
|
||||||
|
+limits.
|
||||||
|
+.
|
||||||
|
.It Sy zvol_volmode Ns = Ns Sy 1 Pq uint
|
||||||
|
Defines zvol block devices behaviour when
|
||||||
|
.Sy volmode Ns = Ns Sy default :
|
||||||
|
diff --git a/module/os/linux/zfs/zvol_os.c b/module/os/linux/zfs/zvol_os.c
|
||||||
|
index 76521c959..7a95b54bd 100644
|
||||||
|
--- a/module/os/linux/zfs/zvol_os.c
|
||||||
|
+++ b/module/os/linux/zfs/zvol_os.c
|
||||||
|
@@ -1620,6 +1620,18 @@ MODULE_PARM_DESC(zvol_prefetch_bytes, "Prefetch N bytes at zvol start+end");
|
||||||
|
module_param(zvol_volmode, uint, 0644);
|
||||||
|
MODULE_PARM_DESC(zvol_volmode, "Default volmode property value");
|
||||||
|
|
||||||
|
+#ifdef HAVE_BLK_MQ
|
||||||
|
+module_param(zvol_blk_mq_queue_depth, uint, 0644);
|
||||||
|
+MODULE_PARM_DESC(zvol_blk_mq_queue_depth, "Default blk-mq queue depth");
|
||||||
|
+
|
||||||
|
+module_param(zvol_use_blk_mq, uint, 0644);
|
||||||
|
+MODULE_PARM_DESC(zvol_use_blk_mq, "Use the blk-mq API for zvols");
|
||||||
|
+
|
||||||
|
+module_param(zvol_blk_mq_blocks_per_thread, uint, 0644);
|
||||||
|
+MODULE_PARM_DESC(zvol_blk_mq_blocks_per_thread,
|
||||||
|
+ "Process volblocksize blocks per thread");
|
||||||
|
+#endif
|
||||||
|
+
|
||||||
|
#ifndef HAVE_BLKDEV_GET_ERESTARTSYS
|
||||||
|
module_param(zvol_open_timeout_ms, uint, 0644);
|
||||||
|
MODULE_PARM_DESC(zvol_open_timeout_ms, "Timeout for ZVOL open retries");
|
||||||
|
diff --git a/tests/zfs-tests/include/tunables.cfg b/tests/zfs-tests/include/tunables.cfg
|
||||||
|
index 8010a9451..80e7bcb3b 100644
|
||||||
|
--- a/tests/zfs-tests/include/tunables.cfg
|
||||||
|
+++ b/tests/zfs-tests/include/tunables.cfg
|
||||||
|
@@ -89,7 +89,7 @@ VDEV_VALIDATE_SKIP vdev.validate_skip vdev_validate_skip
|
||||||
|
VOL_INHIBIT_DEV UNSUPPORTED zvol_inhibit_dev
|
||||||
|
VOL_MODE vol.mode zvol_volmode
|
||||||
|
VOL_RECURSIVE vol.recursive UNSUPPORTED
|
||||||
|
-VOL_USE_BLK_MQ UNSUPPORTED UNSUPPORTED
|
||||||
|
+VOL_USE_BLK_MQ UNSUPPORTED zvol_use_blk_mq
|
||||||
|
XATTR_COMPAT xattr_compat zfs_xattr_compat
|
||||||
|
ZEVENT_LEN_MAX zevent.len_max zfs_zevent_len_max
|
||||||
|
ZEVENT_RETAIN_MAX zevent.retain_max zfs_zevent_retain_max
|
||||||
@@ -0,0 +1,76 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: siv0 <github@nomore.at>
|
||||||
|
Date: Tue, 31 Oct 2023 21:57:54 +0100
|
||||||
|
Subject: [PATCH] Fix nfs_truncate_shares without /etc/exports.d
|
||||||
|
|
||||||
|
Calling nfs_reset_shares on Linux prints a warning:
|
||||||
|
`failed to lock /etc/exports.d/zfs.exports.lock: No such file or
|
||||||
|
directory`
|
||||||
|
when /etc/exports.d does not exist. The directory gets created, when a
|
||||||
|
filesystem is actually exported through nfs_toggle_share and
|
||||||
|
nfs_init_share. The truncation of /etc/exports.d/zfs.exports happens
|
||||||
|
unconditionally when calling `zfs mount -a` (via zfs_do_mount and
|
||||||
|
share_mount in `cmd/zfs/zfs_main.c`).
|
||||||
|
|
||||||
|
Fixing the issue only in the Linux part, since the exports file on
|
||||||
|
freebsd is in `/etc/zfs/`, which seems present on 2 FreeBSD systems I
|
||||||
|
have access to (through `/etc/zfs/compatibility.d/`), while a Debian
|
||||||
|
box does not have the directory even if `/usr/sbin/exportfs` is
|
||||||
|
present through the `nfs-kernel-server` package.
|
||||||
|
|
||||||
|
The code for exports_available is copied from nfs_available above.
|
||||||
|
|
||||||
|
Fixes: ede037cda73675f42b1452187e8dd3438fafc220
|
||||||
|
("Make zfs-share service resilient to stale exports")
|
||||||
|
|
||||||
|
Reviewed-by: Brian Atkinson <batkinson@lanl.gov>
|
||||||
|
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
Closes #15369
|
||||||
|
Closes #15468
|
||||||
|
(cherry picked from commit 41e55b476bcfc90f1ad81c02c5375367fdace9e9)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
---
|
||||||
|
lib/libshare/os/linux/nfs.c | 18 ++++++++++++++++++
|
||||||
|
1 file changed, 18 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/lib/libshare/os/linux/nfs.c b/lib/libshare/os/linux/nfs.c
|
||||||
|
index 004946b0c..3dce81840 100644
|
||||||
|
--- a/lib/libshare/os/linux/nfs.c
|
||||||
|
+++ b/lib/libshare/os/linux/nfs.c
|
||||||
|
@@ -47,6 +47,7 @@
|
||||||
|
|
||||||
|
|
||||||
|
static boolean_t nfs_available(void);
|
||||||
|
+static boolean_t exports_available(void);
|
||||||
|
|
||||||
|
typedef int (*nfs_shareopt_callback_t)(const char *opt, const char *value,
|
||||||
|
void *cookie);
|
||||||
|
@@ -539,6 +540,8 @@ nfs_commit_shares(void)
|
||||||
|
static void
|
||||||
|
nfs_truncate_shares(void)
|
||||||
|
{
|
||||||
|
+ if (!exports_available())
|
||||||
|
+ return;
|
||||||
|
nfs_reset_shares(ZFS_EXPORTS_LOCK, ZFS_EXPORTS_FILE);
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -566,3 +569,18 @@ nfs_available(void)
|
||||||
|
|
||||||
|
return (avail == 1);
|
||||||
|
}
|
||||||
|
+
|
||||||
|
+static boolean_t
|
||||||
|
+exports_available(void)
|
||||||
|
+{
|
||||||
|
+ static int avail;
|
||||||
|
+
|
||||||
|
+ if (!avail) {
|
||||||
|
+ if (access(ZFS_EXPORTS_DIR, F_OK) != 0)
|
||||||
|
+ avail = -1;
|
||||||
|
+ else
|
||||||
|
+ avail = 1;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ return (avail == 1);
|
||||||
|
+}
|
||||||
@@ -0,0 +1,72 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Date: Thu, 9 Nov 2023 16:43:35 -0800
|
||||||
|
Subject: [PATCH] Workaround UBSAN errors for variable arrays
|
||||||
|
|
||||||
|
This gets around UBSAN errors when using arrays at the end of
|
||||||
|
structs. It converts some zero-length arrays to variable length
|
||||||
|
arrays and disables UBSAN checking on certain modules.
|
||||||
|
|
||||||
|
It is based off of the patch from #15460.
|
||||||
|
|
||||||
|
Addresses: #15145
|
||||||
|
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Co-authored-by: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Co-authored-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
|
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
|
---
|
||||||
|
include/os/linux/spl/sys/kmem_cache.h | 2 +-
|
||||||
|
include/sys/vdev_raidz_impl.h | 4 ++--
|
||||||
|
module/Kbuild.in | 4 ++++
|
||||||
|
3 files changed, 7 insertions(+), 3 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/include/os/linux/spl/sys/kmem_cache.h b/include/os/linux/spl/sys/kmem_cache.h
|
||||||
|
index 20eeadc46..82d50b603 100644
|
||||||
|
--- a/include/os/linux/spl/sys/kmem_cache.h
|
||||||
|
+++ b/include/os/linux/spl/sys/kmem_cache.h
|
||||||
|
@@ -108,7 +108,7 @@ typedef struct spl_kmem_magazine {
|
||||||
|
uint32_t skm_refill; /* Batch refill size */
|
||||||
|
struct spl_kmem_cache *skm_cache; /* Owned by cache */
|
||||||
|
unsigned int skm_cpu; /* Owned by cpu */
|
||||||
|
- void *skm_objs[0]; /* Object pointers */
|
||||||
|
+ void *skm_objs[]; /* Object pointers */
|
||||||
|
} spl_kmem_magazine_t;
|
||||||
|
|
||||||
|
typedef struct spl_kmem_obj {
|
||||||
|
diff --git a/include/sys/vdev_raidz_impl.h b/include/sys/vdev_raidz_impl.h
|
||||||
|
index c1037fa12..73c26dff1 100644
|
||||||
|
--- a/include/sys/vdev_raidz_impl.h
|
||||||
|
+++ b/include/sys/vdev_raidz_impl.h
|
||||||
|
@@ -130,7 +130,7 @@ typedef struct raidz_row {
|
||||||
|
uint64_t rr_offset; /* Logical offset for *_io_verify() */
|
||||||
|
uint64_t rr_size; /* Physical size for *_io_verify() */
|
||||||
|
#endif
|
||||||
|
- raidz_col_t rr_col[0]; /* Flexible array of I/O columns */
|
||||||
|
+ raidz_col_t rr_col[]; /* Flexible array of I/O columns */
|
||||||
|
} raidz_row_t;
|
||||||
|
|
||||||
|
typedef struct raidz_map {
|
||||||
|
@@ -139,7 +139,7 @@ typedef struct raidz_map {
|
||||||
|
int rm_nskip; /* RAIDZ sectors skipped for padding */
|
||||||
|
int rm_skipstart; /* Column index of padding start */
|
||||||
|
const raidz_impl_ops_t *rm_ops; /* RAIDZ math operations */
|
||||||
|
- raidz_row_t *rm_row[0]; /* flexible array of rows */
|
||||||
|
+ raidz_row_t *rm_row[]; /* flexible array of rows */
|
||||||
|
} raidz_map_t;
|
||||||
|
|
||||||
|
|
||||||
|
diff --git a/module/Kbuild.in b/module/Kbuild.in
|
||||||
|
index c13217159..b9c284a24 100644
|
||||||
|
--- a/module/Kbuild.in
|
||||||
|
+++ b/module/Kbuild.in
|
||||||
|
@@ -488,6 +488,10 @@ zfs-$(CONFIG_ARM64) += $(addprefix zfs/,$(ZFS_OBJS_ARM64))
|
||||||
|
zfs-$(CONFIG_PPC) += $(addprefix zfs/,$(ZFS_OBJS_PPC_PPC64))
|
||||||
|
zfs-$(CONFIG_PPC64) += $(addprefix zfs/,$(ZFS_OBJS_PPC_PPC64))
|
||||||
|
|
||||||
|
+UBSAN_SANITIZE_zap_leaf.o := n
|
||||||
|
+UBSAN_SANITIZE_zap_micro.o := n
|
||||||
|
+UBSAN_SANITIZE_sa.o := n
|
||||||
|
+
|
||||||
|
# Suppress incorrect warnings from versions of objtool which are not
|
||||||
|
# aware of x86 EVEX prefix instructions used for AVX512.
|
||||||
|
OBJECT_FILES_NON_STANDARD_vdev_raidz_math_avx512bw.o := y
|
||||||
+66
@@ -0,0 +1,66 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
|
Date: Sun, 12 Nov 2023 15:52:25 +0100
|
||||||
|
Subject: [PATCH] zpool status: tighten bounds for noalloc stat availabillity
|
||||||
|
|
||||||
|
When running zfs 2.2.0 userspace utilities with a kernel that still
|
||||||
|
has 2.1.13 modules zpool status adds `(non-allocating)` next to the
|
||||||
|
disk name of a single-disk pool.
|
||||||
|
|
||||||
|
The reason for this seems to be that the patch adding the `vs_pspace` field was
|
||||||
|
backported, but the one adding `vs_noalloc` was not.
|
||||||
|
|
||||||
|
Itself that is not a problem, but in 2.2 `noalloc` was added before `psspace`,
|
||||||
|
so the struct layout between 2.1.13 and 2.2.0 do NOT match anymore...
|
||||||
|
|
||||||
|
I.e., the struct looks like the following at the end for ZFS 2.1.x:
|
||||||
|
|
||||||
|
```
|
||||||
|
typedef struct vdev_stat {
|
||||||
|
hrtime_t vs_timestamp; /* time since vdev load */
|
||||||
|
// snip
|
||||||
|
uint64_t vs_logical_ashift; /* vdev_logical_ashift */
|
||||||
|
uint64_t vs_physical_ashift; /* vdev_physical_ashift */
|
||||||
|
uint64_t vs_pspace; /* physical capacity */
|
||||||
|
} vdev_stat_t;
|
||||||
|
```
|
||||||
|
|
||||||
|
And like the following on ZFS 2.2.x:
|
||||||
|
```
|
||||||
|
typedef struct vdev_stat {
|
||||||
|
hrtime_t vs_timestamp; /* time since vdev load */
|
||||||
|
// snip
|
||||||
|
uint64_t vs_logical_ashift; /* vdev_logical_ashift */
|
||||||
|
uint64_t vs_physical_ashift; /* vdev_physical_ashift */
|
||||||
|
uint64_t vs_noalloc; /* allocations halted? */
|
||||||
|
uint64_t vs_pspace; /* physical capacity */
|
||||||
|
} vdev_stat_t;
|
||||||
|
```
|
||||||
|
|
||||||
|
Resulting in 2.2.x user-space tooling interpreting the `vs_pspace` field from
|
||||||
|
the 2.1.x kernel module as `vs_noalloc` field.
|
||||||
|
|
||||||
|
For now, work-around that discrepancy by coupling the availability of
|
||||||
|
the vs_noalloc field with the one of the vs_pspace one, as when both
|
||||||
|
are returned from the module we can be sure that our struct layout
|
||||||
|
matches again.
|
||||||
|
|
||||||
|
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
|
||||||
|
---
|
||||||
|
cmd/zpool/zpool_main.c | 3 ++-
|
||||||
|
1 file changed, 2 insertions(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/cmd/zpool/zpool_main.c b/cmd/zpool/zpool_main.c
|
||||||
|
index 5507f9d3f..98970abfe 100644
|
||||||
|
--- a/cmd/zpool/zpool_main.c
|
||||||
|
+++ b/cmd/zpool/zpool_main.c
|
||||||
|
@@ -2478,7 +2478,8 @@ print_status_config(zpool_handle_t *zhp, status_cbdata_t *cb, const char *name,
|
||||||
|
|
||||||
|
if (vs->vs_scan_removing != 0) {
|
||||||
|
(void) printf(gettext(" (removing)"));
|
||||||
|
- } else if (VDEV_STAT_VALID(vs_noalloc, vsc) && vs->vs_noalloc != 0) {
|
||||||
|
+ } else if (VDEV_STAT_VALID(vs_pspace, vsc)
|
||||||
|
+ && VDEV_STAT_VALID(vs_noalloc, vsc) && vs->vs_noalloc != 0) {
|
||||||
|
(void) printf(gettext(" (non-allocating)"));
|
||||||
|
}
|
||||||
|
|
||||||
+44
@@ -0,0 +1,44 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: =?UTF-8?q?Martin=20Matu=C5=A1ka?= <mm@FreeBSD.org>
|
||||||
|
Date: Tue, 31 Oct 2023 21:49:41 +0100
|
||||||
|
Subject: [PATCH] Fix block cloning between unencrypted and encrypted datasets
|
||||||
|
|
||||||
|
Block cloning from an encrypted dataset into an unencrypted dataset
|
||||||
|
and vice versa is not possible. The current code did allow cloning
|
||||||
|
unencrypted files into an encrypted dataset causing a panic when
|
||||||
|
these were accessed. Block cloning between encrypted and encrypted
|
||||||
|
is currently supported on the same filesystem only.
|
||||||
|
|
||||||
|
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
|
||||||
|
Reviewed-by: Kay Pedersen <mail@mkwg.de>
|
||||||
|
Reviewed-by: Rob N <robn@despairlabs.com>
|
||||||
|
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
|
||||||
|
Signed-off-by: Martin Matuska <mm@FreeBSD.org>
|
||||||
|
Closes #15464
|
||||||
|
Closes #15465
|
||||||
|
(cherry picked from commit 459c99ff2339a4a514abcf2255f9b3e5324ef09e)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
---
|
||||||
|
module/zfs/zfs_vnops.c | 9 +++++++++
|
||||||
|
1 file changed, 9 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/module/zfs/zfs_vnops.c b/module/zfs/zfs_vnops.c
|
||||||
|
index 40d6c87a7..84e6b10ef 100644
|
||||||
|
--- a/module/zfs/zfs_vnops.c
|
||||||
|
+++ b/module/zfs/zfs_vnops.c
|
||||||
|
@@ -1094,6 +1094,15 @@ zfs_clone_range(znode_t *inzp, uint64_t *inoffp, znode_t *outzp,
|
||||||
|
|
||||||
|
ASSERT(!outzfsvfs->z_replay);
|
||||||
|
|
||||||
|
+ /*
|
||||||
|
+ * Block cloning from an unencrypted dataset into an encrypted
|
||||||
|
+ * dataset and vice versa is not supported.
|
||||||
|
+ */
|
||||||
|
+ if (inos->os_encrypted != outos->os_encrypted) {
|
||||||
|
+ zfs_exit_two(inzfsvfs, outzfsvfs, FTAG);
|
||||||
|
+ return (SET_ERROR(EXDEV));
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
error = zfs_verify_zp(inzp);
|
||||||
|
if (error == 0)
|
||||||
|
error = zfs_verify_zp(outzp);
|
||||||
@@ -0,0 +1,201 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Rich Ercolani <214141+rincebrain@users.noreply.github.com>
|
||||||
|
Date: Thu, 16 Nov 2023 14:35:22 -0500
|
||||||
|
Subject: [PATCH] Add a tunable to disable BRT support.
|
||||||
|
|
||||||
|
Copy the disable parameter that FreeBSD implemented, and extend it to
|
||||||
|
work on Linux as well, until we're sure this is stable.
|
||||||
|
|
||||||
|
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
|
||||||
|
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
|
||||||
|
Signed-off-by: Rich Ercolani <rincebrain@gmail.com>
|
||||||
|
Closes #15529
|
||||||
|
(cherry picked from commit 87e9e828655c250ce064874ff5df16f870c0a52e)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
---
|
||||||
|
include/os/freebsd/zfs/sys/zfs_vfsops_os.h | 1 +
|
||||||
|
include/os/linux/zfs/sys/zfs_vfsops_os.h | 2 ++
|
||||||
|
man/man4/zfs.4 | 5 +++++
|
||||||
|
module/os/freebsd/zfs/zfs_vfsops.c | 4 ++++
|
||||||
|
module/os/freebsd/zfs/zfs_vnops_os.c | 5 +++++
|
||||||
|
module/os/linux/zfs/zfs_vnops_os.c | 4 ++++
|
||||||
|
module/os/linux/zfs/zpl_file_range.c | 5 +++++
|
||||||
|
tests/zfs-tests/include/libtest.shlib | 15 +++++++++++++++
|
||||||
|
tests/zfs-tests/include/tunables.cfg | 1 +
|
||||||
|
.../tests/functional/block_cloning/cleanup.ksh | 4 ++++
|
||||||
|
.../tests/functional/block_cloning/setup.ksh | 5 +++++
|
||||||
|
11 files changed, 51 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/include/os/freebsd/zfs/sys/zfs_vfsops_os.h b/include/os/freebsd/zfs/sys/zfs_vfsops_os.h
|
||||||
|
index 24bb03575..56a0ac96a 100644
|
||||||
|
--- a/include/os/freebsd/zfs/sys/zfs_vfsops_os.h
|
||||||
|
+++ b/include/os/freebsd/zfs/sys/zfs_vfsops_os.h
|
||||||
|
@@ -286,6 +286,7 @@ typedef struct zfid_long {
|
||||||
|
|
||||||
|
extern uint_t zfs_fsyncer_key;
|
||||||
|
extern int zfs_super_owner;
|
||||||
|
+extern int zfs_bclone_enabled;
|
||||||
|
|
||||||
|
extern void zfs_init(void);
|
||||||
|
extern void zfs_fini(void);
|
||||||
|
diff --git a/include/os/linux/zfs/sys/zfs_vfsops_os.h b/include/os/linux/zfs/sys/zfs_vfsops_os.h
|
||||||
|
index b4d5db21f..220466550 100644
|
||||||
|
--- a/include/os/linux/zfs/sys/zfs_vfsops_os.h
|
||||||
|
+++ b/include/os/linux/zfs/sys/zfs_vfsops_os.h
|
||||||
|
@@ -45,6 +45,8 @@ extern "C" {
|
||||||
|
typedef struct zfsvfs zfsvfs_t;
|
||||||
|
struct znode;
|
||||||
|
|
||||||
|
+extern int zfs_bclone_enabled;
|
||||||
|
+
|
||||||
|
/*
|
||||||
|
* This structure emulates the vfs_t from other platforms. It's purpose
|
||||||
|
* is to facilitate the handling of mount options and minimize structural
|
||||||
|
diff --git a/man/man4/zfs.4 b/man/man4/zfs.4
|
||||||
|
index cfadd79d8..32f1765a5 100644
|
||||||
|
--- a/man/man4/zfs.4
|
||||||
|
+++ b/man/man4/zfs.4
|
||||||
|
@@ -1137,6 +1137,11 @@ Selecting any option other than
|
||||||
|
results in vector instructions
|
||||||
|
from the respective CPU instruction set being used.
|
||||||
|
.
|
||||||
|
+.It Sy zfs_bclone_enabled Ns = Ns Sy 1 Ns | Ns 0 Pq int
|
||||||
|
+Enable the experimental block cloning feature.
|
||||||
|
+If this setting is 0, then even if feature@block_cloning is enabled,
|
||||||
|
+attempts to clone blocks will act as though the feature is disabled.
|
||||||
|
+.
|
||||||
|
.It Sy zfs_blake3_impl Ns = Ns Sy fastest Pq string
|
||||||
|
Select a BLAKE3 implementation.
|
||||||
|
.Pp
|
||||||
|
diff --git a/module/os/freebsd/zfs/zfs_vfsops.c b/module/os/freebsd/zfs/zfs_vfsops.c
|
||||||
|
index e8b9ada13..09e18de81 100644
|
||||||
|
--- a/module/os/freebsd/zfs/zfs_vfsops.c
|
||||||
|
+++ b/module/os/freebsd/zfs/zfs_vfsops.c
|
||||||
|
@@ -89,6 +89,10 @@ int zfs_debug_level;
|
||||||
|
SYSCTL_INT(_vfs_zfs, OID_AUTO, debug, CTLFLAG_RWTUN, &zfs_debug_level, 0,
|
||||||
|
"Debug level");
|
||||||
|
|
||||||
|
+int zfs_bclone_enabled = 1;
|
||||||
|
+SYSCTL_INT(_vfs_zfs, OID_AUTO, bclone_enabled, CTLFLAG_RWTUN,
|
||||||
|
+ &zfs_bclone_enabled, 0, "Enable block cloning");
|
||||||
|
+
|
||||||
|
struct zfs_jailparam {
|
||||||
|
int mount_snapshot;
|
||||||
|
};
|
||||||
|
diff --git a/module/os/freebsd/zfs/zfs_vnops_os.c b/module/os/freebsd/zfs/zfs_vnops_os.c
|
||||||
|
index c498a1328..f672deed3 100644
|
||||||
|
--- a/module/os/freebsd/zfs/zfs_vnops_os.c
|
||||||
|
+++ b/module/os/freebsd/zfs/zfs_vnops_os.c
|
||||||
|
@@ -6243,6 +6243,11 @@ zfs_freebsd_copy_file_range(struct vop_copy_file_range_args *ap)
|
||||||
|
int error;
|
||||||
|
uint64_t len = *ap->a_lenp;
|
||||||
|
|
||||||
|
+ if (!zfs_bclone_enabled) {
|
||||||
|
+ mp = NULL;
|
||||||
|
+ goto bad_write_fallback;
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
/*
|
||||||
|
* TODO: If offset/length is not aligned to recordsize, use
|
||||||
|
* vn_generic_copy_file_range() on this fragment.
|
||||||
|
diff --git a/module/os/linux/zfs/zfs_vnops_os.c b/module/os/linux/zfs/zfs_vnops_os.c
|
||||||
|
index 33baac9db..76fac3a02 100644
|
||||||
|
--- a/module/os/linux/zfs/zfs_vnops_os.c
|
||||||
|
+++ b/module/os/linux/zfs/zfs_vnops_os.c
|
||||||
|
@@ -4229,4 +4229,8 @@ EXPORT_SYMBOL(zfs_map);
|
||||||
|
module_param(zfs_delete_blocks, ulong, 0644);
|
||||||
|
MODULE_PARM_DESC(zfs_delete_blocks, "Delete files larger than N blocks async");
|
||||||
|
|
||||||
|
+/* CSTYLED */
|
||||||
|
+module_param(zfs_bclone_enabled, uint, 0644);
|
||||||
|
+MODULE_PARM_DESC(zfs_bclone_enabled, "Enable block cloning");
|
||||||
|
+
|
||||||
|
#endif
|
||||||
|
diff --git a/module/os/linux/zfs/zpl_file_range.c b/module/os/linux/zfs/zpl_file_range.c
|
||||||
|
index c47fe99da..73476ff40 100644
|
||||||
|
--- a/module/os/linux/zfs/zpl_file_range.c
|
||||||
|
+++ b/module/os/linux/zfs/zpl_file_range.c
|
||||||
|
@@ -31,6 +31,8 @@
|
||||||
|
#include <sys/zfs_vnops.h>
|
||||||
|
#include <sys/zfeature.h>
|
||||||
|
|
||||||
|
+int zfs_bclone_enabled = 1;
|
||||||
|
+
|
||||||
|
/*
|
||||||
|
* Clone part of a file via block cloning.
|
||||||
|
*
|
||||||
|
@@ -50,6 +52,9 @@ __zpl_clone_file_range(struct file *src_file, loff_t src_off,
|
||||||
|
fstrans_cookie_t cookie;
|
||||||
|
int err;
|
||||||
|
|
||||||
|
+ if (!zfs_bclone_enabled)
|
||||||
|
+ return (-EOPNOTSUPP);
|
||||||
|
+
|
||||||
|
if (!spa_feature_is_enabled(
|
||||||
|
dmu_objset_spa(ITOZSB(dst_i)->z_os), SPA_FEATURE_BLOCK_CLONING))
|
||||||
|
return (-EOPNOTSUPP);
|
||||||
|
diff --git a/tests/zfs-tests/include/libtest.shlib b/tests/zfs-tests/include/libtest.shlib
|
||||||
|
index 844caa17d..d5d7bb6c8 100644
|
||||||
|
--- a/tests/zfs-tests/include/libtest.shlib
|
||||||
|
+++ b/tests/zfs-tests/include/libtest.shlib
|
||||||
|
@@ -3334,6 +3334,21 @@ function set_tunable_impl
|
||||||
|
esac
|
||||||
|
}
|
||||||
|
|
||||||
|
+function save_tunable
|
||||||
|
+{
|
||||||
|
+ [[ ! -d $TEST_BASE_DIR ]] && return 1
|
||||||
|
+ [[ -e $TEST_BASE_DIR/tunable-$1 ]] && return 2
|
||||||
|
+ echo "$(get_tunable """$1""")" > "$TEST_BASE_DIR"/tunable-"$1"
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+function restore_tunable
|
||||||
|
+{
|
||||||
|
+ [[ ! -e $TEST_BASE_DIR/tunable-$1 ]] && return 1
|
||||||
|
+ val="$(cat $TEST_BASE_DIR/tunable-"""$1""")"
|
||||||
|
+ set_tunable64 "$1" "$val"
|
||||||
|
+ rm $TEST_BASE_DIR/tunable-$1
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
#
|
||||||
|
# Get a global system tunable
|
||||||
|
#
|
||||||
|
diff --git a/tests/zfs-tests/include/tunables.cfg b/tests/zfs-tests/include/tunables.cfg
|
||||||
|
index 80e7bcb3b..a0edad14d 100644
|
||||||
|
--- a/tests/zfs-tests/include/tunables.cfg
|
||||||
|
+++ b/tests/zfs-tests/include/tunables.cfg
|
||||||
|
@@ -90,6 +90,7 @@ VOL_INHIBIT_DEV UNSUPPORTED zvol_inhibit_dev
|
||||||
|
VOL_MODE vol.mode zvol_volmode
|
||||||
|
VOL_RECURSIVE vol.recursive UNSUPPORTED
|
||||||
|
VOL_USE_BLK_MQ UNSUPPORTED zvol_use_blk_mq
|
||||||
|
+BCLONE_ENABLED zfs_bclone_enabled zfs_bclone_enabled
|
||||||
|
XATTR_COMPAT xattr_compat zfs_xattr_compat
|
||||||
|
ZEVENT_LEN_MAX zevent.len_max zfs_zevent_len_max
|
||||||
|
ZEVENT_RETAIN_MAX zevent.retain_max zfs_zevent_retain_max
|
||||||
|
diff --git a/tests/zfs-tests/tests/functional/block_cloning/cleanup.ksh b/tests/zfs-tests/tests/functional/block_cloning/cleanup.ksh
|
||||||
|
index 7ac13adb6..b985445a5 100755
|
||||||
|
--- a/tests/zfs-tests/tests/functional/block_cloning/cleanup.ksh
|
||||||
|
+++ b/tests/zfs-tests/tests/functional/block_cloning/cleanup.ksh
|
||||||
|
@@ -31,4 +31,8 @@ verify_runnable "global"
|
||||||
|
|
||||||
|
default_cleanup_noexit
|
||||||
|
|
||||||
|
+if tunable_exists BCLONE_ENABLED ; then
|
||||||
|
+ log_must restore_tunable BCLONE_ENABLED
|
||||||
|
+fi
|
||||||
|
+
|
||||||
|
log_pass
|
||||||
|
diff --git a/tests/zfs-tests/tests/functional/block_cloning/setup.ksh b/tests/zfs-tests/tests/functional/block_cloning/setup.ksh
|
||||||
|
index 512f5a064..58441bf8f 100755
|
||||||
|
--- a/tests/zfs-tests/tests/functional/block_cloning/setup.ksh
|
||||||
|
+++ b/tests/zfs-tests/tests/functional/block_cloning/setup.ksh
|
||||||
|
@@ -33,4 +33,9 @@ fi
|
||||||
|
|
||||||
|
verify_runnable "global"
|
||||||
|
|
||||||
|
+if tunable_exists BCLONE_ENABLED ; then
|
||||||
|
+ log_must save_tunable BCLONE_ENABLED
|
||||||
|
+ log_must set_tunable32 BCLONE_ENABLED 1
|
||||||
|
+fi
|
||||||
|
+
|
||||||
|
log_pass
|
||||||
@@ -0,0 +1,42 @@
|
|||||||
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
Date: Thu, 16 Nov 2023 11:42:19 -0800
|
||||||
|
Subject: [PATCH] zfs-2.2.1: Disable block cloning by default
|
||||||
|
|
||||||
|
Disable block cloning by default to mitigate possible data corruption
|
||||||
|
(see #15529 and #15526).
|
||||||
|
|
||||||
|
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
|
||||||
|
(cherry picked from commit 479dca51c66a731e637bd2d4f9bba01a05f9ac9f)
|
||||||
|
Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
|
||||||
|
---
|
||||||
|
module/os/freebsd/zfs/zfs_vfsops.c | 2 +-
|
||||||
|
module/os/linux/zfs/zpl_file_range.c | 2 +-
|
||||||
|
2 files changed, 2 insertions(+), 2 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/module/os/freebsd/zfs/zfs_vfsops.c b/module/os/freebsd/zfs/zfs_vfsops.c
|
||||||
|
index 09e18de81..0ac670ed9 100644
|
||||||
|
--- a/module/os/freebsd/zfs/zfs_vfsops.c
|
||||||
|
+++ b/module/os/freebsd/zfs/zfs_vfsops.c
|
||||||
|
@@ -89,7 +89,7 @@ int zfs_debug_level;
|
||||||
|
SYSCTL_INT(_vfs_zfs, OID_AUTO, debug, CTLFLAG_RWTUN, &zfs_debug_level, 0,
|
||||||
|
"Debug level");
|
||||||
|
|
||||||
|
-int zfs_bclone_enabled = 1;
|
||||||
|
+int zfs_bclone_enabled = 0;
|
||||||
|
SYSCTL_INT(_vfs_zfs, OID_AUTO, bclone_enabled, CTLFLAG_RWTUN,
|
||||||
|
&zfs_bclone_enabled, 0, "Enable block cloning");
|
||||||
|
|
||||||
|
diff --git a/module/os/linux/zfs/zpl_file_range.c b/module/os/linux/zfs/zpl_file_range.c
|
||||||
|
index 73476ff40..139c51cf4 100644
|
||||||
|
--- a/module/os/linux/zfs/zpl_file_range.c
|
||||||
|
+++ b/module/os/linux/zfs/zpl_file_range.c
|
||||||
|
@@ -31,7 +31,7 @@
|
||||||
|
#include <sys/zfs_vnops.h>
|
||||||
|
#include <sys/zfeature.h>
|
||||||
|
|
||||||
|
-int zfs_bclone_enabled = 1;
|
||||||
|
+int zfs_bclone_enabled = 0;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Clone part of a file via block cloning.
|
||||||
Vendored
+9
-3
@@ -6,6 +6,12 @@
|
|||||||
0006-dont-symlink-zed-scripts.patch
|
0006-dont-symlink-zed-scripts.patch
|
||||||
0007-Add-systemd-unit-for-importing-specific-pools.patch
|
0007-Add-systemd-unit-for-importing-specific-pools.patch
|
||||||
0008-Patch-move-manpage-arcstat-1-to-arcstat-8.patch
|
0008-Patch-move-manpage-arcstat-1-to-arcstat-8.patch
|
||||||
0009-arcstat-Fix-integer-division-with-python3.patch
|
0009-arc-stat-summary-guard-access-to-l2arc-MFU-MRU-stats.patch
|
||||||
0010-arc-stat-summary-guard-access-to-l2arc-MFU-MRU-stats.patch
|
0010-zvol-Remove-broken-blk-mq-optimization.patch
|
||||||
0011-Avoid-save-restoring-AMX-registers-to-avoid-a-SPR-er.patch
|
0011-Revert-zvol-Temporally-disable-blk-mq.patch
|
||||||
|
0012-Fix-nfs_truncate_shares-without-etc-exports.d.patch
|
||||||
|
0013-Workaround-UBSAN-errors-for-variable-arrays.patch
|
||||||
|
0014-zpool-status-tighten-bounds-for-noalloc-stat-availab.patch
|
||||||
|
0015-Fix-block-cloning-between-unencrypted-and-encrypted-.patch
|
||||||
|
0016-Add-a-tunable-to-disable-BRT-support.patch
|
||||||
|
0017-zfs-2.2.1-Disable-block-cloning-by-default.patch
|
||||||
|
|||||||
Vendored
+1
-9
@@ -60,10 +60,6 @@ override_dh_auto_install:
|
|||||||
@# Install the utilities.
|
@# Install the utilities.
|
||||||
$(MAKE) install DESTDIR='$(CURDIR)/debian/tmp'
|
$(MAKE) install DESTDIR='$(CURDIR)/debian/tmp'
|
||||||
|
|
||||||
# Use upstream's bash completion
|
|
||||||
install -D -t '$(CURDIR)/debian/tmp/usr/share/bash-completion/completions/' \
|
|
||||||
'$(CURDIR)/contrib/bash_completion.d/zfs'
|
|
||||||
|
|
||||||
# Move from bin_dir to /usr/sbin
|
# Move from bin_dir to /usr/sbin
|
||||||
# Remove suffix (.py) as per policy 10.4 - Scripts
|
# Remove suffix (.py) as per policy 10.4 - Scripts
|
||||||
# https://www.debian.org/doc/debian-policy/ch-files.html#s-scripts
|
# https://www.debian.org/doc/debian-policy/ch-files.html#s-scripts
|
||||||
@@ -83,7 +79,6 @@ override_dh_auto_install:
|
|||||||
|
|
||||||
chmod a-x '$(CURDIR)/debian/tmp/etc/zfs/zfs-functions'
|
chmod a-x '$(CURDIR)/debian/tmp/etc/zfs/zfs-functions'
|
||||||
chmod a-x '$(CURDIR)/debian/tmp/etc/default/zfs'
|
chmod a-x '$(CURDIR)/debian/tmp/etc/default/zfs'
|
||||||
chmod a-x '$(CURDIR)/debian/tmp/usr/share/bash-completion/completions/zfs'
|
|
||||||
|
|
||||||
override_dh_python3:
|
override_dh_python3:
|
||||||
dh_python3 -p python3-pyzfs
|
dh_python3 -p python3-pyzfs
|
||||||
@@ -91,9 +86,6 @@ override_dh_python3:
|
|||||||
override_dh_makeshlibs:
|
override_dh_makeshlibs:
|
||||||
dh_makeshlibs -a -V
|
dh_makeshlibs -a -V
|
||||||
|
|
||||||
override_dh_strip:
|
|
||||||
dh_strip --dbgsym-migration='zfs-dbg (<< 2.0.4~)'
|
|
||||||
|
|
||||||
override_dh_auto_clean:
|
override_dh_auto_clean:
|
||||||
find . -name .gitignore -delete
|
find . -name .gitignore -delete
|
||||||
rm -rf zfs-$(DEB_VERSION_UPSTREAM)
|
rm -rf zfs-$(DEB_VERSION_UPSTREAM)
|
||||||
@@ -101,7 +93,7 @@ override_dh_auto_clean:
|
|||||||
@if test -e META.orig; then mv META.orig META; fi
|
@if test -e META.orig; then mv META.orig META; fi
|
||||||
|
|
||||||
override_dh_install:
|
override_dh_install:
|
||||||
find debian/tmp/lib -name *.la -delete
|
find debian/tmp/lib -name '*.la' -delete
|
||||||
dh_install
|
dh_install
|
||||||
|
|
||||||
override_dh_missing:
|
override_dh_missing:
|
||||||
|
|||||||
Vendored
+4
-7
@@ -3,11 +3,8 @@ command-in-sbin-has-manpage-in-incorrect-section
|
|||||||
arch-dep-package-has-big-usr-share
|
arch-dep-package-has-big-usr-share
|
||||||
manpage-without-executable
|
manpage-without-executable
|
||||||
national-encoding *usr/share/zfs/zfs-tests/tests/functional/channel_program/lua_core/tst.lib_table.lua*
|
national-encoding *usr/share/zfs/zfs-tests/tests/functional/channel_program/lua_core/tst.lib_table.lua*
|
||||||
executable-not-elf-or-script *usr/share/zfs/zfs-tests/tests/functional/cli_root/zfs_jail/jail.conf*
|
executable-not-elf-or-script *usr/share/zfs/zfs-tests/tests/functional/cli_root/*
|
||||||
package-contains-documentation-outside-usr-share-doc *usr/share/zfs/zfs-tests/*
|
package-contains-documentation-outside-usr-share-doc *usr/share/zfs/zfs-tests/*
|
||||||
script-not-executable *usr/share/zfs/common.sh*
|
script-not-executable [usr/share/zfs/common.sh]
|
||||||
script-not-executable *usr/share/zfs/zfs-tests/include/default.cfg*
|
script-not-executable [usr/share/zfs/zfs-tests/include/default.cfg]
|
||||||
script-not-executable *usr/share/zfs/zfs-tests/tests/functional/cli_root/zfs_wait/zfs_wait.kshlib*
|
script-not-executable [usr/share/zfs/zfs-tests/tests/functional/*]
|
||||||
script-not-executable *usr/share/zfs/zfs-tests/tests/functional/cli_root/zpool_wait/zpool_wait.kshlib*
|
|
||||||
script-not-executable *usr/share/zfs/zfs-tests/tests/functional/l2arc/l2arc.cfg*
|
|
||||||
script-not-executable *usr/share/zfs/zfs-tests/tests/functional/redacted_send/redacted.kshlib*
|
|
||||||
|
|||||||
Vendored
+7
-4
@@ -14,6 +14,9 @@ lib/systemd/system/zfs-scrub-monthly@.timer
|
|||||||
lib/systemd/system/zfs-scrub-weekly@.timer
|
lib/systemd/system/zfs-scrub-weekly@.timer
|
||||||
lib/systemd/system/zfs-scrub@.service
|
lib/systemd/system/zfs-scrub@.service
|
||||||
lib/systemd/system/zfs-share.service
|
lib/systemd/system/zfs-share.service
|
||||||
|
lib/systemd/system/zfs-trim-monthly@.timer
|
||||||
|
lib/systemd/system/zfs-trim-weekly@.timer
|
||||||
|
lib/systemd/system/zfs-trim@.service
|
||||||
lib/systemd/system/zfs-volume-wait.service
|
lib/systemd/system/zfs-volume-wait.service
|
||||||
lib/systemd/system/zfs-volumes.target
|
lib/systemd/system/zfs-volumes.target
|
||||||
lib/systemd/system/zfs.target
|
lib/systemd/system/zfs.target
|
||||||
@@ -30,8 +33,8 @@ sbin/zpool
|
|||||||
sbin/zstream
|
sbin/zstream
|
||||||
sbin/zstreamdump
|
sbin/zstreamdump
|
||||||
usr/bin/zvol_wait
|
usr/bin/zvol_wait
|
||||||
|
usr/bin/zilstat
|
||||||
usr/lib/modules-load.d/ lib/
|
usr/lib/modules-load.d/ lib/
|
||||||
usr/lib/zfs-linux/zfs_prepare_disk
|
|
||||||
usr/lib/zfs-linux/zpool.d/
|
usr/lib/zfs-linux/zpool.d/
|
||||||
usr/lib/zfs-linux/zpool_influxdb
|
usr/lib/zfs-linux/zpool_influxdb
|
||||||
usr/sbin/arc_summary
|
usr/sbin/arc_summary
|
||||||
@@ -44,6 +47,7 @@ usr/share/man/man1/zvol_wait.1
|
|||||||
usr/share/man/man4/zfs.4
|
usr/share/man/man4/zfs.4
|
||||||
usr/share/man/man4/spl.4
|
usr/share/man/man4/spl.4
|
||||||
usr/share/man/man5/
|
usr/share/man/man5/
|
||||||
|
usr/share/man/man7/vdevprops.7
|
||||||
usr/share/man/man7/zfsconcepts.7
|
usr/share/man/man7/zfsconcepts.7
|
||||||
usr/share/man/man7/zfsprops.7
|
usr/share/man/man7/zfsprops.7
|
||||||
usr/share/man/man7/zpoolconcepts.7
|
usr/share/man/man7/zpoolconcepts.7
|
||||||
@@ -64,12 +68,10 @@ usr/share/man/man8/zfs-get.8
|
|||||||
usr/share/man/man8/zfs-groupspace.8
|
usr/share/man/man8/zfs-groupspace.8
|
||||||
usr/share/man/man8/zfs-hold.8
|
usr/share/man/man8/zfs-hold.8
|
||||||
usr/share/man/man8/zfs-inherit.8
|
usr/share/man/man8/zfs-inherit.8
|
||||||
usr/share/man/man8/zfs-jail.8
|
|
||||||
usr/share/man/man8/zfs-list.8
|
usr/share/man/man8/zfs-list.8
|
||||||
usr/share/man/man8/zfs-load-key.8
|
usr/share/man/man8/zfs-load-key.8
|
||||||
usr/share/man/man8/zfs-mount-generator.8
|
usr/share/man/man8/zfs-mount-generator.8
|
||||||
usr/share/man/man8/zfs-mount.8
|
usr/share/man/man8/zfs-mount.8
|
||||||
usr/share/man/man8/zfs_prepare_disk.8
|
|
||||||
usr/share/man/man8/zfs-program.8
|
usr/share/man/man8/zfs-program.8
|
||||||
usr/share/man/man8/zfs-project.8
|
usr/share/man/man8/zfs-project.8
|
||||||
usr/share/man/man8/zfs-projectspace.8
|
usr/share/man/man8/zfs-projectspace.8
|
||||||
@@ -85,9 +87,9 @@ usr/share/man/man8/zfs-set.8
|
|||||||
usr/share/man/man8/zfs-share.8
|
usr/share/man/man8/zfs-share.8
|
||||||
usr/share/man/man8/zfs-snapshot.8
|
usr/share/man/man8/zfs-snapshot.8
|
||||||
usr/share/man/man8/zfs-unallow.8
|
usr/share/man/man8/zfs-unallow.8
|
||||||
usr/share/man/man8/zfs-unjail.8
|
|
||||||
usr/share/man/man8/zfs-unload-key.8
|
usr/share/man/man8/zfs-unload-key.8
|
||||||
usr/share/man/man8/zfs-unmount.8
|
usr/share/man/man8/zfs-unmount.8
|
||||||
|
usr/share/man/man8/zfs-unzone.8
|
||||||
usr/share/man/man8/zfs-upgrade.8
|
usr/share/man/man8/zfs-upgrade.8
|
||||||
usr/share/man/man8/zfs-userspace.8
|
usr/share/man/man8/zfs-userspace.8
|
||||||
usr/share/man/man8/zfs-wait.8
|
usr/share/man/man8/zfs-wait.8
|
||||||
@@ -126,6 +128,7 @@ usr/share/man/man8/zpool-sync.8
|
|||||||
usr/share/man/man8/zpool-trim.8
|
usr/share/man/man8/zpool-trim.8
|
||||||
usr/share/man/man8/zpool-upgrade.8
|
usr/share/man/man8/zpool-upgrade.8
|
||||||
usr/share/man/man8/zpool-wait.8
|
usr/share/man/man8/zpool-wait.8
|
||||||
|
usr/share/man/man8/zfs-zone.8
|
||||||
usr/share/man/man8/zpool.8
|
usr/share/man/man8/zpool.8
|
||||||
usr/share/man/man8/zstream.8
|
usr/share/man/man8/zstream.8
|
||||||
usr/share/man/man8/zstreamdump.8
|
usr/share/man/man8/zstreamdump.8
|
||||||
|
|||||||
+1
-1
Submodule upstream updated: fb6d532066...95785196f2
Reference in New Issue
Block a user