[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v11 0/9] Take the image size into account when a
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH v11 0/9] Take the image size into account when allocating the L2 cache |
Date: |
Wed, 26 Sep 2018 12:57:55 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 26 Sep 2018 12:48:18 PM CEST, Kevin Wolf wrote:
> By the way, Berto: While reviewing this, I noticed that we don't make
> use of the l2-cache-entry-size feature yet by default. I think you
> said you wanted to do some more tests to find a good default
> (intuitively, I'd expect it around MIN(cluster size, 64k), maybe 128k,
> but of course this needs some benchmarking). Obviously, that's for a
> separate series, though.
Yes, I was thinking about that yesterday after reviewing this series
too... I don't know if I'll have a lot of time to test this, but I'll
try. MIN(cluster_size, 64k) should be a good default probably.
Berto
- [Qemu-devel] [PATCH v11 9/9] qcow2: Explicit number replaced by a constant, (continued)
- [Qemu-devel] [PATCH v11 9/9] qcow2: Explicit number replaced by a constant, Leonid Bloch, 2018/09/24
- [Qemu-devel] [PATCH v11 6/9] qcow2: Increase the default upper limit on the L2 cache size, Leonid Bloch, 2018/09/24
- [Qemu-devel] [PATCH v11 4/9] qcow2: Avoid duplication in setting the refcount cache size, Leonid Bloch, 2018/09/24
- [Qemu-devel] [PATCH v11 5/9] qcow2: Assign the L2 cache relatively to the image size, Leonid Bloch, 2018/09/24
- [Qemu-devel] [PATCH v11 3/9] qcow2: Make sizes more humanly readable, Leonid Bloch, 2018/09/24
- [Qemu-devel] [PATCH v11 2/9] include: Add a lookup table of sizes, Leonid Bloch, 2018/09/24
- [Qemu-devel] [PATCH v11 1/9] qcow2: Options' documentation fixes, Leonid Bloch, 2018/09/24
- Re: [Qemu-devel] [PATCH v11 0/9] Take the image size into account when allocating the L2 cache, Kevin Wolf, 2018/09/26
- Re: [Qemu-devel] [PATCH v11 0/9] Take the image size into account when allocating the L2 cache,
Alberto Garcia <=