2016-07-11 05:11-0500, Suravee Suthikulpanit: > From: Suravee Suthikulpanit <[email protected]> > > This patch introduces avic_ga_log_notifier, which will be called > by IOMMU driver whenever it handles the Guest vAPIC (GA) log entry. > > Signed-off-by: Suravee Suthikulpanit <[email protected]> > --- > diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c > @@ -928,6 +930,45 @@ static void svm_disable_lbrv(struct vcpu_svm *svm) > +static int avic_ga_log_notifier(int avic_tag, int vcpu_id, int vec) > +{ > + unsigned long flags; > + struct kvm_arch *ka = NULL; > + struct kvm_vcpu *vcpu = NULL; > + struct vcpu_svm *svm = NULL; > + > + pr_debug("SVM: %s: avic_tag=%#x, vcpu_id=%#x, vec=%#x\n", > + __func__, avic_tag, vcpu_id, vec); > + > + spin_lock_irqsave(&svm_vm_data_hash_lock, flags); > + hash_for_each_possible(svm_vm_data_hash, ka, hnode, avic_tag) { > + struct kvm *kvm = container_of(ka, struct kvm, arch);
iommu_handle_ga_guest_nr_entry(), the caller, already performed a hash lookup to get ir_data. Cannot we store the vcpu pointer in ir_data, to avoid a second hash lookup and a vcpu lookup? Thanks.
