From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out30-45.freemail.mail.aliyun.com (out30-45.freemail.mail.aliyun.com [115.124.30.45]) by mx.groups.io with SMTP id smtpd.web11.10783.1640948812216760828 for ; Fri, 31 Dec 2021 03:06:53 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: linux.alibaba.com, ip: 115.124.30.45, mailfrom: huangming@linux.alibaba.com) X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R981e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=e01e04426;MF=huangming@linux.alibaba.com;NM=1;PH=DS;RN=7;SR=0;TI=SMTPD_---0V0QXmFy_1640948784; Received: from 842effaa37a8.tbsite.net(mailfrom:huangming@linux.alibaba.com fp:SMTPD_---0V0QXmFy_1640948784) by smtp.aliyun-inc.com(127.0.0.1); Fri, 31 Dec 2021 19:06:49 +0800 From: "Ming Huang" To: devel@edk2.groups.io, sami.mujawar@arm.com, ardb+tianocore@kernel.org, jiewen.yao@intel.com, supreeth.venkatesh@arm.com Cc: ming.huang-@outlook.com, Ming Huang Subject: [PATCH edk2 v3 2/3] StandaloneMmPkg: Replace DEBUG_INFO with DEBUG_ERROR Date: Fri, 31 Dec 2021 19:06:22 +0800 Message-Id: <20211231110623.128888-3-huangming@linux.alibaba.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20211231110623.128888-1-huangming@linux.alibaba.com> References: <20211231110623.128888-1-huangming@linux.alibaba.com> DEBUG_ERROR should be used in error branch. Signed-off-by: Ming Huang --- StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c | 6 +++--- StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c | 12 ++++++------ 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c b/StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c index 165d696f99..5dfaf9d751 100644 --- a/StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c +++ b/StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c @@ -95,7 +95,7 @@ PiMmStandaloneArmTfCpuDriverEntry ( // if ((ARM_SMC_ID_MM_COMMUNICATE != EventId) && (ARM_SVC_ID_FFA_MSG_SEND_DIRECT_REQ != EventId)) { - DEBUG ((DEBUG_INFO, "UnRecognized Event - 0x%x\n", EventId)); + DEBUG ((DEBUG_ERROR, "UnRecognized Event - 0x%x\n", EventId)); return EFI_INVALID_PARAMETER; } @@ -133,7 +133,7 @@ PiMmStandaloneArmTfCpuDriverEntry ( ); if (Status != EFI_SUCCESS) { - DEBUG ((DEBUG_INFO, "Mem alloc failed - 0x%x\n", EventId)); + DEBUG ((DEBUG_ERROR, "Mem alloc failed - 0x%x\n", EventId)); return EFI_OUT_OF_RESOURCES; } @@ -156,7 +156,7 @@ PiMmStandaloneArmTfCpuDriverEntry ( mMmst->CpuSaveState = NULL; if (mMmEntryPoint == NULL) { - DEBUG ((DEBUG_INFO, "Mm Entry point Not Found\n")); + DEBUG ((DEBUG_ERROR, "Mm Entry point Not Found\n")); return EFI_UNSUPPORTED; } diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c b/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c index 10097f792f..fd9c59b4da 100644 --- a/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c +++ b/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c @@ -143,7 +143,7 @@ StandaloneMmCpuInitialize ( // Bail out if the Hoblist could not be found if (Index >= mMmst->NumberOfTableEntries) { - DEBUG ((DEBUG_INFO, "Hoblist not found - 0x%x\n", Index)); + DEBUG ((DEBUG_ERROR, "Hoblist not found - 0x%x\n", Index)); return EFI_OUT_OF_RESOURCES; } @@ -158,7 +158,7 @@ StandaloneMmCpuInitialize ( (VOID **) &CpuDriverEntryPointDesc ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_INFO, "ArmTfCpuDriverEpDesc HOB data extraction failed - 0x%x\n", Status)); + DEBUG ((DEBUG_ERROR, "ArmTfCpuDriverEpDesc HOB data extraction failed - 0x%x\n", Status)); return Status; } @@ -176,7 +176,7 @@ StandaloneMmCpuInitialize ( (VOID **) &NsCommBufMmramRange ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_INFO, "NsCommBufMmramRange HOB data extraction failed - 0x%x\n", Status)); + DEBUG ((DEBUG_ERROR, "NsCommBufMmramRange HOB data extraction failed - 0x%x\n", Status)); return Status; } @@ -195,7 +195,7 @@ StandaloneMmCpuInitialize ( (VOID **) &MpInformationHobData ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_INFO, "MpInformationHob extraction failed - 0x%x\n", Status)); + DEBUG ((DEBUG_ERROR, "MpInformationHob extraction failed - 0x%x\n", Status)); return Status; } @@ -213,7 +213,7 @@ StandaloneMmCpuInitialize ( (VOID **) &mMpInformationHobData ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_INFO, "mMpInformationHobData mem alloc failed - 0x%x\n", Status)); + DEBUG ((DEBUG_ERROR, "mMpInformationHobData mem alloc failed - 0x%x\n", Status)); return Status; } @@ -243,7 +243,7 @@ StandaloneMmCpuInitialize ( (VOID **) &PerCpuGuidedEventContext ); if (EFI_ERROR (Status)) { - DEBUG ((DEBUG_INFO, "PerCpuGuidedEventContext mem alloc failed - 0x%x\n", Status)); + DEBUG ((DEBUG_ERROR, "PerCpuGuidedEventContext mem alloc failed - 0x%x\n", Status)); return Status; } return Status; -- 2.17.1