[edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.

Yuanhao Xie posted 3 patches 5 months ago
Failed in applying to current master (apply log)
UefiCpuPkg/Library/MpInitLib/Microcode.c | 85 +++++++++++++++++++++++++++++++++++++++----------------------------------------------
UefiCpuPkg/Library/MpInitLib/MpLib.c     | 81 ++++++++++++++++++++++++++++++++-------------------------------------------------
UefiCpuPkg/Library/MpInitLib/MpLib.h     | 31 ++++++++++---------------------
3 files changed, 81 insertions(+), 116 deletions(-)
[edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.
Posted by Yuanhao Xie 5 months ago
The DXE stage's Microcode loading process has been eliminated.
Store BSP's MTRR setting only when CpuCount>1.
Extract Dump Microcode Revision as a function

Compare with V3, this version updates the comments,
Move GetMicrocodePatchInfoFromHob removal from patch 3 to patch 1

on
xieyuanh (3):
  UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.
  UefiCpuPkg/MpInitLib: Store MTRRs settings only when  CpuCount>1
  UefiCpuPkg/MpInitLib: Extract Dump Microcode Revision as function.

 UefiCpuPkg/Library/MpInitLib/Microcode.c | 85 +++++++++++++++++++++++++++++++++++++++----------------------------------------------
 UefiCpuPkg/Library/MpInitLib/MpLib.c     | 81 ++++++++++++++++++++++++++++++++-------------------------------------------------
 UefiCpuPkg/Library/MpInitLib/MpLib.h     | 31 ++++++++++---------------------
 3 files changed, 81 insertions(+), 116 deletions(-)

--
2.39.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#111686): https://edk2.groups.io/g/devel/message/111686
Mute This Topic: https://groups.io/mt/102775770/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.
Posted by Ni, Ray 5 months ago
Reviewed-by: Ray Ni <ray.ni@intel.com>

Thanks,
Ray
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yuanhao
> Xie
> Sent: Friday, November 24, 2023 10:57 AM
> To: devel@edk2.groups.io
> Cc: Xie, Yuanhao <yuanhao.xie@intel.com>
> Subject: [edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate
> redundant microcode loading in DXE.
> 
> The DXE stage's Microcode loading process has been eliminated.
> Store BSP's MTRR setting only when CpuCount>1.
> Extract Dump Microcode Revision as a function
> 
> Compare with V3, this version updates the comments,
> Move GetMicrocodePatchInfoFromHob removal from patch 3 to patch 1
> 
> on
> xieyuanh (3):
>   UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.
>   UefiCpuPkg/MpInitLib: Store MTRRs settings only when  CpuCount>1
>   UefiCpuPkg/MpInitLib: Extract Dump Microcode Revision as function.
> 
>  UefiCpuPkg/Library/MpInitLib/Microcode.c | 85
> +++++++++++++++++++++++++++++++++++++++--------------------------------------
> --------
>  UefiCpuPkg/Library/MpInitLib/MpLib.c     | 81
> ++++++++++++++++++++++++++++++++-------------------------------------------------
>  UefiCpuPkg/Library/MpInitLib/MpLib.h     | 31
> ++++++++++---------------------
>  3 files changed, 81 insertions(+), 116 deletions(-)
> 
> --
> 2.39.1.windows.1
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#111693): https://edk2.groups.io/g/devel/message/111693
Mute This Topic: https://groups.io/mt/102775770/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.
Posted by Laszlo Ersek 4 months, 2 weeks ago
On 11/24/23 04:45, Ni, Ray wrote:
> Reviewed-by: Ray Ni <ray.ni@intel.com>

Please go ahead with merging this. The patch set has changed so much
since v2 (which I last reviewed) that I can't perform an incremental
review, and I can't review this from zero at the moment.

Thanks
Laszlo

> 
> Thanks,
> Ray
>> -----Original Message-----
>> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yuanhao
>> Xie
>> Sent: Friday, November 24, 2023 10:57 AM
>> To: devel@edk2.groups.io
>> Cc: Xie, Yuanhao <yuanhao.xie@intel.com>
>> Subject: [edk2-devel] [Patch V4 0/3] UefiCpuPkg/MpInitLib: Eliminate
>> redundant microcode loading in DXE.
>>
>> The DXE stage's Microcode loading process has been eliminated.
>> Store BSP's MTRR setting only when CpuCount>1.
>> Extract Dump Microcode Revision as a function
>>
>> Compare with V3, this version updates the comments,
>> Move GetMicrocodePatchInfoFromHob removal from patch 3 to patch 1
>>
>> on
>> xieyuanh (3):
>>   UefiCpuPkg/MpInitLib: Eliminate redundant microcode loading in DXE.
>>   UefiCpuPkg/MpInitLib: Store MTRRs settings only when  CpuCount>1
>>   UefiCpuPkg/MpInitLib: Extract Dump Microcode Revision as function.
>>
>>  UefiCpuPkg/Library/MpInitLib/Microcode.c | 85
>> +++++++++++++++++++++++++++++++++++++++--------------------------------------
>> --------
>>  UefiCpuPkg/Library/MpInitLib/MpLib.c     | 81
>> ++++++++++++++++++++++++++++++++-------------------------------------------------
>>  UefiCpuPkg/Library/MpInitLib/MpLib.h     | 31
>> ++++++++++---------------------
>>  3 files changed, 81 insertions(+), 116 deletions(-)
>>
>> --
>> 2.39.1.windows.1
>>
>>
>>
>>
>>
> 
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112325): https://edk2.groups.io/g/devel/message/112325
Mute This Topic: https://groups.io/mt/102775770/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-