On 12/08/19 04:33, Miaohe Lin wrote: > new_entry is reassigned a new value next line. So > it's redundant and remove it. > > Signed-off-by: Miaohe Lin <linmiaohe@xxxxxxxxxx> > --- > arch/x86/kvm/svm.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c > index d685491fce4d..e3d3b2128f2b 100644 > --- a/arch/x86/kvm/svm.c > +++ b/arch/x86/kvm/svm.c > @@ -1714,7 +1714,6 @@ static int avic_init_backing_page(struct kvm_vcpu *vcpu) > if (!entry) > return -EINVAL; > > - new_entry = READ_ONCE(*entry); > new_entry = __sme_set((page_to_phys(svm->avic_backing_page) & > AVIC_PHYSICAL_ID_ENTRY_BACKING_PAGE_MASK) | > AVIC_PHYSICAL_ID_ENTRY_VALID_MASK); > Queued, thanks. Paolo