On 2015/10/13 19:55, Laszlo Ersek wrote:
These messages don't report errors, hence they should be emitted as
EFI_D_INFO (or EFI_D_VERBOSE even).

Cc: Star Zeng <[email protected]>
Cc: Elvin Li <[email protected]>
Cc: Drew Jones <[email protected]>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <[email protected]>

Reviewed-by: Star Zeng <[email protected]>

Thanks,
Star
---
  MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c | 6 ++++--
  1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c 
b/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c
index f29c47b..809dff8 100644
--- a/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c
+++ b/MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.c
@@ -1136,21 +1136,22 @@ SmbiosCreateTable (
    EntryPointStructure->TableLength = (UINT16) 
(EntryPointStructure->TableLength + sizeof (EndStructure));
    if (sizeof (EndStructure) > EntryPointStructure->MaxStructureSize) {
      EntryPointStructure->MaxStructureSize = (UINT16) sizeof (EndStructure);
    }

    if ((UINTN) EFI_SIZE_TO_PAGES (EntryPointStructure->TableLength) > 
mPreAllocatedPages) {
      //
      // If new SMBIOS table size exceeds the previous allocated page,
      // it is time to re-allocate memory (below 4GB).
      //
-    DEBUG ((EFI_D_ERROR, "SmbiosCreateTable() re-allocate SMBIOS 32-bit 
table\n"));
+    DEBUG ((EFI_D_INFO, "%a() re-allocate SMBIOS 32-bit table\n",
+      __FUNCTION__));
      if (EntryPointStructure->TableAddress != 0) {
        //
        // Free the previous allocated page
        //
        FreePages (
              (VOID*)(UINTN)EntryPointStructure->TableAddress,
              mPreAllocatedPages
              );
        EntryPointStructure->TableAddress = 0;
        mPreAllocatedPages = 0;
@@ -1304,21 +1305,22 @@ SmbiosCreate64BitTable (
    EndStructure.Header.Handle = SmbiosHandle;
    EndStructure.Tailing[0] = 0;
    EndStructure.Tailing[1] = 0;
    Smbios30EntryPointStructure->TableMaximumSize = (UINT32) 
(Smbios30EntryPointStructure->TableMaximumSize + sizeof (EndStructure));

    if ((UINTN) EFI_SIZE_TO_PAGES (Smbios30EntryPointStructure->TableMaximumSize) 
> mPre64BitAllocatedPages) {
      //
      // If new SMBIOS table size exceeds the previous allocated page,
      // it is time to re-allocate memory at anywhere.
      //
-    DEBUG ((EFI_D_ERROR, "SmbiosCreate64BitTable() re-allocate SMBIOS 64-bit 
table\n"));
+    DEBUG ((EFI_D_INFO, "%a() re-allocate SMBIOS 64-bit table\n",
+      __FUNCTION__));
      if (Smbios30EntryPointStructure->TableAddress != 0) {
        //
        // Free the previous allocated page
        //
        FreePages (
              (VOID*)(UINTN)Smbios30EntryPointStructure->TableAddress,
              mPre64BitAllocatedPages
              );
        Smbios30EntryPointStructure->TableAddress = 0;
        mPre64BitAllocatedPages = 0;


_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to