On Wed, May 12, 2021 at 11:29:06AM +0800, Kefeng Wang wrote:
> > void dump_page(struct page *page, const char *reason)
> > {
> > - __dump_page(page, reason);
> > + if (PagePoisoned(page))
> > + pr_warn("page:%p is uninitialized and poisoned", page);
> > + else
> > + __dump_page(page);
>
> Hi Matthew, dump_page_owenr() should be called when !PagePoisoned, right?
>
>
> > + if (reason)
> > + pr_warn("page dumped because: %s\n", reason);
> > dump_page_owner(page);
> > }
> > EXPORT_SYMBOL(dump_page);
dump_page_owner() is called whether the page is Poisoned or not ...
both before and after this patch. Is there a problem with that?
