Date: Monday, February 21, 2022 @ 06:49:04
  Author: andyrtr
Revision: 437832

upgpkg: linux-lts 5.15.24-2: try to fix NFSv4 not mounting regression - 
FS#73838 / FS#73860

Added:
  linux-lts/trunk/0007_fix_NFSv4_mount_regression.diff
Modified:
  linux-lts/trunk/PKGBUILD

--------------------------------------+
 0007_fix_NFSv4_mount_regression.diff |  243 +++++++++++++++++++++++++++++++++
 PKGBUILD                             |   10 +
 2 files changed, 251 insertions(+), 2 deletions(-)

Added: 0007_fix_NFSv4_mount_regression.diff
===================================================================
--- 0007_fix_NFSv4_mount_regression.diff                                (rev 0)
+++ 0007_fix_NFSv4_mount_regression.diff        2022-02-21 06:49:04 UTC (rev 
437832)
@@ -0,0 +1,243 @@
+From 6f2836341d8a39e1e000572b10959347d7e61fd9 Mon Sep 17 00:00:00 2001
+From: Olga Kornievskaia <ko...@netapp.com>
+Date: Wed, 12 Jan 2022 10:27:38 -0500
+Subject: NFSv4.1 query for fs_location attr on a new file system
+
+[ Upstream commit 1976b2b31462151403c9fc110204fcc2a77bdfd1 ]
+
+Query the server for other possible trunkable locations for a given
+file system on a 4.1+ mount.
+
+v2:
+-- added missing static to nfs4_discover_trunking,
+reported by the kernel test robot
+
+Signed-off-by: Olga Kornievskaia <ko...@netapp.com>
+Signed-off-by: Anna Schumaker <anna.schuma...@netapp.com>
+Signed-off-by: Sasha Levin <sas...@kernel.org>
+---
+ fs/nfs/client.c         |  7 +++++
+ fs/nfs/nfs4_fs.h        |  9 +++---
+ fs/nfs/nfs4proc.c       | 76 ++++++++++++++++++++++++++++++++++++++++++-------
+ fs/nfs/nfs4state.c      |  3 +-
+ include/linux/nfs_xdr.h |  1 +
+ 5 files changed, 81 insertions(+), 15 deletions(-)
+
+diff --git a/fs/nfs/client.c b/fs/nfs/client.c
+index 551833862171f..090b16890e3d6 100644
+--- a/fs/nfs/client.c
++++ b/fs/nfs/client.c
+@@ -860,6 +860,13 @@ int nfs_probe_fsinfo(struct nfs_server *server, struct 
nfs_fh *mntfh, struct nfs
+                       server->namelen = pathinfo.max_namelen;
+       }
+ 
++      if (clp->rpc_ops->discover_trunking != NULL &&
++                      (server->caps & NFS_CAP_FS_LOCATIONS)) {
++              error = clp->rpc_ops->discover_trunking(server, mntfh);
++              if (error < 0)
++                      return error;
++      }
++
+       return 0;
+ }
+ EXPORT_SYMBOL_GPL(nfs_probe_fsinfo);
+diff --git a/fs/nfs/nfs4_fs.h b/fs/nfs/nfs4_fs.h
+index ba78df4b13d94..1a048ee653a11 100644
+--- a/fs/nfs/nfs4_fs.h
++++ b/fs/nfs/nfs4_fs.h
+@@ -261,8 +261,8 @@ struct nfs4_state_maintenance_ops {
+ };
+ 
+ struct nfs4_mig_recovery_ops {
+-      int (*get_locations)(struct inode *, struct nfs4_fs_locations *,
+-              struct page *, const struct cred *);
++      int (*get_locations)(struct nfs_server *, struct nfs_fh *,
++              struct nfs4_fs_locations *, struct page *, const struct cred *);
+       int (*fsid_present)(struct inode *, const struct cred *);
+ };
+ 
+@@ -303,8 +303,9 @@ extern int nfs4_do_close(struct nfs4_state *state, gfp_t 
gfp_mask, int wait);
+ extern int nfs4_server_capabilities(struct nfs_server *server, struct nfs_fh 
*fhandle);
+ extern int nfs4_proc_fs_locations(struct rpc_clnt *, struct inode *, const 
struct qstr *,
+                                 struct nfs4_fs_locations *, struct page *);
+-extern int nfs4_proc_get_locations(struct inode *, struct nfs4_fs_locations *,
+-              struct page *page, const struct cred *);
++extern int nfs4_proc_get_locations(struct nfs_server *, struct nfs_fh *,
++                                 struct nfs4_fs_locations *,
++                                 struct page *page, const struct cred *);
+ extern int nfs4_proc_fsid_present(struct inode *, const struct cred *);
+ extern struct rpc_clnt *nfs4_proc_lookup_mountpoint(struct inode *,
+                                                   struct dentry *,
+diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c
+index 367a1b99b7550..389fa72d4ca98 100644
+--- a/fs/nfs/nfs4proc.c
++++ b/fs/nfs/nfs4proc.c
+@@ -3952,6 +3952,60 @@ int nfs4_server_capabilities(struct nfs_server *server, 
struct nfs_fh *fhandle)
+       return err;
+ }
+ 
++static int _nfs4_discover_trunking(struct nfs_server *server,
++                                 struct nfs_fh *fhandle)
++{
++      struct nfs4_fs_locations *locations = NULL;
++      struct page *page;
++      const struct cred *cred;
++      struct nfs_client *clp = server->nfs_client;
++      const struct nfs4_state_maintenance_ops *ops =
++              clp->cl_mvops->state_renewal_ops;
++      int status = -ENOMEM;
++
++      cred = ops->get_state_renewal_cred(clp);
++      if (cred == NULL) {
++              cred = nfs4_get_clid_cred(clp);
++              if (cred == NULL)
++                      return -ENOKEY;
++      }
++
++      page = alloc_page(GFP_KERNEL);
++      locations = kmalloc(sizeof(struct nfs4_fs_locations), GFP_KERNEL);
++      if (page == NULL || locations == NULL)
++              goto out;
++
++      status = nfs4_proc_get_locations(server, fhandle, locations, page,
++                                       cred);
++      if (status)
++              goto out;
++out:
++      if (page)
++              __free_page(page);
++      kfree(locations);
++      return status;
++}
++
++static int nfs4_discover_trunking(struct nfs_server *server,
++                                struct nfs_fh *fhandle)
++{
++      struct nfs4_exception exception = {
++              .interruptible = true,
++      };
++      struct nfs_client *clp = server->nfs_client;
++      int err = 0;
++
++      if (!nfs4_has_session(clp))
++              goto out;
++      do {
++              err = nfs4_handle_exception(server,
++                              _nfs4_discover_trunking(server, fhandle),
++                              &exception);
++      } while (exception.retry);
++out:
++      return err;
++}
++
+ static int _nfs4_lookup_root(struct nfs_server *server, struct nfs_fh 
*fhandle,
+               struct nfs_fsinfo *info)
+ {
+@@ -7886,18 +7940,18 @@ int nfs4_proc_fs_locations(struct rpc_clnt *client, 
struct inode *dir,
+  * appended to this compound to identify the client ID which is
+  * performing recovery.
+  */
+-static int _nfs40_proc_get_locations(struct inode *inode,
++static int _nfs40_proc_get_locations(struct nfs_server *server,
++                                   struct nfs_fh *fhandle,
+                                    struct nfs4_fs_locations *locations,
+                                    struct page *page, const struct cred *cred)
+ {
+-      struct nfs_server *server = NFS_SERVER(inode);
+       struct rpc_clnt *clnt = server->client;
+       u32 bitmask[2] = {
+               [0] = FATTR4_WORD0_FSID | FATTR4_WORD0_FS_LOCATIONS,
+       };
+       struct nfs4_fs_locations_arg args = {
+               .clientid       = server->nfs_client->cl_clientid,
+-              .fh             = NFS_FH(inode),
++              .fh             = fhandle,
+               .page           = page,
+               .bitmask        = bitmask,
+               .migration      = 1,            /* skip LOOKUP */
+@@ -7943,17 +7997,17 @@ static int _nfs40_proc_get_locations(struct inode 
*inode,
+  * When the client supports GETATTR(fs_locations_info), it can
+  * be plumbed in here.
+  */
+-static int _nfs41_proc_get_locations(struct inode *inode,
++static int _nfs41_proc_get_locations(struct nfs_server *server,
++                                   struct nfs_fh *fhandle,
+                                    struct nfs4_fs_locations *locations,
+                                    struct page *page, const struct cred *cred)
+ {
+-      struct nfs_server *server = NFS_SERVER(inode);
+       struct rpc_clnt *clnt = server->client;
+       u32 bitmask[2] = {
+               [0] = FATTR4_WORD0_FSID | FATTR4_WORD0_FS_LOCATIONS,
+       };
+       struct nfs4_fs_locations_arg args = {
+-              .fh             = NFS_FH(inode),
++              .fh             = fhandle,
+               .page           = page,
+               .bitmask        = bitmask,
+               .migration      = 1,            /* skip LOOKUP */
+@@ -8002,11 +8056,11 @@ static int _nfs41_proc_get_locations(struct inode 
*inode,
+  * -NFS4ERR_LEASE_MOVED is returned if the server still has leases
+  * from this client that require migration recovery.
+  */
+-int nfs4_proc_get_locations(struct inode *inode,
++int nfs4_proc_get_locations(struct nfs_server *server,
++                          struct nfs_fh *fhandle,
+                           struct nfs4_fs_locations *locations,
+                           struct page *page, const struct cred *cred)
+ {
+-      struct nfs_server *server = NFS_SERVER(inode);
+       struct nfs_client *clp = server->nfs_client;
+       const struct nfs4_mig_recovery_ops *ops =
+                                       clp->cl_mvops->mig_recovery_ops;
+@@ -8019,10 +8073,11 @@ int nfs4_proc_get_locations(struct inode *inode,
+               (unsigned long long)server->fsid.major,
+               (unsigned long long)server->fsid.minor,
+               clp->cl_hostname);
+-      nfs_display_fhandle(NFS_FH(inode), __func__);
++      nfs_display_fhandle(fhandle, __func__);
+ 
+       do {
+-              status = ops->get_locations(inode, locations, page, cred);
++              status = ops->get_locations(server, fhandle, locations, page,
++                                          cred);
+               if (status != -NFS4ERR_DELAY)
+                       break;
+               nfs4_handle_exception(server, status, &exception);
+@@ -10516,6 +10571,7 @@ const struct nfs_rpc_ops nfs_v4_clientops = {
+       .free_client    = nfs4_free_client,
+       .create_server  = nfs4_create_server,
+       .clone_server   = nfs_clone_server,
++      .discover_trunking = nfs4_discover_trunking,
+ };
+ 
+ static const struct xattr_handler nfs4_xattr_nfs4_acl_handler = {
+diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c
+index acc1cd3e63a48..51f5cb41e87a4 100644
+--- a/fs/nfs/nfs4state.c
++++ b/fs/nfs/nfs4state.c
+@@ -2097,7 +2097,8 @@ static int nfs4_try_migration(struct nfs_server *server, 
const struct cred *cred
+       }
+ 
+       inode = d_inode(server->super->s_root);
+-      result = nfs4_proc_get_locations(inode, locations, page, cred);
++      result = nfs4_proc_get_locations(server, NFS_FH(inode), locations,
++                                       page, cred);
+       if (result) {
+               dprintk("<-- %s: failed to retrieve fs_locations: %d\n",
+                       __func__, result);
+diff --git a/include/linux/nfs_xdr.h b/include/linux/nfs_xdr.h
+index e9698b6278a52..ecd74cc347974 100644
+--- a/include/linux/nfs_xdr.h
++++ b/include/linux/nfs_xdr.h
+@@ -1805,6 +1805,7 @@ struct nfs_rpc_ops {
+       struct nfs_server *(*create_server)(struct fs_context *);
+       struct nfs_server *(*clone_server)(struct nfs_server *, struct nfs_fh *,
+                                          struct nfs_fattr *, 
rpc_authflavor_t);
++      int     (*discover_trunking)(struct nfs_server *, struct nfs_fh *);
+ };
+ 
+ /*
+-- 
+cgit 
+

Modified: PKGBUILD
===================================================================
--- PKGBUILD    2022-02-21 06:09:46 UTC (rev 437831)
+++ PKGBUILD    2022-02-21 06:49:04 UTC (rev 437832)
@@ -2,7 +2,7 @@
 
 pkgbase=linux-lts
 pkgver=5.15.24
-pkgrel=1
+pkgrel=2
 pkgdesc='LTS Linux'
 url="https://www.kernel.org/";
 arch=(x86_64)
@@ -22,6 +22,7 @@
   0004-cpufreq_intel_pstate_ITMT_support_for_overclocked_system.patch
   0005-Bluetooth_btintel_Fix_bdaddress_comparison_with_garbage_value.patch
   0006-lg-laptop_Recognize_more_models.patch
+  0007_fix_NFSv4_mount_regression.diff
 )
 validpgpkeys=(
   'ABAF11C65A2970B130ABE3C479BE3E4300411886'  # Linus Torvalds
@@ -36,7 +37,8 @@
             '7c7707c738983f3683d76295b496f578996b7341fa39ad334ec2833bfe4b966e'
             '420844779356286057d931e30bbe1dabb8ee52bff575845a8fbf3c34e1a1d29e'
             '3fa8a4af66d5a3b99b48ca979a247c61e81c9b2d3bcdffa9d3895a5532a420b4'
-            '79266c6cc970733fd35881d9a8f0a74c25c00b4d81741b8d4bba6827c48f7c78')
+            '79266c6cc970733fd35881d9a8f0a74c25c00b4d81741b8d4bba6827c48f7c78'
+            'e9527ad81d5b1821a7b17c56cb3abaec85785563f51e448cb3c06f1c68e2966f')
 
 export KBUILD_BUILD_HOST=archlinux
 export KBUILD_BUILD_USER=$pkgbase
@@ -45,6 +47,10 @@
 prepare() {
   cd $_srcname
 
+  # fix NFSv4 mounting issue regression - FS#73838 / FS#73860
+  # 
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/patch/?id=6f2836341d8a39e1e000572b10959347d7e61fd9
+  patch -Rp1 -i ../0007_fix_NFSv4_mount_regression.diff
+
   echo "Setting version..."
   scripts/setlocalversion --save-scmversion
   echo "-$pkgrel" > localversion.10-pkgrel

Reply via email to