MtrrSetAllMtrrs() maybe used by APs to sync BSP's MTRR settings. BSP's MTRR setting should be displayed if EFI_D_CACHE flag is set when MTRR updated. In MtrrSetAllMtrrs(), it's not necessary to display MTRR setting again due to the MTRR settings should be always same among BSP/APs. This updating could avoid APs output MTRR setting at the same time and make display message corrupted.
Cc: Feng Tian <[email protected]> Cc: Michael Kinney <[email protected]> Cc: Laszlo Ersek <[email protected]> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan <[email protected]> Reviewed-by: Laszlo Ersek <[email protected]> Reviewed-by: Michael Kinney <[email protected]> --- UefiCpuPkg/Library/MtrrLib/MtrrLib.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c index 6a6bf76..f667a8f 100644 --- a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c +++ b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c @@ -2103,8 +2103,6 @@ MtrrSetAllMtrrs ( PostMtrrChangeEnableCache (&MtrrContext); - MtrrDebugPrintAllMtrrs (); - return MtrrSetting; } -- 2.7.4.windows.1 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
