From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH RFC V2 3/5] kvm hypervisor : Add two hypercalls to support pv-ticketlock Date: Thu, 27 Oct 2011 12:17:15 +0200 Message-ID: <4EA92FAB.1050607@redhat.com> References: <20111023190307.16364.35381.sendpatchset@oc5400248562.ibm.com> <20111023190558.16364.2136.sendpatchset@oc5400248562.ibm.com> <4EA53A7D.300@redhat.com> <20111024122734.GA10634@linux.vnet.ibm.com> <4EA56385.9040302@redhat.com> <20111024135032.GB10634@linux.vnet.ibm.com> <4EA6FEC2.1060209@linux.vnet.ibm.com> <4EA7E21B.8020805@redhat.com> <4EA85A9D.5060203@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4EA85A9D.5060203@linux.vnet.ibm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Raghavendra K T Cc: Raghavendra K T , KVM , Peter Zijlstra , Srivatsa Vaddagiri , "H. Peter Anvin" , Jeremy Fitzhardinge , Dave Jiang , x86@kernel.org, Ingo Molnar , Rik van Riel , Stefano Stabellini , Xen , Sedat Dilek , Thomas Gleixner , Virtualization , Yinghai Lu , Konrad Rzeszutek Wilk , Greg Kroah-Hartman , LKML , Suzuki Poulose List-Id: virtualization@lists.linuxfoundation.org On 10/26/2011 09:08 PM, Raghavendra K T wrote: > On 10/26/2011 04:04 PM, Avi Kivity wrote: >> On 10/25/2011 08:24 PM, Raghavendra K T wrote: > CCing Ryan also >>> >>> So then do also you foresee the need for directed yield at some point, >>> to address LHP? provided we have good improvements to prove. >> >> Doesn't this patchset completely eliminate lock holder preemption? >> > Basically I was curious whether we can do more better with your > directed yield discussions in https://lkml.org/lkml/2010/8/2/106 . > > I felt we can get little more improvement with doing directed yield to > lock-holder in case of LHP than sleeping. But I may be wrong. > > So wanted to get the feedback, on whether I am thinking in right > direction. i guess donating some time to the lock holder could help, but not by much. The problem with non-pv spinlocks is that you can't just sleep, since no one will wake you up, so you have to actively boost the lock holder. -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.