[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/5] qcow2: Set the default cache-clean-inter
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/5] qcow2: Set the default cache-clean-interval to 30 seconds |
Date: |
Wed, 08 Aug 2018 13:57:26 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 08 Aug 2018 01:49:08 PM CEST, Leonid Bloch wrote:
> On 08/08/2018 02:47 PM, Alberto Garcia wrote:
>> On Wed 08 Aug 2018 09:10:50 AM CEST, Leonid Bloch wrote:
>>> The default cache-clean-interval is set to 30 seconds, in order to lower
>>> the overhead of the qcow2 caches (before the default was 0, i.e.
>>> disabled).
>>>
>>> Signed-off-by: Leonid Bloch <address@hidden>
>>
>> Reviewed-by: Alberto Garcia <address@hidden>
>>
>>> +#define DEFAULT_CACHE_CLEAN_INTERVAL 30 /* seconds */
>>
>> I wonder if it shouldn't be a bit higher, though. Isn't 30 seconds too
>> low for a default?
>
> I also had this thought. What would you suggest?
I don't know, 5 minutes at least. It should discard cache entries that
haven't been used in a while.
It my opinion the default should be for saving memory in VMs that are
mostly idle. If the VM is more or less active I wouldn't want to clean
cache entries that are used frequently.
Berto
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, (continued)
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Alberto Garcia, 2018/08/08
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Leonid Bloch, 2018/08/08
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Alberto Garcia, 2018/08/08
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Leonid Bloch, 2018/08/08
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Alberto Garcia, 2018/08/08
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Leonid Bloch, 2018/08/08
- Re: [Qemu-devel] [PATCH v3 2/5] qcow2: Make the default L2 cache sufficient to cover the entire image, Alberto Garcia, 2018/08/08
[Qemu-devel] [PATCH v3 4/5] qcow2: Set the default cache-clean-interval to 30 seconds, Leonid Bloch, 2018/08/08
[Qemu-devel] [PATCH v3 3/5] qcow2: Resize the cache upon image resizing, Leonid Bloch, 2018/08/08
[Qemu-devel] [PATCH v3 5/5] qcow2: Explicit number replaced by a constant, Leonid Bloch, 2018/08/08