[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v2] qemu: Set reasonable RSS limit on domain startup



On 08/03/2012 02:56 AM, Michal Privoznik wrote:
> If there's a memory leak in qemu or qemu is exploited the host's
> system will sooner or later start trashing instead of killing

s/trashing/thrashing/

> the bad process. This however has impact on performance and other
> guests as well. Therefore we should set a reasonable RSS limit
> even when user hasn't set any. It's better to be secure by default.
> ---
> 
> diff to v1:
> -the expression for reasonable limit adapted to VRAM + XBZRLE
>  (libvirt doesn't override migrate_cache_size yet, so the default
>   64MB should fit, and it did indeed during my testing)

Note that we aren't using XBZRLE migration yet (I am looking at what it
would take to implement that, though), but this should still be reasonable.

ACK.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]