diff options
| -rw-r--r-- | arch/powerpc/platforms/pseries/hotplug-memory.c | 13 | ||||
| -rw-r--r-- | mm/memory_hotplug.c | 4 |
2 files changed, 11 insertions, 6 deletions
diff --git a/arch/powerpc/platforms/pseries/hotplug-memory.c b/arch/powerpc/platforms/pseries/hotplug-memory.c index 11d8e0544ac0..dc0a035e63bb 100644 --- a/arch/powerpc/platforms/pseries/hotplug-memory.c +++ b/arch/powerpc/platforms/pseries/hotplug-memory.c | |||
| @@ -77,7 +77,8 @@ static int pseries_remove_memblock(unsigned long base, unsigned int memblock_siz | |||
| 77 | { | 77 | { |
| 78 | unsigned long start, start_pfn; | 78 | unsigned long start, start_pfn; |
| 79 | struct zone *zone; | 79 | struct zone *zone; |
| 80 | int ret; | 80 | int i, ret; |
| 81 | int sections_to_remove; | ||
| 81 | 82 | ||
| 82 | start_pfn = base >> PAGE_SHIFT; | 83 | start_pfn = base >> PAGE_SHIFT; |
| 83 | 84 | ||
| @@ -97,9 +98,13 @@ static int pseries_remove_memblock(unsigned long base, unsigned int memblock_siz | |||
| 97 | * to sysfs "state" file and we can't remove sysfs entries | 98 | * to sysfs "state" file and we can't remove sysfs entries |
| 98 | * while writing to it. So we have to defer it to here. | 99 | * while writing to it. So we have to defer it to here. |
| 99 | */ | 100 | */ |
| 100 | ret = __remove_pages(zone, start_pfn, memblock_size >> PAGE_SHIFT); | 101 | sections_to_remove = (memblock_size >> PAGE_SHIFT) / PAGES_PER_SECTION; |
| 101 | if (ret) | 102 | for (i = 0; i < sections_to_remove; i++) { |
| 102 | return ret; | 103 | unsigned long pfn = start_pfn + i * PAGES_PER_SECTION; |
| 104 | ret = __remove_pages(zone, start_pfn, PAGES_PER_SECTION); | ||
| 105 | if (ret) | ||
| 106 | return ret; | ||
| 107 | } | ||
| 103 | 108 | ||
| 104 | /* | 109 | /* |
| 105 | * Update memory regions for memory remove | 110 | * Update memory regions for memory remove |
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index 7d0797475a47..56b758ae57d2 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c | |||
| @@ -369,11 +369,11 @@ int __remove_pages(struct zone *zone, unsigned long phys_start_pfn, | |||
| 369 | BUG_ON(phys_start_pfn & ~PAGE_SECTION_MASK); | 369 | BUG_ON(phys_start_pfn & ~PAGE_SECTION_MASK); |
| 370 | BUG_ON(nr_pages % PAGES_PER_SECTION); | 370 | BUG_ON(nr_pages % PAGES_PER_SECTION); |
| 371 | 371 | ||
| 372 | release_mem_region(phys_start_pfn << PAGE_SHIFT, nr_pages * PAGE_SIZE); | ||
| 373 | |||
| 372 | sections_to_remove = nr_pages / PAGES_PER_SECTION; | 374 | sections_to_remove = nr_pages / PAGES_PER_SECTION; |
| 373 | for (i = 0; i < sections_to_remove; i++) { | 375 | for (i = 0; i < sections_to_remove; i++) { |
| 374 | unsigned long pfn = phys_start_pfn + i*PAGES_PER_SECTION; | 376 | unsigned long pfn = phys_start_pfn + i*PAGES_PER_SECTION; |
| 375 | release_mem_region(pfn << PAGE_SHIFT, | ||
| 376 | PAGES_PER_SECTION << PAGE_SHIFT); | ||
| 377 | ret = __remove_section(zone, __pfn_to_section(pfn)); | 377 | ret = __remove_section(zone, __pfn_to_section(pfn)); |
| 378 | if (ret) | 378 | if (ret) |
| 379 | break; | 379 | break; |
