lkml.org 
[lkml]   [2020]   [May]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v3 69/75] x86/realmode: Setup AP jump table
    On Tue, Apr 28, 2020 at 05:17:19PM +0200, Joerg Roedel wrote:
    > From: Tom Lendacky <thomas.lendacky@amd.com>
    >
    > Setup the AP jump table to point to the SEV-ES trampoline code so that
    > the APs can boot.

    Tom, in his laconic way, doesn't want to explain to us why is this even
    needed...

    :)

    /me reads the code

    /me reads the GHCB spec

    aha, it gets it from the HV. And it can be set by the guest too...

    So how about expanding that commit message as to why this is done, why
    needed, etc?

    Thx.

    > diff --git a/arch/x86/realmode/init.c b/arch/x86/realmode/init.c
    > index 262f83cad355..1c5cbfd102d5 100644
    > --- a/arch/x86/realmode/init.c
    > +++ b/arch/x86/realmode/init.c
    > @@ -9,6 +9,7 @@
    > #include <asm/realmode.h>
    > #include <asm/tlbflush.h>
    > #include <asm/crash.h>
    > +#include <asm/sev-es.h>
    >
    > struct real_mode_header *real_mode_header;
    > u32 *trampoline_cr4_features;
    > @@ -107,6 +108,11 @@ static void __init setup_real_mode(void)
    > if (sme_active())
    > trampoline_header->flags |= TH_FLAGS_SME_ACTIVE;
    >
    > + if (sev_es_active()) {
    > + if (sev_es_setup_ap_jump_table(real_mode_header))
    > + panic("Failed to update SEV-ES AP Jump Table");
    > + }
    > +

    So this function gets slowly sprinkled with

    if (sev-something)
    bla

    Please wrap at least those last two into a

    sev_setup_real_mode()

    or so.

    --
    Regards/Gruss,
    Boris.

    https://people.kernel.org/tglx/notes-about-netiquette

    \
     
     \ /
      Last update: 2020-05-29 11:04    [W:4.107 / U:0.520 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site