Re: [PATCH v5 1/6] locking/pvqspinlock: Unconditional PV kick with _Q_SLOW_VAL

2015-08-13 Thread Waiman Long
On 08/08/2015 02:02 AM, Peter Zijlstra wrote: On Fri, Aug 07, 2015 at 11:17:56PM -0400, Waiman Long wrote: If _Q_SLOW_VAL has been set, the vCPU state must have been vcpu_hashed. The extra check at the end of __pv_queued_spin_unlock() is unnecessary and so is removed. This is half the patch it

Re: [PATCH v5 1/6] locking/pvqspinlock: Unconditional PV kick with _Q_SLOW_VAL

2015-08-07 Thread Peter Zijlstra
On Fri, Aug 07, 2015 at 11:17:56PM -0400, Waiman Long wrote: > If _Q_SLOW_VAL has been set, the vCPU state must have been vcpu_hashed. > The extra check at the end of __pv_queued_spin_unlock() is unnecessary > and so is removed. This is half the patch it should be. Because if the load is not need

[PATCH v5 1/6] locking/pvqspinlock: Unconditional PV kick with _Q_SLOW_VAL

2015-08-07 Thread Waiman Long
If _Q_SLOW_VAL has been set, the vCPU state must have been vcpu_hashed. The extra check at the end of __pv_queued_spin_unlock() is unnecessary and so is removed. Signed-off-by: Waiman Long Reviewed-by: Davidlohr Bueso --- kernel/locking/qspinlock_paravirt.h |3 +-- 1 files changed, 1 insert