diff mbox series

KVM: nVMX: Use semi-colon instead of comma for exit-handlers initialization

Message ID 20191118191121.43440-1-liran.alon@oracle.com (mailing list archive)
State New, archived
Headers show
Series KVM: nVMX: Use semi-colon instead of comma for exit-handlers initialization | expand

Commit Message

Liran Alon Nov. 18, 2019, 7:11 p.m. UTC
Reviewed-by: Mark Kanda <mark.kanda@oracle.com>
Signed-off-by: Liran Alon <liran.alon@oracle.com>
---
 arch/x86/kvm/vmx/nested.c | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

Comments

Jim Mattson Nov. 18, 2019, 7:23 p.m. UTC | #1
On Mon, Nov 18, 2019 at 11:11 AM Liran Alon <liran.alon@oracle.com> wrote:
>
> Reviewed-by: Mark Kanda <mark.kanda@oracle.com>
> Signed-off-by: Liran Alon <liran.alon@oracle.com>
Reviewed-by: Jim Mattson <jmattson@google.com>
Paolo Bonzini Nov. 18, 2019, 10:09 p.m. UTC | #2
On 18/11/19 20:11, Liran Alon wrote:
> Reviewed-by: Mark Kanda <mark.kanda@oracle.com>
> Signed-off-by: Liran Alon <liran.alon@oracle.com>
> ---
>  arch/x86/kvm/vmx/nested.c | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c
> index bc11fcbbe12b..229ca7164318 100644
> --- a/arch/x86/kvm/vmx/nested.c
> +++ b/arch/x86/kvm/vmx/nested.c
> @@ -6051,23 +6051,23 @@ __init int nested_vmx_hardware_setup(int (*exit_handlers[])(struct kvm_vcpu *))
>  		init_vmcs_shadow_fields();
>  	}
>  
> -	exit_handlers[EXIT_REASON_VMCLEAR]	= handle_vmclear,
> -	exit_handlers[EXIT_REASON_VMLAUNCH]	= handle_vmlaunch,
> -	exit_handlers[EXIT_REASON_VMPTRLD]	= handle_vmptrld,
> -	exit_handlers[EXIT_REASON_VMPTRST]	= handle_vmptrst,
> -	exit_handlers[EXIT_REASON_VMREAD]	= handle_vmread,
> -	exit_handlers[EXIT_REASON_VMRESUME]	= handle_vmresume,
> -	exit_handlers[EXIT_REASON_VMWRITE]	= handle_vmwrite,
> -	exit_handlers[EXIT_REASON_VMOFF]	= handle_vmoff,
> -	exit_handlers[EXIT_REASON_VMON]		= handle_vmon,
> -	exit_handlers[EXIT_REASON_INVEPT]	= handle_invept,
> -	exit_handlers[EXIT_REASON_INVVPID]	= handle_invvpid,
> -	exit_handlers[EXIT_REASON_VMFUNC]	= handle_vmfunc,
> +	exit_handlers[EXIT_REASON_VMCLEAR]	= handle_vmclear;
> +	exit_handlers[EXIT_REASON_VMLAUNCH]	= handle_vmlaunch;
> +	exit_handlers[EXIT_REASON_VMPTRLD]	= handle_vmptrld;
> +	exit_handlers[EXIT_REASON_VMPTRST]	= handle_vmptrst;
> +	exit_handlers[EXIT_REASON_VMREAD]	= handle_vmread;
> +	exit_handlers[EXIT_REASON_VMRESUME]	= handle_vmresume;
> +	exit_handlers[EXIT_REASON_VMWRITE]	= handle_vmwrite;
> +	exit_handlers[EXIT_REASON_VMOFF]	= handle_vmoff;
> +	exit_handlers[EXIT_REASON_VMON]		= handle_vmon;
> +	exit_handlers[EXIT_REASON_INVEPT]	= handle_invept;
> +	exit_handlers[EXIT_REASON_INVVPID]	= handle_invvpid;
> +	exit_handlers[EXIT_REASON_VMFUNC]	= handle_vmfunc;
>  
>  	kvm_x86_ops->check_nested_events = vmx_check_nested_events;
>  	kvm_x86_ops->get_nested_state = vmx_get_nested_state;
>  	kvm_x86_ops->set_nested_state = vmx_set_nested_state;
> -	kvm_x86_ops->get_vmcs12_pages = nested_get_vmcs12_pages,
> +	kvm_x86_ops->get_vmcs12_pages = nested_get_vmcs12_pages;
>  	kvm_x86_ops->nested_enable_evmcs = nested_enable_evmcs;
>  	kvm_x86_ops->nested_get_evmcs_version = nested_get_evmcs_version;
>  
> 

