Commit 4996eed8 authored Jul 03, 2013 by Toshi Kani Committed by Linus Torvalds Jul 03, 2013

mm/memory_hotplug.c: change normal message to use pr_debug



During early boot-up, iomem_resource is set up from the boot descriptor
table, such as EFI Memory Table and e820.  Later,
acpi_memory_device_add() calls add_memory() for each ACPI memory device
object as it enumerates ACPI namespace.  This add_memory() call is
expected to fail in register_memory_resource() at boot since
iomem_resource has been set up from EFI/e820.  As a result, add_memory()
returns -EEXIST, which acpi_memory_device_add() handles as the normal
case.

This scheme works fine, but the following error message is logged for
every ACPI memory device object during boot-up.

  "System RAM resource %pR cannot be added\n"

This patch changes register_memory_resource() to use pr_debug() for the
message as it shows up under the normal case.

Signed-off-by: Toshi Kani <toshi.kani@hp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent f15bdfa8

mm/memory_hotplug.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -75,7 +75,7 @@ static struct resource *register_memory_resource(u64 start, u64 size)
		res->end = start + size - 1;
		res->flags = IORESOURCE_MEM \| IORESOURCE_BUSY;
		if (request_resource(&iomem_resource, res) < 0) {
		printk("System RAM resource %pR cannot be added\n", res);
		pr_debug("System RAM resource %pR cannot be added\n", res);
		kfree(res);
		res = NULL;
		}

Admin message