[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.kin...@intel.com> > Sent: Wednesday, April 19, 2023 1:55 AM > To: Chang, Abner <abner.ch...@amd.com>; devel@edk2.groups.io > Cc: Gao, Liming <gaolim...@byosoft.com.cn>; Liu, Zhiguang > <zhiguang....@intel.com>; Nickle Wang <nick...@nvidia.com>; Igor > Kulchytskyy <ig...@ami.com>; Oram, Isaac W <isaac.w.o...@intel.com>; > Attar, AbdulLateef (Abdul Lateef) <abdullateef.at...@amd.com>; Tinh > Nguyen <tinhngu...@os.amperecomputing.com>; Kinney, Michael D > <michael.d.kin...@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.ch...@amd.com <abner.ch...@amd.com> > > Sent: Tuesday, April 18, 2023 2:04 AM > > To: devel@edk2.groups.io > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming > > <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; > > Nickle Wang <nick...@nvidia.com>; Igor Kulchytskyy <ig...@ami.com>; > > Oram, Isaac W <isaac.w.o...@intel.com>; Abdul Lateef Attar > > <abdullateef.at...@amd.com>; Tinh Nguyen > > <tinhngu...@os.amperecomputing.com> > > Subject: [PATCH] MdePkg: Add DEBUG_MANAGEABILITY print error level > > > > From: Abner Chang <abner.ch...@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.ch...@amd.com> > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > Cc: Zhiguang Liu <zhiguang....@intel.com> > > Cc: Nickle Wang <nick...@nvidia.com> > > Cc: Igor Kulchytskyy <ig...@ami.com> > > Cc: Isaac Oram <isaac.w.o...@intel.com> > > Cc: Abdul Lateef Attar <abdullateef.at...@amd.com> > > Cc: Tinh Nguyen <tinhngu...@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 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103189): https://edk2.groups.io/g/devel/message/103189 Mute This Topic: https://groups.io/mt/98339907/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
