From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg0-f71.google.com (mail-pg0-f71.google.com [74.125.83.71]) by kanga.kvack.org (Postfix) with ESMTP id 1D0CF6B027A for ; Sat, 31 Mar 2018 14:19:51 -0400 (EDT) Received: by mail-pg0-f71.google.com with SMTP id v8so9238123pgs.9 for ; Sat, 31 Mar 2018 11:19:51 -0700 (PDT) Received: from mga17.intel.com (mga17.intel.com. [192.55.52.151]) by mx.google.com with ESMTPS id o14si7359716pgn.655.2018.03.31.11.19.49 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 31 Mar 2018 11:19:49 -0700 (PDT) Subject: Re: [PATCH 00/11] Use global pages with PTI References: <20180327200719.lvdomez6hszpmo4s@gmail.com> <0d6ea030-ec3b-d649-bad7-89ff54094e25@linux.intel.com> <20180330120920.btobga44wqytlkoe@gmail.com> <20180330121725.zcklh36ulg7crydw@gmail.com> <3cdc23a2-99eb-6f93-6934-f7757fa30a3e@linux.intel.com> <62a0dbae-75eb-6737-6029-4aaf72ebd199@linux.intel.com> <20180331053956.uts5yhxfy7ud4bpf@gmail.com> From: Dave Hansen Message-ID: <2607b1b1-89a7-635c-0c5d-da9f558241f4@linux.intel.com> Date: Sat, 31 Mar 2018 11:19:48 -0700 MIME-Version: 1.0 In-Reply-To: <20180331053956.uts5yhxfy7ud4bpf@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Ingo Molnar Cc: Thomas Gleixner , Linus Torvalds , Linux Kernel Mailing List , linux-mm , Andrea Arcangeli , Andrew Lutomirski , Kees Cook , Hugh Dickins , =?UTF-8?B?SsO8cmdlbiBHcm/Dnw==?= , the arch/x86 maintainers , namit@vmware.com On 03/30/2018 10:39 PM, Ingo Molnar wrote: > There were a couple of valid review comments which need to be addressed as well, > but other than that it all looks good to me and I plan to apply the next > iteration. Testing on that non-PCID systems showed an oddity with parts of the kernel image that are modified later in boot (when we set the kernel image read-only). We split a few of the PMD entries and the the old (early boot) values were being used for userspace. I don't think this is a big deal. The most annoying thing is that it makes it harder to quickly validate that all of the things we set to global *should* be global. I'll put some examples of how this looks in the patch when I repost.