public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Gao, Zhichao" <zhichao.gao@intel.com>
To: "Gao, Liming" <liming.gao@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Wang, Jian J" <jian.j.wang@intel.com>,
	"Wu, Hao A" <hao.a.wu@intel.com>,  "Ni, Ray" <ray.ni@intel.com>,
	"Zeng, Star" <star.zeng@intel.com>,
	Sean Brogan <sean.brogan@microsoft.com>,
	Michael Turner <Michael.Turner@microsoft.com>,
	Bret Barkelew <Bret.Barkelew@microsoft.com>
Subject: Re: [PATCH V6 16/17] MdeModulePkg/PeiDebugLibDebugPpi: Add PEI debug lib
Date: Mon, 1 Apr 2019 00:26:14 +0000	[thread overview]
Message-ID: <3CE959C139B4C44DBEA1810E3AA6F9000B7B9E27@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E417520@SHSMSX104.ccr.corp.intel.com>

This should be implemented in the true debug lib instance such as PeiDxeDebugLibReportStatusCode.

Thanks,
Zhichao
> -----Original Message-----
> From: Gao, Liming
> Sent: Monday, April 1, 2019 8:21 AM
> To: Gao, Zhichao <zhichao.gao@intel.com>; edk2-devel@lists.01.org
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>;
> Ni, Ray <ray.ni@intel.com>; Zeng, Star <star.zeng@intel.com>; Sean Brogan
> <sean.brogan@microsoft.com>; Michael Turner
> <Michael.Turner@microsoft.com>; Bret Barkelew
> <Bret.Barkelew@microsoft.com>
> Subject: RE: [PATCH V6 16/17] MdeModulePkg/PeiDebugLibDebugPpi: Add
> PEI debug lib
> 
> Zhichao:
>   DebugAssert() implementation should be DeadLoop() or Break() bases on
> PCD gEfiMdePkgTokenSpaceGuid.PcdDebugPropertyMask when PPI is not
> loaded.
> 
> Thanks
> Liming
> >-----Original Message-----
> >From: Gao, Zhichao
> >Sent: Monday, April 01, 2019 7:50 AM
> >To: edk2-devel@lists.01.org
> >Cc: Gao, Liming <liming.gao@intel.com>; Wang, Jian J
> ><jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Ni, Ray
> ><ray.ni@intel.com>; Zeng, Star <star.zeng@intel.com>; Sean Brogan
> ><sean.brogan@microsoft.com>; Michael Turner
> ><Michael.Turner@microsoft.com>; Bret Barkelew
> ><Bret.Barkelew@microsoft.com>
> >Subject: [PATCH V6 16/17] MdeModulePkg/PeiDebugLibDebugPpi: Add PEI
> >debug lib
> >
> >From: Liming Gao <liming.gao@intel.com>
> >
> >REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1549
> >
> >Add a PEI debug library instance PeiDebugLibDebugPpi base on DebugPpi.
> >Using the combination of the DebugServicePei and this lib instance can
> >reduce the image size of PEI drivers.
> >
> >Notes: this library instance can be used only the PEIM DebugSerivicePei
> >is runed and install the gEdkiiDebugPpiGuid.
> >And this library contian the depx of gEfiPeiPcdPpiGuid, that means the
> >PcdPei.inf cannot use this library instance. The PcdPei.inf should use
> >the same library instance that the PEIM DebugServicePei consumes.
> >
> >Contributed-under: TianoCore Contribution Agreement 1.1
> >Signed-off-by: Zhichao Gao <zhichao.gao@intel.com>
> >Cc: Jian J Wang <jian.j.wang@intel.com>
> >Cc: Hao Wu <hao.a.wu@intel.com>
> >Cc: Ray Ni <ray.ni@intel.com>
> >Cc: Star Zeng <star.zeng@intel.com>
> >Cc: Liming Gao <liming.gao@intel.com>
> >Cc: Sean Brogan <sean.brogan@microsoft.com>
> >Cc: Michael Turner <Michael.Turner@microsoft.com>
> >Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> >---
> > .../Library/PeiDebugLibDebugPpi/DebugLib.c    | 452
> ++++++++++++++++++
> > .../PeiDebugLibDebugPpi.inf                   |  63 +++
> > 2 files changed, 515 insertions(+)
> > create mode 100644
> >MdeModulePkg/Library/PeiDebugLibDebugPpi/DebugLib.c
> > create mode 100644
> >MdeModulePkg/Library/PeiDebugLibDebugPpi/PeiDebugLibDebugPpi.inf
> >
> >diff --git a/MdeModulePkg/Library/PeiDebugLibDebugPpi/DebugLib.c
> >b/MdeModulePkg/Library/PeiDebugLibDebugPpi/DebugLib.c
> >new file mode 100644
> >index 0000000000..84ba896b53
> >--- /dev/null
> >+++ b/MdeModulePkg/Library/PeiDebugLibDebugPpi/DebugLib.c
> >@@ -0,0 +1,452 @@
> >+/** @file
> >+  PEI debug lib instance base on gEdkiiDebugPpiGuid to save PEIM size.
> >+
> >+  Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> >+
> >+  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
> >+  http://opensource.org/licenses/bsd-license.php
> >+
> >+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> >BASIS,
> >+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> >EXPRESS OR IMPLIED.
> >+
> >+**/
> >+
> >+#include <PiPei.h>
> >+#include <Ppi/Debug.h>
> >+#include <Library/DebugLib.h>
> >+#include <Library/BaseMemoryLib.h>
> >+#include <Library/PcdLib.h>
> >+#include <Library/PeiServicesLib.h>
> >+#include <Library/DebugPrintErrorLevelLib.h>
> >+
> >+/**
> >+  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  ...           Variable argument list whose contents are accessed
> >+                        based on the format string specified by Format.
> >+
> >+**/
> >+VOID
> >+EFIAPI
> >+DebugPrint (
> >+  IN  UINTN        ErrorLevel,
> >+  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.
> >+  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
> >+  )
> >+{
> >+  EFI_STATUS      Status;
> >+  EDKII_DEBUG_PPI *DebugPpi;
> >+
> >+  //
> >+  // If Format is NULL, then ASSERT().
> >+  //
> >+  ASSERT (Format != NULL);
> >+
> >+  //
> >+  // Check driver Debug Level value and global debug level  //  if
> >+ ((ErrorLevel & GetDebugPrintErrorLevel ()) == 0) {
> >+    return;
> >+  }
> >+
> >+  Status = PeiServicesLocatePpi (
> >+              &gEdkiiDebugPpiGuid,
> >+              0,
> >+              NULL,
> >+              (VOID **)&DebugPpi
> >+              );
> >+  if (EFI_ERROR (Status)) {
> >+    return;
> >+  }
> >+
> >+  DebugPpi->DebugBPrint (
> >+              ErrorLevel,
> >+              Format,
> >+              BaseListMarker
> >+              );
> >+}
> >+
> >+
> >+/**
> >+  Worker function that convert a VA_LIST to a BASE_LIST based on a
> >+  Null-terminated format string.
> >+
> >+  @param  Format          Null-terminated format string.
> >+  @param  VaListMarker    VA_LIST style variable argument list consumed
> >+                          by processing Format.
> >+  @param  BaseListMarker  BASE_LIST style variable argument list
> consumed
> >+                          by processing Format.
> >+  @param  Size            The size, in bytes, of the BaseListMarker buffer.
> >+
> >+  @return TRUE   The VA_LIST has been converted to BASE_LIST.
> >+  @return FALSE  The VA_LIST has not been converted to BASE_LIST.
> >+
> >+**/
> >+BOOLEAN
> >+VaListToBaseList (
> >+  IN  CONST CHAR8  *Format,
> >+  IN  VA_LIST      VaListMarker,
> >+  OUT BASE_LIST    BaseListMarker,
> >+  IN  UINTN        Size
> >+  )
> >+{
> >+  BASE_LIST       BaseListStart;
> >+  BOOLEAN         Long;
> >+
> >+  ASSERT (Format != NULL);
> >+
> >+  ASSERT (BaseListMarker != NULL);
> >+
> >+  BaseListStart = BaseListMarker;
> >+
> >+  for (; *Format != '\0'; Format++) {
> >+    //
> >+    // Only format with prefix % is processed.
> >+    //
> >+    if (*Format != '%') {
> >+      continue;
> >+    }
> >+
> >+    Long = FALSE;
> >+
> >+    //
> >+    // Parse Flags and Width
> >+    //
> >+    for (Format++; TRUE; Format++) {
> >+      if (*Format == '.' || *Format == '-' || *Format == '+' || *Format == ' ') {
> >+        //
> >+        // These characters in format field are omitted.
> >+        //
> >+        continue;
> >+      }
> >+      if (*Format >= '0' && *Format <= '9') {
> >+        //
> >+        // These characters in format field are omitted.
> >+        //
> >+        continue;
> >+      }
> >+      if (*Format == 'L' || *Format == 'l') {
> >+        //
> >+        // 'L" or "l" in format field means the number being printed is a UINT64
> >+        //
> >+        Long = TRUE;
> >+        continue;
> >+      }
> >+      if (*Format == '*') {
> >+        //
> >+        // '*' 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);
> >+        continue;
> >+      }
> >+      if (*Format == '\0') {
> >+        //
> >+        // Make no output if Format string terminates unexpectedly when
> >+        // looking up for flag, width, precision and type.
> >+        //
> >+        Format--;
> >+      }
> >+      //
> >+      // When valid argument type detected or format string terminates
> >unexpectedly,
> >+      // the inner loop is done.
> >+      //
> >+      break;
> >+    }
> >+
> >+    //
> >+    // Pack variable arguments into the storage area following
> >EFI_DEBUG_INFO.
> >+    //
> >+    if ((*Format == 'p') && (sizeof (VOID *) > 4)) {
> >+      Long = TRUE;
> >+    }
> >+    if (*Format == 'p' || *Format == 'X' || *Format == 'x' || *Format
> >+ == 'd' ||
> >*Format == 'u') {
> >+      if (Long) {
> >+        BASE_ARG (BaseListMarker, INT64) = VA_ARG (VaListMarker, INT64);
> >+      } else {
> >+        BASE_ARG (BaseListMarker, int) = VA_ARG (VaListMarker, int);
> >+      }
> >+    } else if (*Format == 's' || *Format == 'S' || *Format == 'a' ||
> >+ *Format ==
> >'g' || *Format == 't') {
> >+      BASE_ARG (BaseListMarker, VOID *) = VA_ARG (VaListMarker, VOID *);
> >+    } else if (*Format == 'c') {
> >+      BASE_ARG (BaseListMarker, UINTN) = VA_ARG (VaListMarker, UINTN);
> >+    } else if (*Format == 'r') {
> >+      BASE_ARG (BaseListMarker, RETURN_STATUS) = VA_ARG
> (VaListMarker,
> >RETURN_STATUS);
> >+    }
> >+
> >+    //
> >+    // If the converted BASE_LIST is larger than the size of
> >+ BaseListMarker,
> >then return FALSE
> >+    //
> >+    if (((UINTN)BaseListMarker - (UINTN)BaseListStart) > Size) {
> >+      return FALSE;
> >+    }
> >+  }
> >+
> >+  return TRUE;
> >+}
> >+
> >+
> >+/**
> >+  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
> >+  )
> >+{
> >+  UINT64            BaseListMarker[256 / sizeof (UINT64)];
> >+  BOOLEAN           Converted;
> >+
> >+  //
> >+  // Convert the VaList to BaseList
> >+  //
> >+  Converted = VaListToBaseList (
> >+                Format,
> >+                VaListMarker,
> >+                (BASE_LIST)BaseListMarker,
> >+                sizeof (BaseListMarker) - 8
> >+                );
> >+
> >+  if (!Converted) {
> >+    return;
> >+  }
> >+
> >+  DebugBPrint (ErrorLevel, Format, (BASE_LIST)BaseListMarker); }
> >+
> >+
> >+/**
> >+  Prints an assert message containing a filename, line number, and
> >description.
> >+  This may be followed by a breakpoint or a dead loop.
> >+
> >+  Print a message of the form "ASSERT <FileName>(<LineNumber>):
> ><Description>\n"
> >+  to the debug output device.  If
> >DEBUG_PROPERTY_ASSERT_BREAKPOINT_ENABLED bit of
> >+  PcdDebugProperyMask is set then CpuBreakpoint() is called.
> >+ Otherwise, if  DEBUG_PROPERTY_ASSERT_DEADLOOP_ENABLED bit of
> >PcdDebugProperyMask is set then
> >+  CpuDeadLoop() is called.  If neither of these bits are set, then
> >+ this function  returns immediately after the message is printed to
> >+ the debug output
> >device.
> >+  DebugAssert() must actively prevent recursion.  If DebugAssert() is
> >+ called
> >while
> >+  processing another DebugAssert(), then DebugAssert() must return
> >immediately.
> >+
> >+  If FileName is NULL, then a <FileName> string of "(NULL) Filename"
> >+ is
> >printed.
> >+  If Description is NULL, then a <Description> string of "(NULL)
> >+ Description" is
> >printed.
> >+
> >+  @param  FileName     The pointer to the name of the source file that
> >generated the assert condition.
> >+  @param  LineNumber   The line number in the source file that generated
> >the assert condition
> >+  @param  Description  The pointer to the description of the assert
> condition.
> >+
> >+**/
> >+VOID
> >+EFIAPI
> >+DebugAssert (
> >+  IN CONST CHAR8  *FileName,
> >+  IN UINTN        LineNumber,
> >+  IN CONST CHAR8  *Description
> >+  )
> >+{
> >+  EFI_STATUS      Status;
> >+  EDKII_DEBUG_PPI *DebugPpi;
> >+
> >+  Status = PeiServicesLocatePpi (
> >+              &gEdkiiDebugPpiGuid,
> >+              0,
> >+              NULL,
> >+              (VOID **)&DebugPpi
> >+              );
> >+  if (EFI_ERROR (Status)) {
> >+    return;
> >+  }
> >+
> >+  DebugPpi->DebugAssert (
> >+              FileName,
> >+              LineNumber,
> >+              Description
> >+              );
> >+}
> >+
> >+
> >+/**
> >+  Fills a target buffer with PcdDebugClearMemoryValue, and returns the
> >target buffer.
> >+
> >+  This function fills Length bytes of Buffer with the value specified
> >+ by  PcdDebugClearMemoryValue, and returns Buffer.
> >+
> >+  If Buffer is NULL, then ASSERT().
> >+  If Length is greater than (MAX_ADDRESS - Buffer + 1), then ASSERT().
> >+
> >+  @param   Buffer  The pointer to the target buffer to be filled with
> >PcdDebugClearMemoryValue.
> >+  @param   Length  The number of bytes in Buffer to fill with zeros
> >PcdDebugClearMemoryValue.
> >+
> >+  @return  Buffer  The pointer to the target buffer filled with
> >PcdDebugClearMemoryValue.
> >+
> >+**/
> >+VOID *
> >+EFIAPI
> >+DebugClearMemory (
> >+  OUT VOID  *Buffer,
> >+  IN UINTN  Length
> >+  )
> >+{
> >+  ASSERT (Buffer != NULL);
> >+
> >+  return SetMem (Buffer, Length, PcdGet8
> (PcdDebugClearMemoryValue));
> >+}
> >+
> >+
> >+/**
> >+  Returns TRUE if ASSERT() macros are enabled.
> >+
> >+  This function returns TRUE if the
> >DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of
> >+  PcdDebugProperyMask is set.  Otherwise, FALSE is returned.
> >+
> >+  @retval  TRUE    The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of
> >PcdDebugProperyMask is set.
> >+  @retval  FALSE   The DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED bit of
> >PcdDebugProperyMask is clear.
> >+
> >+**/
> >+BOOLEAN
> >+EFIAPI
> >+DebugAssertEnabled (
> >+  VOID
> >+  )
> >+{
> >+  return (BOOLEAN) ((PcdGet8 (PcdDebugPropertyMask) &
> >DEBUG_PROPERTY_DEBUG_ASSERT_ENABLED) != 0);
> >+}
> >+
> >+
> >+/**
> >+  Returns TRUE if DEBUG() macros are enabled.
> >+
> >+  This function returns TRUE if the
> >DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of
> >+  PcdDebugProperyMask is set.  Otherwise, FALSE is returned.
> >+
> >+  @retval  TRUE    The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of
> >PcdDebugProperyMask is set.
> >+  @retval  FALSE   The DEBUG_PROPERTY_DEBUG_PRINT_ENABLED bit of
> >PcdDebugProperyMask is clear.
> >+
> >+**/
> >+BOOLEAN
> >+EFIAPI
> >+DebugPrintEnabled (
> >+  VOID
> >+  )
> >+{
> >+  return (BOOLEAN) ((PcdGet8 (PcdDebugPropertyMask) &
> >DEBUG_PROPERTY_DEBUG_PRINT_ENABLED) != 0);
> >+}
> >+
> >+
> >+/**
> >+  Returns TRUE if DEBUG_CODE() macros are enabled.
> >+
> >+  This function returns TRUE if the
> >DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of
> >+  PcdDebugProperyMask is set.  Otherwise, FALSE is returned.
> >+
> >+  @retval  TRUE    The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of
> >PcdDebugProperyMask is set.
> >+  @retval  FALSE   The DEBUG_PROPERTY_DEBUG_CODE_ENABLED bit of
> >PcdDebugProperyMask is clear.
> >+
> >+**/
> >+BOOLEAN
> >+EFIAPI
> >+DebugCodeEnabled (
> >+  VOID
> >+  )
> >+{
> >+  return (BOOLEAN) ((PcdGet8 (PcdDebugPropertyMask) &
> >DEBUG_PROPERTY_DEBUG_CODE_ENABLED) != 0);
> >+}
> >+
> >+
> >+/**
> >+  Returns TRUE if DEBUG_CLEAR_MEMORY() macro is enabled.
> >+
> >+  This function returns TRUE if the
> >DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit of
> >+  PcdDebugProperyMask is set.  Otherwise, FALSE is returned.
> >+
> >+  @retval  TRUE    The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit
> of
> >PcdDebugProperyMask is set.
> >+  @retval  FALSE   The DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED bit
> of
> >PcdDebugProperyMask is clear.
> >+
> >+**/
> >+BOOLEAN
> >+EFIAPI
> >+DebugClearMemoryEnabled (
> >+  VOID
> >+  )
> >+{
> >+  return (BOOLEAN) ((PcdGet8 (PcdDebugPropertyMask) &
> >DEBUG_PROPERTY_CLEAR_MEMORY_ENABLED) != 0);
> >+}
> >+
> >+
> >+/**
> >+  Returns TRUE if any one of the bit is set both in ErrorLevel and
> >PcdFixedDebugPrintErrorLevel.
> >+
> >+  This function compares the bit mask of ErrorLevel and
> >PcdFixedDebugPrintErrorLevel.
> >+
> >+  @retval  TRUE    Current ErrorLevel is supported.
> >+  @retval  FALSE   Current ErrorLevel is not supported.
> >+
> >+**/
> >+BOOLEAN
> >+EFIAPI
> >+DebugPrintLevelEnabled (
> >+  IN  CONST UINTN        ErrorLevel
> >+  )
> >+{
> >+  return (BOOLEAN) ((ErrorLevel &
> >PcdGet32(PcdFixedDebugPrintErrorLevel)) != 0);
> >+}
> >+
> >diff --git
> >a/MdeModulePkg/Library/PeiDebugLibDebugPpi/PeiDebugLibDebugPpi.inf
> >b/MdeModulePkg/Library/PeiDebugLibDebugPpi/PeiDebugLibDebugPpi.inf
> >new file mode 100644
> >index 0000000000..827c4a8a70
> >--- /dev/null
> >+++
> >b/MdeModulePkg/Library/PeiDebugLibDebugPpi/PeiDebugLibDebugPpi.inf
> >@@ -0,0 +1,63 @@
> >+## @file
> >+#  Debug Lib instance through DebugServicePei for PEI phase # #  This
> >+module use gEdkiiDebugPpiGuid to implement the PEI DebugLib for
> >reduce
> >+#  the size of PEIMs which consume DebugLib.
> >+#
> >+#  Notes: this library instance can be used only the PEIM #
> >+DebugSerivicePei is runed and install the gEdkiiDebugPpiGuid.
> >+#  And this library contian the depex of gEfiPeiPcdPpiGuid, that #
> >+means the PcdPei.inf cannot use this library instance. The #
> >+PcdPei.inf should use the same library instance that the #  PEIM
> >+DebugServicePei consumes.
> >+#
> >+#  Copyright (c) 2019, Intel Corporation. All rights reserved.<BR> # #
> >+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
> >+#  http://opensource.org/licenses/bsd-license.php
> >+#  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> >BASIS,
> >+#  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> >EXPRESS OR IMPLIED.
> >+#
> >+##
> >+
> >+[Defines]
> >+  INF_VERSION                    = 0x00010005
> >+  BASE_NAME                      = PeiDebugLibDebugPpi
> >+  FILE_GUID                      = 2E08836C-4D1C-42F7-BBBE-EC5D25F1FDD4
> >+  MODULE_TYPE                    = PEIM
> >+  VERSION_STRING                 = 1.0
> >+  LIBRARY_CLASS                  = DebugLib|PEIM
> >+
> >+#
> >+# The following information is for reference only and not required by
> >+the
> >build tools.
> >+#
> >+#  VALID_ARCHITECTURES           = IA32 X64 EBC
> >+#
> >+
> >+[Sources]
> >+  DebugLib.c
> >+
> >+[Packages]
> >+  MdePkg/MdePkg.dec
> >+  MdeModulePkg/MdeModulePkg.dec
> >+
> >+[LibraryClasses]
> >+  PcdLib
> >+  BaseMemoryLib
> >+  DebugPrintErrorLevelLib
> >+  PeiServicesLib
> >+  PeiServicesTablePointerLib
> >+
> >+[Ppis]
> >+  gEdkiiDebugPpiGuid                                        ## CONSUMES
> >+
> >+[Pcd]
> >+  gEfiMdePkgTokenSpaceGuid.PcdDebugClearMemoryValue         ##
> >SOMETIMES_CONSUMES
> >+  gEfiMdePkgTokenSpaceGuid.PcdDebugPropertyMask             ##
> CONSUMES
> >+  gEfiMdePkgTokenSpaceGuid.PcdFixedDebugPrintErrorLevel     ##
> >CONSUMES
> >+
> >+[Depex]
> >+  gEdkiiDebugPpiGuid
> >+
> >--
> >2.21.0.windows.1



  reply	other threads:[~2019-04-01  0:26 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-03-31 23:50 [PATCH V6 00/17] Add new APIs DebugVPrint for DebugLib Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 01/17] MdePkg/DebugLib.h: Add new APIs " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 02/17] MdePkg/BaseDebugLibNull: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 03/17] MdePkg/BaseDebugLibSerialPort: Add new APIs Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 04/17] MdePkg/UefidebugLibConOut: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 05/17] MdePkg/UefiDebugLibStdErr: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 06/17] MdePkg/DxeRuntimeDebugLibSerialPort: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 07/17] MdePkg/UefiDebuglibDebugPortProtocol: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 08/17] ArmPkg/SemiHostingDebugLib: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 09/17] OvmfPkg/PlatformDebugLibIoPort: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 10/17] IntelFsp2Pkg/BaseFspDebugLibSerialPort: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 11/17] IntelFspPkg/BaseFspDebugLibSerialPort: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 12/17] IntelFramworkModulePkg/PeiDxeDebugLibReportStatusCode: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 13/17] MdeModulePkg/PeiDxeDebugLibReportStatusCode: " Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 14/17] MdeModulePkg: Add definitions for EDKII DEBUG PPI Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 15/17] MdeModulePkg: Add a PEIM to install Debug PPI Zhichao Gao
2019-03-31 23:50 ` [PATCH V6 16/17] MdeModulePkg/PeiDebugLibDebugPpi: Add PEI debug lib Zhichao Gao
2019-04-01  0:20   ` Gao, Liming
2019-04-01  0:26     ` Gao, Zhichao [this message]
2019-03-31 23:50 ` [PATCH V6 17/17] MdeModulePkg: Add PEIM and lib to dsc file Zhichao Gao

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3CE959C139B4C44DBEA1810E3AA6F9000B7B9E27@SHSMSX101.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox