]> git.proxmox.com Git - mirror_qemu.git/commit - block/linux-aio.c
linux-aio: limit the batch size using `aio-max-batch` parameter
authorStefano Garzarella <sgarzare@redhat.com>
Wed, 21 Jul 2021 09:42:11 +0000 (11:42 +0200)
committerStefan Hajnoczi <stefanha@redhat.com>
Wed, 21 Jul 2021 12:47:50 +0000 (13:47 +0100)
commitd7ddd0a1618a75b31dc308bb37365ce1da972154
tree246183bcceead8342ab50bbd359c5cc284da534a
parent1793ad0247cad35db1ebbc04fbea0446c30a27ca
linux-aio: limit the batch size using `aio-max-batch` parameter

When there are multiple queues attached to the same AIO context,
some requests may experience high latency, since in the worst case
the AIO engine queue is only flushed when it is full (MAX_EVENTS) or
there are no more queues plugged.

Commit 2558cb8dd4 ("linux-aio: increasing MAX_EVENTS to a larger
hardcoded value") changed MAX_EVENTS from 128 to 1024, to increase
the number of in-flight requests. But this change also increased
the potential maximum batch to 1024 elements.

When there is a single queue attached to the AIO context, the issue
is mitigated from laio_io_unplug() that will flush the queue every
time is invoked since there can't be others queue plugged.

Let's use the new `aio-max-batch` IOThread parameter to mitigate
this issue, limiting the number of requests in a batch.

We also define a default value (32): this value is obtained running
some benchmarks and it represents a good tradeoff between the latency
increase while a request is queued and the cost of the io_submit(2)
system call.

Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
Message-id: 20210721094211.69853-4-sgarzare@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
block/linux-aio.c