On Fri 24 Apr 2020 07:11:08 PM CEST, Eric Blake <address@hidden> wrote:
'write -c 0 64k' followed by 'write -z 16k 16k' would not need to do any
copy on write. The compressed data would remain untouched on disk but
some of the subclusters would have the 'all zeroes' bit set, exactly
like what happens with normal clusters.
It's a special case that avoids COW for write zeroes, but not for
anything else. The moment you write any data (whether to the
zero-above-compressed or the regular compressed portion), the entire
cluster has to be rewritten.
That's right but you can still write zeroes without having to rewrite
anything, and read back the zeroes without having to decompress the
data.
at the same time, I can see where you're coming from in stating that
if it makes management of extended L2 easier to allow zero subclusters
on top of a compressed cluster, then there's no reason to forbid it.
I'm not sure if it makes it easier. Some operations are definitely going
to be easier but maybe we have to add and handle _ZERO_COMPRESSED in
addition to _ZERO_PLAIN and _ZERO_ALLOC (the same for unallocated
subclusters). Or maybe replace QCow2SubclusterType with something
else. I need to evaluate that.