Re: [PATCH v11 29/45] x86/boot: Add Confidential Computing type to setup_data

From: Borislav Petkov
Date: Tue Mar 01 2022 - 04:48:23 EST


On Thu, Feb 24, 2022 at 10:56:09AM -0600, Brijesh Singh wrote:
> +/*
> + * AMD SEV Confidential computing blob structure. The structure is
> + * defined in OVMF UEFI firmware header:
> + * https://github.com/tianocore/edk2/blob/master/OvmfPkg/Include/Guid/ConfidentialComputingSevSnpBlob.h
> + */
> +#define CC_BLOB_SEV_HDR_MAGIC 0x45444d41
> +struct cc_blob_sev_info {
> + u32 magic;
^^^^^^^^^^^^

You said you wanted to rename this:

https://lore.kernel.org/all/12b945a0-b18c-7f2a-52c7-5e22a944d7f5@xxxxxxx/

and further code is checking magic so I'm guessing you wanna rename the
OVMF definition to "Magic" too?

--
Regards/Gruss,
Boris.

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