On Wed, May 12, 2021 at 11:29:06AM +0800, Kefeng Wang wrote:

quoted

  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?

quoted

+	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?