ArmPkg: remove DebugUncachedMemoryAllocationLib
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Thu, 23 Feb 2017 14:38:32 +0000 (14:38 +0000)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Thu, 23 Feb 2017 17:56:10 +0000 (17:56 +0000)
commita76d0e88c39a10cef284eedec3bd4b57c4d83ad2
treed5f5bacff3028d26e44e1c751f77dc05947304f5
parentef2240327863b0d1df79fc73429803ca671316d1
ArmPkg: remove DebugUncachedMemoryAllocationLib

The debug implementation of the UncachedMemoryAllocationLib library
class relies on the creation of an uncached alias of a memory range,
while keeping the original cached mapping, but with read-only attributes
to trap inadvertent write accesses.

This is not a terribly good idea, given that the ARM architecture does
not allow mismatched attributes, and so creating them deliberately is
not something we should encourage by doing it in reference code.

So remove the library, and replace all references to it with a reference
to the non-debug version (unless the platform does not require a resolution
for it in the first place, in which case all UncachedMemoryAllocationLib
references can be removed altogether).

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
ArmPkg/ArmPkg.dsc
ArmPkg/Library/DebugUncachedMemoryAllocationLib/DebugUncachedMemoryAllocationLib.c [deleted file]
ArmPkg/Library/DebugUncachedMemoryAllocationLib/DebugUncachedMemoryAllocationLib.inf [deleted file]
ArmVirtPkg/ArmVirt.dsc.inc
BeagleBoardPkg/BeagleBoardPkg.dsc
Omap35xxPkg/Omap35xxPkg.dsc