Commit 6ab7eba5 authored Aug 17, 2016 by Rui Wang Committed by Ingo Molnar Aug 18, 2016

x86/ioapic: Fix setup_res() failing to get resource



acpi_dev_filter_resource_type() returns 0 on success, and 1 on failure.
A return value of zero means there's a matching resource, so we should
continue within setup_res() to get the resource.

Signed-off-by: Rui Wang <rui.y.wang@intel.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: bhelgaas@google.com
Cc: helgaas@kernel.org
Cc: linux-acpi@vger.kernel.org
Cc: linux-pci@vger.kernel.org
Cc: rjw@rjwysocki.net
Cc: tony.luck@intel.com
Link: http://lkml.kernel.org/r/1471420837-31003-4-git-send-email-rui.y.wang@intel.com


Signed-off-by: Ingo Molnar <mingo@kernel.org>

parent 584c5c42

drivers/acpi/ioapic.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -46,7 +46,7 @@ static acpi_status setup_res(struct acpi_resource acpi_res, void data)
		struct resource_win win;

		res->flags = 0;
		if (acpi_dev_filter_resource_type(acpi_res, IORESOURCE_MEM) == 0)
		if (acpi_dev_filter_resource_type(acpi_res, IORESOURCE_MEM))
		return AE_OK;

		if (!acpi_dev_resource_memory(acpi_res, res)) {

Admin message