On Sat, 2024-12-28 at 22:57 +0800, Kaixiong Yu wrote:
> 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 <yukaixiong@xxxxxxxxxx>
> Reviewed-by: Kees Cook <kees@xxxxxxxxxx>
> Acked-by: Anna Schumaker <anna.schumaker@xxxxxxxxxx>
> Acked-by: Jeff Layton <jlayton@xxxxxxxxxx>
> ---
> 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;
> }
>
> static void
No objection here, but this patch seems unrelated to the rest of the
series. Should this go in via Anna or Trond's tree?
Cheers,
--
Jeff Layton <jlayton@xxxxxxxxxx>
