[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171101085424.cwvc4nrrdhvjc3su@gmail.com>
Date: Wed, 1 Nov 2017 09:54:25 +0100
From: Ingo Molnar <mingo@...nel.org>
To: Dave Hansen <dave.hansen@...ux.intel.com>
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org,
Andy Lutomirski <luto@...nel.org>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Thomas Gleixner <tglx@...utronix.de>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
"H. Peter Anvin" <hpa@...or.com>,
borisBrian Gerst <brgerst@...il.com>,
Denys Vlasenko <dvlasenk@...hat.com>,
Josh Poimboeuf <jpoimboe@...hat.com>,
Thomas Garnier <thgarnie@...gle.com>
Subject: Re: [PATCH 00/23] KAISER: unmap most of the kernel from userspace
page tables
(Filled in the missing Cc: list)
* Dave Hansen <dave.hansen@...ux.intel.com> wrote:
> tl;dr:
>
> KAISER makes it harder to defeat KASLR, but makes syscalls and
> interrupts slower. These patches are based on work from a team at
> Graz University of Technology posted here[1]. The major addition is
> support for Intel PCIDs which builds on top of Andy Lutomorski's PCID
> work merged for 4.14. PCIDs make KAISER's overhead very reasonable
> for a wide variety of use cases.
Ok, while I never thought I'd see the 4g:4g patch come to 64-bit kernels ;-),
this series is a lot better than earlier versions of this feature, and it
solves a number of KASLR timing attacks rather fundamentally.
Beyond the inevitable cavalcade of (solvable) problems that will pop up during
review, one major item I'd like to see addressed is runtime configurability: it
should be possible to switch between a CR3-flushing and a regular syscall and page
table model on the admin level, without restarting the kernel and apps. Distros
really, really don't want to double the number of kernel variants they have.
The 'Kaiser off' runtime switch doesn't have to be as efficient as
CONFIG_KAISER=n, at least initialloy, but at minimum it should avoid the most
expensive page table switching paths in the syscall entry codepaths.
Also, this series should be based on Andy's latest syscall entry cleanup work.
Thanks,
Ingo
Powered by blists - more mailing lists