Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Waiman Long
On 04/14/2016 10:34 AM, Pan Xinhui wrote: hello, Waiman I try your patch, thanks! also I do some improvement. below code diff has been tested, it works for me. :) diff --git a/kernel/locking/qspinlock.c b/kernel/locking/qspinlock.c index ce2f75e..99f31e4 100644 --- a/kernel/locking/qspinlock.c

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Waiman Long
On 04/14/2016 10:34 AM, Pan Xinhui wrote: hello, Waiman I try your patch, thanks! also I do some improvement. below code diff has been tested, it works for me. :) diff --git a/kernel/locking/qspinlock.c b/kernel/locking/qspinlock.c index ce2f75e..99f31e4 100644 --- a/kernel/locking/qspinlock.c

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Pan Xinhui
hello, Waiman I try your patch, thanks! also I do some improvement. below code diff has been tested, it works for me. :) diff --git a/kernel/locking/qspinlock.c b/kernel/locking/qspinlock.c index ce2f75e..99f31e4 100644 --- a/kernel/locking/qspinlock.c +++ b/kernel/locking/qspinlock.c @@ -248,7

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Pan Xinhui
hello, Waiman I try your patch, thanks! also I do some improvement. below code diff has been tested, it works for me. :) diff --git a/kernel/locking/qspinlock.c b/kernel/locking/qspinlock.c index ce2f75e..99f31e4 100644 --- a/kernel/locking/qspinlock.c +++ b/kernel/locking/qspinlock.c @@ -248,7

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Pan Xinhui
On 2016年04月14日 04:59, Waiman Long wrote: > Pan Xinhui was asking for a lock holder cpu argument in pv_wait() > to help the porting of pvqspinlock to PPC. The new argument will can > potentially help hypervisor expediate the execution of the critical > section so that the lock holder vCPU can

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Pan Xinhui
On 2016年04月14日 04:59, Waiman Long wrote: > Pan Xinhui was asking for a lock holder cpu argument in pv_wait() > to help the porting of pvqspinlock to PPC. The new argument will can > potentially help hypervisor expediate the execution of the critical > section so that the lock holder vCPU can

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Pan Xinhui
On 2016年04月14日 08:21, Peter Zijlstra wrote: > On Wed, Apr 13, 2016 at 04:59:20PM -0400, Waiman Long wrote: >> Pan Xinhui was asking for a lock holder cpu argument in pv_wait() >> to help the porting of pvqspinlock to PPC. The new argument will can >> potentially help hypervisor expediate the

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-14 Thread Pan Xinhui
On 2016年04月14日 08:21, Peter Zijlstra wrote: > On Wed, Apr 13, 2016 at 04:59:20PM -0400, Waiman Long wrote: >> Pan Xinhui was asking for a lock holder cpu argument in pv_wait() >> to help the porting of pvqspinlock to PPC. The new argument will can >> potentially help hypervisor expediate the

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-13 Thread Waiman Long
On 04/13/2016 08:21 PM, Peter Zijlstra wrote: On Wed, Apr 13, 2016 at 04:59:20PM -0400, Waiman Long wrote: Pan Xinhui was asking for a lock holder cpu argument in pv_wait() to help the porting of pvqspinlock to PPC. The new argument will can potentially help hypervisor expediate the execution

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-13 Thread Waiman Long
On 04/13/2016 08:21 PM, Peter Zijlstra wrote: On Wed, Apr 13, 2016 at 04:59:20PM -0400, Waiman Long wrote: Pan Xinhui was asking for a lock holder cpu argument in pv_wait() to help the porting of pvqspinlock to PPC. The new argument will can potentially help hypervisor expediate the execution

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-13 Thread Peter Zijlstra
On Wed, Apr 13, 2016 at 04:59:20PM -0400, Waiman Long wrote: > Pan Xinhui was asking for a lock holder cpu argument in pv_wait() > to help the porting of pvqspinlock to PPC. The new argument will can > potentially help hypervisor expediate the execution of the critical > section so that the lock

Re: [PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-13 Thread Peter Zijlstra
On Wed, Apr 13, 2016 at 04:59:20PM -0400, Waiman Long wrote: > Pan Xinhui was asking for a lock holder cpu argument in pv_wait() > to help the porting of pvqspinlock to PPC. The new argument will can > potentially help hypervisor expediate the execution of the critical > section so that the lock

[PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-13 Thread Waiman Long
Pan Xinhui was asking for a lock holder cpu argument in pv_wait() to help the porting of pvqspinlock to PPC. The new argument will can potentially help hypervisor expediate the execution of the critical section so that the lock holder vCPU can release the lock sooner. This patch does just that by

[PATCH] locking/pvqspinlock: Add lock holder CPU argument to pv_wait()

2016-04-13 Thread Waiman Long
Pan Xinhui was asking for a lock holder cpu argument in pv_wait() to help the porting of pvqspinlock to PPC. The new argument will can potentially help hypervisor expediate the execution of the critical section so that the lock holder vCPU can release the lock sooner. This patch does just that by