From: "Chang, Abner" <abner.chang@amd.com>
To: "Attar, AbdulLateef (Abdul Lateef)" <AbdulLateef.Attar@amd.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Attar, AbdulLateef (Abdul Lateef)" <AbdulLateef.Attar@amd.com>,
"Grimes, Paul" <Paul.Grimes@amd.com>,
"Kirkendall, Garrett" <Garrett.Kirkendall@amd.com>,
Eric Dong <eric.dong@intel.com>, Ray Ni <ray.ni@intel.com>,
Rahul Kumar <rahul1.kumar@intel.com>,
Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [PATCH v3 2/6] UefiCpuPkg: Adds SmmSmramSaveStateLib library class
Date: Thu, 19 Jan 2023 04:05:58 +0000 [thread overview]
Message-ID: <MN2PR12MB3966EA87E55152FAC1F65CF7EAC49@MN2PR12MB3966.namprd12.prod.outlook.com> (raw)
In-Reply-To: <3496af23f5bdc7847148150cd7b97fb5d2b54702.1674060385.git.abdattar@amd.com>
[AMD Official Use Only - General]
Reviewed-by: Abner Chang <abner.chang@amd.com>
> -----Original Message-----
> From: Abdul Lateef Attar <abdattar@amd.com>
> Sent: Thursday, January 19, 2023 1:01 AM
> To: devel@edk2.groups.io
> Cc: Attar, AbdulLateef (Abdul Lateef) <AbdulLateef.Attar@amd.com>;
> Grimes, Paul <Paul.Grimes@amd.com>; Kirkendall, Garrett
> <Garrett.Kirkendall@amd.com>; Chang, Abner <Abner.Chang@amd.com>;
> Eric Dong <eric.dong@intel.com>; Ray Ni <ray.ni@intel.com>; Rahul Kumar
> <rahul1.kumar@intel.com>; Gerd Hoffmann <kraxel@redhat.com>
> Subject: [PATCH v3 2/6] UefiCpuPkg: Adds SmmSmramSaveStateLib library
> class
>
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4182
>
> Adds SmmSmramSaveStateLib Library class in UefiCpuPkg.dec.
> Adds function declaration header file.
>
> Cc: Paul Grimes <paul.grimes@amd.com>
> Cc: Garrett Kirkendall <garrett.kirkendall@amd.com>
> Cc: Abner Chang <abner.chang@amd.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Rahul Kumar <rahul1.kumar@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
>
> Signed-off-by: Abdul Lateef Attar <abdattar@amd.com>
> ---
> UefiCpuPkg/UefiCpuPkg.dec | 4 ++
> .../Include/Library/SmmSmramSaveStateLib.h | 70
> +++++++++++++++++++
> 2 files changed, 74 insertions(+)
> create mode 100644 UefiCpuPkg/Include/Library/SmmSmramSaveStateLib.h
>
> diff --git a/UefiCpuPkg/UefiCpuPkg.dec b/UefiCpuPkg/UefiCpuPkg.dec
> index cff239d5283e..1de90b677828 100644
> --- a/UefiCpuPkg/UefiCpuPkg.dec
> +++ b/UefiCpuPkg/UefiCpuPkg.dec
> @@ -2,6 +2,7 @@
> # This Package provides UEFI compatible CPU modules and libraries. # #
> Copyright (c) 2007 - 2022, Intel Corporation. All rights reserved.<BR>+#
> Copyright (C) 2023 Advanced Micro Devices, Inc. All rights reserved.<BR> # #
> SPDX-License-Identifier: BSD-2-Clause-Patent #@@ -65,6 +66,9 @@
> [LibraryClasses.IA32, LibraryClasses.X64]
> ## @libraryclass Provides function for manipulating x86 paging structures.
> CpuPageTableLib|Include/Library/CpuPageTableLib.h + ## @libraryclass
> Provides functions for manipulating Smram savestate registers.+
> SmmSmramSaveSateLib|Include/Library/SmmSmramSaveStateLib.h+ [Guids]
> gUefiCpuPkgTokenSpaceGuid = { 0xac05bf33, 0x995a, 0x4ed4, { 0xaa, 0xb8,
> 0xef, 0x7a, 0xe8, 0xf, 0x5c, 0xb0 }} gMsegSmramGuid = { 0x5802bce4,
> 0xeeee, 0x4e33, { 0xa1, 0x30, 0xeb, 0xad, 0x27, 0xf0, 0xe4, 0x39 }}diff --git
> a/UefiCpuPkg/Include/Library/SmmSmramSaveStateLib.h
> b/UefiCpuPkg/Include/Library/SmmSmramSaveStateLib.h
> new file mode 100644
> index 000000000000..46bc6381bcde
> --- /dev/null
> +++ b/UefiCpuPkg/Include/Library/SmmSmramSaveStateLib.h
> @@ -0,0 +1,70 @@
> +/** @file+Library that provides service to read/write CPU specific smram
> save state registers.++Copyright (c) 2010 - 2019, Intel Corporation. All rights
> reserved.<BR>+Copyright (C) 2023 Advanced Micro Devices, Inc. All rights
> reserved.<BR>++SPDX-License-Identifier: BSD-2-Clause-
> Patent++**/++#ifndef SMM_SMRAM_SAVE_STATE_LIB_H_+#define
> SMM_SMRAM_SAVE_STATE_LIB_H_++#include
> <Protocol/SmmCpu.h>+#include <Uefi/UefiBaseType.h>++/**+ Read an
> SMM Save State register on the target processor. If this function+ returns
> EFI_UNSUPPORTED, then the caller is responsible for reading the+ SMM
> Save Sate register.++ @param[in] CpuIndex The index of the CPU to read
> the SMM Save State. The+ value must be between 0 and the
> NumberOfCpus field in+ the System Management System Table
> (SMST).+ @param[in] Register The SMM Save State register to read.+
> @param[in] Width The number of bytes to read from the CPU save state.+
> @param[out] Buffer Upon return, this holds the CPU register value read+
> from the save state.++ @retval EFI_SUCCESS The register was read
> from Save State.+ @retval EFI_INVALID_PARAMTER Buffer is NULL.+
> @retval EFI_UNSUPPORTED This function does not support reading
> Register.+ @retval EFI_NOT_FOUND If desired Register not
> found.+**/+EFI_STATUS+EFIAPI+SmramSaveStateReadRegister (+ IN
> UINTN CpuIndex,+ IN EFI_SMM_SAVE_STATE_REGISTER
> Register,+ IN UINTN Width,+ OUT VOID
> *Buffer+ );++/**+ Writes an SMM Save State register on the target
> processor. If this function+ returns EFI_UNSUPPORTED, then the caller is
> responsible for writing the+ SMM Save Sate register.++ @param[in]
> CpuIndex The index of the CPU to write the SMM Save State. The+
> value must be between 0 and the NumberOfCpus field in+ the
> System Management System Table (SMST).+ @param[in] Register The
> SMM Save State register to write.+ @param[in] Width The number of
> bytes to write to the CPU save state.+ @param[in] Buffer Upon entry, this
> holds the new CPU register value.++ @retval EFI_SUCCESS The register
> was written to Save State.+ @retval EFI_INVALID_PARAMTER Buffer is
> NULL.+ @retval EFI_UNSUPPORTED This function does not support
> writing Register.+ @retval EFI_NOT_FOUND If desired Register not
> found.+**/+EFI_STATUS+EFIAPI+SmramSaveStateWriteRegister (+ IN
> UINTN CpuIndex,+ IN EFI_SMM_SAVE_STATE_REGISTER
> Register,+ IN UINTN Width,+ IN CONST VOID
> *Buffer+ );++#endif--
> 2.25.1
next prev parent reply other threads:[~2023-01-19 4:06 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-18 17:01 [PATCH v3 0/6] Adds AmdSmmCpuFeaturesLib for AMD Family Abdul Lateef Attar
2023-01-18 17:01 ` [PATCH v3 1/6] MdePkg: Adds AMD SMRAM save state map Abdul Lateef Attar
2023-01-19 4:05 ` Chang, Abner
2023-01-18 17:01 ` [PATCH v3 2/6] UefiCpuPkg: Adds SmmSmramSaveStateLib library class Abdul Lateef Attar
2023-01-19 4:05 ` Chang, Abner [this message]
2023-01-18 17:01 ` [PATCH v3 3/6] UefiCpuPkg: Implements " Abdul Lateef Attar
2023-01-19 3:44 ` Chang, Abner
2023-01-18 17:01 ` [PATCH v3 4/6] UefiCpuPkg/SmmCpuFeaturesLib: Restructure arch-dependent code Abdul Lateef Attar
2023-01-19 4:06 ` [edk2-devel] " Chang, Abner
2023-01-18 17:01 ` [PATCH v3 5/6] UefiCpuPkg: Initial implementation of AMD's SmmCpuFeaturesLib Abdul Lateef Attar
2023-01-19 4:08 ` Chang, Abner
2023-01-18 17:01 ` [PATCH v3 6/6] UefiCpuPkg: Implements SmmCpuFeaturesLib for AMD Family Abdul Lateef Attar
2023-01-19 4:10 ` Chang, Abner
2023-01-19 4:17 ` [PATCH v3 0/6] Adds AmdSmmCpuFeaturesLib " Chang, Abner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MN2PR12MB3966EA87E55152FAC1F65CF7EAC49@MN2PR12MB3966.namprd12.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox