On 2/22/24 03:39, Zhou Jianfeng wrote: > Some IN OUT parameters in CpuPageTableMap.c were mistakenly marked as IN. > "IN" replaced with "IN OUT" in the following interfaces: > > PageTableLibSetPte4K(): Pte4K > PageTableLibSetPleB(): PleB > PageTableLibSetPle(): Ple > PageTableLibSetPnle(): Pnle > > Reviewed-by: Ray Ni <[email protected]> > Signed-off-by: Zhou Jianfeng <[email protected]> > Cc: Ray Ni <[email protected]> > Cc: Laszlo Ersek <[email protected]> > Cc: Rahul Kumar <[email protected]> > Cc: Gerd Hoffmann <[email protected]> > --- > .../Library/CpuPageTableLib/CpuPageTableMap.c | 32 +++++++++---------- > 1 file changed, 16 insertions(+), 16 deletions(-) > > diff --git a/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableMap.c > b/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableMap.c > index ae4caf8dfe..2ea40666cc 100644 > --- a/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableMap.c > +++ b/UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableMap.c > @@ -20,10 +20,10 @@ > **/ > VOID > PageTableLibSetPte4K ( > - IN IA32_PTE_4K *Pte4K, > - IN UINT64 Offset, > - IN IA32_MAP_ATTRIBUTE *Attribute, > - IN IA32_MAP_ATTRIBUTE *Mask > + IN OUT IA32_PTE_4K *Pte4K, > + IN UINT64 Offset, > + IN IA32_MAP_ATTRIBUTE *Attribute, > + IN IA32_MAP_ATTRIBUTE *Mask > ) > { > IA32_PTE_4K LocalPte4K; > @@ -94,10 +94,10 @@ PageTableLibSetPte4K ( > **/ > VOID > PageTableLibSetPleB ( > - IN IA32_PAGE_LEAF_ENTRY_BIG_PAGESIZE *PleB, > - IN UINT64 Offset, > - IN IA32_MAP_ATTRIBUTE *Attribute, > - IN IA32_MAP_ATTRIBUTE *Mask > + IN OUT IA32_PAGE_LEAF_ENTRY_BIG_PAGESIZE *PleB, > + IN UINT64 Offset, > + IN IA32_MAP_ATTRIBUTE *Attribute, > + IN IA32_MAP_ATTRIBUTE *Mask > ) > { > IA32_PAGE_LEAF_ENTRY_BIG_PAGESIZE LocalPleB; > @@ -171,11 +171,11 @@ PageTableLibSetPleB ( > **/ > VOID > PageTableLibSetPle ( > - IN UINTN Level, > - IN IA32_PAGING_ENTRY *Ple, > - IN UINT64 Offset, > - IN IA32_MAP_ATTRIBUTE *Attribute, > - IN IA32_MAP_ATTRIBUTE *Mask > + IN UINTN Level, > + IN OUT IA32_PAGING_ENTRY *Ple, > + IN UINT64 Offset, > + IN IA32_MAP_ATTRIBUTE *Attribute, > + IN IA32_MAP_ATTRIBUTE *Mask > ) > { > if (Level == 1) { > @@ -195,9 +195,9 @@ PageTableLibSetPle ( > **/ > VOID > PageTableLibSetPnle ( > - IN IA32_PAGE_NON_LEAF_ENTRY *Pnle, > - IN IA32_MAP_ATTRIBUTE *Attribute, > - IN IA32_MAP_ATTRIBUTE *Mask > + IN OUT IA32_PAGE_NON_LEAF_ENTRY *Pnle, > + IN IA32_MAP_ATTRIBUTE *Attribute, > + IN IA32_MAP_ATTRIBUTE *Mask > ) > { > IA32_PAGE_NON_LEAF_ENTRY LocalPnle; > -- > 2.31.1.windows.1 >
Reviewed-by: Laszlo Ersek <[email protected]> I can't start merging this immediately; we probably have a long list of patches that can and should now be merged, after the stable tag. It would be nice if patch submitters could ping the maintainers on their patch sets that are now ready for merging; we should wait for such pings for a few days, then mass-merge the bunch. Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115927): https://edk2.groups.io/g/devel/message/115927 Mute This Topic: https://groups.io/mt/104524856/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
