[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 8/8] docs: document how to configure the qcow2 L
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH 8/8] docs: document how to configure the qcow2 L2/refcount caches |
Date: |
Mon, 11 May 2015 15:30:42 +0200 |
User-agent: |
Notmuch/0.13.2 (http://notmuchmail.org) Emacs/23.2.1 (i486-pc-linux-gnu) |
On Mon 11 May 2015 03:23:25 PM CEST, Max Reitz <address@hidden> 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
- [Qemu-devel] [PATCH v3 0/8] qcow2 L2/refcount cache improvements, Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 2/8] qcow2: simplify qcow2_cache_put() and qcow2_cache_entry_mark_dirty(), Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 3/8] qcow2: use an LRU algorithm to replace entries from the L2 cache, Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 5/8] qcow2: use a hash to look for entries in the L2 cache, Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 8/8] docs: document how to configure the qcow2 L2/refcount caches, Alberto Garcia, 2015/05/11
- Re: [Qemu-devel] [PATCH 8/8] docs: document how to configure the qcow2 L2/refcount caches, Eric Blake, 2015/05/11
- [Qemu-devel] [PATCH 7/8] qcow2: style fixes in qcow2-cache.c, Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 6/8] qcow2: make qcow2_cache_put() a void function, Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 1/8] qcow2: use one single memory block for the L2/refcount cache tables, Alberto Garcia, 2015/05/11
- [Qemu-devel] [PATCH 4/8] qcow2: remove qcow2_cache_find_entry_to_replace(), Alberto Garcia, 2015/05/11
- Re: [Qemu-devel] [PATCH v3 0/8] qcow2 L2/refcount cache improvements, Kevin Wolf, 2015/05/12