[PATCH] x86: Add a kvm module parameter check for vmware_backdoors test

[PATCH] x86: Add a kvm module parameter check for vmware_backdoors test

[Babu Moger]

vmware_backdoors test fails if the kvm module parameter
enable_vmware_backdoor is not set to Y. Add a check before
running the test.

Suggested-by: Huang2, Wei <Wei.Huang2@amd.com>
Signed-off-by: Babu Moger <babu.moger@amd.com>
---
 x86/unittests.cfg |    1 +
 1 file changed, 1 insertion(+)

diff --git a/x86/unittests.cfg b/x86/unittests.cfg
index 51e4ba5..aae1523 100644
--- a/x86/unittests.cfg
+++ b/x86/unittests.cfg
@@ -164,6 +164,7 @@ check = /proc/sys/kernel/nmi_watchdog=0
 [vmware_backdoors]
 file = vmware_backdoors.flat
 extra_params = -machine vmport=on -cpu host
+check = /sys/module/kvm/parameters/enable_vmware_backdoor=Y
 arch = x86_64
 
 [port80]

Re: [PATCH] x86: Add a kvm module parameter check for vmware_backdoors test

[Liran Alon]

> On 15 Jan 2020, at 21:55, Babu Moger <babu.moger@amd.com> wrote:
> 
> vmware_backdoors test fails if the kvm module parameter
> enable_vmware_backdoor is not set to Y. Add a check before
> running the test.
> 
> Suggested-by: Huang2, Wei <Wei.Huang2@amd.com>
> Signed-off-by: Babu Moger <babu.moger@amd.com>
> ---
> x86/unittests.cfg |    1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/x86/unittests.cfg b/x86/unittests.cfg
> index 51e4ba5..aae1523 100644
> --- a/x86/unittests.cfg
> +++ b/x86/unittests.cfg
> @@ -164,6 +164,7 @@ check = /proc/sys/kernel/nmi_watchdog=0
> [vmware_backdoors]
> file = vmware_backdoors.flat
> extra_params = -machine vmport=on -cpu host
> +check = /sys/module/kvm/parameters/enable_vmware_backdoor=Y
> arch = x86_64
> 
> [port80]
> 

Nice!
I didn’t notice there is a “check” feature when I first introduced vmware_backdoor tests.
Sorry for that.

Reviewed-by: Liran Alon <liran.alon@oracle.com>

-Liran