On 05/17/2018 04:41 PM, Jiri Denemark wrote:
> On Sat, May 05, 2018 at 12:48:47 -0500, Chris Venteicher wrote:
>> S390 uses qemu-kvm in /usr/libexec.
> 
> That's incorrect, /usr/libexec/qemu-kvm is the native binary on
> RHEL/CentOS. That is it's arch matches host arch.
> 
>> ---
>>  src/qemu/qemu_capabilities.c | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
>> index 9ffbf91a90..ac7569679c 100644
>> --- a/src/qemu/qemu_capabilities.c
>> +++ b/src/qemu/qemu_capabilities.c
>> @@ -687,6 +687,11 @@ virQEMUCapsFindBinaryForArch(virArch hostarch,
>>              goto out;
>>      }
>>  
>> +    /* Fourth attempt: try 'qemu-kvm' */
>> +    if ((ret = virQEMUCapsFindBinary("%s", "/usr/libexec/qemu-kvm")) != 
>> NULL)
>> +        goto out;
>> +
>> +
>>   out:
>>      return ret;
>>  }
> 
> NACK, this is handled elsewhere.

This is going to sound silly since the string "redhat" is found everywhere in 
these
emails, but do you think it's worthwhile to also add the ability to probe the 
other 
native binaries that are used by other distros?

For example (and I believe is why you mention "this is handled elsewhere") 
virQEMUCapsInitGuest (src/qemu/qemu_capabilities.c) will check for the 
different "kvmbins" 
for RHEL, Fedora, and Debian/Ubuntu. A little bit of work would allow us to 
reuse this to 
get the appropriate qemu caps for the other distros.

A "virQEMUCapsFindNativeBinary" or something like that comes to mind.

Just a thought.

> 
> --
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
> 


-- 
Respectfully,
- Collin Walling

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to