Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-09 Thread Raghavendra K T
On 04/09/2014 12:45 AM, Waiman Long wrote: Yes, I am able to reproduce the hang problem with ebizzy. BTW, could you try to apply the attached patch file on top of the v8 patch series to see if it can fix the hang problem? Ran the benchmarks with the fix and I am not seeing hang so far. ebizzy

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-09 Thread Raghavendra K T
On 04/09/2014 12:45 AM, Waiman Long wrote: Yes, I am able to reproduce the hang problem with ebizzy. BTW, could you try to apply the attached patch file on top of the v8 patch series to see if it can fix the hang problem? Ran the benchmarks with the fix and I am not seeing hang so far. ebizzy

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-08 Thread Waiman Long
On 04/07/2014 01:51 PM, Raghavendra K T wrote: On 04/07/2014 10:08 PM, Waiman Long wrote: On 04/07/2014 02:14 AM, Raghavendra K T wrote: [...] But I am seeing hang in overcommit cases. Gdb showed that many vcpus are halted and there was no progress. Suspecting the problem /race with halting,

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-08 Thread Waiman Long
On 04/07/2014 01:51 PM, Raghavendra K T wrote: On 04/07/2014 10:08 PM, Waiman Long wrote: On 04/07/2014 02:14 AM, Raghavendra K T wrote: [...] But I am seeing hang in overcommit cases. Gdb showed that many vcpus are halted and there was no progress. Suspecting the problem /race with halting,

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-07 Thread Raghavendra K T
On 04/07/2014 10:08 PM, Waiman Long wrote: On 04/07/2014 02:14 AM, Raghavendra K T wrote: [...] But I am seeing hang in overcommit cases. Gdb showed that many vcpus are halted and there was no progress. Suspecting the problem /race with halting, I removed the halt() part of kvm_hibernate(). I

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-07 Thread Waiman Long
On 04/07/2014 02:14 AM, Raghavendra K T wrote: I tested the v7,v8 of qspinlock with unfair config on kvm guest. I was curious about unfair locks performance in undercommit cases. (overcommit case is expected to perform well) But I am seeing hang in overcommit cases. Gdb showed that many

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-07 Thread Raghavendra K T
On 04/02/2014 06:57 PM, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in content. v7->v8: - Remove one unneeded atomic operation from the

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-07 Thread Raghavendra K T
On 04/02/2014 06:57 PM, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in content. v7-v8: - Remove one unneeded atomic operation from the

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-07 Thread Waiman Long
On 04/07/2014 02:14 AM, Raghavendra K T wrote: I tested the v7,v8 of qspinlock with unfair config on kvm guest. I was curious about unfair locks performance in undercommit cases. (overcommit case is expected to perform well) But I am seeing hang in overcommit cases. Gdb showed that many

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-07 Thread Raghavendra K T
On 04/07/2014 10:08 PM, Waiman Long wrote: On 04/07/2014 02:14 AM, Raghavendra K T wrote: [...] But I am seeing hang in overcommit cases. Gdb showed that many vcpus are halted and there was no progress. Suspecting the problem /race with halting, I removed the halt() part of kvm_hibernate(). I

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Fri, Apr 04, 2014 at 01:58:15PM -0400, Konrad Rzeszutek Wilk wrote: > On Fri, Apr 04, 2014 at 01:13:17PM -0400, Waiman Long wrote: > > On 04/04/2014 12:55 PM, Konrad Rzeszutek Wilk wrote: > > >On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: > > >>On 04/03/2014 01:23 PM, Konrad

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Fri, Apr 04, 2014 at 01:13:17PM -0400, Waiman Long wrote: > On 04/04/2014 12:55 PM, Konrad Rzeszutek Wilk wrote: > >On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: > >>On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: > >>>On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Waiman Long
On 04/04/2014 12:55 PM, Konrad Rzeszutek Wilk wrote: On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: > On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: > >On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: > >>On 04/02/2014 04:35 PM, Waiman Long wrote: > >>>On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: > On Wed,

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Wed, Apr 02, 2014 at 10:32:01AM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: > > N.B. Sorry for the duplicate. This patch series were resent as the > > original one was rejected by the vger.kernel.org list server > > due to long

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Wed, Apr 02, 2014 at 10:32:01AM -0400, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header.

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Waiman Long
On 04/04/2014 12:55 PM, Konrad Rzeszutek Wilk wrote: On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Fri, Apr 04, 2014 at 01:13:17PM -0400, Waiman Long wrote: On 04/04/2014 12:55 PM, Konrad Rzeszutek Wilk wrote: On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-04 Thread Konrad Rzeszutek Wilk
On Fri, Apr 04, 2014 at 01:58:15PM -0400, Konrad Rzeszutek Wilk wrote: On Fri, Apr 04, 2014 at 01:13:17PM -0400, Waiman Long wrote: On 04/04/2014 12:55 PM, Konrad Rzeszutek Wilk wrote: On Thu, Apr 03, 2014 at 10:57:18PM -0400, Waiman Long wrote: On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-03 Thread Waiman Long
On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-03 Thread Konrad Rzeszutek Wilk
On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: > On 04/02/2014 04:35 PM, Waiman Long wrote: > >On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: > >>On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: > >>>N.B. Sorry for the duplicate. This patch series were resent as the

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-03 Thread Konrad Rzeszutek Wilk
On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-03 Thread Waiman Long
On 04/03/2014 01:23 PM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 10:10:17PM -0400, Waiman Long wrote: On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Waiman Long
On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Waiman Long
On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Konrad Rzeszutek Wilk
On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: > N.B. Sorry for the duplicate. This patch series were resent as the > original one was rejected by the vger.kernel.org list server > due to long header. There is no change in content. > > v7->v8: > - Remove one unneeded

[PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Waiman Long
N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in content. v7->v8: - Remove one unneeded atomic operation from the slowpath, thus improving performance. -

[PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Waiman Long
N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in content. v7-v8: - Remove one unneeded atomic operation from the slowpath, thus improving performance. -

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Konrad Rzeszutek Wilk
On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in content. v7-v8: - Remove one unneeded atomic

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Waiman Long
On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server due to long header. There is no change in

Re: [PATCH v8 00/10] qspinlock: a 4-byte queue spinlock with PV support

2014-04-02 Thread Waiman Long
On 04/02/2014 04:35 PM, Waiman Long wrote: On 04/02/2014 10:32 AM, Konrad Rzeszutek Wilk wrote: On Wed, Apr 02, 2014 at 09:27:29AM -0400, Waiman Long wrote: N.B. Sorry for the duplicate. This patch series were resent as the original one was rejected by the vger.kernel.org list server