UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
Support PAE paging for PageTableParse API in CpuPageTableLib.
Signed-off-by: Dun Tan <dun.tan@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>
---
UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c b/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c
index e66961e122..65490751ab 100644
--- a/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c
+++ b/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c
@@ -270,10 +270,9 @@ PageTableParse (
IA32_MAP_ENTRY OneEntry;
UINTN MaxLevel;
- if ((PagingMode == Paging32bit) || (PagingMode == PagingPae) || (PagingMode >= PagingModeMax)) {
+ if ((PagingMode == Paging32bit) || (PagingMode >= PagingModeMax)) {
//
// 32bit paging is never supported.
- // PAE paging will be supported later.
//
return RETURN_UNSUPPORTED;
}
--
2.31.1.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94968): https://edk2.groups.io/g/devel/message/94968
Mute This Topic: https://groups.io/mt/94253417/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
I didn't enable the PAE support because I found slightly differences in logic that creates the PAE paging structure. I agree that PageTableParse() supporting PAE is more straightforward. Reviewed-by: Ray Ni <ray.ni@Intel.com> > -----Original Message----- > From: Tan, Dun <dun.tan@intel.com> > Sent: Tuesday, October 11, 2022 2:00 PM > To: devel@edk2.groups.io > Cc: Dong, Eric <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Kumar, > Rahul R <rahul.r.kumar@intel.com> > Subject: [PATCH] UefiCpuPkg/CpuPageTableLib:Support PAE paging for > PageTableParse > > Support PAE paging for PageTableParse API in CpuPageTableLib. > > Signed-off-by: Dun Tan <dun.tan@intel.com> > Cc: Eric Dong <eric.dong@intel.com> > Cc: Ray Ni <ray.ni@intel.com> > Cc: Rahul Kumar <rahul1.kumar@intel.com> > --- > UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c > b/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c > index e66961e122..65490751ab 100644 > --- a/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c > +++ b/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableParse.c > @@ -270,10 +270,9 @@ PageTableParse ( > IA32_MAP_ENTRY OneEntry; > UINTN MaxLevel; > > - if ((PagingMode == Paging32bit) || (PagingMode == PagingPae) || > (PagingMode >= PagingModeMax)) { > + if ((PagingMode == Paging32bit) || (PagingMode >= PagingModeMax)) { > // > // 32bit paging is never supported. > - // PAE paging will be supported later. > // > return RETURN_UNSUPPORTED; > } > -- > 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94974): https://edk2.groups.io/g/devel/message/94974 Mute This Topic: https://groups.io/mt/94253417/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.