Linux Virtualization - Apr 2014 - [PATCH v9 00/19] qspinlock: a 4-byte queue spinlock with PV support

On 04/17/2014 01:23 PM, Konrad Rzeszutek Wilk wrote:
> On Thu, Apr 17, 2014 at 11:03:52AM -0400, Waiman Long wrote:
>> v8->v9:
>>    - Integrate PeterZ's version of the queue spinlock patch with
some
>>      modification:
>>      http://lkml.kernel.org/r/20140310154236.038181843 at infradead.org
>>    - Break the more complex patches into smaller ones to ease review
effort.
>>    - Fix a racing condition in the PV qspinlock code.
> I am not seeing anything mentioning that the overcommit scenario
> for KVM and Xen had been fixed. Or was the 'racing condition' said
> issue?
>
> Thanks.
The hanging is caused by a racing condition which should be fixed in the 
v9 patch. Please let me know if you are still seeing it.

-Longman

On Thu, Apr 17, 2014 at 09:48:36PM -0400, Waiman Long
wrote:
> On 04/17/2014 01:23 PM, Konrad Rzeszutek Wilk wrote:
> >On Thu, Apr 17, 2014 at 11:03:52AM -0400, Waiman Long wrote:
> >>v8->v9:
> >>   - Integrate PeterZ's version of the queue spinlock patch
with some
> >>     modification:
> >>     http://lkml.kernel.org/r/20140310154236.038181843 at
infradead.org
> >>   - Break the more complex patches into smaller ones to ease
review effort.
> >>   - Fix a racing condition in the PV qspinlock code.
> >I am not seeing anything mentioning that the overcommit scenario
> >for KVM and Xen had been fixed. Or was the 'racing condition'
said
> >issue?
> >
> >Thanks.
> 
> The hanging is caused by a racing condition which should be fixed in
> the v9 patch. Please let me know if you are still seeing it.
OK, is there a git tree with these patches to easily slurp them up?


Thanks!
> 
> -Longman

On 04/18/2014 09:18 AM, Konrad Rzeszutek Wilk wrote:
> On Thu, Apr 17, 2014 at 09:48:36PM -0400, Waiman Long wrote:
>> On 04/17/2014 01:23 PM, Konrad Rzeszutek Wilk wrote:
>>> On Thu, Apr 17, 2014 at 11:03:52AM -0400, Waiman Long wrote:
>>>> v8->v9:
>>>>    - Integrate PeterZ's version of the queue spinlock patch
with some
>>>>      modification:
>>>>      http://lkml.kernel.org/r/20140310154236.038181843 at
infradead.org
>>>>    - Break the more complex patches into smaller ones to ease
review effort.
>>>>    - Fix a racing condition in the PV qspinlock code.
>>> I am not seeing anything mentioning that the overcommit scenario
>>> for KVM and Xen had been fixed. Or was the 'racing
condition' said
>>> issue?
>>>
>>> Thanks.
>> The hanging is caused by a racing condition which should be fixed in
>> the v9 patch. Please let me know if you are still seeing it.
> OK, is there a git tree with these patches to easily slurp them up?
>
I am sorry that I don't have a public git tree with the qspinlock 
patches. However, I have made a consolidated patch (patches 1-19) in the 
attached file. Hopefully that will make it easier to apply the patch. 
BTW, it has to be on top of 3.15-rc1 or later version. This may also be 
a conflict in the xen/spinlock.c file.

-Longman
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: qspinlock-consolidated-patches
URL:
<http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20140418/58a2042e/attachment-0001.ksh>