On Wed, 2016-10-19 at 18:59 +0530, Shivaprasad G Bhat wrote: > virQEMUCapsSupportsChardev existing checks returns true > for spapr-vty alone. Instead verify spapr-vty validity > and let the logic to return true for other device types > so that virtio-console passes. > > The non-pseries machines dont have spapr-vio-bus. So, the > function always returned false for them before. > > Fixes - https://bugzilla.redhat.com/show_bug.cgi?id=1257813 > > Signed-off-by: Shivaprasad G Bhat <sbhat@xxxxxxxxxxxxxxxxxx> > --- > src/qemu/qemu_capabilities.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) I finally got around to review this, sorry for the delay. There is definitely some cleaning up to do in the part of libvirt your patch touches, but the fix works and doesn't affect other architectures so there's no point in holding it off any longer. ACK and pushed. -- Andrea Bolognani / Red Hat / Virtualization -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
