NFSD: replace __get_free_page() with kmalloc() in nfsd_buffered_readdir()

nfsd_buffered_readdir() allocates a staging buffer with __get_free_page().

kmalloc() is a better API for such use and it also provides better
scalability and more debugging possibilities.

Replace use of __get_free_page() with kmalloc().

Signed-off-by: Mike Rapoport (Microsoft) <rppt@kernel.org>
Link: https://patch.msgid.link/20260523-b4-fs-v1-7-275e36a83f0e@kernel.org
Acked-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>

diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
index eafdf7b7890fdee55ddf802d040363f33a7be628..c99e54b23cd9bbd1d8a122d7918b0dd38cd87528 100644 (file)
--- a/fs/nfsd/vfs.c
+++ b/fs/nfsd/vfs.c
@@ -2407,7 +2407,7 @@ static __be32 nfsd_buffered_readdir(struct file *file, struct svc_fh *fhp,
        loff_t offset;
        struct readdir_data buf = {
                .ctx.actor = nfsd_buffered_filldir,
-               .dirent = (void *)__get_free_page(GFP_KERNEL)
+               .dirent = kmalloc(PAGE_SIZE, GFP_KERNEL)
        };
 
        if (!buf.dirent)
@@ -2458,7 +2458,7 @@ static __be32 nfsd_buffered_readdir(struct file *file, struct svc_fh *fhp,
                offset = vfs_llseek(file, 0, SEEK_CUR);
        }
 
-       free_page((unsigned long)(buf.dirent));
+       kfree((buf.dirent));
 
        if (host_err)
                return nfserrno(host_err);