From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753290AbeCaSTv (ORCPT ); Sat, 31 Mar 2018 14:19:51 -0400 Received: from mga11.intel.com ([192.55.52.93]:26743 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753017AbeCaSTt (ORCPT ); Sat, 31 Mar 2018 14:19:49 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,387,1517904000"; d="scan'208";a="38394138" Subject: Re: [PATCH 00/11] Use global pages with PTI To: Ingo Molnar 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> 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 From: Dave Hansen Message-ID: <2607b1b1-89a7-635c-0c5d-da9f558241f4@linux.intel.com> Date: Sat, 31 Mar 2018 11:19:48 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 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: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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.