Thread (24 messages) 24 messages, 3 authors, 2025-02-05

Re: [PATCH v5 -next 11/16] sunrpc: simplify rpcauth_cache_shrink_count()

From: Joel Granados <joel.granados@kernel.org>
Date: 2025-01-17 09:55:19
Also in: linux-fsdevel, linux-mm, linux-nfs, linux-security-module, linux-sh, lkml

On Sat, Jan 11, 2025 at 03:07:46PM +0800, Kaixiong Yu wrote:
quoted hunk ↗ jump to hunk
It is inappropriate to use sysctl_vfs_cache_pressure here.
The sysctl is documented as: This percentage value controls
the tendency of the kernel to reclaim the memory which is used
for caching of directory and inode objects.

So, simplify result of rpcauth_cache_shrink_count() to
"return number_cred_unused;".

Signed-off-by: Kaixiong Yu <redacted>
Reviewed-by: Kees Cook <kees@kernel.org>
Acked-by: Anna Schumaker <redacted>
Acked-by: Jeff Layton <jlayton@kernel.org>
---
v4:
 - Simplify result of rpcauth_cache_shrink_count().
---
---
 net/sunrpc/auth.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/sunrpc/auth.c b/net/sunrpc/auth.c
index 04534ea537c8..5a827afd8e3b 100644
--- a/net/sunrpc/auth.c
+++ b/net/sunrpc/auth.c
@@ -489,7 +489,7 @@ static unsigned long
 rpcauth_cache_shrink_count(struct shrinker *shrink, struct shrink_control *sc)
 
 {
-	return number_cred_unused * sysctl_vfs_cache_pressure / 100;
+	return number_cred_unused;
This one is not related to the "moving sysctls out of kenrel/sysctl.c"
but I'll keep it here because of the Acks received.

 }
 
 static void
-- 
2.34.1
-- 

Joel Granados
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help