diff options
Diffstat (limited to 'mm')
-rw-r--r-- | mm/memory_hotplug.c | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index 1218f73890b6..b8c11e063ff0 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c | |||
@@ -1484,10 +1484,13 @@ bool is_mem_section_removable(unsigned long start_pfn, unsigned long nr_pages) | |||
1484 | 1484 | ||
1485 | /* | 1485 | /* |
1486 | * Confirm all pages in a range [start, end) belong to the same zone. | 1486 | * Confirm all pages in a range [start, end) belong to the same zone. |
1487 | * When true, return its valid [start, end). | ||
1487 | */ | 1488 | */ |
1488 | int test_pages_in_a_zone(unsigned long start_pfn, unsigned long end_pfn) | 1489 | int test_pages_in_a_zone(unsigned long start_pfn, unsigned long end_pfn, |
1490 | unsigned long *valid_start, unsigned long *valid_end) | ||
1489 | { | 1491 | { |
1490 | unsigned long pfn, sec_end_pfn; | 1492 | unsigned long pfn, sec_end_pfn; |
1493 | unsigned long start, end; | ||
1491 | struct zone *zone = NULL; | 1494 | struct zone *zone = NULL; |
1492 | struct page *page; | 1495 | struct page *page; |
1493 | int i; | 1496 | int i; |
@@ -1509,14 +1512,20 @@ int test_pages_in_a_zone(unsigned long start_pfn, unsigned long end_pfn) | |||
1509 | page = pfn_to_page(pfn + i); | 1512 | page = pfn_to_page(pfn + i); |
1510 | if (zone && page_zone(page) != zone) | 1513 | if (zone && page_zone(page) != zone) |
1511 | return 0; | 1514 | return 0; |
1515 | if (!zone) | ||
1516 | start = pfn + i; | ||
1512 | zone = page_zone(page); | 1517 | zone = page_zone(page); |
1518 | end = pfn + MAX_ORDER_NR_PAGES; | ||
1513 | } | 1519 | } |
1514 | } | 1520 | } |
1515 | 1521 | ||
1516 | if (zone) | 1522 | if (zone) { |
1523 | *valid_start = start; | ||
1524 | *valid_end = end; | ||
1517 | return 1; | 1525 | return 1; |
1518 | else | 1526 | } else { |
1519 | return 0; | 1527 | return 0; |
1528 | } | ||
1520 | } | 1529 | } |
1521 | 1530 | ||
1522 | /* | 1531 | /* |
@@ -1843,6 +1852,7 @@ static int __ref __offline_pages(unsigned long start_pfn, | |||
1843 | long offlined_pages; | 1852 | long offlined_pages; |
1844 | int ret, drain, retry_max, node; | 1853 | int ret, drain, retry_max, node; |
1845 | unsigned long flags; | 1854 | unsigned long flags; |
1855 | unsigned long valid_start, valid_end; | ||
1846 | struct zone *zone; | 1856 | struct zone *zone; |
1847 | struct memory_notify arg; | 1857 | struct memory_notify arg; |
1848 | 1858 | ||
@@ -1853,10 +1863,10 @@ static int __ref __offline_pages(unsigned long start_pfn, | |||
1853 | return -EINVAL; | 1863 | return -EINVAL; |
1854 | /* This makes hotplug much easier...and readable. | 1864 | /* This makes hotplug much easier...and readable. |
1855 | we assume this for now. .*/ | 1865 | we assume this for now. .*/ |
1856 | if (!test_pages_in_a_zone(start_pfn, end_pfn)) | 1866 | if (!test_pages_in_a_zone(start_pfn, end_pfn, &valid_start, &valid_end)) |
1857 | return -EINVAL; | 1867 | return -EINVAL; |
1858 | 1868 | ||
1859 | zone = page_zone(pfn_to_page(start_pfn)); | 1869 | zone = page_zone(pfn_to_page(valid_start)); |
1860 | node = zone_to_nid(zone); | 1870 | node = zone_to_nid(zone); |
1861 | nr_pages = end_pfn - start_pfn; | 1871 | nr_pages = end_pfn - start_pfn; |
1862 | 1872 | ||