Re: [PATCH 3/8] mm/debug: Factor PagePoisoned out of __dump_page

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 2021/5/12 11:38, Matthew Wilcox wrote:
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?

struct page_ext *page_ext = lookup_page_ext(page);

  unsigned long pfn = page_to_pfn(page);
  struct mem_section *section = __pfn_to_section(pfn);
  if (!section->page_ext)

If page is Poisoned, I guess the section maybe NULL,
so section->page_ext may meet NULL pointer dereference,
is it possible?



.





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux