File 0013-mountd-Don-t-proactively-add-export-info-w... of Package nfs-utils

Overview Repositories Revisions Requests Users Attributes Meta

File 0013-mountd-Don-t-proactively-add-export-info-when-fh-inf.patch of Package nfs-utils (Revision f1367d66852d0ee408821ea2ee7d89d6)

Currently displaying revision f1367d66852d0ee408821ea2ee7d89d6 , Show latest

From 8f06f5025be898d8e3f8b29ab4ef3f7c1178221a Mon Sep 17 00:00:00 2001
From: NeilBrown <neil@brown.name>
Date: Mon, 15 Mar 2021 09:39:59 -0400
Subject: [PATCH] mountd: Don't proactively add export info when fh info is
 requested.

When an "nfsd.fh" request is received from the kernel, we map the
file-handle prefix to a path name and report that (as required) and then
also add "nfsd.export" information with export flags applicable to that
path.

This is not necessary and was added as a perceived optimisation.
When updating data already in the kernel, it is unlikely to help as the
kernel can be expected to ask for both details at much the same time.
With NFSv3, new information is normally added by a MOUNT rpc request, so
this is irrelevant.
With NFSv4, the kernel requests the "nfsd.export" information when
walking down from ROOT, *before* requesting the nfsd.fh information, so
this "optimisation" causes unnecessary work.

A future patch will add logging of authentication requests, and this
double-handling would result in extra unnecessary log messages.

As this "optimisation" appears to have no practical value and some
(small) cost, let's remove it.

Signed-off-by: NeilBrown <neil@brown.name>
Signed-off-by: Steve Dickson <steved@redhat.com>
---
 utils/mountd/cache.c |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

--- a/utils/mountd/cache.c
+++ b/utils/mountd/cache.c
@@ -62,7 +62,6 @@ enum nfsd_fsid {
  * Record is terminated with newline.
  *
  */
-static int cache_export_ent(char *buf, int buflen, char *domain, struct exportent *exp, char *path);
 
 #define INITIAL_MANAGED_GROUPS 100
 
@@ -790,7 +789,7 @@ static void nfsd_fh(int f)
 	    !is_mountpoint(found->e_mountpoint[0]?
 			   found->e_mountpoint:
 			   found->e_path)) {
-		/* Cannot export this yet 
+		/* Cannot export this yet
 		 * should log a warning, but need to rate limit
 		   xlog(L_WARNING, "%s not exported as %d not a mountpoint",
 		   found->e_path, found->e_mountpoint);
@@ -805,10 +804,6 @@ static void nfsd_fh(int f)
 		goto out;
 	}
 
-	if (found)
-		if (cache_export_ent(buf, sizeof(buf), dom, found, found_path) < 0)
-			found = 0;
-
 	bp = buf; blen = sizeof(buf);
 	qword_add(&bp, &blen, dom);
 	qword_addint(&bp, &blen, fsidtype);

Places

File 0013-mountd-Don-t-proactively-add-export-info-when-fh-inf.patch of Package nfs-utils (Revision f1367d66852d0ee408821ea2ee7d89d6)

Places