Re: [edk2] [PATCH v1 3/6] MdeModulePkg: Map persistent (NV) memory

2018-10-13 Thread Laszlo Ersek
(+Grant) On 10/12/18 16:40, Sami Mujawar wrote: > Some platforms are able to preserve a memory range across > system resets. This memory can be used for the Non-Volatile > variables storage. The PcdEmuVariableNvStoreReserved is > used to specify this option. This is the wrong approach, for two

Re: [edk2] [PATCH v1 3/6] MdeModulePkg: Map persistent (NV) memory

2018-10-13 Thread Zeng, Star
Hi Sami, Thanks for the patch. I just checked this patch and roughly checked [PATCH v1 5/6] ArmVirtPkg: Add kvmtool platform driver. I would like to suggest that the code added in this patch to be moved into KvmtoolPlatformDxe which is added by [PATCH v1 5/6] as KvmtoolPlatformDxe is the

[edk2] [PATCH v1 3/6] MdeModulePkg: Map persistent (NV) memory

2018-10-12 Thread Sami Mujawar
Some platforms are able to preserve a memory range across system resets. This memory can be used for the Non-Volatile variables storage. The PcdEmuVariableNvStoreReserved is used to specify this option. This patch enables mapping of this Non-Volatile memory range as runtime memory so that the