]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ublk: remove "can't touch 'ublk_io' any more" comments
authorCaleb Sander Mateos <csander@purestorage.com>
Fri, 30 Jan 2026 15:25:30 +0000 (08:25 -0700)
committerJens Axboe <axboe@kernel.dk>
Sat, 31 Jan 2026 13:38:43 +0000 (06:38 -0700)
The struct ublk_io is in fact accessed in __ublk_complete_rq() after the
comment. But it's not racy to access the ublk_io between clearing its
UBLK_IO_FLAG_OWNED_BY_SRV flag and completing the request, as no other
thread can use the ublk_io in the meantime.

Signed-off-by: Caleb Sander Mateos <csander@purestorage.com>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/ublk_drv.c

index 4fe754e7d1e8b6fd2339209cebe289ff4dfeed48..3c918db4905c7c8a7ff3afc9a53703b4b3aeb099 100644 (file)
@@ -3349,7 +3349,6 @@ static int ublk_ch_uring_cmd_local(struct io_uring_cmd *cmd,
                        io_buffer_unregister_bvec(cmd, buf_idx, issue_flags);
                compl = ublk_need_complete_req(ub, io);
 
-               /* can't touch 'ublk_io' any more */
                if (req_op(req) == REQ_OP_ZONE_APPEND)
                        req->__sector = addr;
                if (compl)
@@ -3681,7 +3680,6 @@ static int ublk_batch_commit_io(struct ublk_queue *ubq,
                return ret;
        }
 
-       /* can't touch 'ublk_io' any more */
        if (buf_idx != UBLK_INVALID_BUF_IDX)
                io_buffer_unregister_bvec(data->cmd, buf_idx, data->issue_flags);
        if (req_op(req) == REQ_OP_ZONE_APPEND)