Gaurav, You sent to the old list. Adding the new edk2-devel list
> > > -----Original Message----- > > From: Gaurav Jain <gaurav.j...@nxp.com> > > Sent: Friday, April 17, 2020 2:55 AM > > To: edk2-de...@lists.01.org > > Cc: Eric Jin <eric....@intel.com>; Supreeth Venkatesh > > <supreeth.venkat...@arm.com>; Samer El-Haj-Mahmoud <Samer.El-Haj- > > mahm...@arm.com>; Prasanth Pulla <prasanth.pu...@arm.com>; Pankaj > > Bansal <pankaj.ban...@nxp.com> > > Subject: RE: [edk2-test PATCH] SctPkg: Updated Start Address Alignment > > code > > > > Hi > > > > Gentle Reminder!! > > Please help to review patch. > > > > Regards > > Gaurav Jain > > > > > -----Original Message----- > > > From: Gaurav Jain <gaurav.j...@nxp.com> > > > Sent: Thursday, April 9, 2020 6:33 PM > > > To: edk2-de...@lists.01.org > > > Cc: Eric Jin <eric....@intel.com>; Supreeth Venkatesh > > > <supreeth.venkat...@arm.com>; Samer El-Haj-Mahmoud <Samer.El-Haj- > > > mahm...@arm.com>; Prasanth Pulla <prasanth.pu...@arm.com>; Pankaj > > > Bansal <pankaj.ban...@nxp.com>; Gaurav Jain <gaurav.j...@nxp.com> > > > Subject: [edk2-test PATCH] SctPkg: Updated Start Address Alignment > > > code > > > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2671 > > > > > > AllocatePages Functionality test. > > > Updated Start Address Calculation for aligning to 64k. > > > > > > Signed-off-by: Gaurav Jain <gaurav.j...@nxp.com> > > > --- > > > .../BlackBoxTest/MemoryAllocationServicesBBTestFunction.c | 14 > > > +++++++-- > > > ----- > > > 1 file changed, 7 insertions(+), 7 deletions(-) > > > > > > diff --git a/uefi- > > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/MemoryAllocationServices/B > > > la ck BoxTest/MemoryAllocationServicesBBTestFunction.c b/uefi- > > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/MemoryAllocationServices/B > > > la ck BoxTest/MemoryAllocationServicesBBTestFunction.c > > > index d18fe1f..a42cd9a 100644 > > > --- a/uefi- > > > sct/SctPkg/TestCase/UEFI/EFI/BootServices/MemoryAllocationServices/B > > > la ck BoxTest/MemoryAllocationServicesBBTestFunction.c > > > +++ b/uefi-sct/SctPkg/TestCase/UEFI/EFI/BootServices/MemoryAllocatio > > > +++ nS er > > > +++ vices/BlackBoxTest/MemoryAllocationServicesBBTestFunction.c > > > @@ -706,7 +706,7 @@ BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start; > > > @@ -836,7 +836,7 @@ BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start; > > > @@ -959,7 +959,7 @@ BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start + (SctLShiftU64 (PageNum/3, EFI_PAGE_SHIFT) > > > & 0xFFFFFFFFFFFF0000); @@ -1082,7 +1082,7 @@ > > > BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start + (SctLShiftU64 (PageNum * 2 / 3, > > > EFI_PAGE_SHIFT) & 0xFFFFFFFFFFFF0000); @@ -1212,7 +1212,7 @@ > > > BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start; > > > @@ -1335,7 +1335,7 @@ BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start; > > > @@ -1474,7 +1474,7 @@ BBTestAllocatePagesInterfaceTest ( > > > if (PageNum <= 0x10) { > > > break; > > > } > > > - Start = (Start + 0x10000) & 0xFFFFFFFFFFFF0000; > > > + Start = (Start + 0xFFFF) & 0xFFFFFFFFFFFF0000; > > > PageNum = PageNum - EFI_SIZE_TO_PAGES(0x10000); > > > > > > Memory = Start; > > > -- > > > 2.7.4 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#57507): https://edk2.groups.io/g/devel/message/57507 Mute This Topic: https://groups.io/mt/73082849/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-