The zone type check was a leftover from the cleanup that plumbed altmap
through the memory hotplug path, i.e. commit da024512a1fa "mm: pass the
vmem_altmap to arch_remove_memory and __remove_pages".

Cc: Michal Hocko <mho...@suse.com>
Cc: Logan Gunthorpe <log...@deltatee.com>
Cc: Pavel Tatashin <pasha.tatas...@soleen.com>
Reviewed-by: David Hildenbrand <da...@redhat.com>
Reviewed-by: Oscar Salvador <osalva...@suse.de>
Signed-off-by: Dan Williams <dan.j.willi...@intel.com>
---
 mm/memory_hotplug.c |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c
index 647859a1d119..4b882c57781a 100644
--- a/mm/memory_hotplug.c
+++ b/mm/memory_hotplug.c
@@ -535,11 +535,8 @@ void __remove_pages(struct zone *zone, unsigned long 
phys_start_pfn,
        unsigned long map_offset = 0;
        int sections_to_remove;
 
-       /* In the ZONE_DEVICE case device driver owns the memory region */
-       if (is_dev_zone(zone)) {
-               if (altmap)
-                       map_offset = vmem_altmap_offset(altmap);
-       }
+       if (altmap)
+               map_offset = vmem_altmap_offset(altmap);
 
        clear_zone_contiguous(zone);
 

Reply via email to