On 11/24/23 04:45, Ni, Ray wrote: > Reviewed-by: Ray Ni <ray...@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....@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
