Paul,
On Thu, 20 Sep 2018, Paul Menzel wrote:
> As always, thank you for the quick response.
Thank you for providing the info!
> Am 19.09.2018 um 10:09 schrieb Thomas Gleixner:
> > On Wed, 19 Sep 2018, Paul Menzel wrote:
> > >
> > > With Linux 4.19-rc4+ and `CONFIG_DEBUG_WX=y`, I see the message below on
> > > the ASRock E350M1.
> > >
> > > > [ 1.813378] Freeing unused kernel image memory: 1112K
> > > > [ 1.818662] Write protecting the kernel text: 8708k
> > > > [ 1.818987] Write protecting the kernel read-only data: 2864k
> > > > [ 1.818989] NX-protecting the kernel data: 5628k
> > > > [ 1.819265] ------------[ cut here ]------------
> > > > [ 1.819272] x86/mm: Found insecure W+X mapping at address
> > > > (ptrval)/0xc00a0000
> > >
> > > I do not notice any problems with the system, but maybe something can be
> > > done
> > > to get rid of these.
> >
> > Can you please enable CONFIG_X86_PTDUMP and provide the output of the files
> > in /sys/kernel/debug/page_tables/ ?
>
> By accident, I noticed that this issue does not happen with GRUB as coreboot
> payload, and only with SeaBIOS. (I only tested on the ASRock E350M1.) A
> coreboot developer said, that SeaBIOS does not do mapping though.
Interesting, but I can't spot what causes that.
Can you please apply the patch below, and provide full dmesg of a seabios
and a grub boot along with the page table files for each?
Thanks,
tglx
8<------------------
diff --git a/arch/x86/mm/init.c b/arch/x86/mm/init.c
index faca978ebf9d..2190d40d99a5 100644
--- a/arch/x86/mm/init.c
+++ b/arch/x86/mm/init.c
@@ -794,6 +794,7 @@ void free_kernel_image_pages(void *begin, void *end)
unsigned long len_pages = (end_ul - begin_ul) >> PAGE_SHIFT;
+ pr_info("Freeing init [mem %#010lx-%#010lx]\n", begin_ul, end_ul - 1);
free_init_pages("unused kernel image", begin_ul, end_ul);
/*
diff --git a/arch/x86/mm/init_32.c b/arch/x86/mm/init_32.c
index 979e0a02cbe1..651447261798 100644
--- a/arch/x86/mm/init_32.c
+++ b/arch/x86/mm/init_32.c
@@ -915,6 +915,7 @@ static void mark_nxdata_nx(void)
*/
unsigned long size = (((unsigned long)__init_end + HPAGE_SIZE) & HPAGE_MASK) - start;
+ pr_info("NX data: 0x%08lx - 0x%08lx\n", start, start + size - 1);
if (__supported_pte_mask & _PAGE_NX)
printk(KERN_INFO "NX-protecting the kernel data: %luk\n", size >> 10);
set_pages_nx(virt_to_page(start), size >> PAGE_SHIFT);
@@ -925,6 +926,7 @@ void mark_rodata_ro(void)
unsigned long start = PFN_ALIGN(_text);
unsigned long size = PFN_ALIGN(_etext) - start;
+ pr_info("RO text: 0x%08lx - 0x%08lx\n", start, start + size - 1);
set_pages_ro(virt_to_page(start), size >> PAGE_SHIFT);
printk(KERN_INFO "Write protecting the kernel text: %luk\n",
size >> 10);
@@ -942,6 +944,7 @@ void mark_rodata_ro(void)
start += size;
size = (unsigned long)__end_rodata - start;
+ pr_info("RO data: 0x%08lx - 0x%08lx\n", start, start + size - 1);
set_pages_ro(virt_to_page(start), size >> PAGE_SHIFT);
printk(KERN_INFO "Write protecting the kernel read-only data: %luk\n",
size >> 10);
