From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sasha Levin Subject: Re: [PATCH RFC V2 5/5] kvm guest : pv-ticketlocks support for linux guests running on KVM hypervisor Date: Mon, 24 Oct 2011 12:01:54 +0200 Message-ID: <1319450514.5660.7.camel@lappy> References: <20111023190307.16364.35381.sendpatchset@oc5400248562.ibm.com> <20111023190753.16364.31164.sendpatchset@oc5400248562.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Greg Kroah-Hartman , "H. Peter Anvin" , Gleb Natapov , Virtualization , Jeremy Fitzhardinge , x86@kernel.org, KVM , Dave Jiang , Thomas Gleixner , Stefano Stabellini , Yinghai Lu , Sedat Dilek , Ingo Molnar , Marcelo Tosatti , Xen , Avi Kivity , Rik van Riel , Konrad Rzeszutek Wilk , LKML , Suzuki Poulose , Srivatsa Vaddagiri , Peter Zijlstra To: Raghavendra K T Return-path: Received: from mail-ey0-f174.google.com ([209.85.215.174]:37650 "EHLO mail-ey0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753773Ab1JXJDb (ORCPT ); Mon, 24 Oct 2011 05:03:31 -0400 In-Reply-To: <20111023190753.16364.31164.sendpatchset@oc5400248562.ibm.com> Sender: kvm-owner@vger.kernel.org List-ID: On Mon, 2011-10-24 at 00:37 +0530, Raghavendra K T wrote: > This patch extends Linux guests running on KVM hypervisor to support > pv-ticketlocks. Very early during bootup, paravirtualied KVM guest detects if > the hypervisor has required feature (KVM_FEATURE_WAIT_FOR_KICK) to support > pv-ticketlocks. If so, support for pv-ticketlocks is registered via pv_lock_ops. > > Signed-off-by: Srivatsa Vaddagiri > Signed-off-by: Suzuki Poulose > Signed-off-by: Raghavendra K T > --- > diff --git a/arch/x86/include/asm/kvm_para.h b/arch/x86/include/asm/kvm_para.h > index 2874c19..c7f34b7 100644 > --- a/arch/x86/include/asm/kvm_para.h > +++ b/arch/x86/include/asm/kvm_para.h > @@ -195,10 +195,18 @@ void kvm_async_pf_task_wait(u32 token); > void kvm_async_pf_task_wake(u32 token); > u32 kvm_read_and_reset_pf_reason(void); > extern void kvm_disable_steal_time(void); > -#else > + > +#ifdef CONFIG_PARAVIRT_SPINLOCKS > +void __init kvm_guest_early_init(void); > +#else /* CONFIG_PARAVIRT_SPINLOCKS */ > +#define kvm_guest_early_init() do { } while (0) This should be defined as an empty function. -- Sasha.