From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl0-f72.google.com (mail-pl0-f72.google.com [209.85.160.72]) by kanga.kvack.org (Postfix) with ESMTP id 1991D6B0003 for ; Thu, 19 Jul 2018 09:23:19 -0400 (EDT) Received: by mail-pl0-f72.google.com with SMTP id y8-v6so4547873plp.17 for ; Thu, 19 Jul 2018 06:23:19 -0700 (PDT) Received: from mail-sor-f65.google.com (mail-sor-f65.google.com. [209.85.220.65]) by mx.google.com with SMTPS id k12-v6sor1643652pgo.192.2018.07.19.06.23.18 for (Google Transport Security); Thu, 19 Jul 2018 06:23:18 -0700 (PDT) Date: Thu, 19 Jul 2018 16:23:12 +0300 From: "Kirill A. Shutemov" Subject: Re: [PATCHv5 08/19] x86/mm: Introduce variables to store number, shift and mask of KeyIDs Message-ID: <20180719132312.75lduymla2uretax@kshutemo-mobl1> References: <20180717112029.42378-1-kirill.shutemov@linux.intel.com> <20180717112029.42378-9-kirill.shutemov@linux.intel.com> <1edc05b0-8371-807e-7cfa-6e8f61ee9b70@intel.com> <20180719102130.b4f6b6v5wg3modtc@kshutemo-mobl1> <20180719131245.sxnqsgzvkqriy3o2@kshutemo-mobl1> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Thomas Gleixner Cc: Dave Hansen , "Kirill A. Shutemov" , Ingo Molnar , x86@kernel.org, "H. Peter Anvin" , Tom Lendacky , Kai Huang , Jacob Pan , linux-kernel@vger.kernel.org, linux-mm@kvack.org On Thu, Jul 19, 2018 at 03:18:03PM +0200, Thomas Gleixner wrote: > On Thu, 19 Jul 2018, Kirill A. Shutemov wrote: > > On Thu, Jul 19, 2018 at 02:37:35PM +0200, Thomas Gleixner wrote: > > > On Thu, 19 Jul 2018, Kirill A. Shutemov wrote: > > > > On Wed, Jul 18, 2018 at 04:19:10PM -0700, Dave Hansen wrote: > > > > > > } else { > > > > > > /* > > > > > > * Reset __PHYSICAL_MASK. > > > > > > @@ -591,6 +592,9 @@ static void detect_tme(struct cpuinfo_x86 *c) > > > > > > * between CPUs. > > > > > > */ > > > > > > physical_mask = (1ULL << __PHYSICAL_MASK_SHIFT) - 1; > > > > > > + mktme_keyid_mask = 0; > > > > > > + mktme_keyid_shift = 0; > > > > > > + mktme_nr_keyids = 0; > > > > > > } > > > > > > > > > > Should be unnecessary. These are zeroed by the compiler. > > > > > > > > No. detect_tme() called for each CPU in the system. > > > > > > And then the variables are cleared out while other CPUs can access them? > > > How is that supposed to work? > > > > This code path only matter in patalogical case: when MKTME configuation is > > inconsitent between CPUs. Basically if BIOS screwed things up we disable > > MKTME. > > I still don't see how that's supposed to work. > > When the inconsistent CPU is brought up _AFTER_ MKTME is enabled, then how > does clearing the variables help? It does not magically make all the other > stuff go away. We don't actually enable MKTME in kernel. BIOS does. Kernel makes choose to use it or not. Current design targeted to be used by userspace. So until init we don't have any other stuff to go away. We can just pretend that MKTME was never there. -- Kirill A. Shutemov