From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org (mail.linuxfoundation.org [140.211.169.12]) by ozlabs.org (Postfix) with ESMTP id 2F86A2C0169 for ; Sun, 18 Aug 2013 09:14:11 +1000 (EST) Date: Sat, 17 Aug 2013 16:15:52 -0700 From: Greg KH To: Toshi Kani Subject: Re: [PATCH 2/4] hotplug, x86: Add hotplug lock to missing places Message-ID: <20130817231552.GA20169@kroah.com> References: <1376768819-28975-1-git-send-email-toshi.kani@hp.com> <1376768819-28975-3-git-send-email-toshi.kani@hp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1376768819-28975-3-git-send-email-toshi.kani@hp.com> Cc: fenghua.yu@intel.com, bp@suse.de, x86@kernel.org, linux-kernel@vger.kernel.org, rjw@sisk.pl, isimatu.yasuaki@jp.fujitsu.com, mingo@redhat.com, srivatsa.bhat@linux.vnet.ibm.com, tglx@linutronix.de, hpa@linux.intel.com, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, Aug 17, 2013 at 01:46:57PM -0600, Toshi Kani wrote: > lock_device_hotplug() serializes CPU & Memory online/offline and hotplug > operations. However, this lock is not held in the debug interfaces below > that initiate CPU online/offline operations. > > - _debug_hotplug_cpu(), cpu0 hotplug test interface enabled by > CONFIG_DEBUG_HOTPLUG_CPU0. > - cpu_probe_store() and cpu_release_store(), cpu hotplug test interface > enabled by CONFIG_ARCH_CPU_PROBE_RELEASE. > > This patch changes the above interfaces to hold lock_device_hotplug(). > > Signed-off-by: Toshi Kani > --- > arch/x86/kernel/topology.c | 2 ++ > drivers/base/cpu.c | 16 ++++++++++++++-- > 2 files changed, 16 insertions(+), 2 deletions(-) Acked-by: Greg Kroah-Hartman