diff mbox series

x86: Add a kvm module parameter check for vmware_backdoors test

Message ID 157911811499.8078.14507140648862605986.stgit@naples-babu.amd.com (mailing list archive)
State New, archived
Headers show
Series x86: Add a kvm module parameter check for vmware_backdoors test | expand

Commit Message

Moger, Babu Jan. 15, 2020, 7:55 p.m. UTC
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(+)

Comments

Liran Alon Jan. 15, 2020, 10:16 p.m. UTC | #1
> 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
diff mbox series

Patch

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]