Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-09 Thread Ian Jackson
Stefano Stabellini writes (Re: [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests): On Mon, 8 Jun 2015, Ian Jackson wrote: I think the right answer is probably to start all the qemus in parallel, and count (or otherwise record) the spawn outcomes. They are already started in parallel

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-09 Thread Ian Jackson
Stefano Stabellini writes (Re: [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests): That is of great help, but this code is quite complex and I don't understand it. I tried to complete the missing bits but I think I ended up with something quite far from what you had in mind. At least it

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-08 Thread Stefano Stabellini
On Mon, 8 Jun 2015, Ian Jackson wrote: Stefano Stabellini writes (Re: [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests): Do you have any suggestions on how I could fix this function for the purpose of this series? I would be quite happy to leave fixing qdisk_spawn_outcome in libxl.c

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-08 Thread Stefano Stabellini
On Sun, 7 Jun 2015, Wei Liu wrote: On Thu, Jun 04, 2015 at 12:28:19PM +0100, Stefano Stabellini wrote: Starts a second QEMU to provide PV backends in userspace to HVM guests. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- tools/libxl/libxl_create.c | 18

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-08 Thread Ian Jackson
Stefano Stabellini writes (Re: [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests): On Sun, 7 Jun 2015, Wei Liu wrote: Also it looks very similar to the function with the same name in libxl.c. I'm not sure why that doesn't propagate rc either... I think that's a bug. Right, I based

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-08 Thread Stefano Stabellini
On Mon, 8 Jun 2015, Ian Jackson wrote: Stefano Stabellini writes (Re: [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests): On Sun, 7 Jun 2015, Wei Liu wrote: Also it looks very similar to the function with the same name in libxl.c. I'm not sure why that doesn't propagate rc either...

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-08 Thread Ian Jackson
Stefano Stabellini writes (Re: [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests): Do you have any suggestions on how I could fix this function for the purpose of this series? I would be quite happy to leave fixing qdisk_spawn_outcome in libxl.c to you :-) Sure. TBH as I wrote earlier,

Re: [Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-07 Thread Wei Liu
On Thu, Jun 04, 2015 at 12:28:19PM +0100, Stefano Stabellini wrote: Starts a second QEMU to provide PV backends in userspace to HVM guests. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- tools/libxl/libxl_create.c | 18 ++ tools/libxl/libxl_dm.c

[Xen-devel] [PATCH v2 5/5] libxl: spawns two QEMUs for HVM guests

2015-06-04 Thread Stefano Stabellini
Starts a second QEMU to provide PV backends in userspace to HVM guests. Signed-off-by: Stefano Stabellini stefano.stabell...@eu.citrix.com --- tools/libxl/libxl_create.c | 18 ++ tools/libxl/libxl_dm.c |9 - 2 files changed, 26 insertions(+), 1 deletion(-) diff