> + } else { > + MemoryAttrMask = EFI_MEMORY_XP; > + for (Index = 0; Index < mSmmCpuSmramRangeCount; Index++) { > + Base = mSmmCpuSmramRanges[Index].CpuStart; > + if ((Base > PreviousAddress) && mXdSupported) {
Is "mXdSupported" check really needed? But you didn't add that check for the last remaining range. ConvertMemoryPageAttributes() can handle the case when XD is not supported by CPU. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#105632): https://edk2.groups.io/g/devel/message/105632 Mute This Topic: https://groups.io/mt/98922942/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-