[pve-devel] [PATCH zfsonlinux] update zfs submodule to 2.1.9 and refresh patches

Stoiko Ivanov s.ivanov at proxmox.com
Thu Jan 26 12:06:02 CET 2023


mostly drop patches we already applied intermittently

Signed-off-by: Stoiko Ivanov <s.ivanov at proxmox.com>
---
This supersedes the patch from
https://lists.proxmox.com/pipermail/pve-devel/2023-January/055555.html

2.1.9 was released yesterday and addresses exactly the 2 issues with 2.1.8
noted there

did some minimal testing (as before):
* running ztest on a VM
* having a few containers with replication setup and migrating one of them
* creating a PBS-backup of a VM and then again (to verify that incremental
  dirty-bitmap backups work)
all as expected.

 ...o-the-zed-binary-on-the-systemd-unit.patch |   4 +-
 ...itramfs-Fix-legacy-mountpoint-rootfs.patch |  58 --------
 ...dd-workaround-for-broken-Linux-pipes.patch |  68 ---------
 ...ssion-checks-for-extended-attributes.patch |  91 ------------
 ...Linux-6.1-compat-open-inside-tmpfile.patch | 137 ------------------
 debian/patches/series                         |   4 -
 upstream                                      |   2 +-
 7 files changed, 3 insertions(+), 361 deletions(-)
 delete mode 100644 debian/patches/0012-initramfs-Fix-legacy-mountpoint-rootfs.patch
 delete mode 100644 debian/patches/0013-Add-workaround-for-broken-Linux-pipes.patch
 delete mode 100644 debian/patches/0014-skip-permission-checks-for-extended-attributes.patch
 delete mode 100644 debian/patches/0015-Linux-6.1-compat-open-inside-tmpfile.patch

diff --git a/debian/patches/0003-Fix-the-path-to-the-zed-binary-on-the-systemd-unit.patch b/debian/patches/0003-Fix-the-path-to-the-zed-binary-on-the-systemd-unit.patch
index ebc1850b..b1f78182 100644
--- a/debian/patches/0003-Fix-the-path-to-the-zed-binary-on-the-systemd-unit.patch
+++ b/debian/patches/0003-Fix-the-path-to-the-zed-binary-on-the-systemd-unit.patch
@@ -13,7 +13,7 @@ Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
  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
-index 008075138..570e27707 100644
+index be80025a4..20ce8e632 100644
 --- a/etc/systemd/system/zfs-zed.service.in
 +++ b/etc/systemd/system/zfs-zed.service.in
 @@ -4,7 +4,7 @@ Documentation=man:zed(8)
@@ -22,6 +22,6 @@ index 008075138..570e27707 100644
  [Service]
 -ExecStart=@sbindir@/zed -F
 +ExecStart=/usr/sbin/zed -F
- Restart=on-abort
+ Restart=always
  
  [Install]
