Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-11-02 Thread Gao, Liming
g, Star >Subject: RE: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all >module types > >Liming, > >Will you help push the patch series? > > >Thanks, >Star > >-Original Message- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf O

Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-11-01 Thread Zeng, Star
, Liming Subject: Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types Jeff: Thanks for your case. PrePiMemoryAllocationLib is a MemoryAllocationLib implementation that doesn't depend on PEI or DXE service. So, MemoryAllocationLib may be used in BASE type. And, I

Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-11-01 Thread Gao, Liming
56:42 AM To: Jeff Brasen; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Gao, Liming; Kinney, Michael D Subject: RE: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types SortLib depends on MemoryAllocationLib. How can it be a BASE type library? Can we do it

Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-10-31 Thread Jeff Brasen
, Michael D Subject: RE: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types SortLib depends on MemoryAllocationLib. How can it be a BASE type library? Can we do it in two options? 1. Change the SortLib API to accept a temporary buffer from caller. So the MemoryAllocationlIb

Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-10-31 Thread Ni, Ruiyu
internally. E.g.: UINT8 Buffer[MAX_ELEMENT_SIZE]; Thanks/Ray > -Original Message- > From: edk2-devel On Behalf Of Jeff > Brasen > Sent: Wednesday, October 31, 2018 5:31 AM > To: edk2-devel@lists.01.org > Cc: Jeff Brasen > Subject: [edk2] [PATCH 2/2] MdeModulePkg/Ba

Re: [edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-10-30 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Jeff Brasen > Sent: Tuesday, October 30, 2018 2:31 PM > To: edk2-devel@lists.01.org > Cc: Jeff Brasen > Subject: [edk2] [PATCH 2/2] MdeModulePk

[edk2] [PATCH 2/2] MdeModulePkg/BaseSortLib: Enable for all module types

2018-10-30 Thread Jeff Brasen
Expose BaseSortLib for use in SEC and PEI phases. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jeff Brasen --- MdeModulePkg/Library/BaseSortLib/BaseSortLib.inf | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MdeModulePkg/Library/BaseSortLib/BaseS