[PATCH v2 0/3] linux-aio: allow block devices to limit aio-max-batch

Stefano Garzarella posted 3 patches 2 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211026162346.253081-1-sgarzare@redhat.com
Maintainers: Hanna Reitz <hreitz@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>
qapi/block-core.json    |  7 +++++++
include/block/raw-aio.h |  6 ++++--
block/file-posix.c      | 14 ++++++++++++--
block/linux-aio.c       | 38 +++++++++++++++++++++++++++-----------
4 files changed, 50 insertions(+), 15 deletions(-)
[PATCH v2 0/3] linux-aio: allow block devices to limit aio-max-batch
Posted by Stefano Garzarella 2 years, 6 months ago
v1: https://lore.kernel.org/qemu-devel/20210923143100.182979-1-sgarzare@redhat.com
v2:
- @aio-max-batch documentation rewrite [Stefan, Kevin]
- added Stefan and Kevin R-b tags

Commit d7ddd0a161 ("linux-aio: limit the batch size using
`aio-max-batch` parameter") added a way to limit the batch size
of Linux AIO backend for the entire AIO context.

The same AIO context can be shared by multiple devices, so
latency-sensitive devices may want to limit the batch size even
more to avoid increasing latency.

This series add the `aio-max-batch` option to the file backend,
and use it in laio_co_submit() and laio_io_unplug() to limit the
Linux AIO batch size more than the limit set by the AIO context.

Stefano Garzarella (3):
  file-posix: add `aio-max-batch` option
  linux-aio: add `dev_max_batch` parameter to laio_co_submit()
  linux-aio: add `dev_max_batch` parameter to laio_io_unplug()

 qapi/block-core.json    |  7 +++++++
 include/block/raw-aio.h |  6 ++++--
 block/file-posix.c      | 14 ++++++++++++--
 block/linux-aio.c       | 38 +++++++++++++++++++++++++++-----------
 4 files changed, 50 insertions(+), 15 deletions(-)

-- 
2.31.1


Re: [PATCH v2 0/3] linux-aio: allow block devices to limit aio-max-batch
Posted by Kevin Wolf 2 years, 6 months ago
Am 26.10.2021 um 18:23 hat Stefano Garzarella geschrieben:
> v1: https://lore.kernel.org/qemu-devel/20210923143100.182979-1-sgarzare@redhat.com
> v2:
> - @aio-max-batch documentation rewrite [Stefan, Kevin]
> - added Stefan and Kevin R-b tags
> 
> Commit d7ddd0a161 ("linux-aio: limit the batch size using
> `aio-max-batch` parameter") added a way to limit the batch size
> of Linux AIO backend for the entire AIO context.
> 
> The same AIO context can be shared by multiple devices, so
> latency-sensitive devices may want to limit the batch size even
> more to avoid increasing latency.
> 
> This series add the `aio-max-batch` option to the file backend,
> and use it in laio_co_submit() and laio_io_unplug() to limit the
> Linux AIO batch size more than the limit set by the AIO context.

Thanks, applied to the block branch.

Kevin