Re: [edk2] [Patch v2 2/2] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation.

2018-08-09 Thread Dong, Eric
Re: [edk2] [Patch v2 2/2] UefiCpuPkg/CpuS3DataDxe: Change > Memory Type and address limitation. > > Hello Eric, > > On 08/08/18 09:40, Eric Dong wrote: > > Because CpuS3Data memory will be copy to smram at SmmReadToLock > point, > > so the memory type no need to be ACP

Re: [edk2] [Patch v2 2/2] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation.

2018-08-08 Thread Ni, Ruiyu
> (3) The "UefiCpuPkg/Include/AcpiCpuData.h" header states that > ACPI_CPU_DATA (the structure itself) "must be allocated below 4GB from > memory of type EfiACPIMemoryNVS". > > If we have determined that this is no longer necessary, then: > - we should first update the documentation in "AcpiCpuDat

Re: [edk2] [Patch v2 2/2] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation.

2018-08-08 Thread Laszlo Ersek
Hello Eric, On 08/08/18 09:40, Eric Dong wrote: > Because CpuS3Data memory will be copy to smram at SmmReadToLock point, > so the memory type no need to be ACPI NVS type, also the address not > limit to below 4G. > This change remove the limit of ACPI NVS memory type and below 4G. > > Pass OS boot

[edk2] [Patch v2 2/2] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation.

2018-08-08 Thread Eric Dong
Because CpuS3Data memory will be copy to smram at SmmReadToLock point, so the memory type no need to be ACPI NVS type, also the address not limit to below 4G. This change remove the limit of ACPI NVS memory type and below 4G. Pass OS boot and resume from S3 test. Bugz: https://bugzilla.tianocore.