> Am 06.07.2021 um 15:19 schrieb Kevin Wolf <kw...@redhat.com>:
>
> Am 02.07.2021 um 19:23 hat Ilya Dryomov geschrieben:
>> This series migrates the qemu rbd driver from the old aio emulation
>> to native coroutines and adds write zeroes support which is important
>> for block operations.
>>
>> To achieve this we first bump the librbd requirement to the already
>> outdated luminous release of ceph to get rid of some wrappers and
>> ifdef'ry in the code.
>
> Thanks, applied to the block branch.
>
> I've only had a very quick look at the patches, but I think there is one
> suggestion for a cleanup I can make: The qemu_rbd_finish_bh()
> indirection is probably unnecessary now because aio_co_wake() is thread
> safe.
But this is new, isn’t it?
We also have this indirection in iscsi and nfs drivers I think.
Does it matter that the completion callback is called from an librbd thread?
Will the coroutine continue to run in the right thread?
I will have a decent look after my vacation.
Anyway, Thanks for applying,
Peter
>
> (Also, if I were the responsible maintainer, I would prefer true/false
> rather than 0/1 for bools, but that's minor. :-))
>
> Kevin
>