diff options
author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-10-13 13:57:25 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-10-13 13:57:25 -0400 |
commit | 8346aa765e14348b7b436825b3c4740895a2fe1a (patch) | |
tree | db2644417bc07d6b39a20dac27e98622f43c5d46 | |
parent | dbe2256ddd8e8420c254c79f4045c41cb5f4bd6b (diff) |
Revert "mm: Check if section present during memory block (un)registering"
This reverts commit 7568fb63f57ac8672f8bf2018171255441238882 as it's
already in Linus's tree through a different patch.
Reported-by: Tony Luck <tony.luck@intel.com>
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: stable@vger.kernel.org #v3.15
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/base/node.c | 31 |
1 files changed, 4 insertions, 27 deletions
diff --git a/drivers/base/node.c b/drivers/base/node.c index ae98ebaf6313..560751bad294 100644 --- a/drivers/base/node.c +++ b/drivers/base/node.c | |||
@@ -390,18 +390,7 @@ int register_mem_sect_under_node(struct memory_block *mem_blk, int nid) | |||
390 | sect_end_pfn = section_nr_to_pfn(mem_blk->end_section_nr); | 390 | sect_end_pfn = section_nr_to_pfn(mem_blk->end_section_nr); |
391 | sect_end_pfn += PAGES_PER_SECTION - 1; | 391 | sect_end_pfn += PAGES_PER_SECTION - 1; |
392 | for (pfn = sect_start_pfn; pfn <= sect_end_pfn; pfn++) { | 392 | for (pfn = sect_start_pfn; pfn <= sect_end_pfn; pfn++) { |
393 | int page_nid, scn_nr; | 393 | int page_nid; |
394 | |||
395 | /* | ||
396 | * memory block could have several absent sections from start. | ||
397 | * skip pfn range from absent section | ||
398 | */ | ||
399 | scn_nr = pfn_to_section_nr(pfn); | ||
400 | if (!present_section_nr(scn_nr)) { | ||
401 | pfn = round_down(pfn + PAGES_PER_SECTION, | ||
402 | PAGES_PER_SECTION) - 1; | ||
403 | continue; | ||
404 | } | ||
405 | 394 | ||
406 | /* | 395 | /* |
407 | * memory block could have several absent sections from start. | 396 | * memory block could have several absent sections from start. |
@@ -447,22 +436,10 @@ int unregister_mem_sect_under_nodes(struct memory_block *mem_blk, | |||
447 | return -ENOMEM; | 436 | return -ENOMEM; |
448 | nodes_clear(*unlinked_nodes); | 437 | nodes_clear(*unlinked_nodes); |
449 | 438 | ||
450 | sect_start_pfn = section_nr_to_pfn(mem_blk->start_section_nr); | 439 | sect_start_pfn = section_nr_to_pfn(phys_index); |
451 | sect_end_pfn = section_nr_to_pfn(mem_blk->end_section_nr); | 440 | sect_end_pfn = sect_start_pfn + PAGES_PER_SECTION - 1; |
452 | sect_end_pfn += PAGES_PER_SECTION - 1; | ||
453 | for (pfn = sect_start_pfn; pfn <= sect_end_pfn; pfn++) { | 441 | for (pfn = sect_start_pfn; pfn <= sect_end_pfn; pfn++) { |
454 | int nid, scn_nr; | 442 | int nid; |
455 | |||
456 | /* | ||
457 | * memory block could have several absent sections from start. | ||
458 | * skip pfn range from absent section | ||
459 | */ | ||
460 | scn_nr = pfn_to_section_nr(pfn); | ||
461 | if (!present_section_nr(scn_nr)) { | ||
462 | pfn = round_down(pfn + PAGES_PER_SECTION, | ||
463 | PAGES_PER_SECTION) - 1; | ||
464 | continue; | ||
465 | } | ||
466 | 443 | ||
467 | nid = get_nid_for_pfn(pfn); | 444 | nid = get_nid_for_pfn(pfn); |
468 | if (nid < 0) | 445 | if (nid < 0) |