Re: [PATCH v5 0/5] MdeModulePkg/CapsulePei: Optimize the CapsulePei

["Wu, Hao A" <hao.a.wu@intel.com>]

> -----Original Message-----
> From: Gao, Zhichao
> Sent: Monday, June 10, 2019 1:21 PM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J; Wu, Hao A; Ni, Ray; Zeng, Star; Gao, Liming; Sean Brogan;
> Michael Turner; Bret Barkelew; Leif Lindholm
> Subject: [PATCH v5 0/5] MdeModulePkg/CapsulePei: Optimize the
> CapsulePei
> 
> v1:
> Sperate the capsule check function from GetCapsuleDescriptors
> and name it to AreCapsulesStaged.
> Rename GetCapsuleDescriptors to GetScatterGatherHeadEntries.
> And optimize its to remove the duplicated code.
> 
> V2:
> Clarify the changes of the patch and descript in the commit
> message.
> 
> V3:
> Add the code change repo location in Mu and add description of
> the changes on it.
> Do not need to remain 2 byte for the null-terminate while using
> UnicodeValueToStringS.
> 
> V4:
> Separate the patch into 2 part. first part is changes from MU.
> Second part is the substantial change on it.
> 
> V5:
> Separate the substantial change in 2/2 base on the purpose.


With the comment from Leif handled, for the series:
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>

Best Regards,
Hao Wu


> 
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>
> Cc: Michael Turner <Michael.Turner@microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> 
> Bret Barkelew (1):
>   MdeModulePkg/CapsulePei: Optimize the CapsulePei
> 
> Zhichao Gao (4):
>   MdeMoudlePkg/CapsulePei: Fix coding style issue
>   MdeMoudlePkg/CapsulePei: Optimize AreCapsulesStaged
>   MdeMoudlePkg/CapsulePei: Optimize GetScatterGatherHeadEntries
>   MdeMoudlePkg/CapsulePei: No need to remain space for null-terminate
> 
>  MdeModulePkg/Universal/CapsulePei/Capsule.h   |   1 +
>  .../Universal/CapsulePei/CapsulePei.inf       |   1 +
>  .../Universal/CapsulePei/UefiCapsule.c        | 356 +++++++++---------
>  3 files changed, 190 insertions(+), 168 deletions(-)
> 
> --
> 2.21.0.windows.1