[IA64] Drop debug test/printk that some special pages are marked reserved
authorTony Luck <tony.luck@intel.com>
Thu, 2 Jul 2015 19:12:53 +0000 (12:12 -0700)
committerTony Luck <tony.luck@intel.com>
Thu, 2 Jul 2015 21:30:33 +0000 (14:30 -0700)
In commit 92923ca3aace "mm: meminit: only set page reserved in the memblock region"
we dropped setting the reserved bits for all pages. This results in some warnings
on ia64:

put_kernel_page: page at 0xe000000005588000 not in reserved memory
put_kernel_page: page at 0xe000000005588000 not in reserved memory
put_kernel_page: page at 0xe000000005580000 not in reserved memory
put_kernel_page: page at 0xe000000005580000 not in reserved memory
put_kernel_page: page at 0xe000000005580000 not in reserved memory
put_kernel_page: page at 0xe000000005580000 not in reserved memory

the two different pages match up with two objects from the loaded kernel
that get mapped by arch/ia64/mm/init.c:setup_gate()

a000000101588000 D __start_gate_section
a000000101580000 D empty_zero_page

In a discussion with Mel Gorman:
  http://lkml.kernel.org/r/20150526102219.GB13750%40suse.de
he suggested that while the preferred approach might be to
set the reserved bit for these pages, it would also be OK
to just drop the test:
   "as it's a debugging check that is ia-64 specific"

After hunting around a bit and failin to find a good place to mark these
pages as reserved - I decided to just delete the test.

Signed-off-by: Tony Luck <tony.luck@intel.com>
arch/ia64/mm/init.c

index 7f3028965064b1af467bfe93028e1277006fabc1..97e48b0eefc7c18f54f0d1ee76860eed43a53c4e 100644 (file)
@@ -215,10 +215,6 @@ put_kernel_page (struct page *page, unsigned long address, pgprot_t pgprot)
        pmd_t *pmd;
        pte_t *pte;
 
-       if (!PageReserved(page))
-               printk(KERN_ERR "put_kernel_page: page at 0x%p not in reserved memory\n",
-                      page_address(page));
-
        pgd = pgd_offset_k(address);            /* note: this is NOT pgd_offset()! */
 
        {