[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/10] block: Make raw_co_get_allocated_file_size asynchro
From: |
Fabiano Rosas |
Subject: |
Re: [PATCH v2 00/10] block: Make raw_co_get_allocated_file_size asynchronous |
Date: |
Mon, 03 Jul 2023 10:55:34 -0300 |
Fabiano Rosas <farosas@suse.de> writes:
> Hi,
>
> The major change from the last version is that this time I'm moving
> all of the callers of bdrv_get_allocated_file_size() into
> coroutines. I had to make some temporary changes to avoid asserts
> while not all the callers were converted.
>
> I tried my best to explain why I think the changes are safe. To avoid
> changing too much of the code I added a change that removes the
> dependency of qmp_query_block from hmp_nbd_server_start, that way I
> don't need to move all of the nbd code into a coroutine as well.
>
> Based on:
> [PATCH v2 00/11] block: Re-enable the graph lock
> https://lore.kernel.org/r/20230605085711.21261-1-kwolf@redhat.com
>
> changes:
>
> - fixed duplicated commit message [Lin]
> - clarified why we need to convert info-block [Claudio]
> - added my rationale of why the changes are safe [Eric]
> - converted all callers to coroutines [Kevin]
> - made hmp_nbd_server_start don't depend on qmp_query_block
>
> CI run: https://gitlab.com/farosas/qemu/-/pipelines/895525156
Ping, this seems to have fallen through the cracks.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v2 00/10] block: Make raw_co_get_allocated_file_size asynchronous,
Fabiano Rosas <=