Re: [edk2-devel] [PATCH] MdePkg: Add DEBUG_MANAGEABILITY print error level

["Michael D Kinney" <michael.d.kinney@intel.com>]

Merged

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chang, Abner via groups.io
> Sent: Friday, April 21, 2023 1:10 AM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; Nickle Wang <nicklew@nvidia.com>; Igor
> Kulchytskyy <igork@ami.com>; Oram, Isaac W <isaac.w.oram@intel.com>; Attar, AbdulLateef (Abdul Lateef)
> <AbdulLateef.Attar@amd.com>; Tinh Nguyen <tinhnguyen@os.amperecomputing.com>
> Subject: Re: [edk2-devel] [PATCH] MdePkg: Add DEBUG_MANAGEABILITY print error level
> 
> [AMD Official Use Only - General]
> 
> 
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Michael
> > D Kinney via groups.io
> > Sent: Friday, April 21, 2023 5:09 AM
> > To: devel@edk2.groups.io; Chang, Abner <Abner.Chang@amd.com>
> > Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> > <zhiguang.liu@intel.com>; Nickle Wang <nicklew@nvidia.com>; Igor
> > Kulchytskyy <igork@ami.com>; Oram, Isaac W <isaac.w.oram@intel.com>;
> > Attar, AbdulLateef (Abdul Lateef) <AbdulLateef.Attar@amd.com>; Tinh
> > Nguyen <tinhnguyen@os.amperecomputing.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > Subject: Re: [edk2-devel] [PATCH] MdePkg: Add DEBUG_MANAGEABILITY
> > print error level
> >
> > Caution: This message originated from an External Source. Use proper
> > caution when opening attachments, clicking links, or responding.
> >
> >
> > Hi Abner,
> >
> > This seems like a reasonable additional to the debug levels.
> >
> > Do you intend to mix this debug level with DEBUG_ERROR, DEBUG_WARN,
> > DEBUG_INFO, DEBUG_VERBOSE.
> > DEBUG_INFO | DEBUG_LOAD example:
> We still use DEBUG_INFO for the function entry point. DEBUG_VERBOSE with DEBUG_MANAGEABILITY may useful if user would like to
> have full debug information with only DEBUG_VERBOSE enabled.
> 
> >
> >
> > https://github.com/tianocore/edk2/blob/9bf79303ae5cb4d0e14ed7a219107
> > b53e2ecdcd0/StandaloneMmPkg/Core/Dispatcher.c#L279
> >
> > There may be challenges defining what is a manageability related component
> > and what is not, but that can be handled through code reviews.
> >
> > Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> This is the PR that passed CI, please help to merge it.
> https://github.com/tianocore/edk2/pull/4294
> 
> Thanks
> Abner
> 
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chang,
> > > Abner via groups.io
> > > Sent: Tuesday, April 18, 2023 7:06 PM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>;
> > > devel@edk2.groups.io
> > > Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> > > <zhiguang.liu@intel.com>; Nickle Wang <nicklew@nvidia.com>; Igor
> > > Kulchytskyy <igork@ami.com>; Oram, Isaac W <isaac.w.oram@intel.com>;
> > > Attar, AbdulLateef (Abdul Lateef) <AbdulLateef.Attar@amd.com>; Tinh
> > > Nguyen <tinhnguyen@os.amperecomputing.com>
> > > Subject: Re: [edk2-devel] [PATCH] MdePkg: Add DEBUG_MANAGEABILITY
> > > print error level
> > >
> > > [AMD Official Use Only - General]
> > >
> > > I was hesitated having a new debug level for just RedfishPkg, until I
> > > realized not only RedfishPkg but also ManageabilityPKg and
> > > edk2-redfish-client need a specific debug level. The scope look to me
> > > big enough to create a new individual debug level due to lot of debug
> > > information for those modules and people don’t want to see VERBOSE is
> > > enabled on all modules just because PcdDebugPrintErrorLevel is
> > > overwritten in platform DSC in most of platform development practices.
> > There are many modules and multiple layers of driver stack in the above
> > packages, especially to the upcoming package under edk2-redfish-client repo.
> > It seems to me it would be pesky to developers for turning on VERBOSE on
> > certain modules one by one.
> > >
> > > Abner
> > >
> > > > -----Original Message-----
> > > > From: Kinney, Michael D <michael.d.kinney@intel.com>
> > > > Sent: Wednesday, April 19, 2023 1:55 AM
> > > > To: Chang, Abner <Abner.Chang@amd.com>; devel@edk2.groups.io
> > > > Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> > > > <zhiguang.liu@intel.com>; Nickle Wang <nicklew@nvidia.com>; Igor
> > > > Kulchytskyy <igork@ami.com>; Oram, Isaac W
> > <isaac.w.oram@intel.com>;
> > > > Attar, AbdulLateef (Abdul Lateef) <AbdulLateef.Attar@amd.com>; Tinh
> > > > Nguyen <tinhnguyen@os.amperecomputing.com>; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>
> > > > Subject: RE: [PATCH] MdePkg: Add DEBUG_MANAGEABILITY print error
> > > > level
> > > >
> > > > Caution: This message originated from an External Source. Use proper
> > > > caution when opening attachments, clicking links, or responding.
> > > >
> > > >
> > > > Do we really need a new debug level?
> > > >
> > > > The DEBUG_INIT/WARN/INFO/VERBOSE levels can be enabled/disabled
> > in
> > > > individual modules.
> > > >
> > > > I imagine even within the set of manageability components you will
> > > > want to use those different levels to adjust verbosity of the debug log.
> > > >
> > > > Mike
> > > >
> > > > > -----Original Message-----
> > > > > From: abner.chang@amd.com <abner.chang@amd.com>
> > > > > Sent: Tuesday, April 18, 2023 2:04 AM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > > > > <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> > > > > <zhiguang.liu@intel.com>; Nickle Wang <nicklew@nvidia.com>; Igor
> > > > > Kulchytskyy <igork@ami.com>; Oram, Isaac W
> > > > > <isaac.w.oram@intel.com>; Abdul Lateef Attar
> > > > > <AbdulLateef.Attar@amd.com>; Tinh Nguyen
> > > > > <tinhnguyen@os.amperecomputing.com>
> > > > > Subject: [PATCH] MdePkg: Add DEBUG_MANAGEABILITY print error
> > level
> > > > >
> > > > > From: Abner Chang <abner.chang@amd.com>
> > > > >
> > > > > Bug #4414
> > > > >
> > > > > Add DEBUG_MANAGEABILITY print error lever to output debug
> > message
> > > > > of detailed manageability related module information, such as
> > > > > - RedfishPkg:
> > > > >   - HTTP header/request/response
> > > > >   - JSON plain text
> > > > >   - Refish resource
> > > > >   - Redfish Host interface information
> > > > >   - Redfish credential information
> > > > >   - Platform configuration to Redfish mapping
> > > > >   - etc.
> > > > > - ManageabilityPKg
> > > > >   - Protocol payload of MCTP/PLDM/IPMI
> > > > >   - Payload of transport interface transfers
> > > > >   - IPMI BLOB transfer
> > > > >   - etc.
> > > > > - RedfishClinetPkg
> > > > >   - Redfish feature driver dispatcher
> > > > >   - Redfish BIOS attributes
> > > > >   - Platform configuration (HII) to
> > > > >     Redfish property information
> > > > >   - Redfish C structure information
> > > > >   - etc.
> > > > >
> > > > > Signed-off-by: Abner Chang <abner.chang@amd.com>
> > > > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > > > > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > > > > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > > > > Cc: Nickle Wang <nicklew@nvidia.com>
> > > > > Cc: Igor Kulchytskyy <igork@ami.com>
> > > > > Cc: Isaac Oram <isaac.w.oram@intel.com>
> > > > > Cc: Abdul Lateef Attar <AbdulLateef.Attar@amd.com>
> > > > > Cc: Tinh Nguyen <tinhnguyen@os.amperecomputing.com>
> > > > > ---
> > > > >  MdePkg/MdePkg.dec                 |  1 +
> > > > >  MdePkg/Include/Library/DebugLib.h | 42
> > > > > ++++++++++++++++---------------
> > > > >  2 files changed, 23 insertions(+), 20 deletions(-)
> > > > >
> > > > > diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index
> > > > > e70182bf7fb..a7566b688d3 100644
> > > > > --- a/MdePkg/MdePkg.dec
> > > > > +++ b/MdePkg/MdePkg.dec
> > > > > @@ -2282,6 +2282,7 @@
> > > > >    #  BIT20 - Global Coherency Database changes message.<BR>
> > > > >    #  BIT21 - Memory range cachability changes message.<BR>
> > > > >    #  BIT22 - Detailed debug message.<BR>
> > > > > +  #  BIT23 - Manageability messages. BR>
> > > > >    #  BIT31 - Error message.<BR>
> > > > >    # @Prompt Debug Message Print Level.
> > > > >    # @Expression  0x80000002 |
> > > > > (gEfiMdePkgTokenSpaceGuid.PcdDebugPrintErrorLevel & 0x7F84AA00)
> > ==
> > > > 0
> > > > > diff --git a/MdePkg/Include/Library/DebugLib.h
> > > > > b/MdePkg/Include/Library/DebugLib.h
> > > > > index 9110be2f41b..12de1584c41 100644
> > > > > --- a/MdePkg/Include/Library/DebugLib.h
> > > > > +++ b/MdePkg/Include/Library/DebugLib.h
> > > > > @@ -29,26 +29,28 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> > > > > // // Declare bits for PcdDebugPrintErrorLevel and the ErrorLevel
> > > > > parameter of DebugPrint()  //
> > > > > -#define DEBUG_INIT      0x00000001  // Initialization
> > > > > -#define DEBUG_WARN      0x00000002  // Warnings
> > > > > -#define DEBUG_LOAD      0x00000004  // Load events
> > > > > -#define DEBUG_FS        0x00000008  // EFI File system
> > > > > -#define DEBUG_POOL      0x00000010  // Alloc & Free (pool)
> > > > > -#define DEBUG_PAGE      0x00000020  // Alloc & Free (page)
> > > > > -#define DEBUG_INFO      0x00000040  // Informational debug messages
> > > > > -#define DEBUG_DISPATCH  0x00000080  // PEI/DXE/SMM Dispatchers
> > > > > -#define DEBUG_VARIABLE  0x00000100  // Variable
> > > > > -#define DEBUG_BM        0x00000400  // Boot Manager
> > > > > -#define DEBUG_BLKIO     0x00001000  // BlkIo Driver
> > > > > -#define DEBUG_NET       0x00004000  // Network Io Driver
> > > > > -#define DEBUG_UNDI      0x00010000  // UNDI Driver
> > > > > -#define DEBUG_LOADFILE  0x00020000  // LoadFile
> > > > > -#define DEBUG_EVENT     0x00080000  // Event messages
> > > > > -#define DEBUG_GCD       0x00100000  // Global Coherency Database
> > > > changes
> > > > > -#define DEBUG_CACHE     0x00200000  // Memory range cachability
> > > > changes
> > > > > -#define DEBUG_VERBOSE   0x00400000  // Detailed debug messages
> > that
> > > > may
> > > > > -                                    // significantly impact boot performance
> > > > > -#define DEBUG_ERROR  0x80000000     // Error
> > > > > +#define DEBUG_INIT           0x00000001  // Initialization
> > > > > +#define DEBUG_WARN           0x00000002  // Warnings
> > > > > +#define DEBUG_LOAD           0x00000004  // Load events
> > > > > +#define DEBUG_FS             0x00000008  // EFI File system
> > > > > +#define DEBUG_POOL           0x00000010  // Alloc & Free (pool)
> > > > > +#define DEBUG_PAGE           0x00000020  // Alloc & Free (page)
> > > > > +#define DEBUG_INFO           0x00000040  // Informational debug
> > messages
> > > > > +#define DEBUG_DISPATCH       0x00000080  // PEI/DXE/SMM
> > Dispatchers
> > > > > +#define DEBUG_VARIABLE       0x00000100  // Variable
> > > > > +#define DEBUG_BM             0x00000400  // Boot Manager
> > > > > +#define DEBUG_BLKIO          0x00001000  // BlkIo Driver
> > > > > +#define DEBUG_NET            0x00004000  // Network Io Driver
> > > > > +#define DEBUG_UNDI           0x00010000  // UNDI Driver
> > > > > +#define DEBUG_LOADFILE       0x00020000  // LoadFile
> > > > > +#define DEBUG_EVENT          0x00080000  // Event messages
> > > > > +#define DEBUG_GCD            0x00100000  // Global Coherency Database
> > > > changes
> > > > > +#define DEBUG_CACHE          0x00200000  // Memory range cachability
> > > > changes
> > > > > +#define DEBUG_VERBOSE        0x00400000  // Detailed debug
> > messages
> > > > that may
> > > > > +                                         // significantly impact
> > > > > +boot performance #define DEBUG_MANAGEABILITY  0x00800000  //
> > > > > +Detailed
> > > > debug and payload message of manageability
> > > > > +                                         // related modules, such Redfish, IPMI, MCTP and
> > etc.
> > > > > +#define DEBUG_ERROR          0x80000000  // Error
> > > > >
> > > > >  //
> > > > >  // Aliases of debug message mask bits
> > > > > --
> > > > > 2.37.1.windows.1
> > >
> > >
> > >
> > >
> >
> >
> >
> >
> >
> 
> 
> 
>