From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933116AbcJ3Qpg (ORCPT ); Sun, 30 Oct 2016 12:45:36 -0400 Received: from mail.kernel.org ([198.145.29.136]:48492 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932654AbcJ3QpY (ORCPT ); Sun, 30 Oct 2016 12:45:24 -0400 Message-Id: <20161030164519.682263781@goodmis.org> User-Agent: quilt/0.63-1 Date: Sun, 30 Oct 2016 12:45:11 -0400 From: Steven Rostedt To: linux-kernel@vger.kernel.org, linux-rt-users Cc: Thomas Gleixner , Carsten Emde , Sebastian Andrzej Siewior , John Kacur , Paul Gortmaker , kbuild test robot Subject: [PATCH RT 4/6] x86/apic: get rid of "warning: acpi_ioapic_lock defined but not used" References: <20161030164507.542995196@goodmis.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Disposition: inline; filename=0004-x86-apic-get-rid-of-warning-acpi_ioapic_lock-defined.patch Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 4.4.27-rt38-rc1 stable review patch. If anyone has any objections, please let me know. ------------------ From: Sebastian Andrzej Siewior kbuild test robot reported this against the -RT tree: | In file included from include/linux/mutex.h:30:0, | from include/linux/notifier.h:13, | from include/linux/memory_hotplug.h:6, | from include/linux/mmzone.h:777, | from include/linux/gfp.h:5, | from include/linux/slab.h:14, | from include/linux/resource_ext.h:19, | from include/linux/acpi.h:26, | from arch/x86/kernel/acpi/boot.c:27: |>> arch/x86/kernel/acpi/boot.c:90:21: warning: 'acpi_ioapic_lock' defined but not used [-Wunused-variable] | static DEFINE_MUTEX(acpi_ioapic_lock); | ^ | include/linux/mutex_rt.h:27:15: note: in definition of macro 'DEFINE_MUTEX' | struct mutex mutexname = __MUTEX_INITIALIZER(mutexname) ^~~~~~~~~ which is also true (as in non-used) for !RT but the compiler does not emit a warning. Reported-by: kbuild test robot Signed-off-by: Sebastian Andrzej Siewior Signed-off-by: Steven Rostedt --- arch/x86/kernel/acpi/boot.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/x86/kernel/acpi/boot.c b/arch/x86/kernel/acpi/boot.c index e75907601a41..a29fc4f84fc4 100644 --- a/arch/x86/kernel/acpi/boot.c +++ b/arch/x86/kernel/acpi/boot.c @@ -87,7 +87,9 @@ static u64 acpi_lapic_addr __initdata = APIC_DEFAULT_PHYS_BASE; * ->ioapic_mutex * ->ioapic_lock */ +#ifdef CONFIG_X86_IO_APIC static DEFINE_MUTEX(acpi_ioapic_lock); +#endif /* -------------------------------------------------------------------------- Boot-time Configuration -- 2.9.3