Re: [Xen-devel] [PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain

2018-01-29 Thread Ian Jackson
Roger Pau Monné writes ("Re: [Xen-devel] [PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain"): > On Fri, Jan 26, 2018 at 07:23:52PM +, Andrew Cooper wrote: ... > > I'm not the check is overly old to begin with.  We

Re: [Xen-devel] [PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain

2018-01-26 Thread Roger Pau Monné
On Fri, Jan 26, 2018 at 07:23:52PM +, Andrew Cooper wrote: > On 26/01/18 19:11, Ian Jackson wrote: > > Andrew Cooper writes ("[PATCH] tools/libxl: Fix assertion failure when > > trying to build a nested-virt PVH domain"): > >> xl: libxl.c:339: libxl_defbool_val: Assertion > >>

Re: [Xen-devel] [PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain

2018-01-26 Thread Andrew Cooper
On 26/01/18 19:11, Ian Jackson wrote: > Andrew Cooper writes ("[PATCH] tools/libxl: Fix assertion failure when trying > to build a nested-virt PVH domain"): >> xl: libxl.c:339: libxl_defbool_val: Assertion >> `!libxl_defbool_is_default(db)' failed. >> >> This happens because

Re: [Xen-devel] [PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain

2018-01-26 Thread Ian Jackson
Andrew Cooper writes ("[PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain"): > xl: libxl.c:339: libxl_defbool_val: Assertion `!libxl_defbool_is_default(db)' > failed. > > This happens because initiate_domain_create() checks for type != HVM, then > pokes at

[Xen-devel] [PATCH] tools/libxl: Fix assertion failure when trying to build a nested-virt PVH domain

2018-01-26 Thread Andrew Cooper
xl: libxl.c:339: libxl_defbool_val: Assertion `!libxl_defbool_is_default(db)' failed. This happens because initiate_domain_create() checks for type != HVM, then pokes at the hvm union. Check for == HVM instead so the union access is correctly guarded. Signed-off-by: Andrew Cooper