Queued, thanks.

Paolo
diff mbox series

Patch

diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c
index bc11fcbbe12b..229ca7164318 100644
--- a/arch/x86/kvm/vmx/nested.c
+++ b/arch/x86/kvm/vmx/nested.c
@@ -6051,23 +6051,23 @@  __init int nested_vmx_hardware_setup(int (*exit_handlers[])(struct kvm_vcpu *))
 		init_vmcs_shadow_fields();
 	}
 
-	exit_handlers[EXIT_REASON_VMCLEAR]	= handle_vmclear,
-	exit_handlers[EXIT_REASON_VMLAUNCH]	= handle_vmlaunch,
-	exit_handlers[EXIT_REASON_VMPTRLD]	= handle_vmptrld,
-	exit_handlers[EXIT_REASON_VMPTRST]	= handle_vmptrst,
-	exit_handlers[EXIT_REASON_VMREAD]	= handle_vmread,
-	exit_handlers[EXIT_REASON_VMRESUME]	= handle_vmresume,
-	exit_handlers[EXIT_REASON_VMWRITE]	= handle_vmwrite,
-	exit_handlers[EXIT_REASON_VMOFF]	= handle_vmoff,
-	exit_handlers[EXIT_REASON_VMON]		= handle_vmon,
-	exit_handlers[EXIT_REASON_INVEPT]	= handle_invept,
-	exit_handlers[EXIT_REASON_INVVPID]	= handle_invvpid,
-	exit_handlers[EXIT_REASON_VMFUNC]	= handle_vmfunc,
+	exit_handlers[EXIT_REASON_VMCLEAR]	= handle_vmclear;
+	exit_handlers[EXIT_REASON_VMLAUNCH]	= handle_vmlaunch;
+	exit_handlers[EXIT_REASON_VMPTRLD]	= handle_vmptrld;
+	exit_handlers[EXIT_REASON_VMPTRST]	= handle_vmptrst;
+	exit_handlers[EXIT_REASON_VMREAD]	= handle_vmread;
+	exit_handlers[EXIT_REASON_VMRESUME]	= handle_vmresume;
+	exit_handlers[EXIT_REASON_VMWRITE]	= handle_vmwrite;
+	exit_handlers[EXIT_REASON_VMOFF]	= handle_vmoff;
+	exit_handlers[EXIT_REASON_VMON]		= handle_vmon;
+	exit_handlers[EXIT_REASON_INVEPT]	= handle_invept;
+	exit_handlers[EXIT_REASON_INVVPID]	= handle_invvpid;
+	exit_handlers[EXIT_REASON_VMFUNC]	= handle_vmfunc;
 
 	kvm_x86_ops->check_nested_events = vmx_check_nested_events;
 	kvm_x86_ops->get_nested_state = vmx_get_nested_state;
 	kvm_x86_ops->set_nested_state = vmx_set_nested_state;
-	kvm_x86_ops->get_vmcs12_pages = nested_get_vmcs12_pages,
+	kvm_x86_ops->get_vmcs12_pages = nested_get_vmcs12_pages;
 	kvm_x86_ops->nested_enable_evmcs = nested_enable_evmcs;
 	kvm_x86_ops->nested_get_evmcs_version = nested_get_evmcs_version;