Re: [PATCH v4 11/11] KVM: arm/arm64: timer: remove request-less vcpu kick

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 01/06/2017 13:09, Andrew Jones wrote:
>>> -	kvm_vcpu_kick(vcpu);
>>> +	swake_up(kvm_arch_vcpu_wq(vcpu));
>> We have kvm_vcpu_wake_up().  Why not use that?
> The are two differences between swake_up(kvm_arch_vcpu_wq(vcpu)) and
> kvm_vcpu_wake_up(vcpu)
>  1. kvm_vcpu_wake_up() has a return value: true on wake up, else false
>  2. kvm_vcpu_wake_up() increments the halt_wakeup stat when the vcpu
>     is awaken
> 
> (1) doesn't really matter, but (2) might. Hmm, I think we do want to
> increment that stat in this case though, so I should change this.

Yep.

> Also, we have another use of swake_up(kvm_arch_vcpu_wq(vcpu)), in
> kvm_arm_resume_guest(), but there I don't think we want to increment
> the halt stat, so that one is probably OK.

I would define a __kvm_vcpu_wake_up if you don't want the stat.

Paolo



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux