On Fri, Mar 12, 2021 at 7:43 AM Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: > > On 12/03/21 16:37, Sean Christopherson wrote: > > On Thu, Mar 11, 2021, Ben Gardon wrote: > >> The pt passed into handle_removed_tdp_mmu_page does not need RCU > >> protection, as it is not at any risk of being freed by another thread at > >> that point. However, the implicit cast from tdp_sptep_t to u64 * dropped > >> the __rcu annotation without a proper rcu_derefrence. Fix this by > >> passing the pt as a tdp_ptep_t and then rcu_dereferencing it in > >> the function. > >> > >> Suggested-by: Sean Christopherson <seanjc@xxxxxxxxxx> > >> Reported-by: kernel test robot <lkp@xxxxxxxxx> > > > > Should be <lkp@xxxxxxxxx>. Looks like you've been taking pointers from Paolo :-) I'll update that in v2. I was a little confused because I was looking at the report archived on Spinics, where all the domains are xxxxxxxx. Didn't notice that all the emails had been redacted like that. > > The day someone starts confusing employers in CCs you should tell them > "I see you have constructed a new email sending alias. Your skills are > now complete". > > Paolo > > > https://lkml.org/lkml/2019/6/17/1210 > > > > Other than that, > > > > Reviewed-by: Sean Christopherson <seanjc@xxxxxxxxxx> > > > >> Signed-off-by: Ben Gardon <bgardon@xxxxxxxxxx> > > >
