From: Kees Cook <keescook@chromium.org>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: "Tobin C . Harding" <me@tobin.cc>,
Tycho Andersen <tycho@tycho.ws>,
Arvind Sankar <nivedita@alum.mit.edu>,
kernel-hardening@lists.openwall.com,
Dave Hansen <dave.hansen@linux.intel.com>,
Andy Lutomirski <luto@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
x86@kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] x86/mm/init_32: Stop printing the virtual memory layout
Date: Thu, 5 Mar 2020 09:28:48 -0800 [thread overview]
Message-ID: <202003050927.530CC2D6DD@keescook> (raw)
In-Reply-To: <20200305150152.831697-1-nivedita@alum.mit.edu>
On Thu, Mar 05, 2020 at 10:01:52AM -0500, Arvind Sankar wrote:
> For security, don't display the kernel's virtual memory layout.
>
> Kees Cook points out:
> "These have been entirely removed on other architectures, so let's
> just do the same for ia32 and remove it unconditionally."
>
> 071929dbdd86 ("arm64: Stop printing the virtual memory layout")
> 1c31d4e96b8c ("ARM: 8820/1: mm: Stop printing the virtual memory layout")
> 31833332f798 ("m68k/mm: Stop printing the virtual memory layout")
> fd8d0ca25631 ("parisc: Hide virtual kernel memory layout")
> adb1fe9ae2ee ("mm/page_alloc: Remove kernel address exposure in free_reserved_area()")
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Thanks!
(*randomly choosing an x86 maintainer to aim this patch at; hi Thomas!*)
Acked-by: Kees Cook <keescook@chromium.org>
-Kees
> ---
> arch/x86/mm/init_32.c | 38 --------------------------------------
> 1 file changed, 38 deletions(-)
>
> diff --git a/arch/x86/mm/init_32.c b/arch/x86/mm/init_32.c
> index 23df4885bbed..8ae0272c1c51 100644
> --- a/arch/x86/mm/init_32.c
> +++ b/arch/x86/mm/init_32.c
> @@ -788,44 +788,6 @@ void __init mem_init(void)
> x86_init.hyper.init_after_bootmem();
>
> mem_init_print_info(NULL);
> - printk(KERN_INFO "virtual kernel memory layout:\n"
> - " fixmap : 0x%08lx - 0x%08lx (%4ld kB)\n"
> - " cpu_entry : 0x%08lx - 0x%08lx (%4ld kB)\n"
> -#ifdef CONFIG_HIGHMEM
> - " pkmap : 0x%08lx - 0x%08lx (%4ld kB)\n"
> -#endif
> - " vmalloc : 0x%08lx - 0x%08lx (%4ld MB)\n"
> - " lowmem : 0x%08lx - 0x%08lx (%4ld MB)\n"
> - " .init : 0x%08lx - 0x%08lx (%4ld kB)\n"
> - " .data : 0x%08lx - 0x%08lx (%4ld kB)\n"
> - " .text : 0x%08lx - 0x%08lx (%4ld kB)\n",
> - FIXADDR_START, FIXADDR_TOP,
> - (FIXADDR_TOP - FIXADDR_START) >> 10,
> -
> - CPU_ENTRY_AREA_BASE,
> - CPU_ENTRY_AREA_BASE + CPU_ENTRY_AREA_MAP_SIZE,
> - CPU_ENTRY_AREA_MAP_SIZE >> 10,
> -
> -#ifdef CONFIG_HIGHMEM
> - PKMAP_BASE, PKMAP_BASE+LAST_PKMAP*PAGE_SIZE,
> - (LAST_PKMAP*PAGE_SIZE) >> 10,
> -#endif
> -
> - VMALLOC_START, VMALLOC_END,
> - (VMALLOC_END - VMALLOC_START) >> 20,
> -
> - (unsigned long)__va(0), (unsigned long)high_memory,
> - ((unsigned long)high_memory - (unsigned long)__va(0)) >> 20,
> -
> - (unsigned long)&__init_begin, (unsigned long)&__init_end,
> - ((unsigned long)&__init_end -
> - (unsigned long)&__init_begin) >> 10,
> -
> - (unsigned long)&_etext, (unsigned long)&_edata,
> - ((unsigned long)&_edata - (unsigned long)&_etext) >> 10,
> -
> - (unsigned long)&_text, (unsigned long)&_etext,
> - ((unsigned long)&_etext - (unsigned long)&_text) >> 10);
>
> /*
> * Check boundaries twice: Some fundamental inconsistencies can
> --
> 2.24.1
>
--
Kees Cook
next prev parent reply other threads:[~2020-03-05 17:28 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-26 21:50 [PATCH] x86/mm/init_32: Don't print out kernel memory layout if KASLR Arvind Sankar
2020-02-29 23:51 ` Kees Cook
2020-03-01 0:11 ` Arvind Sankar
2020-03-02 18:38 ` Kees Cook
2020-03-05 15:05 ` [PATCH] microblaze: Stop printing the virtual memory layout Arvind Sankar
2020-03-05 20:50 ` Tycho Andersen
2020-03-09 10:09 ` Michal Simek
2020-03-05 15:06 ` [PATCH] nds32/mm: " Arvind Sankar
2020-03-05 20:50 ` Tycho Andersen
2020-03-05 15:08 ` [PATCH] powerpc/32: " Arvind Sankar
2020-03-05 20:50 ` Tycho Andersen
2022-03-09 19:35 ` Christophe Leroy
2020-03-05 15:10 ` [PATCH] sh: " Arvind Sankar
2020-03-05 15:18 ` John Paul Adrian Glaubitz
2020-03-05 15:38 ` Joe Perches
2020-03-05 15:41 ` John Paul Adrian Glaubitz
2020-03-05 15:46 ` Arvind Sankar
2020-03-05 15:49 ` John Paul Adrian Glaubitz
2020-03-05 15:56 ` Arvind Sankar
2020-03-05 20:51 ` Tycho Andersen
2020-03-05 20:56 ` John Paul Adrian Glaubitz
2020-03-05 21:17 ` Kees Cook
2020-03-05 17:34 ` Kees Cook
2020-03-06 8:04 ` Geert Uytterhoeven
2020-03-08 12:17 ` Kaiwan N Billimoria
2020-03-05 15:11 ` [PATCH] xtensa/mm: " Arvind Sankar
2020-03-05 20:51 ` Tycho Andersen
2020-03-06 7:54 ` Max Filippov
2020-03-01 0:22 ` [PATCH v2] x86/mm/init_32: " Arvind Sankar
2020-03-02 15:29 ` Tycho Andersen
2020-03-02 18:39 ` Kees Cook
2020-03-05 15:01 ` [PATCH v3] " Arvind Sankar
2020-03-05 17:28 ` Kees Cook [this message]
2020-03-05 20:49 ` Tycho Andersen
2020-03-05 23:00 ` [tip: x86/mm] x86/mm/init/32: " tip-bot2 for Arvind Sankar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202003050927.530CC2D6DD@keescook \
--to=keescook@chromium.org \
--cc=dave.hansen@linux.intel.com \
--cc=kernel-hardening@lists.openwall.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@kernel.org \
--cc=me@tobin.cc \
--cc=nivedita@alum.mit.edu \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=tycho@tycho.ws \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox