On 08/15/2016 07:28 AM, Pavel Hrdina wrote: > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1364843 > Similar to 4/5 a few extra words here would help so one doesn't have to chase into the bz to attempt to figure out what's being fixed and why moving the Setup earlier does what you want. ACK series - please do add a few more commit words though John > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx> > --- > src/qemu/qemu_process.c | 10 +++++++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c > index b3c6400..2b857bb 100644 > --- a/src/qemu/qemu_process.c > +++ b/src/qemu/qemu_process.c > @@ -4213,6 +4213,13 @@ qemuProcessSetupGraphics(virQEMUDriverPtr driver, > size_t i; > int ret = -1; > > + for (i = 0; i < vm->def->ngraphics; i++) { > + graphics = vm->def->graphics[i]; > + > + if (qemuProcessGraphicsSetupListen(driver, graphics, vm) < 0) > + goto cleanup; > + } > + > if (allocate) { > for (i = 0; i < vm->def->ngraphics; i++) { > graphics = vm->def->graphics[i]; > @@ -4227,9 +4234,6 @@ qemuProcessSetupGraphics(virQEMUDriverPtr driver, > > if (qemuProcessGraphicsAllocatePorts(driver, graphics, allocate) < 0) > goto cleanup; > - > - if (qemuProcessGraphicsSetupListen(driver, graphics, vm) < 0) > - goto cleanup; > } > > ret = 0; > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list