回复: [edk2-devel] [PATCH ovmf 1/5] MdePkg/Register/Amd: Define all bits from MSR_SEV_STATUS_REGISTER

["gaoliming" <gaoliming@byosoft.com.cn>]

Alexey:


> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Alexey
> Kardashevskiy via groups.io
> 发送时间: 2022年12月1日 10:35
> 收件人: devel@edk2.groups.io
> 抄送: Ard Biesheuvel <ardb+tianocore@kernel.org>; Jiewen Yao
> <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd
> Hoffmann <kraxel@redhat.com>; Brijesh Singh <brijesh.singh@amd.com>;
> Erdem Aktas <erdemaktas@google.com>; James Bottomley
> <jejb@linux.ibm.com>; Min Xu <min.m.xu@intel.com>; Tom Lendacky
> <thomas.lendacky@amd.com>; Alexey Kardashevskiy <aik@amd.com>
> 主题: [edk2-devel] [PATCH ovmf 1/5] MdePkg/Register/Amd: Define all bits
> from MSR_SEV_STATUS_REGISTER
> 
> We will need soon DebugSwap but others likely too.
> 
> Signed-off-by: Alexey Kardashevskiy <aik@amd.com>
> ---
>  MdePkg/Include/Register/Amd/Fam17Msr.h | 57 +++++++++++++++++++-
>  1 file changed, 56 insertions(+), 1 deletion(-)
> 
> diff --git a/MdePkg/Include/Register/Amd/Fam17Msr.h
> b/MdePkg/Include/Register/Amd/Fam17Msr.h
> index bb4e143e2456..f9474e6776f2 100644
> --- a/MdePkg/Include/Register/Amd/Fam17Msr.h
> +++ b/MdePkg/Include/Register/Amd/Fam17Msr.h
> @@ -121,7 +121,62 @@ typedef union {
>      ///
>      UINT32    SevSnpBit : 1;
> 
> -    UINT32    Reserved2 : 29;
> +    ///
> +    /// [Bit 3] The guest was run with the Virtual TOM feature enabled in
> SEV_FEATURES[1]
> +    ///
> +    UINT32    vTOM_Enabled : 1;
> +
This name doesn't follow name convention. You can create pull request to see
CI test result. 

Thanks
Liming
> +    ///
> +    /// [Bit 4] The guest was run with the ReflectVC feature enabled in
> SEV_FEATURES[2]
> +    ///
> +    UINT32    ReflectVC : 1;
> +
> +    ///
> +    /// [Bit 5] The guest was run with the Restricted Injection feature
> enabled in SEV_FEATURES[3]
> +    ///
> +    UINT32    RestrictedInjection : 1;
> +
> +    ///
> +    /// [Bit 6] The guest was run with the Alternate Injection feature
> enabled in SEV_FEATURES[4]
> +    ///
> +    UINT32    AlternateInjection : 1;
> +
> +    ///
> +    /// [Bit 7] This guest was run with debug register swapping enabled
in
> SEV_FEATURES[5]
> +    ///
> +    UINT32    DebugSwap : 1;
> +
> +    ///
> +    /// [Bit 8]  This guest was run with the PreventHostIBS feature
> enabled in SEV_FEATURES[6]
> +    ///
> +    UINT32    PreventHostIBS : 1;
> +
> +    ///
> +    /// [Bit 9] The guest was run with the BTB isolation feature enabled
in
> SEV_FEATURES[7]
> +    ///
> +    UINT32    SNPBTBIsolation : 1;
> +
> +    ///
> +    /// [Bit 10]
> +    ///
> +    UINT32    Reserved0 : 1;
> +
> +    ///
> +    /// [Bit 11] The guest was run with the Secure TSC feature enabled in
> SEV_FEATURES[9]
> +    ///
> +    UINT32    SecureTsc : 1;
> +
> +    ///
> +    /// [Bits 12 13 14 15]
> +    ///
> +    UINT32    Reserved1 : 4;
> +
> +    ///
> +    /// [Bit 16] The guest was run with the VMSA Register Protection
> feature enabled in SEV_FEATURES[14]
> +    ///
> +    UINT32    VmsaRegProt_Enabled : 1;
> +
> +    UINT32    Reserved2 : 15;
>    } Bits;
>    ///
>    /// All bit fields as a 32-bit value
> --
> 2.38.1
> 
> 
> 
> 
>