• Pavel Begunkov's avatar
    io_uring: remove file batch-get optimisation · 62906e89
    Pavel Begunkov authored
    For requests with non-fixed files, instead of grabbing just one
    reference, we get by the number of left requests, so the following
    requests using the same file can take it without atomics.
    
    However, it's not all win. If there is one request in the middle
    not using files or having a fixed file, we'll need to put back the left
    references. Even worse if an application submits requests dealing with
    different files, it will do a put for each new request, so doubling the
    number of atomics needed. Also, even if not used, it's still takes some
    cycles in the submission path.
    
    If a file used many times, it rather makes sense to pre-register it, if
    not, we may fall in the described pitfall. So, this optimisation is a
    matter of use case. Go with the simpliest code-wise way, remove it.
    Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
    Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
    62906e89
io_uring.c 252 KB