[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v2 1/9] qapi/block-core: add option for io_u
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [RFC PATCH v2 1/9] qapi/block-core: add option for io_uring |
Date: |
Fri, 24 May 2019 18:17:03 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Stefan Hajnoczi <address@hidden> writes:
> On Fri, May 24, 2019 at 07:33:29PM +0530, Aarushi Mehta wrote:
>> Signed-off-by: Aarushi Mehta <address@hidden>
>> ---
>> qapi/block-core.json | 6 ++++--
>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/qapi/block-core.json b/qapi/block-core.json
>> index 7ccbfff9d0..0e927b247d 100644
>> --- a/qapi/block-core.json
>> +++ b/qapi/block-core.json
>> @@ -2776,11 +2776,13 @@
>> #
>> # @threads: Use qemu's thread pool
>> # @native: Use native AIO backend (only Linux and Windows)
>> +# @io_uring: Use linux io_uring (only Linux)
>> #
>> -# Since: 2.9
>> +# Since: 4.1
>
> Please leave "Since: 2.9" unchanged and mark only @io_uring "Since:
> 4.1". This way users can see that this specific option was added in
> 4.1 and the rest has been available since 2.9.
>
>> ##
>> { 'enum': 'BlockdevAioOptions',
>> - 'data': [ 'threads', 'native' ] }
>> + 'data': [ 'threads', 'native',
>> + { 'name': 'io_uring', 'if': 'defined(CONFIG_LINUX)' } ] }
>
> What about CONFIG_LINUX_IO_URING? That way the feature wouldn't be
> advertized on Linux systems that don't offer io_uring.
Yes, please.
Also, drop (only Linux) from the doc comment. As is, qemu-qmp-ref.txt
looks like this:
'io_uring'
Use linux io_uring (only Linux)
If: 'defined(CONFIG_LINUX)'
> Note that in order to do this you'll need to move the ./configure
> patches before this patch (or squash them into this patch).
- [Qemu-devel] [RFC PATCH v2 5/9] util/async: add aio interfaces for io_uring, (continued)
- [Qemu-devel] [RFC PATCH v2 5/9] util/async: add aio interfaces for io_uring, Aarushi Mehta, 2019/05/24
- [Qemu-devel] [RFC PATCH v2 4/9] stubs: add aio interface stubs for io_uring, Aarushi Mehta, 2019/05/24
- [Qemu-devel] [RFC PATCH v2 3/9] include/block: declare interfaces for io_uring, Aarushi Mehta, 2019/05/24
- [Qemu-devel] [RFC PATCH v2 8/9] block/file-posix: extends to use with io_uring, Aarushi Mehta, 2019/05/24
- [Qemu-devel] [RFC PATCH v2 1/9] qapi/block-core: add option for io_uring, Aarushi Mehta, 2019/05/24
- [Qemu-devel] [RFC PATCH v2 9/9] configure: permit use of io_uring, Aarushi Mehta, 2019/05/24
- [Qemu-devel] [RFC PATCH v2 2/9] block/block: add BDRV flag for io_uring, Aarushi Mehta, 2019/05/24