On 06/19/2015 06:31 AM, Paolo Bonzini wrote:
>
>
> On 12/06/2015 07:34, Wei Huang wrote:
>> -static void trigger_pmi(struct irq_work *irq_work)
>> +static void kvm_pmi_trigger_fn(struct irq_work *irq_work)
>> {
>> - struct kvm_pmu *pmu = container_of(irq_work, struct kvm_pmu,
>> - irq_work);
>> - struct kvm_vcpu *vcpu = container_of(pmu, struct kvm_vcpu,
>> - arch.pmu);
>> + struct kvm_pmu *pmu = container_of(irq_work, struct kvm_pmu, irq_work);
>> + struct kvm_vcpu *vcpu = pmu_to_vcpu(pmu);
>>
>> - kvm_deliver_pmi(vcpu);
>> + kvm_pmu_deliver_pmi(vcpu);
>> }
>>
>
> On the other hand, this patch really should have been split further.
>
> For example, just looking at this snippet you can see that the
> introduction of pmu_to_vcpu and the function renames should have been
> separate from the creation of the vPMU interface.
Drew Jones brought up in a separate email. I will split it into a
smaller pieces, less intrusive in each step.
Thanks,
-Wei
>
> Paolo
>
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
