io_uring: remove empty cleanup of OP_OPEN* reqs
authorPavel Begunkov <asml.silence@gmail.com>
Wed, 15 Jul 2020 09:46:50 +0000 (12:46 +0300)
committerJens Axboe <axboe@kernel.dk>
Fri, 24 Jul 2020 19:00:44 +0000 (13:00 -0600)
A switch in __io_clean_op() doesn't have default, it's pointless to list
opcodes that doesn't do any cleanup. Remove IORING_OP_OPEN* from there.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/io_uring.c

index a06d5b9..8d6f1c4 100644 (file)
@@ -5473,9 +5473,6 @@ static void __io_clean_op(struct io_kiocb *req)
                if (req->flags & REQ_F_BUFFER_SELECTED)
                        kfree(req->sr_msg.kbuf);
                break;
-       case IORING_OP_OPENAT:
-       case IORING_OP_OPENAT2:
-               break;
        case IORING_OP_SPLICE:
        case IORING_OP_TEE:
                io_put_file(req, req->splice.file_in,