From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752442AbdKZPNf (ORCPT ); Sun, 26 Nov 2017 10:13:35 -0500 Received: from mail-wr0-f195.google.com ([209.85.128.195]:35018 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752254AbdKZPNe (ORCPT ); Sun, 26 Nov 2017 10:13:34 -0500 X-Google-Smtp-Source: AGs4zMYpeBiBm7IfVq0jNtplQDVY2xkjq3T/oS9Ot1OZOpKJQlzN06A5v9zfLW1owcnASsymkOH5dw== Date: Sun, 26 Nov 2017 16:13:30 +0100 From: Ingo Molnar To: Dave Hansen Cc: Peter Zijlstra , linux-kernel@vger.kernel.org, Andy Lutomirski , Thomas Gleixner , "H . Peter Anvin" , Borislav Petkov , Linus Torvalds Subject: Re: [PATCH 25/43] x86/mm/kaiser: Unmap kernel from userspace page tables (core patch) Message-ID: <20171126151330.otk6576fi5aae64h@gmail.com> References: <20171124091448.7649-1-mingo@kernel.org> <20171124091448.7649-26-mingo@kernel.org> <20171124121638.ej2flgfvctw2cqer@hirez.programming.kicks-ass.net> <36922832-274f-89fa-d39d-b73e75292216@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <36922832-274f-89fa-d39d-b73e75292216@linux.intel.com> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Dave Hansen wrote: > On 11/24/2017 04:16 AM, Peter Zijlstra wrote: > > On Fri, Nov 24, 2017 at 10:14:30AM +0100, Ingo Molnar wrote: > >> +The minimalistic kernel portion of the user page tables try to > >> +map only what is needed to enter/exit the kernel such as the > >> +entry/exit functions themselves and the interrupt descriptor > >> +table (IDT). > > > > There are a few unnecessary things that get mapped > >> +such as the first C function when entering an interrupt (see > >> +comments in kaiser.c). > > > > If I understood Andy's patches correctly, this should no longer be > > required. Is this text still correct? > > It is out of date. We were able to remove the irq handler mappings. I have updated the Documentation/x86/kaiser.txt file accordingly. Thanks, Ingo