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

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 fo

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 vide

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 This is IMO a bugfi

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 * 1