linux-kernel - Re: [PATCH 00/24] x86/mm: Add KAISER support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 27 Nov 2017 14:59:05 +0100
From:   Borislav Petkov <bp@...en8.de>
To:     Thomas Gleixner <tglx@...utronix.de>
Cc:     Ingo Molnar <mingo@...nel.org>, linux-kernel@...r.kernel.org,
        Dave Hansen <dave.hansen@...ux.intel.com>,
        Andy Lutomirski <luto@...capital.net>,
        "H . Peter Anvin" <hpa@...or.com>,
        Peter Zijlstra <peterz@...radead.org>,
        Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: [PATCH 00/24] x86/mm: Add KAISER support

On Mon, Nov 27, 2017 at 02:57:17PM +0100, Thomas Gleixner wrote:
> On Mon, 27 Nov 2017, Borislav Petkov wrote:
> > On Mon, Nov 27, 2017 at 11:48:59AM +0100, Ingo Molnar wrote:
> > > Sending this out for another round of review.
> > > 
> > > These are the main KAISER patches, on top of the preparatory series I just sent.
> > > This includes KAISER support plus enhancements on top, all the latest code.
> > > 
> > > ( All review feedback so far is incorporated - if anything is missing please
> > >   let me know. )
> > 
> > Something's missing:
> > 
> > arch/x86/boot/compressed/pagetable.o: In function `kernel_ident_mapping_init':
> > pagetable.c:(.text+0x3cc): undefined reference to `kaiser_enabled'
> > pagetable.c:(.text+0x400): undefined reference to `__supported_pte_mask'
> > pagetable.c:(.text+0x473): undefined reference to `system_state'
> > make[2]: *** [arch/x86/boot/compressed/vmlinux] Error 1
> > make[1]: *** [arch/x86/boot/compressed/vmlinux] Error 2
> > make: *** [bzImage] Error 2
> > make: *** Waiting for unfinished jobs....
> > 
> > I suspect it is that core patch but need to stare at it more. .config
> > attached.
> 
> That's the core patch plus the run time enablement I did.

Right, and I think the fix is as simple as this because we don't care
about kaiser in the bootstrap kernel:

---
diff --git a/arch/x86/boot/compressed/pagetable.c b/arch/x86/boot/compressed/pagetable.c
index d5364ca2e3f9..63eaa71ad9f1 100644
--- a/arch/x86/boot/compressed/pagetable.c
+++ b/arch/x86/boot/compressed/pagetable.c
@@ -22,6 +22,7 @@
  * the SME support to avoid any compile and link errors.
  */
 #undef CONFIG_AMD_MEM_ENCRYPT
+#undef CONFIG_KAISER
 
 #include "misc.h"
 


-- 
Regards/Gruss,
    Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.