Re: [libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-07-02 Thread Jiri Denemark
On Fri, Jun 28, 2013 at 17:24:42 -0400, Laine Stump wrote: On 06/28/2013 11:04 AM, Jiri Denemark wrote: --- src/qemu/qemu_cgroup.c | 21 ++--- src/qemu/qemu_domain.c | 29 + src/qemu/qemu_domain.h | 2 ++ 3 files changed, 33 insertions(+),

Re: [libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-07-02 Thread Paolo Bonzini
Il 02/07/2013 08:34, Jiri Denemark ha scritto: I'm not sure if that's the right think to do or not but it's certainly better than before when memory locking limit completely ignore the need for VRAM and per-disk cache. Unfortunately, the original formula was suggested by Avi, who moved to new

Re: [libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-07-02 Thread Jiri Denemark
On Tue, Jul 02, 2013 at 12:34:44 +0200, Paolo Bonzini wrote: Il 02/07/2013 08:34, Jiri Denemark ha scritto: I'm not sure if that's the right think to do or not but it's certainly better than before when memory locking limit completely ignore the need for VRAM and per-disk cache.

Re: [libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-07-02 Thread Paolo Bonzini
Il 02/07/2013 13:40, Jiri Denemark ha scritto: On Tue, Jul 02, 2013 at 12:34:44 +0200, Paolo Bonzini wrote: Il 02/07/2013 08:34, Jiri Denemark ha scritto: I'm not sure if that's the right think to do or not but it's certainly better than before when memory locking limit completely ignore the

Re: [libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-07-02 Thread Alex Williamson
On Tue, 2013-07-02 at 13:51 +0200, Paolo Bonzini wrote: Il 02/07/2013 13:40, Jiri Denemark ha scritto: On Tue, Jul 02, 2013 at 12:34:44 +0200, Paolo Bonzini wrote: Il 02/07/2013 08:34, Jiri Denemark ha scritto: I'm not sure if that's the right think to do or not but it's certainly better

[libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-06-28 Thread Jiri Denemark
--- src/qemu/qemu_cgroup.c | 21 ++--- src/qemu/qemu_domain.c | 29 + src/qemu/qemu_domain.h | 2 ++ 3 files changed, 33 insertions(+), 19 deletions(-) diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c index 5f54ca6..22bf78e 100644 ---

Re: [libvirt] [PATCH 1/3] qemu: Move memory limit computation to a reusable function

2013-06-28 Thread Laine Stump
On 06/28/2013 11:04 AM, Jiri Denemark wrote: --- src/qemu/qemu_cgroup.c | 21 ++--- src/qemu/qemu_domain.c | 29 + src/qemu/qemu_domain.h | 2 ++ 3 files changed, 33 insertions(+), 19 deletions(-) diff --git a/src/qemu/qemu_cgroup.c