[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] target/i386: kvm: print info when the kernel doesn't support
|
From: |
Philippe Mathieu-Daudé |
|
Subject: |
Re: [PATCH] target/i386: kvm: print info when the kernel doesn't support ioctl(KVM_CAP_GET_MSR_FEATURES) |
|
Date: |
Fri, 3 Jan 2020 15:53:24 +0100 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 |
On 1/3/20 3:32 PM, Li Qiang wrote:
The ioctl(KVM_CAP_GET_MSR_FEATURES) is quite new. In old platform that
Can you refer to the kernel version? "is quite new (introduced in vX.Y)"
doesn't support this ioctl will sometimes make the user confusion. For
example, when we do nested virtualiztion using host-passthrough model
"virtualization"
the VM will has quite different cpu feature with the host.
I'm not native English speaking but I'd use:
"... will have quite different features than the host"
Signed-off-by: Li Qiang <address@hidden>
---
target/i386/kvm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/i386/kvm.c b/target/i386/kvm.c
index 0b511906e3..9688f7a167 100644
--- a/target/i386/kvm.c
+++ b/target/i386/kvm.c
@@ -1916,6 +1916,8 @@ static int kvm_get_supported_feature_msrs(KVMState *s)
}
if (!kvm_check_extension(s, KVM_CAP_GET_MSR_FEATURES)) {
+ info_report("ioctl(KVM_CAP_GET_MSR_FEATURES) is "
+ "not supported by this kernel.");
Please remove the trailing '.'
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
return 0;
}