powerpc/pseries/cmm: Switch to balloon_page_alloc()
authorDavid Hildenbrand <david@redhat.com>
Thu, 31 Oct 2019 14:29:30 +0000 (15:29 +0100)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 13 Nov 2019 05:58:02 +0000 (16:58 +1100)
balloon_page_alloc() will use GFP_HIGHUSER_MOVABLE in case we have
CONFIG_BALLOON_COMPACTION. This is now possible, as balloon pages are
movable with CONFIG_BALLOON_COMPACTION. Without
CONFIG_BALLOON_COMPACTION, GFP_HIGHUSER is used.

Note that apart from that, balloon_page_alloc() uses the following
flags:
    __GFP_NOMEMALLOC | __GFP_NORETRY | __GFP_NOWARN
And current code used:
    GFP_NOIO | __GFP_NOWARN | __GFP_NORETRY | __GFP_NOMEMALLOC

GFP_HIGHUSER/GFP_HIGHUSER_MOVABLE include
    __GFP_RECLAIM | __GFP_IO | __GFP_FS | __GFP_HARDWALL | __GFP_HIGHMEM

GFP_NOIO is __GFP_RECLAIM.

With CONFIG_BALLOON_COMPACTION, we essentially add:
    __GFP_IO | __GFP_FS | __GFP_HARDWALL | __GFP_HIGHMEM | __GFP_MOVABLE

Without CONFIG_BALLOON_COMPACTION, we essentially add:
    __GFP_IO | __GFP_FS | __GFP_HARDWALL | __GFP_HIGHMEM

I assume this is fine, as this is what all other balloon compaction
users use. If it turns out to be a problem, we could add __GFP_MOVABLE
manually if we have CONFIG_BALLOON_COMPACTION.

Signed-off-by: David Hildenbrand <david@redhat.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20191031142933.10779-10-david@redhat.com
arch/powerpc/platforms/pseries/cmm.c

index 0663ddad964af95b248934ec5a2ed6e41ff1fb05..86eb845b4737de136b4dfc71be6ccc0e72480d29 100644 (file)
@@ -147,8 +147,7 @@ static long cmm_alloc_pages(long nr)
                        break;
                }
 
-               page = alloc_page(GFP_NOIO | __GFP_NOWARN | __GFP_NORETRY |
-                                 __GFP_NOMEMALLOC);
+               page = balloon_page_alloc();
                if (!page)
                        break;
                rc = plpar_page_set_loaned(page);