Re: [PATCH 0/6] x86-64: Remove global variables from boot

David Woodhouse
2023-02-23T13:46:16Z

[-- Attachment #1: Type: text/plain, Size: 1369 bytes --]

On Wed, 2023-02-22 at 17:12 -0500, Brian Gerst wrote:
> This is on top of the parallel boot v10 series.
>
> Remove the global variables initial_gs, initial_stack, and
> early_gdt_descr from the 64-bit boot code.  The stack, GDT, and GSBASE
> can be determined from the CPU number.
>
> Brian Gerst (6):
>   x86/smpboot: Use CPU number instead of APIC ID for single CPU startup
>   x86/smpboot: Use current_task to get idle thread

I think those first two should be folded into the 'x86/smpboot: Support
parallel startup of secondary CPUs' patch rather than follow-on
patches?

>   x86/smpboot: Remove initial_stack on 64-bit
>   x86/smpbppt: Remove early_gdt_descr on 64-bit
>   x86/smpboot: Remove initial_gs
>   x86/smpboot: Simplify boot CPU setup

Those four probably make sense to come separately. For each of them,

Reviewed-by: David Woodhouse <dwmw@amazon.co.uk>

I've pulled in the v10 series from Usama, squashed the first two as I
suggested, added the last four on top to do some testing:
https://git.infradead.org/users/dwmw2/linux.git/shortlog/refs/heads/parallel-6.2-rc8-part1

Rather than overthinking the SoB chain, I've left Usama's signoff as
the last on the original series, and on the assumption that Usama will
continue posting, I'll give them the choice of my R-b or S-o-B on what
are now the final four.

[-- Attachment #2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 5965 bytes --]