Re: [PATCH 3/3] mm: don't panic when links can't be created in sysfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Le 11/09/2020 à 16:01, Greg Kroah-Hartman a écrit :
On Fri, Sep 11, 2020 at 03:48:31PM +0200, Laurent Dufour wrote:
At boot time, or when doing memory hot-add operations, if the links in
sysfs can't be created, the system is still able to run, so just report the
error in the kernel log.

Since the number of memory blocks managed could be high, the messages are
rate limited.

As a consequence, link_mem_sections() has no status to report anymore.

Signed-off-by: Laurent Dufour <ldufour@xxxxxxxxxxxxx>
Cc: David Hildenbrand <david@xxxxxxxxxx>
---
  drivers/base/node.c  | 25 +++++++++++++++++--------
  include/linux/node.h | 17 ++++++++---------
  mm/memory_hotplug.c  |  5 ++---
  3 files changed, 27 insertions(+), 20 deletions(-)

diff --git a/drivers/base/node.c b/drivers/base/node.c
index 862516c5a5ae..749a1c8ea992 100644
--- a/drivers/base/node.c
+++ b/drivers/base/node.c
@@ -811,12 +811,21 @@ static int register_mem_sect_under_node(struct memory_block *mem_blk,
  		ret = sysfs_create_link_nowarn(&node_devices[nid]->dev.kobj,
  					&mem_blk->dev.kobj,
  					kobject_name(&mem_blk->dev.kobj));
-		if (ret)
-			return ret;
+		if (ret && ret != -EEXIST)
+			pr_err_ratelimited(
+				"can't create %s to %s link in sysfs (%d)\n",
+				kobject_name(&node_devices[nid]->dev.kobj),
+				kobject_name(&mem_blk->dev.kobj), ret);

dev_err_ratelimited()?

Thanks Greg, I was not aware about this.

Same elsewhere in this patch.

thanks,

greg k-h







[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux