From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mx.groups.io with SMTP id smtpd.web11.9224.1678230247321217669 for ; Tue, 07 Mar 2023 15:04:07 -0800 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=IZQpl+Jy; spf=pass (domain: intel.com, ip: 134.134.136.31, mailfrom: darbin.reyes@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1678230247; x=1709766247; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=G83w+YG1VTzwnv7mAq1ZH4jgxTUU34u1vP3y2jmnnrg=; b=IZQpl+JyYaoKxcpisSjdG9wAG2uEJuABZOCUYMpDsz/KI+fB9F6JSVI/ XcrK6qg6+nhsk2H1UB4AUtWqb1w8IpyR66hirFvhOwr7jWQ0MUeWmeAoS TxQPqhNztmxT59kBAf0hO1lEpcFlL4dICWXtKBQwhUF0eB1evvI5nOTDm 0yBkADAVyAljj5prCUR22MgHcYOeHVZUezBotT1+3wWUAZfZtaLf6iZbp htSn+7KQ2FvxxhxtOLcaOjWfROACELfRmk+zjoYTp/L/3RNA66j0mefbv CJmdwdBMl6X5VWp+kimFhcYcrBmFzZEiU7mMGeIsGxpgSlQlfzuFUf+uD Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10642"; a="398575975" X-IronPort-AV: E=Sophos;i="5.98,242,1673942400"; d="scan'208";a="398575975" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Mar 2023 15:04:06 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10642"; a="922553253" X-IronPort-AV: E=Sophos;i="5.98,242,1673942400"; d="scan'208";a="922553253" Received: from darbinre-desk.amr.corp.intel.com ([10.53.80.90]) by fmsmga006.fm.intel.com with ESMTP; 07 Mar 2023 15:04:06 -0800 From: "Darbin Reyes" To: devel@edk2.groups.io Cc: Darbin Reyes , Jacob Narey Subject: [PATCH] UefiCpuPkg/MicrocodeMeasurementDxe: Fix exception Date: Tue, 7 Mar 2023 15:04:03 -0800 Message-Id: <7649d8fa34ad6b7f893fdd49efd4c3a7f2125fb6.1678229331.git.darbin.reyes@intel.com> X-Mailer: git-send-email 2.38.1.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4360 An incorrect format specifier is being used in a DEBUG print, specifically, a variable of type EFI_STATUS was being printed with the %a format specifier (pointer to an ASCII string), thus the value of the Status variable was being treated as the address of a string, leading to a CPU exception, when encountered this bug manifests itself as a hang near "Ready to Boot Event", with the last DEBUG print being "INFO: Got MicrocodePatchHob with microcode patches starting address" followed by a CPU Exception dump. Signed-off-by: Darbin Reyes Reviewed-by: Jacob Narey --- UefiCpuPkg/MicrocodeMeasurementDxe/MicrocodeMeasurementDxe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/UefiCpuPkg/MicrocodeMeasurementDxe/MicrocodeMeasurementDxe.c b= /UefiCpuPkg/MicrocodeMeasurementDxe/MicrocodeMeasurementDxe.c index 762ca159ff..5fd3b3365c 100644 --- a/UefiCpuPkg/MicrocodeMeasurementDxe/MicrocodeMeasurementDxe.c +++ b/UefiCpuPkg/MicrocodeMeasurementDxe/MicrocodeMeasurementDxe.c @@ -238,7 +238,7 @@ MeasureMicrocodePatches ( TotalMicrocodeSize)=0D );=0D } else {=0D - DEBUG ((DEBUG_ERROR, "ERROR: TpmMeasureAndLogData failed with status %= a!\n", Status));=0D + DEBUG ((DEBUG_ERROR, "ERROR: TpmMeasureAndLogData failed with status %= r!\n", Status));=0D }=0D =0D FreePool (Offsets);=0D --=20 2.38.1.windows.1