[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v5 00/42] block: Deal with filters
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v5 00/42] block: Deal with filters |
Date: |
Thu, 13 Jun 2019 18:12:06 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 13.06.19 17:28, Vladimir Sementsov-Ogievskiy wrote:
> 13.06.2019 1:09, Max Reitz wrote:
>> Hi,
>>
>> When we introduced filters, we did it a bit casually. Sure, we talked a
>> lot about them before, but that was mostly discussion about where
>> implicit filters should be added to the graph (note that we currently
>> only have two implicit filters, those being mirror and commit). But in
>> the end, we really just designated some drivers filters (Quorum,
>> blkdebug, etc.) and added some specifically (throttle, COR), without
>> really looking through the block layer to see where issues might occur.
>>
>> It turns out vast areas of the block layer just don’t know about filters
>> and cannot really handle them. Many cases will work in practice, in
>> others, well, too bad, you cannot use some feature because some part
>> deep inside the block layer looks at your filters and thinks they are
>> format nodes.
>>
>> This is one reason why this series is needed. Over time (since v1), a
>> second reason has made its way in:
>>
>> bs->file is not necessarily the place where a node’s data is stored.
>> qcow2 now has external data files, and currently there is no way for the
>> general block layer to know that the data is not stored in bs->file.
>> Right now, I do not think that has any real consequences (all functions
>> that need access to the actual data storage file should only do so as a
>> fallback if the driver does not provide some functionality, but qcow2
>> should provide it all), but it still shows that we need some way to let
>> the general block layer know about such data files. (Also, I will need
>> this for v1 of my “Inquire images’ rotational info” series.)
>>
>> I won’t go on and on about this series now, I think the patches pretty
>> much speak for themselves now. If the cover letter gets too long,
>> nobody reads it anyway (see previous versions).
>>
>>
>> *** This series depends on some others. ***
>>
>> Dependencies:
>> - [PATCH 0/4] block: Keep track of parent quiescing
>> - [PATCH 0/2] vl: Drain before (block) job cancel when quitting
>> - [PATCH v2 0/2] blockdev: Overlays are not snapshots
>>
>> Based-on: <address@hidden>
>> Based-on: <address@hidden>
>> Based-on: <address@hidden>
>
> Could you please export a branch?
Sure:
https://git.xanclic.moe/XanClic/qemu child-access-functions-v5
Or:
https://github.com/XanClic/qemu child-access-functions-v5
(And the base branch is:
https://git.xanclic.moe/XanClic/qemu child-access-functions-base
https://github.com/XanClic/qemu child-access-functions-base
)
>> v5:
>> - Split the huge patches 2 and 3 from the previous version into many
>> smaller patches to maintain the potential reviewers’ sanity [Vladimir]
>
> Thank you! In spite of frightening amount of patches, reviewing became a lot
> simpler.
I had hoped making it exactly 42 patches would make it a bit more welcoming.
Again, thanks a lot for reviewing!
Max
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-block] [PATCH v5 34/42] block: Inline bdrv_co_block_status_from_*(), (continued)
- [Qemu-block] [PATCH v5 36/42] iotests: Add tests for mirror @replaces loops, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 38/42] iotests: Let complete_and_wait() work with commit, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 39/42] iotests: Add filter commit test cases, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 37/42] block: Leave BDS.backing_file constant, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 40/42] iotests: Add filter mirror test cases, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 41/42] iotests: Add test for commit in sub directory, Max Reitz, 2019/06/12
- [Qemu-block] [PATCH v5 42/42] iotests: Test committing to overridden backing, Max Reitz, 2019/06/12
- Re: [Qemu-block] [PATCH v5 00/42] block: Deal with filters, Vladimir Sementsov-Ogievskiy, 2019/06/13
- Re: [Qemu-block] [PATCH v5 00/42] block: Deal with filters,
Max Reitz <=