Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 9 Sep 2025 15:58:00 GMT
From:      Olivier Certner <olce@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 67f8bd5986a4 - main - nfs: newnfs_setroot(): Remove an obsolete comment
Message-ID:  <202509091558.589Fw0hR039325@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by olce:

URL: https://cgit.FreeBSD.org/src/commit/?id=67f8bd5986a42a173ef9082abfcd523b1a3ac056

commit 67f8bd5986a42a173ef9082abfcd523b1a3ac056
Author:     Olivier Certner <olce@FreeBSD.org>
AuthorDate: 2025-08-26 10:26:36 +0000
Commit:     Olivier Certner <olce@FreeBSD.org>
CommitDate: 2025-09-09 15:56:47 +0000

    nfs: newnfs_setroot(): Remove an obsolete comment
    
    This comment is obsolete, as:
    1. This code is FreeBSD-specific and is not shared with other BSDs.
    2. With our recent changes in commit be1f7435ef218b1d ("kern: start
       tracking cr_gid outside of cr_groups[]"), all of NetBSD, OpenBSD and
       FreeBSD have the effective GID in a separate field (DragonFlyBSD
       remains to this day an outlier).
    
    MFC after:      9 days
    Sponsored by:   The FreeBSD Foundation
    Differential Revision:  https://reviews.freebsd.org/D52254
---
 sys/fs/nfs/nfs_commonport.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/sys/fs/nfs/nfs_commonport.c b/sys/fs/nfs/nfs_commonport.c
index e5fdb395c9f7..862780741ee7 100644
--- a/sys/fs/nfs/nfs_commonport.c
+++ b/sys/fs/nfs/nfs_commonport.c
@@ -371,8 +371,6 @@ nfsrv_atroot(struct vnode *vp, uint64_t *retp)
 
 /*
  * Set the credentials to refer to root.
- * If only the various BSDen could agree on whether cr_gid is a separate
- * field or cr_groups[0]...
  */
 void
 newnfs_setroot(struct ucred *cred)



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