> 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.
