Re: [PATCH v13 00/11] Parallel CPU bringup for x86_64
From: David Woodhouse
Date: Wed Mar 08 2023 - 07:21:17 EST
On Wed, 2023-03-08 at 12:15 +0000, David Laight wrote:
>
> > /*
> > * Encrypted guests other than SEV-ES (in the future) will need to
> > * implement an early way of finding the APIC ID, since they will
> > * presumably block direct CPUID too. Be kind to our future selves
> > * by warning here instead of just letting them break. Parallel
> > * startup doesn't have to be in the first round of enabling patches
> > * for any such technology.
> > */
> > if (cc_platform_has(CC_ATTR_GUEST_STATE_ENCRYPT) || !has_sev_es) {
> > pr_info("Disabling parallel bringup due to guest memory encryption\n");
> > return false;
> > }
>
> That looks wrong, won't has_sev_es almost always be false
> so it prints the message and returns?
> Maybe s/||/&&/ ?
D'oh! Fixed; thanks.
Attachment:
smime.p7s
Description: S/MIME cryptographic signature