On 05/05/2015 02:03 PM, Laine Stump wrote: > The commandline generator for a virtio-serial device had a hardcoded > printf of "virtio-serial%d" as the id of the virtio-serial > controller. This patch changes it to use the alias of the controller > instead. Because the function that finds a controller alias requires a > pointer to the domainDef in order to get the list of controllers, the > arglist of a few functions had to have this added. > > Once this was done, the literal string QEMU_VIRTIO_SERIAL_PREFIX was > no longer needed, so it has been removed. > --- > src/qemu/qemu_command.c | 28 +++++++++++++++++----------- > src/qemu/qemu_command.h | 1 - > 2 files changed, 17 insertions(+), 12 deletions(-) > ACK - although a couple of notes (a cscope search on "bus=" finds : qemuUSBId - where there are decision points to use usb or usb%d qemuBuildSCSIHostdevDevStr - where bus=scsi%d is used Can/Should either or both of those be changed as well? John -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list