On Mon 11 May 2015 03:23:25 PM CEST, Max Reitz <mre...@redhat.com> wrote:

>> +   disk_size = l2_cache_size * cluster_size / 8
>> +   disk_size = refcount_cache_size * cluster_size / 2
>
> Only with the default of refcount_bits=16. In the general case, it's
> refcount_cache_size * cluster_size * 8 / refcount_bits.

I actually omitted that on purpose because I didn't want to go into too
many details and I hadn't realized that it's now possible to configure
refcount_bits. I will correct the document.

I wonder what happens then to this DEFAULT_L2_REFCOUNT_SIZE_RATIO of 4,
because that's only valid if refcount_bits == 16.

Is the user supposed to calculate that ratio manually and configure both
cache sizes?

> Apart from this minor thing, looks good. Well, except that maybe it
> should be preferred to use .qcow2 as the extension for qcow2 files
> (instead of .img).

Sure, I can change that too.

Berto

Reply via email to