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

[Borislav Petkov]

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.