Re: [PATCH 1/3] mm/page_alloc: Use dump_page() directly

From: Matthew Wilcox
Date: Tue May 11 2021 - 23:14:57 EST


On Wed, May 12, 2021 at 11:10:55AM +0800, Kefeng Wang wrote:
> Simplfy use dump_page().

https://lore.kernel.org/linux-mm/20210430145549.2662354-3-willy@xxxxxxxxxxxxx/

> Signed-off-by: Kefeng Wang <wangkefeng.wang@xxxxxxxxxx>
> ---
> mm/page_alloc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/mm/page_alloc.c b/mm/page_alloc.c
> index a2fe714aed93..f23702e7c564 100644
> --- a/mm/page_alloc.c
> +++ b/mm/page_alloc.c
> @@ -658,8 +658,7 @@ static void bad_page(struct page *page, const char *reason)
>
> pr_alert("BUG: Bad page state in process %s pfn:%05lx\n",
> current->comm, page_to_pfn(page));
> - __dump_page(page, reason);
> - dump_page_owner(page);
> + dump_page(page, reason);
>
> print_modules();
> dump_stack();
> --
> 2.26.2
>