Re: [PATCH] KVM: vmx: remove unreachable statement in vmx_get_msr_feature()

[Liran Alon]

> On 5 Dec 2019, at 4:24, linmiaohe <linmiaohe@xxxxxxxxxx> wrote:
> 
> From: Miaohe Lin <linmiaohe@xxxxxxxxxx>
> 
> We have no way to reach the final statement, remove it.
> Signed-off-by: Miaohe Lin <linmiaohe@xxxxxxxxxx>
> ---
> arch/x86/kvm/vmx/vmx.c | 2 --
> 1 file changed, 2 deletions(-)
> 
> diff --git a/arch/x86/kvm/vmx/vmx.c b/arch/x86/kvm/vmx/vmx.c
> index e7ea332ad1e8..e58a0daf0f86 100644
> --- a/arch/x86/kvm/vmx/vmx.c
> +++ b/arch/x86/kvm/vmx/vmx.c
> @@ -1781,8 +1781,6 @@ static int vmx_get_msr_feature(struct kvm_msr_entry *msr)
> 	default:
> 		return 1;
> 	}
> -
> -	return 0;
> }

I personally just prefer to remove the âdefaultâ case and change this âreturn 0;â to âreturn 1;â.
But itâs a matter of taste of course.