> > > + ASSERT (SizeOfMemorySpace <= 52);
> > > +
> > > //
> > > - // Calculate the table entries of PML4E and PDPTE.
> > > + // Calculate the table entries of PML5E, PML4E and PDPTE.
> > > //
> > > NumberOfPml5Entries = 1;
> > > - if (SizeOfMemorySpace > 48) {
> > > + if (Enable5LevelPaging && (SizeOfMemorySpace > 48)) {
> > > NumberOfPml5Entries = (UINTN)LShiftU64 (1, SizeOfMemorySpace - 48);
> > > - SizeOfMemorySpace = 48;
> > > }
> > >
> > > + SizeOfMemorySpace = SizeOfMemorySpace > 48 ? 48 :
> > > SizeOfMemorySpace;
>
> if (SizeOfMemorySpace > 48) {
> if (Enable5LevelPaging) {
> NumberOfPml5Entries = ...
> }
> SizeOfMemorySpace = 48
> }
>
> That is a much more readable version.
I had the same thought. New version is consistent with the logic below.
>
> The only effect I can see is that this avoids creating page tables which
> would not be used anyway.
>
> Can you explain where the hangs mentioned in the subject line are coming
> from and why the patch fixes them?
>
> take care,
> Gerd
>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#98684): https://edk2.groups.io/g/devel/message/98684
Mute This Topic: https://groups.io/mt/96045489/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
