Re: [Xen-devel] [PATCH V2 1/1] libxl: set stub domain size based on VRAM size

2015-07-13 Thread Ian Jackson
Ian Jackson writes (Re: [PATCH V2 1/1] libxl: set stub domain size based on VRAM size): Eric Shelton writes ([PATCH V2 1/1] libxl: set stub domain size based on VRAM size): Allocate additional memory to the stub domain for qemu-traditional if more than 4 MB is assigned to the video adapter

Re: [Xen-devel] [PATCH V2 1/1] libxl: set stub domain size based on VRAM size

2015-07-13 Thread Ian Campbell
On Mon, 2015-07-13 at 11:22 +0100, Ian Jackson wrote: Ian Jackson writes (Re: [PATCH V2 1/1] libxl: set stub domain size based on VRAM size): Eric Shelton writes ([PATCH V2 1/1] libxl: set stub domain size based on VRAM size): Allocate additional memory to the stub domain for

Re: [Xen-devel] [PATCH V2 1/1] libxl: set stub domain size based on VRAM size

2015-07-13 Thread Ian Jackson
Ian Campbell writes (Re: [PATCH V2 1/1] libxl: set stub domain size based on VRAM size): On Mon, 2015-07-13 at 11:22 +0100, Ian Jackson wrote: In a moment I am going to send a v3 which uses max_t and uint64_t (which is the type of the memkb fields and also obviously correct). Eric already

Re: [Xen-devel] [PATCH V2 1/1] libxl: set stub domain size based on VRAM size

2015-07-13 Thread Ian Jackson
Eric Shelton writes ([PATCH V2 1/1] libxl: set stub domain size based on VRAM size): Allocate additional memory to the stub domain for qemu-traditional if more than 4 MB is assigned to the video adapter to avoid out of memory condition for QEMU. Acked-by: Ian Jackson ian.jack...@eu.citrix.com

[Xen-devel] [PATCH V2 1/1] libxl: set stub domain size based on VRAM size

2015-07-10 Thread Eric Shelton
Allocate additional memory to the stub domain for qemu-traditional if more than 4 MB is assigned to the video adapter to avoid out of memory condition for QEMU. Signed-off-by: Eric Shelton eshel...@pobox.com --- tools/libxl/libxl_dm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

Re: [Xen-devel] [PATCH V2 1/1] libxl: set stub domain size based on VRAM size

2015-07-10 Thread Samuel Thibault
Eric Shelton, le Fri 10 Jul 2015 18:14:32 -0400, a écrit : -dm_config-b_info.max_memkb = 32 * 1024; +dm_config-b_info.max_memkb = 28 * 1024 + +max(guest_config-b_info.video_memkb, 4096); I'm actually wondering whether just using +dm_config-b_info.max_memkb = 28 * 1024 +