ArmPlatformPkg/MemoryInitPeiLib: revert "don't reserve primary FV in memory"
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Wed, 28 Feb 2018 15:27:15 +0000 (15:27 +0000)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Wed, 28 Feb 2018 16:10:49 +0000 (16:10 +0000)
commite7f0a829c97b7133e245f5e4d1fec5c7e28784e1
treead896f744126624e3ff2fa35860e11a826d0c4b2
parentf3b314331ce1557d51bbd6930ca9bec01ceefdd7
ArmPlatformPkg/MemoryInitPeiLib: revert "don't reserve primary FV in memory"

Commit 8ae5fc182941 ("ArmPlatformPkg/MemoryInitPeiLib: don't reserve
primary FV in memory") deleted the code that removes the memory covering
the primary firmware volume from the memory map. The assumption was that
this is no longer necessary now that we no longer expose compression and
PE/COFF loader library code from the PrePi module to DXE core.

However, the FV is still declared, and so code may attempt to access it
anyway, which may cause unexpected results depending on whether the
memory has been reused for other purposes in the mean time. So revert
this change in preparation of dealing with this in a better way.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
ArmPlatformPkg/MemoryInitPei/MemoryInitPeiLib.c