From: Max Reitz <mreitz@redhat.com>
To: Alberto Garcia <berto@igalia.com>, qemu-devel@nongnu.org
Cc: qemu-block@nongnu.org, Kevin Wolf <kwolf@redhat.com>,
Eric Blake <eblake@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v2 2/2] docs: Document the new default sizes of the qcow2 caches
Date: Mon, 16 Apr 2018 18:15:05 +0200 [thread overview]
Message-ID: <b6e01b00-8e5d-cbf4-b0c7-e6f7d65cd0cc@redhat.com> (raw)
In-Reply-To: <e8ecd7dd92900230c2027c75ea0be1aac264a676.1521016003.git.berto@igalia.com>
[-- Attachment #1: Type: text/plain, Size: 3218 bytes --]
On 2018-03-14 09:29, Alberto Garcia wrote:
> We have just reduced the refcount cache size to the minimum unless
> the user explicitly requests a larger one, so we have to update the
> documentation to reflect this change.
>
> Signed-off-by: Alberto Garcia <berto@igalia.com>
> Reviewed-by: Eric Blake <eblake@redhat.com>
> ---
> docs/qcow2-cache.txt | 31 ++++++++++++++-----------------
> 1 file changed, 14 insertions(+), 17 deletions(-)
>
> diff --git a/docs/qcow2-cache.txt b/docs/qcow2-cache.txt
> index 170191a242..c640d45d06 100644
> --- a/docs/qcow2-cache.txt
> +++ b/docs/qcow2-cache.txt
> @@ -116,31 +116,28 @@ There are three options available, and all of them take bytes:
> "refcount-cache-size": maximum size of the refcount block cache
> "cache-size": maximum size of both caches combined
>
> -There are two things that need to be taken into account:
> +There are a few things that need to be taken into account:
>
> - Both caches must have a size that is a multiple of the cluster size
> (or the cache entry size: see "Using smaller cache sizes" below).
>
> - - If you only set one of the options above, QEMU will automatically
> - adjust the others so that the L2 cache is 4 times bigger than the
> - refcount cache.
> + - The default L2 cache size is 8 clusters or 1MB (whichever is more),
> + and the minimum is 2 clusters (or 2 cache entries, see below).
>
> -This means that these options are equivalent:
> + - The default (and minimum) refcount cache size is 4 clusters.
>
> - -drive file=hd.qcow2,l2-cache-size=2097152
> - -drive file=hd.qcow2,refcount-cache-size=524288
> - -drive file=hd.qcow2,cache-size=2621440
> + - If only "cache-size" is specified then QEMU will assign as much
> + memory as possible to the L2 cache before increasing the refcount
> + cache size.
>
> -The reason for this 1/4 ratio is to ensure that both caches cover the
> -same amount of disk space. Note however that this is only valid with
> -the default value of refcount_bits (16). If you are using a different
> -value you might want to calculate both cache sizes yourself since QEMU
> -will always use the same 1/4 ratio.
> +Unlike L2 tables, refcount blocks are not used during normal I/O but
> +only during allocations and internal snapshots. In most cases they are
> +accessed sequentially (even during random guest I/O) so increasing the
> +refcount cache size won't have any measurable effect in performance.
Could you add a note that it can have a measurable effect when using
internal snapshots and the user may want to think about increasing its
size when making heavy use of them?
Max
> -It's also worth mentioning that there's no strict need for both caches
> -to cover the same amount of disk space. The refcount cache is used
> -much less often than the L2 cache, so it's perfectly reasonable to
> -keep it small.
> +Before QEMU 2.12 the refcount cache had a default size of 1/4 of the
> +L2 cache size. This resulted in unnecessarily large caches, so now the
> +refcount cache is as small as possible unless overridden by the user.
>
>
> Using smaller cache entries
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
prev parent reply other threads:[~2018-04-16 16:15 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-14 8:29 [Qemu-devel] [PATCH v2 0/2] Give the refcount cache the minimum possible size by default Alberto Garcia
2018-03-14 8:29 ` [Qemu-devel] [PATCH v2 1/2] qcow2: " Alberto Garcia
2018-04-13 15:00 ` Max Reitz
2018-04-16 13:56 ` Alberto Garcia
2018-04-16 14:05 ` Max Reitz
2018-04-16 14:31 ` Alberto Garcia
2018-04-16 16:14 ` Max Reitz
2018-03-14 8:29 ` [Qemu-devel] [PATCH v2 2/2] docs: Document the new default sizes of the qcow2 caches Alberto Garcia
2018-04-16 16:15 ` Max Reitz [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b6e01b00-8e5d-cbf4-b0c7-e6f7d65cd0cc@redhat.com \
--to=mreitz@redhat.com \
--cc=berto@igalia.com \
--cc=eblake@redhat.com \
--cc=kwolf@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).