diff options
| author |   Wei Yang <richardw.yang@linux.intel.com> | 2019-09-23 15:36:27 -0700 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@linux-foundation.org> | 2019-09-24 15:54:09 -0700 |
| commit | c1cbc3eebf7a9ae46473a66710c0859aaafc1607 (patch) | |
| tree | 701fabe4c243c8c44d3f5ab0bfd9a09fd607d0b6 /mm/sparse.c | |
| parent | mm/sparse.c: fix ALIGN() without power of 2 in sparse_buffer_alloc() (diff) | |
| download | linux-dev-c1cbc3eebf7a9ae46473a66710c0859aaafc1607.tar.xz linux-dev-c1cbc3eebf7a9ae46473a66710c0859aaafc1607.zip | |
mm/sparse.c: use __nr_to_section(section_nr) to get mem_section
__pfn_to_section is defined as __nr_to_section(pfn_to_section_nr(pfn)).
Since we already get section_nr, it is not necessary to get mem_section
from start_pfn. By doing so, we reduce one redundant operation.
Link: http://lkml.kernel.org/r/20190809010242.29797-1-richardw.yang@linux.intel.com
Signed-off-by: Wei Yang <richardw.yang@linux.intel.com>
Reviewed-by: Anshuman Khandual <anshuman.khandual@arm.com>
Tested-by: Anshuman Khandual <anshuman.khandual@arm.com>
Cc: Oscar Salvador <osalvador@suse.de>
Cc: Pavel Tatashin <pasha.tatashin@oracle.com>
Cc: Michal Hocko <mhocko@suse.com>
Cc: David Hildenbrand <david@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/sparse.c')
| -rw-r--r-- | mm/sparse.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/sparse.c b/mm/sparse.c index 79355a86064f..a1679024ab5c 100644 --- a/mm/sparse.c +++ b/mm/sparse.c @@ -877,7 +877,7 @@ int __meminit sparse_add_section(int nid, unsigned long start_pfn, */ page_init_poison(pfn_to_page(start_pfn), sizeof(struct page) * nr_pages); - ms = __pfn_to_section(start_pfn); + ms = __nr_to_section(section_nr); set_section_nid(section_nr, nid); section_mark_present(ms); |