From: Aurelien Degremont Date: Mon, 2 Jan 2023 16:04:31 +0000 (+0000) Subject: LU-16438 llite: remove false outdated comment X-Git-Tag: 2.15.54~122 X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=557bb0004d7ea1d877a44cccdb3d72b9437e47c5;p=fs%2Flustre-release.git LU-16438 llite: remove false outdated comment Old commit 99727c7a1 from Lustre 2.6 changed ll_i2gids() behavior without updating the function documentation accordingly. Fix it as this is confusing. Test-Parameters: trivial Fixes: 99727c7 ("LU-4476 kernel: support process namespace containers") Change-Id: Iccc50fe6ac9e02de9bae7fd8f91e3e73ff45e327 Signed-off-by: Aurelien Degremont Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/49539 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Andreas Dilger Reviewed-by: Sebastien Buisson Reviewed-by: Oleg Drokin --- diff --git a/lustre/llite/namei.c b/lustre/llite/namei.c index 24cb5a8..9d2e37e 100644 --- a/lustre/llite/namei.c +++ b/lustre/llite/namei.c @@ -534,10 +534,7 @@ __u32 ll_i2suppgid(struct inode *i) return (__u32) __kgid_val(INVALID_GID); } -/* Pack the required supplementary groups into the supplied groups array. - * If we don't need to use the groups from the target inode(s) then we - * instead pack one or more groups from the user's supplementary group - * array in case it might be useful. Not needed if doing an MDS-side upcall. */ +/* Pack the required supplementary groups into the supplied groups array. */ void ll_i2gids(__u32 *suppgids, struct inode *i1, struct inode *i2) { LASSERT(i1 != NULL);