[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160416084228.GA30071@gmail.com>
Date: Sat, 16 Apr 2016 10:42:28 +0200
From: Ingo Molnar <mingo@...nel.org>
To: Kees Cook <keescook@...omium.org>
Cc: Baoquan He <bhe@...hat.com>, Yinghai Lu <yinghai@...nel.org>,
Ard Biesheuvel <ard.biesheuvel@...aro.org>,
Matt Redfearn <matt.redfearn@...tec.com>,
"x86@...nel.org" <x86@...nel.org>,
"H. Peter Anvin" <hpa@...or.com>, Ingo Molnar <mingo@...hat.com>,
Borislav Petkov <bp@...en8.de>,
Vivek Goyal <vgoyal@...hat.com>,
Andy Lutomirski <luto@...nel.org>, lasse.collin@...aani.org,
Andrew Morton <akpm@...ux-foundation.org>,
Dave Young <dyoung@...hat.com>,
"kernel-hardening@...ts.openwall.com"
<kernel-hardening@...ts.openwall.com>,
LKML <linux-kernel@...r.kernel.org>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH v5 03/21] x86, KASLR: Drop
CONFIG_RANDOMIZE_BASE_MAX_OFFSET
* Kees Cook <keescook@...omium.org> wrote:
> > Also note the assertive tone: if this Kconfig feature is eanbled, we say that
> > the kernel address _will_ be randomized, and we should make sure it is. (If
> > for some weird reason randomization fails we should warn prominently during
> > bootup.)
>
> This will need some thought... is it better to fail to boot or to boot without
> entropy? As-is, it's possible to randomly position the kernel base address at
> exactly the location it was going to boot without KASLR too, yet this is still
> considered random...
I think we should boot but print a prominent warning. On real systems it's not
supposed to happen, right? So we want to know if it happens, but don't want to
hassle the user with breaking the system by not booting.
> > Also, could we always mix the i8254 timer into this as well, not just when
> > RDTSC is unavailable?
>
> IIRC, hpa explicitly did not want to do this when he was making
> suggestions on this area. I would need to dig out the thread -- I
> can't find it now. I'd prefer to leave this as-is, since changing it
> would add yet another variable to the behavioral changes of this
> series.
Sure, can stay as-is for now.
Thanks,
Ingo
Powered by blists - more mailing lists