[edk2-devel] [PATCH 02/13] MdePkg/Amd: add white spaces to retain alignment for future expansion
Laszlo Ersek
lersek at redhat.com
Tue May 11 08:36:41 UTC 2021
On 05/07/21 22:38, Brijesh Singh wrote:
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3275
>
> Version 2 of the GHCB spec introduces several new SNP-specific NAEs.
> Unfortunately, the names for those NAEs break the alignment. Add some
> white spaces so that the SNP support patches do not break the alignment.
>
> Cc: James Bottomley <jejb at linux.ibm.com>
> Cc: Min Xu <min.m.xu at intel.com>
> Cc: Jiewen Yao <jiewen.yao at intel.com>
> Cc: Tom Lendacky <thomas.lendacky at amd.com>
> Cc: Jordan Justen <jordan.l.justen at intel.com>
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>
> Cc: Laszlo Ersek <lersek at redhat.com>
> Cc: Erdem Aktas <erdemaktas at google.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu at intel.com>
> Signed-off-by: Brijesh Singh <brijesh.singh at amd.com>
> ---
> MdePkg/Include/Register/Amd/Fam17Msr.h | 10 +++++-----
> MdePkg/Include/Register/Amd/Ghcb.h | 12 ++++++------
> 2 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/MdePkg/Include/Register/Amd/Fam17Msr.h b/MdePkg/Include/Register/Amd/Fam17Msr.h
> index 716d52fd508d..7368ce7af02a 100644
> --- a/MdePkg/Include/Register/Amd/Fam17Msr.h
> +++ b/MdePkg/Include/Register/Amd/Fam17Msr.h
> @@ -53,11 +53,11 @@ typedef union {
> UINT64 GhcbPhysicalAddress;
> } MSR_SEV_ES_GHCB_REGISTER;
>
> -#define GHCB_INFO_SEV_INFO 1
> -#define GHCB_INFO_SEV_INFO_GET 2
> -#define GHCB_INFO_CPUID_REQUEST 4
> -#define GHCB_INFO_CPUID_RESPONSE 5
> -#define GHCB_INFO_TERMINATE_REQUEST 256
> +#define GHCB_INFO_SEV_INFO 1
> +#define GHCB_INFO_SEV_INFO_GET 2
> +#define GHCB_INFO_CPUID_REQUEST 4
> +#define GHCB_INFO_CPUID_RESPONSE 5
> +#define GHCB_INFO_TERMINATE_REQUEST 256
>
> #define GHCB_TERMINATE_GHCB 0
> #define GHCB_TERMINATE_GHCB_GENERAL 0
> diff --git a/MdePkg/Include/Register/Amd/Ghcb.h b/MdePkg/Include/Register/Amd/Ghcb.h
> index ccdb662af7a7..712dc8e769c0 100644
> --- a/MdePkg/Include/Register/Amd/Ghcb.h
> +++ b/MdePkg/Include/Register/Amd/Ghcb.h
> @@ -49,12 +49,12 @@
> //
> // VMG Special Exit Codes
> //
> -#define SVM_EXIT_MMIO_READ 0x80000001ULL
> -#define SVM_EXIT_MMIO_WRITE 0x80000002ULL
> -#define SVM_EXIT_NMI_COMPLETE 0x80000003ULL
> -#define SVM_EXIT_AP_RESET_HOLD 0x80000004ULL
> -#define SVM_EXIT_AP_JUMP_TABLE 0x80000005ULL
> -#define SVM_EXIT_UNSUPPORTED 0x8000FFFFULL
> +#define SVM_EXIT_MMIO_READ 0x80000001ULL
> +#define SVM_EXIT_MMIO_WRITE 0x80000002ULL
> +#define SVM_EXIT_NMI_COMPLETE 0x80000003ULL
> +#define SVM_EXIT_AP_RESET_HOLD 0x80000004ULL
> +#define SVM_EXIT_AP_JUMP_TABLE 0x80000005ULL
> +#define SVM_EXIT_UNSUPPORTED 0x8000FFFFULL
>
> //
> // IOIO Exit Information
>
Based on Liming's feedback at
<https://listman.redhat.com/archives/edk2-devel-archive/2021-May/msg00157.html>,
I'll change the subject of this patch to:
MdePkg/Register/Amd: realign macros with more space for future expansion
(72 chars)
Reviewed-by: Laszlo Ersek <lersek at redhat.com>
Thanks,
Laszlo
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74959): https://edk2.groups.io/g/devel/message/74959
Mute This Topic: https://groups.io/mt/82665179/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list