From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <5342DCF3.7080408@citrix.com> Date: Mon, 7 Apr 2014 18:14:27 +0100 From: David Vrabel MIME-Version: 1.0 To: CC: , , Konrad Rzeszutek Wilk , Steven Rostedt , , Boris Ostrovsky Subject: Re: [PATCH] xen/spinlock: Don't enable them unconditionally. References: <1396637284-6312-1-git-send-email-konrad@kernel.org> In-Reply-To: <1396637284-6312-1-git-send-email-konrad@kernel.org> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: On 04/04/14 19:48, konrad@kernel.org wrote: > From: Konrad Rzeszutek Wilk > > The git commit a945928ea2709bc0e8e8165d33aed855a0110279 > ('xen: Do not enable spinlocks before jump_label_init() has executed') > was added to deal with the jump machinery. Earlier the code > that turned on the jump label was only called by Xen specific > functions. But now that it had been moved to the initcall machinery > it gets called on Xen, KVM, and baremetal - ouch!. And the detection > machinery to only call it on Xen wasn't remembered in the heat > of merge window excitement. > > This means that the slowpath is enabled on baremetal while it should > not be. Applied to devel/for-linus-3.15. Thanks. David