Re: [libvirt] [PATCH] tools: virt-host-validate: fix KVM check on s390
On 03/21/2016 07:55 AM, Bjoern Walk wrote:
Since kernel version 4.5, s390 has the 'sie' flag to declare
its
hardware virtualization support. Let's make virt-host-validate aware so
this check is passed correctly.
Signed-off-by: Bjoern Walk <bwalk(a)linux.vnet.ibm.com>
Reviewed-by: Boris Fiuczynski <fiuczy(a)linux.vnet.ibm.com>
---
tools/virt-host-validate-qemu.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
I've never seen the output of /proc/cpuinfo on an S390, but looking at
the source code I found it does seem to print with the "features" for a
cpuinfo on an s390 machine (as opposed the "flags" for a vmx/svm machine).
The virHostValidateHasCPUFlag doesn't really distinguish "flags" or
"features" field being specified as a label (although I imagine that
could have internationalization impact if we tried to limit that more).
Anyway, before I push this - I wanted to see if there were any "other"
thoughts regarding letting this be a bit more generic. My one concern is
some field name some day has "sie" added (e.g. "easier" or
"transient"
would match).
One change that may help with that would be to check "sie " instead of
"sie" (similarly "svm" could be "svm " and "vmx"
could be "vmx "). The
source code prints the output as "..."%s ", int_hwcap_str[i]..."
John
diff --git a/tools/virt-host-validate-qemu.c
b/tools/virt-host-validate-qemu.c
index a9f6c1e..01fb24b 100644
--- a/tools/virt-host-validate-qemu.c
+++ b/tools/virt-host-validate-qemu.c
@@ -31,7 +31,8 @@ int virHostValidateQEMU(void)
virHostMsgCheck("QEMU", "%s", ("for hardware
virtualization"));
if (virHostValidateHasCPUFlag("svm") ||
- virHostValidateHasCPUFlag("vmx")) {
+ virHostValidateHasCPUFlag("vmx") ||
+ virHostValidateHasCPUFlag("sie")) {
virHostMsgPass();
if (virHostValidateDeviceExists("QEMU", "/dev/kvm",
VIR_HOST_VALIDATE_FAIL,