]> git.proxmox.com Git - mirror_edk2.git/commit
UefiCpuPkg/AcpiCpuData.h: Remove AcpiNVS and Below 4G limitation.
authorEric Dong <eric.dong@intel.com>
Fri, 10 Aug 2018 02:28:50 +0000 (10:28 +0800)
committerEric Dong <eric.dong@intel.com>
Thu, 16 Aug 2018 00:42:00 +0000 (08:42 +0800)
commitfcd92f16af8fd61d220fbd58af3198083180ae6e
treea432fcb5c631158999583f74f35a88f3b75ac749
parent293f876618c319ca08687e55e126332216185ad6
UefiCpuPkg/AcpiCpuData.h: Remove AcpiNVS and Below 4G limitation.

ACPI_CPU_DATA structure first introduced to save data in
normal boot phase. Also this data will be used in S3 phase
by one PEI driver. So in first phase, this data is been
defined to use ACPI NVS memory type and must below 4G.

Later in order to fix potential security issue,
PiSmmCpuDxeSmm driver added logic to copy ACPI_CPU_DATA
(except ResetVector and Stack buffer) to  smram at smm
ready to lock point. ResetVector must below 1M and Stack
buffer is write only in S3 phase, so these two fields not
copy to smram. Also PiSmmCpuDxeSmm driver owned the task
to restore the CPU setting and it's a SMM driver.

After above change, the acpi nvs memory type and below 4G
limitation is no longer needed.

This change remove the limitation in the comments for
ACPI_CPU_DATA definition.

Cc: Marvin Häuser <Marvin.Haeuser@outlook.com>
Cc: Fan Jeff <vanjeff_919@hotmail.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>
Tested-by: Laszlo Ersek <lersek@redhat.com>
UefiCpuPkg/Include/AcpiCpuData.h