From e293bc24ed3bcffe8227d6a9b2ff3711954fce44 Mon Sep 17 00:00:00 2001 From: Star Zeng Date: Thu, 29 Oct 2015 01:13:07 +0000 Subject: [PATCH] MdeModulePkg SmbiosMeasurementDxe: Use EFI_D_VERBOSE for internal dump functions Use EFI_D_VERBOSE instead of EFI_D_INFO in InternalDumpData() and InternalDumpHex(). And also add DEBUG_CODE wrapper to InternalDumpHex() call. It is to avoid the bother from the internal verbose debug information. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng Reviewed-by: Jiewen Yao git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18693 6f19259b-4bc3-4df7-8a09-765794883524 --- .../SmbiosMeasurementDxe.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c b/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c index e82b028116..95a3aebefd 100644 --- a/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c +++ b/MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurementDxe.c @@ -125,7 +125,7 @@ InternalDumpData ( { UINTN Index; for (Index = 0; Index < Size; Index++) { - DEBUG ((EFI_D_INFO, "%02x", (UINTN)Data[Index])); + DEBUG ((EFI_D_VERBOSE, "%02x", (UINTN)Data[Index])); } } @@ -152,15 +152,15 @@ InternalDumpHex ( Count = Size / COLUME_SIZE; Left = Size % COLUME_SIZE; for (Index = 0; Index < Count; Index++) { - DEBUG ((EFI_D_INFO, "%04x: ", Index * COLUME_SIZE)); + DEBUG ((EFI_D_VERBOSE, "%04x: ", Index * COLUME_SIZE)); InternalDumpData (Data + Index * COLUME_SIZE, COLUME_SIZE); - DEBUG ((EFI_D_INFO, "\n")); + DEBUG ((EFI_D_VERBOSE, "\n")); } if (Left != 0) { - DEBUG ((EFI_D_INFO, "%04x: ", Index * COLUME_SIZE)); + DEBUG ((EFI_D_VERBOSE, "%04x: ", Index * COLUME_SIZE)); InternalDumpData (Data + Index * COLUME_SIZE, Left); - DEBUG ((EFI_D_INFO, "\n")); + DEBUG ((EFI_D_VERBOSE, "\n")); } } @@ -271,7 +271,7 @@ FilterSmbiosEntry ( UINTN StringLen; DEBUG ((EFI_D_INFO, "Smbios Table (Type - %d):\n", ((SMBIOS_STRUCTURE *)TableEntry)->Type)); - InternalDumpHex (TableEntry, TableEntrySize); + DEBUG_CODE (InternalDumpHex (TableEntry, TableEntrySize);); FilterStruct = GetFilterStructByType (((SMBIOS_STRUCTURE *)TableEntry)->Type); if (FilterStruct != NULL) { @@ -297,7 +297,7 @@ FilterSmbiosEntry ( } DEBUG ((EFI_D_INFO, "Filter Smbios Table (Type - %d):\n", ((SMBIOS_STRUCTURE *)TableEntry)->Type)); - InternalDumpHex (TableEntry, TableEntrySize); + DEBUG_CODE (InternalDumpHex (TableEntry, TableEntrySize);); } /** @@ -531,7 +531,7 @@ MeasureSmbiosTable ( if (SmbiosTableAddress != NULL) { DEBUG ((DEBUG_INFO, "The Smbios Table starts at: 0x%x\n", SmbiosTableAddress)); DEBUG ((DEBUG_INFO, "The Smbios Table size: 0x%x\n", TableLength)); - InternalDumpHex ((UINT8 *)(UINTN)SmbiosTableAddress, TableLength); + DEBUG_CODE (InternalDumpHex ((UINT8 *)(UINTN)SmbiosTableAddress, TableLength);); TableAddress = AllocateCopyPool ((UINTN)TableLength, (VOID *)(UINTN)SmbiosTableAddress); if (TableAddress == NULL) { @@ -542,7 +542,7 @@ MeasureSmbiosTable ( DEBUG ((DEBUG_INFO, "The final Smbios Table starts at: 0x%x\n", TableAddress)); DEBUG ((DEBUG_INFO, "The final Smbios Table size: 0x%x\n", TableLength)); - InternalDumpHex (TableAddress, TableLength); + DEBUG_CODE (InternalDumpHex (TableAddress, TableLength);); HandoffTables.NumberOfTables = 1; if (Smbios3Table != NULL) { -- 2.39.2