[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] qcow2: Add DEFAULT_L2_CACHE_CLUSTERS
|
From: |
Alberto Garcia |
|
Subject: |
Re: [Qemu-devel] [PATCH 3/3] qcow2: Add DEFAULT_L2_CACHE_CLUSTERS |
|
Date: |
Mon, 01 Jun 2015 19:16:03 +0200 |
|
User-agent: |
Notmuch/0.13.2 (http://notmuchmail.org) Emacs/23.2.1 (i486-pc-linux-gnu) |
On Mon 01 Jun 2015 06:09:19 PM CEST, Max Reitz wrote:
> If a relatively large cluster size is chosen, the default of 1 MB L2
> cache is not really appropriate. In this case, unless overridden by the
> user, the default cache size should not be determined by its size in
> bytes but by the number of L2 tables (clusters) it is supposed to
> contain.
>
> Note that without this patch, MIN_L2_CACHE_SIZE will effectively take
> over the same role. However, providing space for just two L2 tables is
> not enough to be the default.
>
> Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Alberto Garcia <address@hidden>
Berto