diff --git a/debian/patches/0012-initramfs-Fix-legacy-mountpoint-rootfs.patch b/debian/patches/0012-initramfs-Fix-legacy-mountpoint-rootfs.patch
deleted file mode 100644
index e26941d2..00000000
--- a/debian/patches/0012-initramfs-Fix-legacy-mountpoint-rootfs.patch
+++ /dev/null
@@ -1,58 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Ryan Moeller <ryan at iXsystems.com>
-Date: Mon, 12 Dec 2022 13:23:06 -0500
-Subject: [PATCH] initramfs: Fix legacy mountpoint rootfs
-
-Legacy mountpoint datasets should not pass `-o zfsutil` to `mount.zfs`.
-Fix the logic in `mount_fs()` to not forget we have a legacy mountpoint
-when checking for an `org.zol:mountpoint` userprop.
-
-Reviewed-by: Richard Yao <ryao at gentoo.org>
-Reviewed-by: Brian Behlendorf <behlendorf1 at llnl.gov>
-Signed-off-by: Ryan Moeller <ryan at iXsystems.com>
-Closes #14274
-(cherry picked from commit 786ff6a6cb33226b4f4292c7569b9093286f74d9)
-(cherry picked from commit f28c7302cb77a8cb6112690d0eed0bf4adeeae15)
-Signed-off-by: Stoiko Ivanov <s.ivanov at proxmox.com>
----
- contrib/initramfs/scripts/zfs | 20 ++++++++------------
- 1 file changed, 8 insertions(+), 12 deletions(-)
-
-diff --git a/contrib/initramfs/scripts/zfs b/contrib/initramfs/scripts/zfs
-index 9169adf5f..c7548fa07 100644
---- a/contrib/initramfs/scripts/zfs
-+++ b/contrib/initramfs/scripts/zfs
-@@ -331,25 +331,21 @@ mount_fs()
- 		# Can't use the mountpoint property. Might be one of our
- 		# clones. Check the 'org.zol:mountpoint' property set in
- 		# clone_snap() if that's usable.
--		mountpoint=$(get_fs_value "$fs" org.zol:mountpoint)
--		if [ "$mountpoint" = "legacy" ] ||
--		   [ "$mountpoint" = "none" ] ||
--		   [ "$mountpoint" = "-" ]
-+		mountpoint1=$(get_fs_value "$fs" org.zol:mountpoint)
-+		if [ "$mountpoint1" = "legacy" ] ||
-+		   [ "$mountpoint1" = "none" ] ||
-+		   [ "$mountpoint1" = "-" ]
- 		then
- 			if [ "$fs" != "${ZFS_BOOTFS}" ]; then
- 				# We don't have a proper mountpoint and this
- 				# isn't the root fs.
- 				return 0
--			else
--				# Last hail-mary: Hope 'rootmnt' is set!
--				mountpoint=""
- 			fi
--		fi
--
--		# If it's not a legacy filesystem, it can only be a
--		# native one...
--		if [ "$mountpoint" = "legacy" ]; then
- 			ZFS_CMD="mount.zfs"
-+			# Last hail-mary: Hope 'rootmnt' is set!
-+			mountpoint=""
-+		else
-+			mountpoint="$mountpoint1"
- 		fi
- 	fi
- 
diff --git a/debian/patches/0013-Add-workaround-for-broken-Linux-pipes.patch b/debian/patches/0013-Add-workaround-for-broken-Linux-pipes.patch
deleted file mode 100644
index ce4d75cd..00000000
--- a/debian/patches/0013-Add-workaround-for-broken-Linux-pipes.patch
+++ /dev/null
@@ -1,68 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Rich Ercolani <214141+rincebrain at users.noreply.github.com>
-Date: Mon, 9 May 2022 19:33:55 -0400
-Subject: [PATCH] Add workaround for broken Linux pipes
-
-Linux has an unresolved hang if you resize a pipe with bytes
-in it.
-
-Since there's no obvious way to detect this happening, added a
-workaround to disable resizing the pipe buffer if you set an
-environment variable.
-
-Reviewed-by: Brian Behlendorf <behlendorf1 at llnl.gov>
-Signed-off-by: Rich Ercolani <rincebrain at gmail.com>
-Closes #13309
-(cherry picked from commit e84a2ed7a8b9e6458c25c6e93a93601eaf4128eb)
-Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
----
- lib/libzfs/os/linux/libzfs_sendrecv_os.c | 16 ++++++++++++++++
- man/man8/zfs.8                           |  8 ++++++++
- 2 files changed, 24 insertions(+)
-
-diff --git a/lib/libzfs/os/linux/libzfs_sendrecv_os.c b/lib/libzfs/os/linux/libzfs_sendrecv_os.c
-index ac7ab95a3..8fc02fff2 100644
---- a/lib/libzfs/os/linux/libzfs_sendrecv_os.c
-+++ b/lib/libzfs/os/linux/libzfs_sendrecv_os.c
-@@ -35,6 +35,22 @@
- void
- libzfs_set_pipe_max(int infd)
- {
-+#if __linux__
-+	/*
-+	 * Sadly, Linux has an unfixed deadlock if you do SETPIPE_SZ on a pipe
-+	 * with data in it.
-+	 * cf. #13232, https://bugzilla.kernel.org/show_bug.cgi?id=212295
-+	 *
-+	 * And since the problem is in waking up the writer, there's nothing
-+	 * we can do about it from here.
-+	 *
-+	 * So if people want to, they can set this, but they
-+	 * may regret it...
-+	 */
-+	if (getenv("ZFS_SET_PIPE_MAX") == NULL)
-+		return;
-+#endif
-+
- 	FILE *procf = fopen("/proc/sys/fs/pipe-max-size", "re");
- 
- 	if (procf != NULL) {
-diff --git a/man/man8/zfs.8 b/man/man8/zfs.8
-index fca1ba00d..a5c944169 100644
---- a/man/man8/zfs.8
-+++ b/man/man8/zfs.8
-@@ -713,6 +713,14 @@ to use
- to mount ZFS datasets.
- This option is provided for backwards compatibility with older ZFS versions.
- .El
-+.Bl -tag -width "ZFS_SET_PIPE_MAX"
-+.It Sy ZFS_SET_PIPE_MAX
-+Tells
-+.Nm zfs
-+to set the maximum pipe size for sends/recieves.
-+Disabled by default on Linux
-+due to an unfixed deadlock in Linux's pipe size handling code.
-+.El
- .
- .Sh INTERFACE STABILITY
- .Sy Committed .
diff --git a/debian/patches/0014-skip-permission-checks-for-extended-attributes.patch b/debian/patches/0014-skip-permission-checks-for-extended-attributes.patch
deleted file mode 100644
index 0b5f6cd9..00000000
--- a/debian/patches/0014-skip-permission-checks-for-extended-attributes.patch
+++ /dev/null
@@ -1,91 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Ameer Hamza <ahamza at ixsystems.com>
-Date: Wed, 23 Nov 2022 01:28:13 +0500
-Subject: [PATCH] skip permission checks for extended attributes
-
-zfs_zaccess_trivial() calls the generic_permission() to read
-xattr attributes. This causes deadlock if called from
-zpl_xattr_set_dir() context as xattr and the dent locks are
-already held in this scenario. This commit skips the permissions
-checks for extended attributes since the Linux VFS stack already
-checks it before passing us the control.
-
-Signed-off-by: Ameer Hamza <ahamza at ixsystems.com>
-(cherry picked from commit 75fbe7eb99c3d20403c79c7e24a97997c424313a)
-Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
----
- module/os/linux/zfs/zfs_dir.c                                | 3 ---
- module/os/linux/zfs/zfs_vnops_os.c                           | 3 ++-
- module/os/linux/zfs/zpl_xattr.c                              | 2 +-
- tests/zfs-tests/tests/functional/acl/posix/posix_004_pos.ksh | 3 +++
- 4 files changed, 6 insertions(+), 5 deletions(-)
-
-diff --git a/module/os/linux/zfs/zfs_dir.c b/module/os/linux/zfs/zfs_dir.c
-index 82b32d1cc..8ad5454b5 100644
---- a/module/os/linux/zfs/zfs_dir.c
-+++ b/module/os/linux/zfs/zfs_dir.c
-@@ -1066,9 +1066,6 @@ zfs_make_xattrdir(znode_t *zp, vattr_t *vap, znode_t **xzpp, cred_t *cr)
- 
- 	*xzpp = NULL;
- 
--	if ((error = zfs_zaccess(zp, ACE_WRITE_NAMED_ATTRS, 0, B_FALSE, cr)))
--		return (error);
--
- 	if ((error = zfs_acl_ids_create(zp, IS_XATTR, vap, cr, NULL,
- 	    &acl_ids)) != 0)
- 		return (error);
-diff --git a/module/os/linux/zfs/zfs_vnops_os.c b/module/os/linux/zfs/zfs_vnops_os.c
-index 796a86c25..ae0401e60 100644
---- a/module/os/linux/zfs/zfs_vnops_os.c
-+++ b/module/os/linux/zfs/zfs_vnops_os.c
-@@ -579,6 +579,7 @@ zfs_create(znode_t *dzp, char *name, vattr_t *vap, int excl,
- 	boolean_t	fuid_dirtied;
- 	boolean_t	have_acl = B_FALSE;
- 	boolean_t	waited = B_FALSE;
-+	boolean_t	skip_acl = (flag & ATTR_NOACLCHECK) ? B_TRUE : B_FALSE;
- 
- 	/*
- 	 * If we have an ephemeral id, ACL, or XVATTR then
-@@ -651,7 +652,7 @@ top:
- 		 * Create a new file object and update the directory
- 		 * to reference it.
- 		 */
--		if ((error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, B_FALSE, cr))) {
-+		if ((error = zfs_zaccess(dzp, ACE_ADD_FILE, 0, skip_acl, cr))) {
- 			if (have_acl)
- 				zfs_acl_ids_free(&acl_ids);
- 			goto out;
-diff --git a/module/os/linux/zfs/zpl_xattr.c b/module/os/linux/zfs/zpl_xattr.c
-index bd5d5803f..9d758d2d0 100644
---- a/module/os/linux/zfs/zpl_xattr.c
-+++ b/module/os/linux/zfs/zpl_xattr.c
-@@ -496,7 +496,7 @@ zpl_xattr_set_dir(struct inode *ip, const char *name, const void *value,
- 		vap->va_gid = crgetgid(cr);
- 
- 		error = -zfs_create(dxzp, (char *)name, vap, 0, 0644, &xzp,
--		    cr, 0, NULL);
-+		    cr, ATTR_NOACLCHECK, NULL);
- 		if (error)
- 			goto out;
- 	}
-diff --git a/tests/zfs-tests/tests/functional/acl/posix/posix_004_pos.ksh b/tests/zfs-tests/tests/functional/acl/posix/posix_004_pos.ksh
-index 6c6b592fb..8aa2cf496 100755
---- a/tests/zfs-tests/tests/functional/acl/posix/posix_004_pos.ksh
-+++ b/tests/zfs-tests/tests/functional/acl/posix/posix_004_pos.ksh
-@@ -35,6 +35,7 @@
- # STRATEGY:
- #	1. Prepare an appropriate ACL on the test directory
- #	2. Change the owner of the directory
-+#	3. Reset and set the ACLs for test directory owned by the user
- #
- 
- verify_runnable "both"
-@@ -44,6 +45,8 @@ log_must setfacl -d -m u:$ZFS_ACL_STAFF1:rwx $TESTDIR
- log_must setfacl -b $TESTDIR
- 
- log_must chown $ZFS_ACL_STAFF1 $TESTDIR
-+log_must setfacl -b $TESTDIR
-+log_must setfacl -d -m u:$ZFS_ACL_STAFF1:rwx $TESTDIR
- log_must chown 0 $TESTDIR
- 
- log_pass "chown works with POSIX ACLs"
diff --git a/debian/patches/0015-Linux-6.1-compat-open-inside-tmpfile.patch b/debian/patches/0015-Linux-6.1-compat-open-inside-tmpfile.patch
deleted file mode 100644
index 778e7912..00000000
--- a/debian/patches/0015-Linux-6.1-compat-open-inside-tmpfile.patch
+++ /dev/null
@@ -1,137 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Antonio Russo <aerusso at aerusso.net>
-Date: Sat, 31 Dec 2022 07:51:32 -0700
-Subject: [PATCH] Linux 6.1 compat: open inside tmpfile()
-
-Linux 863f144 modified the .tmpfile interface to pass a struct file,
-rather than a struct dentry, and expect the tmpfile implementation to
-open inside of tmpfile().
-
-This patch implements a configuration test that checks for this new API
-and appropriately sets a HAVE_TMPFILE_DENTRY flag that tracks this old
-API.  Contingent on this flag, the appropriate API is implemented.
-
-Reviewed-by: Richard Yao <richard.yao at alumni.stonybrook.edu>
-Reviewed-by: Brian Behlendorf <behlendorf1 at llnl.gov>
-Signed-off-by: Antonio Russo <aerusso at aerusso.net>
-Closes #14301
-Closes #14343
-(cherry picked from commit d27c81847b43584483b5509ff352e7e727b0ce87)
-Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
----
- config/kernel-tmpfile.m4        | 32 +++++++++++++++++++++++++++-----
- module/os/linux/zfs/zpl_inode.c | 15 +++++++++++++++
- 2 files changed, 42 insertions(+), 5 deletions(-)
-
-diff --git a/config/kernel-tmpfile.m4 b/config/kernel-tmpfile.m4
-index 45c2e6cee..acb7ea1e8 100644
---- a/config/kernel-tmpfile.m4
-+++ b/config/kernel-tmpfile.m4
-@@ -3,11 +3,25 @@ dnl # 3.11 API change
- dnl # Add support for i_op->tmpfile
- dnl #
- AC_DEFUN([ZFS_AC_KERNEL_SRC_TMPFILE], [
-+	dnl #
-+	dnl # 6.1 API change
-+	dnl # use struct file instead of struct dentry
-+	dnl #
-+	ZFS_LINUX_TEST_SRC([inode_operations_tmpfile], [
-+		#include <linux/fs.h>
-+		int tmpfile(struct user_namespace *userns,
-+		    struct inode *inode, struct file *file,
-+		    umode_t mode) { return 0; }
-+		static struct inode_operations
-+		    iops __attribute__ ((unused)) = {
-+			.tmpfile = tmpfile,
-+		};
-+	],[])
- 	dnl #
- 	dnl # 5.11 API change
- 	dnl # add support for userns parameter to tmpfile
- 	dnl #
--	ZFS_LINUX_TEST_SRC([inode_operations_tmpfile_userns], [
-+	ZFS_LINUX_TEST_SRC([inode_operations_tmpfile_dentry_userns], [
- 		#include <linux/fs.h>
- 		int tmpfile(struct user_namespace *userns,
- 		    struct inode *inode, struct dentry *dentry,
-@@ -17,7 +31,7 @@ AC_DEFUN([ZFS_AC_KERNEL_SRC_TMPFILE], [
- 			.tmpfile = tmpfile,
- 		};
- 	],[])
--	ZFS_LINUX_TEST_SRC([inode_operations_tmpfile], [
-+	ZFS_LINUX_TEST_SRC([inode_operations_tmpfile_dentry], [
- 			#include <linux/fs.h>
- 			int tmpfile(struct inode *inode, struct dentry *dentry,
- 			    umode_t mode) { return 0; }
-@@ -30,16 +44,24 @@ AC_DEFUN([ZFS_AC_KERNEL_SRC_TMPFILE], [
- 
- AC_DEFUN([ZFS_AC_KERNEL_TMPFILE], [
- 	AC_MSG_CHECKING([whether i_op->tmpfile() exists])
--	ZFS_LINUX_TEST_RESULT([inode_operations_tmpfile_userns], [
-+	ZFS_LINUX_TEST_RESULT([inode_operations_tmpfile], [
- 		AC_MSG_RESULT(yes)
- 		AC_DEFINE(HAVE_TMPFILE, 1, [i_op->tmpfile() exists])
- 		AC_DEFINE(HAVE_TMPFILE_USERNS, 1, [i_op->tmpfile() has userns])
- 	],[
--		ZFS_LINUX_TEST_RESULT([inode_operations_tmpfile], [
-+		ZFS_LINUX_TEST_RESULT([inode_operations_tmpfile_dentry_userns], [
- 			AC_MSG_RESULT(yes)
- 			AC_DEFINE(HAVE_TMPFILE, 1, [i_op->tmpfile() exists])
-+			AC_DEFINE(HAVE_TMPFILE_USERNS, 1, [i_op->tmpfile() has userns])
-+			AC_DEFINE(HAVE_TMPFILE_DENTRY, 1, [i_op->tmpfile() uses old dentry signature])
- 		],[
--			AC_MSG_RESULT(no)
-+			ZFS_LINUX_TEST_RESULT([inode_operations_tmpfile_dentry], [
-+				AC_MSG_RESULT(yes)
-+				AC_DEFINE(HAVE_TMPFILE, 1, [i_op->tmpfile() exists])
-+				AC_DEFINE(HAVE_TMPFILE_DENTRY, 1, [i_op->tmpfile() uses old dentry signature])
-+			],[
-+				AC_MSG_RESULT(no)
-+			])
- 		])
- 	])
- ])
-diff --git a/module/os/linux/zfs/zpl_inode.c b/module/os/linux/zfs/zpl_inode.c
-index a0615af8d..fece7886e 100644
---- a/module/os/linux/zfs/zpl_inode.c
-+++ b/module/os/linux/zfs/zpl_inode.c
-@@ -224,12 +224,17 @@ zpl_mknod(struct inode *dir, struct dentry *dentry, umode_t mode,
- 
- #ifdef HAVE_TMPFILE
- static int
-+#ifndef HAVE_TMPFILE_DENTRY
-+zpl_tmpfile(struct user_namespace *userns, struct inode *dir,
-+    struct file *file, umode_t mode)
-+#else
- #ifdef HAVE_TMPFILE_USERNS
- zpl_tmpfile(struct user_namespace *userns, struct inode *dir,
-     struct dentry *dentry, umode_t mode)
- #else
- zpl_tmpfile(struct inode *dir, struct dentry *dentry, umode_t mode)
- #endif
-+#endif
- {
- 	cred_t *cr = CRED();
- 	struct inode *ip;
-@@ -252,11 +257,21 @@ zpl_tmpfile(struct inode *dir, struct dentry *dentry, umode_t mode)
- 	if (error == 0) {
- 		/* d_tmpfile will do drop_nlink, so we should set it first */
- 		set_nlink(ip, 1);
-+#ifndef HAVE_TMPFILE_DENTRY
-+		d_tmpfile(file, ip);
-+
-+		error = zpl_xattr_security_init(ip, dir,
-+		    &file->f_path.dentry->d_name);
-+#else
- 		d_tmpfile(dentry, ip);
- 
- 		error = zpl_xattr_security_init(ip, dir, &dentry->d_name);
-+#endif
- 		if (error == 0)
- 			error = zpl_init_acl(ip, dir);
-+#ifndef HAVE_TMPFILE_DENTRY
-+		error = finish_open_simple(file, error);
-+#endif
- 		/*
- 		 * don't need to handle error here, file is already in
- 		 * unlinked set.
diff --git a/debian/patches/series b/debian/patches/series
index 4032f1ee..d2770d39 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -9,7 +9,3 @@
 0009-Patch-move-manpage-arcstat-1-to-arcstat-8.patch
 0010-arcstat-Fix-integer-division-with-python3.patch
 0011-arc-stat-summary-guard-access-to-l2arc-MFU-MRU-stats.patch
-0012-initramfs-Fix-legacy-mountpoint-rootfs.patch
-0013-Add-workaround-for-broken-Linux-pipes.patch
-0014-skip-permission-checks-for-extended-attributes.patch
-0015-Linux-6.1-compat-open-inside-tmpfile.patch
diff --git a/upstream b/upstream
index 21bd7661..92e0d9d1 160000
--- a/upstream
+++ b/upstream
@@ -1 +1 @@
-Subproject commit 21bd7661334cd865d17934bebbcaf8d3356279ee
+Subproject commit 92e0d9d183ce6752cd52f7277c8321d81df9ffee
-- 
2.30.2






More information about the pve-devel mailing list