From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peter Zijlstra <peterz@infradead.org>
Subject: Re: [PATCH][retry 3] Add support for Pause Filtering to AMD
 SVM
Date: Wed, 20 May 2009 10:54:17 +0200
Message-ID: <1242809657.26820.559.camel@twins>
References: <200905050909.58583.mark.langsdorf@amd.com>
	 <200905071000.14038.mark.langsdorf@amd.com>
	 <200905081203.55484.mark.langsdorf@amd.com>
	 <200905191356.37071.mark.langsdorf@amd.com>
	 <1242806386.26820.549.camel@twins>  <4A13C199.5020400@redhat.com>
	 <1242808954.26820.556.camel@twins>  <4A13C40B.8080608@redhat.com>
Mime-Version: 1.0
Content-Type: text/plain
Content-Transfer-Encoding: 7bit
Cc: Mark Langsdorf <mark.langsdorf@amd.com>,
	Joerg Roedel <joerg.roedel@amd.com>, kvm@vger.kernel.org,
	linux-kernel@vger.kernel.org
To: Avi Kivity <avi@redhat.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from bombadil.infradead.org ([18.85.46.34]:43342 "EHLO
	bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1752861AbZETIyQ (ORCPT <rfc822;kvm@vger.kernel.org>);
	Wed, 20 May 2009 04:54:16 -0400
In-Reply-To: <4A13C40B.8080608@redhat.com>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On Wed, 2009-05-20 at 11:49 +0300, Avi Kivity wrote:
> Peter Zijlstra wrote:
> >>> That said, I still thoroughly dislike this whole approach.
> >>>   
> >>>       
> >> Can you explain why?  We have a thread that has detected that it's 
> >> spinning.  Keeping on spinning is a waste of cpu time.  Why not let 
> >> something else use the cpu?
> >>     
> >
> > Because its a polling interface. I much prefer it if we were to get a
> > wakeup notification when the vcpu holding the lock releases it.
> >   
> 
> It's a fully virtualized guest.  There's no way to get this without 
> patching the guest kernel.

Yes there is.. virtualized monitor-wait stuff coupled with a
monitor-wait based spinlock implementation.

Once we go change silicon, you might as well do it right.