[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 1/6] Stream block job involves copy-on-read f
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v5 1/6] Stream block job involves copy-on-read filter driver |
Date: |
Wed, 9 Jan 2019 14:13:54 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 08.01.19 14:45, Vladimir Sementsov-Ogievskiy wrote:
> 30.12.2018 23:09, Andrey Shinkevich wrote:
>> The copy-on-read filter is applied to block-stream operation.
>> It is necessary for further block discard option.
>
> If we move to c-o-r filter in stream, I think we should:
> 1. get rid of COPY_ON_READ flag in stream code (so, this thing to be done in
> one place)
> 2. add base parameter to c-o-r filter, to not copy things that are below base.
>
> Current behavior of stream is enabling copy-on-read on bs during stream if
> base == NULL.
> So, if we just add c-o-r filter, we will copy extra blocks. On the other
> hand, if we
> support base parameter in c-o-r filter we will involve guest reads to the
> stream
> process for cases with base != NULL, so stream operation should become more
> efficient.
>
> So for me the following sequence seems reasonable:
>
> - prepare iotests for graph changes
> - add base parameter to c-o-r filter
> - use c-o-r in stream and drop COPY_ON_READ flag from stream code
>
> then (may be in separate series) add discard functionality. And looks like
> again, it should
> be property of c-o-r filter, not stream job itself, to involve guest reads to
> discarding too.
>
> In general, Max, is that correspond to what you mean?
Yep, that sounds good to me.
Max
signature.asc
Description: OpenPGP digital signature