Re: [External] [PATCH v2 0/5] x86-64: Remove global variables from boot

[Usama Arif]

On 25/02/2023 12:55, David Woodhouse wrote:
> On 25 February 2023 12:47:33 GMT, Brian Gerst <brgerst@xxxxxxxxx> wrote:
> > On Sat, Feb 25, 2023 at 5:20 AM David Woodhouse <dwmw2@xxxxxxxxxxxxx> wrote:
> > > On Fri, 2023-02-24 at 16:38 -0500, Brian Gerst wrote:
> > > > Removing the globals before the parallel boot series, would be the
> > > > best option IMO.  That would make the transition simpler.
> > >
> > > Looks like this:
> > >
> > > https://git.infradead.org/users/dwmw2/linux.git/shortlog/refs/heads/parallel-6.2-rc8-brfirst
> > >
> > > Passes basic smoke testing in qemu, including suspend to RAM and
> > > offlining CPU0.
> >
> > Looks good, thanks.
>
> Aside from the fact that I forgot to put the tr_lock into the other startup paths that load the realmode %esp. But I'll let Usama do that now.
Yeah looks good! I am testing with the macro diff for tr_lock from 
https://lore.kernel.org/all/05e27a7a-1faa-944e-2764-6ab4d620fb8f@xxxxxxxxxxxxx/. 
If it all works, happy for me to send out v12 with it?

> I can probably work out how to test the AMD SEV path; is there a way I can test the startup_64 MADT one?

I guess the easiest way is to create a TDX VM on Sapphire Rapids which I 
believe mostly Intel people have access to right now? Maybe we can post 
v12 and someone from Intel could just quickly verify if it boots with 
it? I have added Yuan from the other thread in here who pointed it out 
initially.