Re: [PATCH 1/7] KVM: nVMX: Prepend "nested_" to check_vmentry_{pre,post}reqs()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 16, 2018 at 08:01:48PM -0500, Krish Sadhukhan wrote:
> .. as they are used only in nested context.
> 
> Signed-off-by: Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx>
> Reviewed-by: Liran Alon <liran.alon@xxxxxxxxxx>
> Reviewed-by: Mihai Carabas <mihai.carabas@xxxxxxxxxx>
> Reviewed-by: Mark Kanda <mark.kanda@xxxxxxxxxx>
> ---
>  arch/x86/kvm/vmx.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
> index e665aa7..411fcd2 100644
> --- a/arch/x86/kvm/vmx.c
> +++ b/arch/x86/kvm/vmx.c
> @@ -12380,7 +12380,7 @@ static int nested_vmx_check_nmi_controls(struct vmcs12 *vmcs12)
>  	return 0;
>  }
>  
> -static int check_vmentry_prereqs(struct kvm_vcpu *vcpu, struct vmcs12 *vmcs12)
> +static int nested_check_vmentry_prereqs(struct kvm_vcpu *vcpu, struct vmcs12 *vmcs12)

Probably worth putting a newline before vmcs12.

>  {
>  	struct vcpu_vmx *vmx = to_vmx(vcpu);
>  
> @@ -12542,7 +12542,7 @@ static int nested_vmx_check_vmcs_link_ptr(struct kvm_vcpu *vcpu,
>  	return r;
>  }
>  
> -static int check_vmentry_postreqs(struct kvm_vcpu *vcpu, struct vmcs12 *vmcs12,
> +static int nested_check_vmentry_postreqs(struct kvm_vcpu *vcpu, struct vmcs12 *vmcs12,
>  				  u32 *exit_qual)

If you put the newline between vcpu and vmcs12 you can squeak into 80
chars and maintain alignment, e.g.: 

static int nested_check_vmentry_postreqs(struct kvm_vcpu *vcpu,
					 struct vmcs12 *vmcs12, u32 *exit_qual)

>  {
>  	bool ia32e;
> @@ -12746,7 +12746,7 @@ static int nested_vmx_run(struct kvm_vcpu *vcpu, bool launch)
>  		goto out;
>  	}
>  
> -	ret = check_vmentry_prereqs(vcpu, vmcs12);
> +	ret = nested_check_vmentry_prereqs(vcpu, vmcs12);
>  	if (ret) {
>  		nested_vmx_failValid(vcpu, ret);
>  		goto out;
> @@ -12761,7 +12761,7 @@ static int nested_vmx_run(struct kvm_vcpu *vcpu, bool launch)
>  	 */
>  	skip_emulated_instruction(vcpu);
>  
> -	ret = check_vmentry_postreqs(vcpu, vmcs12, &exit_qual);
> +	ret = nested_check_vmentry_postreqs(vcpu, vmcs12, &exit_qual);
>  	if (ret) {
>  		nested_vmx_entry_failure(vcpu, vmcs12,
>  					 EXIT_REASON_INVALID_STATE, exit_qual);
> @@ -14085,8 +14085,8 @@ static int vmx_set_nested_state(struct kvm_vcpu *vcpu,
>  			return -EINVAL;
>  	}
>  
> -	if (check_vmentry_prereqs(vcpu, vmcs12) ||
> -	    check_vmentry_postreqs(vcpu, vmcs12, &exit_qual))
> +	if (nested_check_vmentry_prereqs(vcpu, vmcs12) ||
> +	    nested_check_vmentry_postreqs(vcpu, vmcs12, &exit_qual))
>  		return -EINVAL;
>  
>  	vmx->nested.dirty_vmcs12 = true;
> -- 
> 2.9.5
> 



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux