From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.126; helo=mga18.intel.com; envelope-from=zhichao.gao@intel.com; receiver=edk2-devel@lists.01.org Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 60CAB208AE9A9 for ; Tue, 19 Mar 2019 08:26:41 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Mar 2019 08:26:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,498,1544515200"; d="scan'208";a="330011690" Received: from gaozhic-mobl.ccr.corp.intel.com ([10.249.174.112]) by fmsmga005.fm.intel.com with ESMTP; 19 Mar 2019 08:26:38 -0700 From: Zhichao Gao To: edk2-devel@lists.01.org Cc: Jian J Wang , Hao Wu , Ray Ni , Star Zeng , Liming Gao , Sean Brogan , Michael Turner , Bret Barkelew Date: Tue, 19 Mar 2019 23:25:45 +0800 Message-Id: <20190319152549.16104-14-zhichao.gao@intel.com> X-Mailer: git-send-email 2.16.2.windows.1 In-Reply-To: <20190319152549.16104-1-zhichao.gao@intel.com> References: <20190319152549.16104-1-zhichao.gao@intel.com> Subject: [PATCH V3 13/17] MdeModulePkg/PeiDxeDebugLibReportStatusCode: Add new APIs X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Mar 2019 15:26:41 -0000 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1395 Add new APIs' implementation (DebugVPrint, DebugBPrint) in the DebugLib instance. These APIs would expose print routines with VaList parameter and BaseList parameter. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhichao Gao Cc: Jian J Wang Cc: Hao Wu Cc: Ray Ni Cc: Star Zeng Cc: Liming Gao Cc: Sean Brogan Cc: Michael Turner Cc: Bret Barkelew --- .../PeiDxeDebugLibReportStatusCode/DebugLib.c | 144 ++++++++++++++++++--- 1 file changed, 128 insertions(+), 16 deletions(-) diff --git a/MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/DebugLib.c b/MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/DebugLib.c index 6f0f416273..f1d31cb619 100644 --- a/MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/DebugLib.c +++ b/MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/DebugLib.c @@ -4,7 +4,7 @@ Note that if the debug message length is larger than the maximum allowable record length, then the debug message will be ignored directly. - Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.
+ Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -27,6 +27,12 @@ #include #include +// +// VA_LIST can not initialize to NULL for all compiler, so we use this to +// indicate a null VA_LIST +// +VA_LIST mVaListNull; + /** Prints a debug message to the debug output device if the specified error level is enabled. @@ -52,12 +58,43 @@ DebugPrint ( IN CONST CHAR8 *Format, ... ) +{ + VA_LIST Marker; + + VA_START (Marker, Format); + DebugVPrint (ErrorLevel, Format, Marker); + VA_END (Marker); +} + +/** + Prints a debug message to the debug output device if the specified + error level is enabled base on Null-terminated format string and a + VA_LIST argument list or a BASE_LIST argument list. + + If any bit in ErrorLevel is also set in DebugPrintErrorLevelLib function + GetDebugPrintErrorLevel (), then print the message specified by Format and + the associated variable argument list to the debug output device. + + If Format is NULL, then ASSERT(). + + @param ErrorLevel The error level of the debug message. + @param Format Format string for the debug message to print. + @param VaListMarker VA_LIST marker for the variable argument list. + @param BaseListMarker BASE_LIST marker for the variable argument list. + +**/ +VOID +DebugPrintMarker ( + IN UINTN ErrorLevel, + IN CONST CHAR8 *Format, + IN VA_LIST VaListMarker, + IN BASE_LIST BaseListMarker + ) { UINT64 Buffer[(EFI_STATUS_CODE_DATA_MAX_SIZE / sizeof (UINT64)) + 1]; EFI_DEBUG_INFO *DebugInfo; UINTN TotalSize; - VA_LIST VaListMarker; - BASE_LIST BaseListMarker; + BASE_LIST BaseListMarkerPointer; CHAR8 *FormatString; BOOLEAN Long; @@ -96,7 +133,7 @@ DebugPrint ( // If the TotalSize is larger than the maximum record size, then return // if (TotalSize > sizeof (Buffer)) { - TotalSize = sizeof (Buffer); + return; } // @@ -109,7 +146,7 @@ DebugPrint ( // DebugInfo = (EFI_DEBUG_INFO *)(Buffer) + 1; DebugInfo->ErrorLevel = (UINT32)ErrorLevel; - BaseListMarker = (BASE_LIST)(DebugInfo + 1); + BaseListMarkerPointer = (BASE_LIST)(DebugInfo + 1); FormatString = (CHAR8 *)((UINT64 *)(DebugInfo + 1) + 12); // @@ -125,7 +162,6 @@ DebugPrint ( // of format in DEBUG string, which is followed by the DEBUG format string. // Here we will process the variable arguments and pack them in this area. // - VA_START (VaListMarker, Format); // // Use the actual format string. @@ -167,7 +203,11 @@ DebugPrint ( // '*' in format field means the precision of the field is specified by // a UINTN argument in the argument list. // - BASE_ARG (BaseListMarker, UINTN) = VA_ARG (VaListMarker, UINTN); + if (BaseListMarker == NULL) { + BASE_ARG (BaseListMarkerPointer, UINTN) = VA_ARG (VaListMarker, UINTN); + } else { + BASE_ARG (BaseListMarkerPointer, UINTN) = BASE_ARG (BaseListMarker, UINTN); + } continue; } if (*Format == '\0') { @@ -192,16 +232,36 @@ DebugPrint ( } if (*Format == 'p' || *Format == 'X' || *Format == 'x' || *Format == 'd' || *Format == 'u') { if (Long) { - BASE_ARG (BaseListMarker, INT64) = VA_ARG (VaListMarker, INT64); + if (BaseListMarker == NULL) { + BASE_ARG (BaseListMarkerPointer, INT64) = VA_ARG (VaListMarker, INT64); + } else { + BASE_ARG (BaseListMarkerPointer, INT64) = BASE_ARG (BaseListMarker, INT64); + } } else { - BASE_ARG (BaseListMarker, int) = VA_ARG (VaListMarker, int); + if (BaseListMarker == NULL) { + BASE_ARG (BaseListMarkerPointer, int) = VA_ARG (VaListMarker, int); + } else { + BASE_ARG (BaseListMarkerPointer, int) = BASE_ARG (BaseListMarker, int); + } } } else if (*Format == 's' || *Format == 'S' || *Format == 'a' || *Format == 'g' || *Format == 't') { - BASE_ARG (BaseListMarker, VOID *) = VA_ARG (VaListMarker, VOID *); + if (BaseListMarker == NULL) { + BASE_ARG (BaseListMarkerPointer, VOID *) = VA_ARG (VaListMarker, VOID *); + } else { + BASE_ARG (BaseListMarkerPointer, VOID *) = BASE_ARG (BaseListMarker, VOID *); + } } else if (*Format == 'c') { - BASE_ARG (BaseListMarker, UINTN) = VA_ARG (VaListMarker, UINTN); + if (BaseListMarker == NULL) { + BASE_ARG (BaseListMarkerPointer, UINTN) = VA_ARG (VaListMarker, UINTN); + } else { + BASE_ARG (BaseListMarkerPointer, UINTN) = BASE_ARG (BaseListMarker, UINTN); + } } else if (*Format == 'r') { - BASE_ARG (BaseListMarker, RETURN_STATUS) = VA_ARG (VaListMarker, RETURN_STATUS); + if (BaseListMarker == NULL) { + BASE_ARG (BaseListMarkerPointer, RETURN_STATUS) = VA_ARG (VaListMarker, RETURN_STATUS); + } else { + BASE_ARG (BaseListMarkerPointer, RETURN_STATUS) = BASE_ARG (BaseListMarker, RETURN_STATUS); + } } // @@ -209,17 +269,15 @@ DebugPrint ( // This indicates that the DEBUG() macro is passing in more argument than can be handled by // the EFI_DEBUG_INFO record // - ASSERT ((CHAR8 *)BaseListMarker <= FormatString); + ASSERT ((CHAR8 *)BaseListMarkerPointer <= FormatString); // // If the converted BASE_LIST is larger than the 12 * sizeof (UINT64) allocated bytes, then return // - if ((CHAR8 *)BaseListMarker > FormatString) { - VA_END (VaListMarker); + if ((CHAR8 *)BaseListMarkerPointer > FormatString) { return; } } - VA_END (VaListMarker); // // Send the DebugInfo record @@ -235,6 +293,60 @@ DebugPrint ( ); } +/** + Prints a debug message to the debug output device if the specified + error level is enabled. + + If any bit in ErrorLevel is also set in DebugPrintErrorLevelLib function + GetDebugPrintErrorLevel (), then print the message specified by Format and + the associated variable argument list to the debug output device. + + If Format is NULL, then ASSERT(). + + @param ErrorLevel The error level of the debug message. + @param Format Format string for the debug message to print. + @param VaListMarker VA_LIST marker for the variable argument list. + +**/ +VOID +EFIAPI +DebugVPrint ( + IN UINTN ErrorLevel, + IN CONST CHAR8 *Format, + IN VA_LIST VaListMarker + ) +{ + DebugPrintMarker (ErrorLevel, Format, VaListMarker, NULL); +} + +/** + Prints a debug message to the debug output device if the specified + error level is enabled. + This function use BASE_LIST which would provide a more compatible + service than VA_LIST. + + If any bit in ErrorLevel is also set in DebugPrintErrorLevelLib function + GetDebugPrintErrorLevel (), then print the message specified by Format and + the associated variable argument list to the debug output device. + + If Format is NULL, then ASSERT(). + + @param ErrorLevel The error level of the debug message. + @param Format Format string for the debug message to print. + @param BaseListMarker BASE_LIST marker for the variable argument list. + +**/ +VOID +EFIAPI +DebugBPrint ( + IN UINTN ErrorLevel, + IN CONST CHAR8 *Format, + IN BASE_LIST BaseListMarker + ) +{ + DebugPrintMarker (ErrorLevel, Format, mVaListNull, BaseListMarker); +} + /** Prints an assert message containing a filename, line number, and description. This may be followed by a breakpoint or a dead loop. -- 2.16.2.windows.1