path: root/mm/page_alloc.c
diff options
authorOscar Salvador <>2021-05-04 18:35:33 -0700
committerLinus Torvalds <>2021-05-05 11:27:22 -0700
commiteb14d4eefdc4f0051a63973124f431798e16a8b2 (patch)
tree882020053ee16096f10ee0655c116cb89ea84558 /mm/page_alloc.c
parentae37c7ff79f1f030e28ec76c46ee032f8fd07607 (diff)
mm,page_alloc: drop unnecessary checks from pfn_range_valid_contig
pfn_range_valid_contig() bails out when it finds an in-use page or a hugetlb page, among other things. We can drop the in-use page check since __alloc_contig_pages can migrate away those pages, and the hugetlb page check can go too since isolate_migratepages_range is now capable of dealing with hugetlb pages. Either way, those checks are racy so let the end function handle it when the time comes. Link: Signed-off-by: Oscar Salvador <> Suggested-by: David Hildenbrand <> Reviewed-by: David Hildenbrand <> Acked-by: Mike Kravetz <> Acked-by: Michal Hocko <> Cc: Muchun Song <> Cc: Vlastimil Babka <> Signed-off-by: Andrew Morton <> Signed-off-by: Linus Torvalds <>
Diffstat (limited to 'mm/page_alloc.c')
1 files changed, 0 insertions, 6 deletions
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 656f86a98554..80fa6e0f9ed9 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -8898,12 +8898,6 @@ static bool pfn_range_valid_contig(struct zone *z, unsigned long start_pfn,
if (PageReserved(page))
return false;
- if (page_count(page) > 0)
- return false;
- if (PageHuge(page))
- return false;
return true;