All of lore.kernel.org
 help / color / mirror / Atom feed
From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
To: "H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@elte.hu>
Cc: Andrew Jones <drjones@redhat.com>,
	Xen Devel <xen-devel@lists.xensource.com>,
	KVM <kvm@vger.kernel.org>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	Peter Zijlstra <peterz@infradead.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Stephan Diestelhorst <stephan.diestelhorst@amd.com>,
	the arch/x86 maintainers <x86@kernel.org>,
	Srivatsa Vaddagiri <vatsa@linux.vnet.ibm.com>,
	Virtualization <virtualization@lists.linux-foundation.org>,
	Andi Kleen <andi@firstfloor.org>,
	Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>,
	Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>,
	Attilio Rao <attilio.rao@citrix.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Avi Kivity <avi@redhat.com>
Subject: [PATCH RFC V7 2/12] x86/ticketlock: don't inline _spin_unlock when using paravirt spinlocks
Date: Fri, 20 Apr 2012 01:42:59 +0530	[thread overview]
Message-ID: <20120419201256.5411.59675.sendpatchset@codeblue> (raw)
In-Reply-To: <20120419201209.5411.43877.sendpatchset@codeblue>

From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> 

The code size expands somewhat, and its better to just call
a function rather than inline it.

Thanks Jeremy for original version of ARCH_NOINLINE_SPIN_UNLOCK config patch, 
which is simplified.

Suggested-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
---
 arch/x86/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 1d14cc6..35eb2e4 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -597,6 +597,7 @@ config PARAVIRT
 config PARAVIRT_SPINLOCKS
 	bool "Paravirtualization layer for spinlocks"
 	depends on PARAVIRT && SMP && EXPERIMENTAL
+	select UNINLINE_SPIN_UNLOCK
 	---help---
 	  Paravirtualized spinlocks allow a pvops backend to replace the
 	  spinlock implementation with something virtualization-friendly

WARNING: multiple messages have this Message-ID (diff)
From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
To: "H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@elte.hu>
Cc: Attilio Rao <attilio.rao@citrix.com>,
	Srivatsa Vaddagiri <vatsa@linux.vnet.ibm.com>,
	the arch/x86 maintainers <x86@kernel.org>,
	Peter Zijlstra <peterz@infradead.org>,
	Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Virtualization <virtualization@lists.linux-foundation.org>,
	Xen Devel <xen-devel@lists.xensource.com>,
	KVM <kvm@vger.kernel.org>,
	Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>,
	Andi Kleen <andi@firstfloor.org>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	Andrew Jones <drjones@redhat.com>, Avi Kivity <avi@redhat.com>,
	Stephan Diestelhorst <stephan.diestelhorst@amd.com>,
	LKML <linux-kernel@vger.kernel.org>
Subject: [PATCH RFC V7 2/12]  x86/ticketlock: don't inline _spin_unlock when using paravirt spinlocks
Date: Fri, 20 Apr 2012 01:42:59 +0530	[thread overview]
Message-ID: <20120419201256.5411.59675.sendpatchset@codeblue> (raw)
In-Reply-To: <20120419201209.5411.43877.sendpatchset@codeblue>

From: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com> 

The code size expands somewhat, and its better to just call
a function rather than inline it.

Thanks Jeremy for original version of ARCH_NOINLINE_SPIN_UNLOCK config patch, 
which is simplified.

Suggested-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Raghavendra K T <raghavendra.kt@linux.vnet.ibm.com>
---
 arch/x86/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 1d14cc6..35eb2e4 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -597,6 +597,7 @@ config PARAVIRT
 config PARAVIRT_SPINLOCKS
 	bool "Paravirtualization layer for spinlocks"
 	depends on PARAVIRT && SMP && EXPERIMENTAL
+	select UNINLINE_SPIN_UNLOCK
 	---help---
 	  Paravirtualized spinlocks allow a pvops backend to replace the
 	  spinlock implementation with something virtualization-friendly


  parent reply	other threads:[~2012-04-19 20:12 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-19 20:12 [PATCH RFC V7 0/12] Paravirtualized ticketlocks Raghavendra K T
2012-04-19 20:12 ` Raghavendra K T
2012-04-19 20:12 ` [PATCH RFC V7 1/12] x86/spinlock: replace pv spinlocks with pv ticketlocks Raghavendra K T
2012-04-19 20:12   ` Raghavendra K T
2012-04-19 20:12 ` Raghavendra K T [this message]
2012-04-19 20:12   ` [PATCH RFC V7 2/12] x86/ticketlock: don't inline _spin_unlock when using paravirt spinlocks Raghavendra K T
2012-04-19 20:13 ` [PATCH RFC V7 3/12] x86/ticketlock: collapse a layer of functions Raghavendra K T
2012-04-19 20:13   ` Raghavendra K T
2012-04-19 20:13 ` [PATCH RFC V7 4/12] xen: defer spinlock setup until boot CPU setup Raghavendra K T
2012-04-19 20:13   ` Raghavendra K T
2012-04-19 20:14 ` [PATCH RFC V7 5/12] xen/pvticketlock: Xen implementation for PV ticket locks Raghavendra K T
2012-04-19 20:14   ` Raghavendra K T
2012-04-19 20:14 ` [PATCH RFC V7 6/12] xen/pvticketlocks: add xen_nopvspin parameter to disable xen pv ticketlocks Raghavendra K T
2012-04-19 20:14   ` Raghavendra K T
2012-04-19 20:15 ` [PATCH RFC V7 7/12] x86/pvticketlock: use callee-save for lock_spinning Raghavendra K T
2012-04-19 20:15 ` Raghavendra K T
2012-04-19 20:15 ` [PATCH RFC V7 8/12] x86/pvticketlock: when paravirtualizing ticket locks, increment by 2 Raghavendra K T
2012-04-19 20:15 ` Raghavendra K T
2012-04-19 20:15 ` [PATCH RFC V7 9/12] split out rate limiting from jump_label.h Raghavendra K T
2012-04-19 20:15   ` Raghavendra K T
2012-04-19 20:16 ` [PATCH RFC V7 10/12] x86/ticketlock: add slowpath logic Raghavendra K T
2012-04-19 20:16 ` Raghavendra K T
2012-04-19 20:16 ` [PATCH RFC V7 11/12] xen/pvticketlock: allow interrupts to be enabled while blocking Raghavendra K T
2012-04-19 20:16 ` Raghavendra K T
2012-04-19 20:17 ` [PATCH RFC V7 12/12] xen: enable PV ticketlocks on HVM Xen Raghavendra K T
2012-04-19 20:17   ` Raghavendra K T
2012-05-01 12:50 ` [Xen-devel] [PATCH RFC V7 0/12] Paravirtualized ticketlocks Ian Campbell
2012-05-01 12:50   ` Ian Campbell
2012-05-01 13:04   ` Raghavendra K T
2012-05-01 13:04     ` Raghavendra K T

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120419201256.5411.59675.sendpatchset@codeblue \
    --to=raghavendra.kt@linux.vnet.ibm.com \
    --cc=andi@firstfloor.org \
    --cc=attilio.rao@citrix.com \
    --cc=avi@redhat.com \
    --cc=drjones@redhat.com \
    --cc=hpa@zytor.com \
    --cc=jeremy.fitzhardinge@citrix.com \
    --cc=konrad.wilk@oracle.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=peterz@infradead.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=stephan.diestelhorst@amd.com \
    --cc=torvalds@linux-foundation.org \
    --cc=vatsa@linux.vnet.ibm.com \
    --cc=virtualization@lists.linux-foundation.org \
    --cc=x86@kernel.org \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.