public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath
@ 2026-04-14 14:18 Jeff Layton
  2026-04-14 21:13 ` Chuck Lever
  2026-04-15  5:29 ` Christoph Hellwig
  0 siblings, 2 replies; 3+ messages in thread
From: Jeff Layton @ 2026-04-14 14:18 UTC (permalink / raw)
  To: Chuck Lever, NeilBrown, Olga Kornievskaia, Dai Ngo, Tom Talpey
  Cc: linux-nfs, linux-kernel, Christoph Hellwig, Jeff Layton

The rework of how block layouts were checked moved the check for
NFSEXP_PNFS out of nfsd4_setup_layout_type() and into the callers. That
patch didn't account for the new call in nfsd4_setup_layout_type().

Cc: Christoph Hellwig <hch@lst.de>
Fixes: f85460b2aa22 ("exportfs,nfsd: rework checking for layout-based block device access support")
Reported-by: Olga Kornievskaia <okorniev@redhat.com>
Signed-off-by: Jeff Layton <jlayton@kernel.org>
---
This patch needs to go into Chuck's nfsd-testing branch, ideally folded
into the patch in the Fixes: line. Otherwise, pnfs is enabled on every
xfs export discovered via netlink.
---
 fs/nfsd/export.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c
index e6a250576048..f562d383439b 100644
--- a/fs/nfsd/export.c
+++ b/fs/nfsd/export.c
@@ -1017,7 +1017,8 @@ static int nfsd_nl_parse_one_export(struct cache_detail *cd,
 			goto out_uuid;
 		err = 0;
 
-		nfsd4_setup_layout_type(&exp);
+		if (exp.ex_flags & NFSEXP_PNFS)
+			nfsd4_setup_layout_type(&exp);
 	}
 
 	expp = svc_export_lookup(&exp);

---
base-commit: b858b275543b35cb0fc2bf863da017de92ef192b
change-id: 20260414-pnfs-exp-fix-b1eb96d05634

Best regards,
-- 
Jeff Layton <jlayton@kernel.org>


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath
  2026-04-14 14:18 [PATCH] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath Jeff Layton
@ 2026-04-14 21:13 ` Chuck Lever
  2026-04-15  5:29 ` Christoph Hellwig
  1 sibling, 0 replies; 3+ messages in thread
From: Chuck Lever @ 2026-04-14 21:13 UTC (permalink / raw)
  To: NeilBrown, Olga Kornievskaia, Dai Ngo, Tom Talpey, Jeff Layton
  Cc: Chuck Lever, linux-nfs, linux-kernel, Christoph Hellwig

From: Chuck Lever <chuck.lever@oracle.com>

On Tue, 14 Apr 2026 07:18:05 -0700, Jeff Layton wrote:
> The rework of how block layouts were checked moved the check for
> NFSEXP_PNFS out of nfsd4_setup_layout_type() and into the callers. That
> patch didn't account for the new call in nfsd4_setup_layout_type().
> 
> 

Applied to nfsd-testing, thanks!

For the moment Christoph's series is still in my tree.

[1/1] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath
      commit: b8cefe5d302f351c252a833dd8c7e8f4a0cd5028

--
Chuck Lever <chuck.lever@oracle.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath
  2026-04-14 14:18 [PATCH] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath Jeff Layton
  2026-04-14 21:13 ` Chuck Lever
@ 2026-04-15  5:29 ` Christoph Hellwig
  1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2026-04-15  5:29 UTC (permalink / raw)
  To: Jeff Layton
  Cc: Chuck Lever, NeilBrown, Olga Kornievskaia, Dai Ngo, Tom Talpey,
	linux-nfs, linux-kernel, Christoph Hellwig

Looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-04-15  5:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-14 14:18 [PATCH] nfsd: fix handling of NFSEXP_PNFS in the netlink codepath Jeff Layton
2026-04-14 21:13 ` Chuck Lever
2026-04-15  5:29 ` Christoph Hellwig

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox