From: Trond Myklebust Date: Sat, 3 Jan 2026 17:15:00 +0000 (-0500) Subject: NFS/localio: Cleanup the nfs_local_pgio_done() parameters X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5fcd95831d9751435e3617e870d706cb9070b55a;p=thirdparty%2Fkernel%2Flinux.git NFS/localio: Cleanup the nfs_local_pgio_done() parameters Remove the redundant 'force' parameter. Signed-off-by: Trond Myklebust Signed-off-by: Anna Schumaker --- diff --git a/fs/nfs/localio.c b/fs/nfs/localio.c index 00bbac6c9fe40..7f5e7f0e3e00d 100644 --- a/fs/nfs/localio.c +++ b/fs/nfs/localio.c @@ -517,8 +517,7 @@ nfs_local_pgio_init(struct nfs_pgio_header *hdr, hdr->task.tk_start = ktime_get(); } -static bool -nfs_local_pgio_done(struct nfs_local_kiocb *iocb, long status, bool force) +static bool nfs_local_pgio_done(struct nfs_local_kiocb *iocb, long status) { struct nfs_pgio_header *hdr = iocb->hdr; @@ -533,9 +532,6 @@ nfs_local_pgio_done(struct nfs_local_kiocb *iocb, long status, bool force) hdr->task.tk_status = status; } - if (force) - return true; - BUG_ON(atomic_read(&iocb->n_iters) <= 0); return atomic_dec_and_test(&iocb->n_iters); } @@ -651,7 +647,7 @@ static void nfs_local_read_aio_complete(struct kiocb *kiocb, long ret) container_of(kiocb, struct nfs_local_kiocb, kiocb); /* AIO completion of DIO read should always be last to complete */ - if (unlikely(!nfs_local_pgio_done(iocb, ret, false))) + if (unlikely(!nfs_local_pgio_done(iocb, ret))) return; nfs_local_pgio_aio_complete(iocb); /* Calls nfs_local_read_aio_complete_work */ @@ -683,7 +679,7 @@ static void nfs_local_call_read(struct work_struct *work) if (status == -EIOCBQUEUED) continue; /* Break on completion, errors, or short reads */ - if (nfs_local_pgio_done(iocb, status, false) || status < 0 || + if (nfs_local_pgio_done(iocb, status) || status < 0 || (size_t)status < iov_iter_count(&iocb->iters[i])) { nfs_local_read_iocb_done(iocb); break; @@ -840,7 +836,7 @@ static void nfs_local_write_aio_complete(struct kiocb *kiocb, long ret) container_of(kiocb, struct nfs_local_kiocb, kiocb); /* AIO completion of DIO write should always be last to complete */ - if (unlikely(!nfs_local_pgio_done(iocb, ret, false))) + if (unlikely(!nfs_local_pgio_done(iocb, ret))) return; nfs_local_pgio_aio_complete(iocb); /* Calls nfs_local_write_aio_complete_work */ @@ -876,7 +872,7 @@ static void nfs_local_call_write(struct work_struct *work) if (status == -EIOCBQUEUED) continue; /* Break on completion, errors, or short writes */ - if (nfs_local_pgio_done(iocb, status, false) || status < 0 || + if (nfs_local_pgio_done(iocb, status) || status < 0 || (size_t)status < iov_iter_count(&iocb->iters[i])) { nfs_local_write_iocb_done(iocb); break;