Drew reported that the ArmVirtQemu platform logs several annoying / confusing messages even when it is built with -D DEBUG_PRINT_ERROR_LEVEL=0x80000000 (i.e., enabling only EFI_D_ERROR in PcdDebugPrintErrorLevel).
There seems to be a constant arms race in edk2 between people who want platform builds to be as silent as possible, and people who want *their* messages to be emitted. For distributors, the solution is obviously to distribute two firmware builds, one verbose and another silent (not unlike DEBUG vs. RELEASE builds), whereas for developers, the PcdDebugPrintErrorLevel PCD can be set in the DSC file for each module in separation. Hard-coding EFI_D_ERROR in the source for various "important" informative messages is probably the worst workaround, so let's not do that. This patch series downgrades a number of EFI_D_ERROR messages to EFI_D_INFO, in NorFlashDxe, FaultTolerantWriteDxe, and SmbiosDxe. Cc: Ard Biesheuvel <[email protected]> Cc: Liming Gao <[email protected]> Cc: Drew Jones <[email protected]> Cc: Elvin Li <[email protected]> Cc: Leif Lindholm <[email protected]> Cc: Star Zeng <[email protected]> Cc: Yehuda Yitschak <[email protected]> Thanks Laszlo Laszlo Ersek (3): ArmPlatformPkg: NorFlashDxe: mellow DEBUG messages about flash reinit MdeModulePkg: FaultTolerantWriteDxe: mellow DEBUGs about workspace reinit MdeModulePkg: SmbiosDxe: soften DEBUG messages about table reallocation ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashFvbDxe.c | 18 ++++++++++++------ MdeModulePkg/Universal/FaultTolerantWriteDxe/FtwMisc.c | 6 ++++-- MdeModulePkg/Universal/FaultTolerantWriteDxe/UpdateWorkingBlock.c | 2 +- MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c | 6 ++++-- 4 files changed, 21 insertions(+), 11 deletions(-) -- 1.8.3.1 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
