diff options
author | Dun Tan <dun.tan@intel.com> | 2024-05-07 17:00:56 +0800 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2024-06-04 12:38:54 +0000 |
commit | 839bd179735284592ba8f0879d2cbf07e0cb585a (patch) | |
tree | e05cab6684197c774efc79f1222c0c7f360ed474 /MdeModulePkg | |
parent | 077760fec40c2e5fcae274cc609d97aee12e5d56 (diff) | |
download | edk2-839bd179735284592ba8f0879d2cbf07e0cb585a.tar.gz edk2-839bd179735284592ba8f0879d2cbf07e0cb585a.tar.bz2 edk2-839bd179735284592ba8f0879d2cbf07e0cb585a.zip |
UefiCpuPkg:fix issue when splitting paging entry
This patch is to fix issue when splitting leaf paging
entry in CpuPageTableLib code.
In previous code, before we assign the new child paging
structure address to the content of splitted paging entry,
PageTableLibSetPnle() is called to make sure the bit7 is
set to 0, which indicate the previous leaf entry is
changed to non-leaf entry now. There is a gap between
we change the bit7 and we assign the new child paging
structure address to the content of the splitted paging
entry. If the address of code execution or data access
happens to be in the range covered by the splitted paging
entry, this gap may cause issue.
In this patch, we prepare the new paging entry content
value in a local variable and assign the value to the
splitted paging entry at once. The volatile keyword
is used to ensure that no optimization will occur in
compilation.
Signed-off-by: Dun Tan <dun.tan@intel.com>
Reviewed-by: Ray Ni <ray.ni@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Jiaxin Wu <jiaxin.wu@intel.com>
Cc: Zhou Jianfeng <jianfeng.zhou@intel.com>
Diffstat (limited to 'MdeModulePkg')
0 files changed, 0 insertions, 0 deletions