NFSv4.2/pnfs: Don't use READ_PLUS with pNFS yet
authorTrond Myklebust <trond.myklebust@hammerspace.com>
Thu, 10 Dec 2020 14:34:34 +0000 (09:34 -0500)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Mon, 14 Dec 2020 11:51:08 +0000 (06:51 -0500)
We have no way of tracking server READ_PLUS support in pNFS for now, so
just disable it.

Reported-by: "Mkrtchyan, Tigran" <tigran.mkrtchyan@desy.de>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
fs/nfs/nfs4proc.c

index 7ab40d0e6a74ca9a5254e6f80bc33568490e8ead..61a07dcb963de1753a3afd45d25f38c7a29c5f28 100644 (file)
@@ -5320,17 +5320,17 @@ static int nfs4_read_done(struct rpc_task *task, struct nfs_pgio_header *hdr)
 }
 
 #ifdef CONFIG_NFS_V4_2
-static void nfs42_read_plus_support(struct nfs_server *server, struct rpc_message *msg)
+static void nfs42_read_plus_support(struct nfs_pgio_header *hdr,
+                                   struct rpc_message *msg)
 {
-       if (server->caps & NFS_CAP_READ_PLUS)
+       /* Note: We don't use READ_PLUS with pNFS yet */
+       if (nfs_server_capable(hdr->inode, NFS_CAP_READ_PLUS) && !hdr->ds_clp)
                msg->rpc_proc = &nfs4_procedures[NFSPROC4_CLNT_READ_PLUS];
-       else
-               msg->rpc_proc = &nfs4_procedures[NFSPROC4_CLNT_READ];
 }
 #else
-static void nfs42_read_plus_support(struct nfs_server *server, struct rpc_message *msg)
+static void nfs42_read_plus_support(struct nfs_pgio_header *hdr,
+                                   struct rpc_message *msg)
 {
-       msg->rpc_proc = &nfs4_procedures[NFSPROC4_CLNT_READ];
 }
 #endif /* CONFIG_NFS_V4_2 */
 
@@ -5340,7 +5340,8 @@ static void nfs4_proc_read_setup(struct nfs_pgio_header *hdr,
        hdr->timestamp   = jiffies;
        if (!hdr->pgio_done_cb)
                hdr->pgio_done_cb = nfs4_read_done_cb;
-       nfs42_read_plus_support(NFS_SERVER(hdr->inode), msg);
+       msg->rpc_proc = &nfs4_procedures[NFSPROC4_CLNT_READ];
+       nfs42_read_plus_support(hdr, msg);
        nfs4_init_sequence(&hdr->args.seq_args, &hdr->res.seq_res, 0, 0);
 }