All of lore.kernel.org
 help / color / mirror / Atom feed
* + mm-change-normal-message-to-use-pr_debug.patch added to -mm tree
@ 2013-05-28 22:31 akpm
  0 siblings, 0 replies; only message in thread
From: akpm @ 2013-05-28 22:31 UTC (permalink / raw)
  To: mm-commits, toshi.kani

Subject: + mm-change-normal-message-to-use-pr_debug.patch added to -mm tree
To: toshi.kani@hp.com
From: akpm@linux-foundation.org
Date: Tue, 28 May 2013 15:31:39 -0700


The patch titled
     Subject: mm/memory_hotplug.c: change normal message to use pr_debug
has been added to the -mm tree.  Its filename is
     mm-change-normal-message-to-use-pr_debug.patch

Before you just go and hit "reply", please:
   a) Consider who else should be cc'ed
   b) Prefer to cc a suitable mailing list as well
   c) Ideally: find the original patch on the mailing list and do a
      reply-to-all to that, adding suitable additional cc's

*** Remember to use Documentation/SubmitChecklist when testing your code ***

The -mm tree is included into linux-next and is updated
there every 3-4 working days

------------------------------------------------------
From: Toshi Kani <toshi.kani@hp.com>
Subject: 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>
---

 mm/memory_hotplug.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -puN mm/memory_hotplug.c~mm-change-normal-message-to-use-pr_debug mm/memory_hotplug.c
--- a/mm/memory_hotplug.c~mm-change-normal-message-to-use-pr_debug
+++ a/mm/memory_hotplug.c
@@ -75,7 +75,7 @@ static struct resource *register_memory_
 	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;
 	}
_

Patches currently in -mm which might be from toshi.kani@hp.com are

linux-next.patch
mm-change-normal-message-to-use-pr_debug.patch


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2013-05-28 22:31 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-28 22:31 + mm-change-normal-message-to-use-pr_debug.patch added to -mm tree akpm

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.