@@ -334,6 +334,22 @@ static ssize_t valid_zones_show(struct device *dev,
* online nodes otherwise the page_zone is not reliable
*/
if (mem->state == MEM_ONLINE) {
+#ifdef CONFIG_ZONE_DEVICE
+ struct resource res;
+ int ret;
+
+ /* adjust start_pfn for dax pmem kmem */
+ ret = find_next_iomem_res(start_pfn << PAGE_SHIFT,
+ ((start_pfn + nr_pages) << PAGE_SHIFT) - 1,
+ IORESOURCE_SYSTEM_RAM | IORESOURCE_BUSY,
+ IORES_DESC_PERSISTENT_MEMORY,
+ false, &res);
+ if (!ret && PFN_UP(res.start) > start_pfn) {
+ nr_pages -= PFN_UP(res.start) - start_pfn;
+ start_pfn = PFN_UP(res.start);
+ }
+#endif
+
/*
* The block contains more than one zone can not be offlined.
* This can happen e.g. for ZONE_DMA and ZONE_DMA32
@@ -999,6 +999,20 @@ int try_online_node(int nid)
static int check_hotplug_memory_range(u64 start, u64 size)
{
+#ifdef CONFIG_ZONE_DEVICE
+ struct resource res;
+ int ret;
+
+ /* Allow pmem kmem not to align with block size */
+ ret = find_next_iomem_res(start, start + size - 1,
+ IORESOURCE_SYSTEM_RAM | IORESOURCE_BUSY,
+ IORES_DESC_PERSISTENT_MEMORY,
+ false, &res);
+ if (!ret) {
+ return 0;
+ }
+#endif
+
/* memory range must be block size aligned */
if (!size || !IS_ALIGNED(start, memory_block_size_bytes()) ||
!IS_ALIGNED(size, memory_block_size_bytes())) {
@@ -1481,19 +1495,42 @@ static int __ref __offline_pages(unsigned long start_pfn,
mem_hotplug_begin();
/*
- * Don't allow to offline memory blocks that contain holes.
+ * Don't allow to offline memory blocks that contain holes except
+ * for pmem.
* Consequently, memory blocks with holes can never get onlined
* via the hotplug path - online_pages() - as hotplugged memory has
* no holes. This way, we e.g., don't have to worry about marking
* memory holes PG_reserved, don't need pfn_valid() checks, and can
* avoid using walk_system_ram_range() later.
+ * When dax pmem is used as RAM (kmem), holes at the beginning is
+ * allowed.
*/
walk_system_ram_range(start_pfn, end_pfn - start_pfn, &nr_pages,
count_system_ram_pages_cb);
if (nr_pages != end_pfn - start_pfn) {
+#ifdef CONFIG_ZONE_DEVICE
+ struct resource res;
+
+ /* Allow pmem kmem not to align with block size */
+ ret = find_next_iomem_res(start_pfn << PAGE_SHIFT,
+ (end_pfn << PAGE_SHIFT) - 1,
+ IORESOURCE_SYSTEM_RAM | IORESOURCE_BUSY,
+ IORES_DESC_PERSISTENT_MEMORY,
+ false, &res);
+ if (ret) {
+ ret = -EINVAL;
+ reason = "memory holes";
+ goto failed_removal;
+ }
+
+ /* adjust start_pfn for dax pmem kmem */
+ start_pfn = PFN_UP(res.start);
+ end_pfn = PFN_DOWN(res.end + 1);
+#else
ret = -EINVAL;
reason = "memory holes";
goto failed_removal;
+#endif
}
/* This makes hotplug much easier...and readable.
When dax pmem is probed as RAM device on arm64, previously, kmem_start in dev_dax_kmem_probe() should be aligned with 1G memblock size on arm64 due to SECTION_SIZE_BITS(30). There will be some meta data at the beginning/end of the iomem space, e.g. namespace info and nvdimm label: 240000000-33fdfffff : Persistent Memory 240000000-2403fffff : namespace0.0 280000000-2bfffffff : dax0.0 280000000-2bfffffff : System RAM Hence it makes the whole kmem space not aligned with memory_block_size for both start addr and end addr. Hence there is a big gap when kmem is added into memory block which causes big memory space wasting. This changes it by relaxing the alignment check for dax pmem kmem in the path of online/offline memory blocks. Signed-off-by: Jia He <justin.he@arm.com> --- drivers/base/memory.c | 16 ++++++++++++++++ mm/memory_hotplug.c | 39 ++++++++++++++++++++++++++++++++++++++- 2 files changed, 54 insertions(+), 1 deletion(-)