[libvirt] [PATCHv1 5/7] qemu_capabilities: Find QEMU binary for S390 arch

[Collin Walling]

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 at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
> 


-- 
Respectfully,
- Collin Walling