mm/page_alloc.c: fix regression with deferred struct page init
authorJuergen Gross <>
Thu, 4 Jul 2019 22:14:36 +0000 (15:14 -0700)
committerLinus Torvalds <>
Fri, 5 Jul 2019 02:12:07 +0000 (11:12 +0900)
Commit 0e56acae4b4d ("mm: initialize MAX_ORDER_NR_PAGES at a time
instead of doing larger sections") is causing a regression on some
systems when the kernel is booted as Xen dom0.

The system will just hang in early boot.

Reason is an endless loop in get_page_from_freelist() in case the first
zone looked at has no free memory.  deferred_grow_zone() is always
returning true due to the following code snipplet:

  /* If the zone is empty somebody else may have cleared out the zone */
  if (!deferred_init_mem_pfn_range_in_zone(&i, zone, &spfn, &epfn,
                                           first_deferred_pfn)) {
          pgdat->first_deferred_pfn = ULONG_MAX;
          pgdat_resize_unlock(pgdat, &flags);
          return true;

This in turn results in the loop as get_page_from_freelist() is assuming
forward progress can be made by doing some more struct page

Fixes: 0e56acae4b4d ("mm: initialize MAX_ORDER_NR_PAGES at a time instead of doing larger sections")
Signed-off-by: Juergen Gross <>
Suggested-by: Alexander Duyck <>
Acked-by: Alexander Duyck <>
Cc: Michal Hocko <>
Cc: Pavel Tatashin <>
Cc: Mike Rapoport <>
Signed-off-by: Andrew Morton <>
Signed-off-by: Linus Torvalds <>

index d66bc8a..8e3bc94 100644 (file)
@@ -1826,7 +1826,8 @@ deferred_grow_zone(struct zone *zone, unsigned int order)
                                                 first_deferred_pfn)) {
                pgdat->first_deferred_pfn = ULONG_MAX;
                pgdat_resize_unlock(pgdat, &flags);
-               return true;
+               /* Retry only once. */
+               return first_deferred_pfn != ULONG_MAX;