Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 23 Aug 2025 21:23:42 GMT
From:      Rick Macklem <rmacklem@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 248daca0b3c3 - stable/14 - nfscl: Fix GetACL so that it does not want a reply cached
Message-ID:  <202508232123.57NLNg3h097851@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/14 has been updated by rmacklem:

URL: https://cgit.FreeBSD.org/src/commit/?id=248daca0b3c3bd411a4629047e21cc03532890c0

commit 248daca0b3c3bd411a4629047e21cc03532890c0
Author:     Rick Macklem <rmacklem@FreeBSD.org>
AuthorDate: 2025-08-04 23:36:28 +0000
Commit:     Rick Macklem <rmacklem@FreeBSD.org>
CommitDate: 2025-08-23 21:21:02 +0000

    nfscl: Fix GetACL so that it does not want a reply cached
    
    Without this patch, an NFSv4.1/4.2 GetACL RPC requests that
    the session cache the reply.  In some cases, the reply may
    be too large to cache, resulting in a NFS4ERR_X
    error from the server.
    
    Since a GetACL is idempotent, disable reply caching for it,
    by setting that it can generate a large reply.
    Tested against a Linux server with a large ACL on a file.
    
    (cherry picked from commit 51affb7e97161e260f6d99cb0904daf7cc9f51a7)
---
 sys/fs/nfs/nfs_commonsubs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/fs/nfs/nfs_commonsubs.c b/sys/fs/nfs/nfs_commonsubs.c
index 560720911e84..bde9be0885e0 100644
--- a/sys/fs/nfs/nfs_commonsubs.c
+++ b/sys/fs/nfs/nfs_commonsubs.c
@@ -218,7 +218,7 @@ NFSD_VNET_DEFINE_STATIC(u_char *, nfsrv_dnsname) = NULL;
  */
 static int nfs_bigreply[NFSV42_NPROCS] = { 0, 0, 0, 1, 0, 1, 1, 0, 0, 0, 0,
     0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
-    0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0,
+    0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 0,
     1, 0, 0, 1, 0, 0, 0, 0, 0 };
 
 /* local functions */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202508232123.57NLNg3h097851>