]> git.proxmox.com Git - mirror_edk2.git/commit
ArmPkg/AArch64Mmu: don't let table entries inherit XN permission bits
authorArd Biesheuvel <ard.biesheuvel@linaro.org>
Wed, 20 Apr 2016 08:07:20 +0000 (10:07 +0200)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Tue, 10 May 2016 12:44:40 +0000 (14:44 +0200)
commitd1ec2b2f78b752b9dfc06843f994cbfad134ea63
treecb3777e2f058e073bf64b2e8476762c6751a5d49
parentb64e44cc098ffd03ccbf1a635ae405ae98971419
ArmPkg/AArch64Mmu: don't let table entries inherit XN permission bits

When we split a block entry into a table entry, the UXN/PXN/XN permission
attributes are inherited both by the new table entry and by the new block
entries at the next level down. Unlike the NS bit, which only affects the
next level of lookup, the XN table bits supersede the permissions of the
final translation, and setting the permissions at multiple levels is not
only redundant, it also prevents us from lifting XN restrictions on a
subregion of the original block entry by simply clearing the appropriate
bits at the lowest level.

So drop the code that sets the UXN/PXN/XN bits on the table entries.

Reported-by: "Oliyil Kunnil, Vishal" <vishalo@qti.qualcomm.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
ArmPkg/Library/ArmLib/AArch64/AArch64Mmu.c