mm: skip memory block registration for ZONE_DEVICE
authorDan Williams <dan.j.williams@intel.com>
Sat, 16 Jan 2016 00:56:17 +0000 (16:56 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sat, 16 Jan 2016 01:56:32 +0000 (17:56 -0800)
Prevent userspace from trying and failing to online ZONE_DEVICE pages
which are meant to never be onlined.

For example on platforms with a udev rule like the following:

  SUBSYSTEM=="memory", ACTION=="add", ATTR{state}=="offline", ATTR{state}="online"

...will generate futile attempts to online the ZONE_DEVICE sections.
Example kernel messages:

    Built 1 zonelists in Node order, mobility grouping on.  Total pages: 1004747
    Policy zone: Normal
    online_pages [mem 0x248000000-0x24fffffff] failed

Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/base/memory.c
include/linux/mm.h

index 619fe584a44ccbc294395f66448973b19595ed5e..213456c2b1236a6eb4949155f087a8d139c5aa4e 100644 (file)
@@ -647,6 +647,13 @@ static int add_memory_block(int base_section_nr)
        return 0;
 }
 
+static bool is_zone_device_section(struct mem_section *ms)
+{
+       struct page *page;
+
+       page = sparse_decode_mem_map(ms->section_mem_map, __section_nr(ms));
+       return is_zone_device_page(page);
+}
 
 /*
  * need an interface for the VM to add new memory regions,
@@ -657,6 +664,9 @@ int register_new_memory(int nid, struct mem_section *section)
        int ret = 0;
        struct memory_block *mem;
 
+       if (is_zone_device_section(section))
+               return 0;
+
        mutex_lock(&mem_sysfs_mutex);
 
        mem = find_memory_block(section);
@@ -693,6 +703,9 @@ static int remove_memory_section(unsigned long node_id,
 {
        struct memory_block *mem;
 
+       if (is_zone_device_section(section))
+               return 0;
+
        mutex_lock(&mem_sysfs_mutex);
        mem = find_memory_block(section);
        unregister_mem_sect_under_nodes(mem, __section_nr(section));
index 0ef5f21735af59b08cd4915a7456fa64ee8341dd..d9fe12d45c217c822296b12bb7d78cd82fc8c30d 100644 (file)
@@ -674,6 +674,18 @@ static inline enum zone_type page_zonenum(const struct page *page)
        return (page->flags >> ZONES_PGSHIFT) & ZONES_MASK;
 }
 
+#ifdef CONFIG_ZONE_DEVICE
+static inline bool is_zone_device_page(const struct page *page)
+{
+       return page_zonenum(page) == ZONE_DEVICE;
+}
+#else
+static inline bool is_zone_device_page(const struct page *page)
+{
+       return false;
+}
+#endif
+
 #if defined(CONFIG_SPARSEMEM) && !defined(CONFIG_SPARSEMEM_VMEMMAP)
 #define SECTION_IN_PAGE_FLAGS
 #endif