Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upstream/dynamic tables smbios #10

Open
wants to merge 2 commits into
base: main-edk2-stable202205
Choose a base branch
from

Conversation

gmahadevan
Copy link
Contributor

Enable SMBIOS Table Generation using Dynamic Tables Package.
There are 2 commits:

DynamicTablesPkg: Add SMBIOS table generation
3239a7b
This patch makes changes to the DynamicTableManagerDxe to install SMBIOS tables and includes a change to enable installing multiple SMBIOS tables. Also we move the SMBIOS and ACPI table installations into their respective Protocol Ready callbacks as it is possible that ACPI isn't present on all platforms and in those cases SMBIOS tables won't be installed.
DynamicTablesPkg: Add Smbios Type17 Table generator Lib
5bb1334)
This patch is an example of an SMBIOS table that can be installed using DynamicTablesPkg. A new CM object describing the memory device is added to the ARM CM Object space, The library Gets the CM Object and sets up the Type17 record using the CM Object describing the memory device.

Add the SMBIOS Table generator code to the DynamicTablesPkg.

This change includes adding new logic to the DynamicTableManager
to process and add SMBIOS tables and augmenting the existing SMBIOS
Factory generator to include installing multiple SMBIOS tables.
Also included is running the SMBIOS and ACPI table generation as part
of the SMBIOS and ACPI protocol GUID callback notifications respectively.

Signed-off-by: Girish Mahadevan <[email protected]>
Add a new library to register the SMBIOS Type17 Table Generator
Library.

Signed-off-by: Girish Mahadevan <[email protected]>
@jbrasen jbrasen added Pending EDK2 Review EDK2 maintainer review needed Pending EDK2 merge Change is not in tianocore/master Pending EDK2 stable release Change is not in most recent edk2-stable tag labels Sep 8, 2022
@jbrasen jbrasen added In NVIDIA repo Changes are part of our builds prior to stable tag and removed Pending EDK2 merge Change is not in tianocore/master Pending EDK2 stable release Change is not in most recent edk2-stable tag labels Sep 18, 2023
jgarver pushed a commit that referenced this pull request Oct 12, 2024
This patch does not impact functionality. It aims to clarify the
synchronization flow between the BSP and APs to enhance code
readability and understanding:

Steps #6 and #11 are the basic synchronization requirements for all
cases.

Steps #1 is additional requirements if the MmCpuSyncModeTradition
mode is selected.

Steps #1, #2, #3, #4, #5, #7, #8, #9, and #10 are additional
requirements if the system needs to configure the MTRR.

Steps #9 and #10 are additional requirements if the system needs to
support the mSmmDebugAgentSupport.

Signed-off-by: Jiaxin Wu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In NVIDIA repo Changes are part of our builds prior to stable tag Pending EDK2 Review EDK2 maintainer review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants