diff options
author | Jens Axboe <axboe@kernel.dk> | 2021-10-21 17:22:35 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2021-10-25 18:36:24 +0200 |
commit | 6b19b766e8f077f29cdb47da5003469a85bbfb9c (patch) | |
tree | 8ae1c68714c880a644ac05094491b74d86e9f590 /fs/aio.c | |
parent | usb: remove res2 argument from gadget code completions (diff) | |
download | linux-6b19b766e8f077f29cdb47da5003469a85bbfb9c.tar.xz linux-6b19b766e8f077f29cdb47da5003469a85bbfb9c.zip |
fs: get rid of the res2 iocb->ki_complete argument
The second argument was only used by the USB gadget code, yet everyone
pays the overhead of passing a zero to be passed into aio, where it
ends up being part of the aio res2 value.
Now that everybody is passing in zero, kill off the extra argument.
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/aio.c')
-rw-r--r-- | fs/aio.c | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -1417,7 +1417,7 @@ static void aio_remove_iocb(struct aio_kiocb *iocb) spin_unlock_irqrestore(&ctx->ctx_lock, flags); } -static void aio_complete_rw(struct kiocb *kiocb, long res, long res2) +static void aio_complete_rw(struct kiocb *kiocb, long res) { struct aio_kiocb *iocb = container_of(kiocb, struct aio_kiocb, rw); @@ -1437,7 +1437,7 @@ static void aio_complete_rw(struct kiocb *kiocb, long res, long res2) } iocb->ki_res.res = res; - iocb->ki_res.res2 = res2; + iocb->ki_res.res2 = 0; iocb_put(iocb); } @@ -1508,7 +1508,7 @@ static inline void aio_rw_done(struct kiocb *req, ssize_t ret) ret = -EINTR; fallthrough; default: - req->ki_complete(req, ret, 0); + req->ki_complete(req, ret); } } |