[libvirt] [PATCH v1 4/7] qemu: Use correct bus type for input devices

Farhan Ali alifm at linux.vnet.ibm.com
Fri Mar 16 19:08:41 UTC 2018



On 03/16/2018 09:39 AM, John Ferlan wrote:
> 
> 
> On 03/08/2018 11:07 AM, Farhan Ali wrote:
>> commit 7210cef452db 'qemu: build command line for virtio input devices'
>> introduced an error, by checking if input bus type is
>> VIR_DOMAIN_DISK_BUS_VIRTIO.
>>
>> Fix it by using the correct bus type for input devices.
>>
>> Signed-off-by: Farhan Ali <alifm at linux.vnet.ibm.com>
>> Reviewed-by: Boris Fiuczynski <fiuczy at linux.vnet.ibm.com>
>> ---
>>   src/qemu/qemu_domain_address.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
> 
> Nice catch!  Been around awhile too.  Doesn't look like there was a test
> to catch it either.

Hopefully my virtio ccw input tests should be enough to cover this?

> 
> Reviewed-by: John Ferlan <jferlan at redhat.com>
> 
> John
> 
> (I'll push this along with patch 1 since it's separable)
> 


Thanks so much for all your review, I really appreciate all your 
feeedback :)




More information about the libvir-list mailing list