From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rik van Riel Subject: Re: Preemptable Ticket Spinlock Date: Mon, 22 Apr 2013 15:56:42 -0400 Message-ID: <517595FA.800@redhat.com> References: <51745650.9050204@redhat.com> <1366631460.4443.3.camel@laptop> <51753289.70406@redhat.com> <1366660147.6454.6.camel@laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Jiannan Ouyang , LKML , Raghavendra K T , Avi Kivity , Gleb Natapov , Ingo Molnar , Marcelo Tosatti , Srikar , "H. Peter Anvin" , "Nikunj A. Dadhania" , KVM , Thomas Gleixner , Chegu Vinod , "Andrew M. Theurer" , Srivatsa Vaddagiri , Andrew Jones , Karen Noel To: Peter Zijlstra Return-path: In-Reply-To: <1366660147.6454.6.camel@laptop> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 04/22/2013 03:49 PM, Peter Zijlstra wrote: > On Mon, 2013-04-22 at 08:52 -0400, Rik van Riel wrote: >> If the native spin_lock code has been called already at >> that time, the native code would still need to be modified >> to increment the ticket number by 2, so we end up with a >> compatible value in each spin lock's .tickets field, and >> prevent a deadlock after we switch over to the paravirt >> variant. > > I thought the stuff already made it upstream, but apparently not; the > lastest posting I'm aware of is here: > > https://lkml.org/lkml/2012/5/2/105 > > That stuff changes the normal ticket increment as well.. Jiannan, It looks like the patch above could make a good patch 1 (or 2) in your patch series :) -- All rights reversed