Michael: I think this change is helpful. For this patch set, Acked-by: Liming Gao <[email protected]>
Thanks Liming > -----邮件原件----- > 发件人: [email protected] <[email protected]> 代表 Michael > Kubacki > 发送时间: 2023年8月30日 7:27 > 收件人: [email protected] > 抄送: Abner Chang <[email protected]>; Alexei Fedorov > <[email protected]>; Ard Biesheuvel <[email protected]>; > Gerd Hoffmann <[email protected]>; Igor Kulchytskyy <[email protected]>; > Jian J Wang <[email protected]>; Jiewen Yao <[email protected]>; > Jordan Justen <[email protected]>; Leif Lindholm > <[email protected]>; Liming Gao <[email protected]>; > Michael D Kinney <[email protected]>; Nickle Wang > <[email protected]>; Pierre Gondois <[email protected]>; Sami > Mujawar <[email protected]>; Sean Brogan > <[email protected]> > 主题: Re: [edk2-devel] [PATCH v1 0/7] Add DebugMacroCheck > > Hi all, > > Now that the stable tag is out, I would appreciate reviews for this series. > > Thanks, > Michael > > On 8/14/2023 4:48 PM, Michael Kubacki wrote: > > From: Michael Kubacki <[email protected]> > > > > Adds a new script and build plugin called DebugMacroCheck. > > > > The script verifies that the number of print specifiers match the > > number of arguments in DEBUG() calls. > > > > Overview: > > > > - Build plugin: BuildPlugin/DebugMacroCheckBuildPlugin.py > > - Runs on any build target that is not NO-TARGET > > - Standalone script: DebugMacroCheck.py > > - Run `DebugMacroCheck.py --help` to see command line options > > - Unit tests: > > - Tests/test_DebugMacroCheck.py > > - Can be run with: > > `python -m unittest discover > -s ./.pytool/Plugin/DebugMacroCheck/tests -v` > > - Also visible in VS Code Test Explorer > > > > Background: > > > > The tool has been constantly run against edk2 derived code for about > > a year now. During that time, its found over 20 issues in edk2, over > > 50 issues in various vendor code, and numerous other issues specific > > to Project Mu. > > > > See the following series for a batch of issues previously fixed in > > edk2 discovered by the tool: > > > > https://edk2.groups.io/g/devel/message/93104 > > > > I've received interest from vendors to place it in edk2 to > > immediately find issues in the upstream and make it easier for edk2 > > consumers to directly acquire it. That led to this patch series. > > > > This would run in edk2 as a build plugin. All issues in the edk2 > > codebase have been resolved so this would find new issues before > > they are merged into the codebase. > > > > The script is meant to be portable so it can be run as a build plugin > > or dropped as a standalone script into other environments alongside > > the unit tests. > > > > Series Overview: > > > > - Fixes outstanding issues in RedfishPkg > > - Adds the `regex` PIP module to pip-requirements.txt > > - Adds exceptions for debug macro usage in ArmVirtPkg, > > DynamicTablesPkg, and SecurityPkg > > - Disables the plugin in OvmfPkg per maintainer's previous > > preferences > > - Adds the plugin > > > > The plugin (this series) is running with passing CI results as shown > > in this PR: > > https://github.com/tianocore/edk2/pull/4736 > > > > Cc: Abner Chang <[email protected]> > > Cc: Alexei Fedorov <[email protected]> > > Cc: Ard Biesheuvel <[email protected]> > > Cc: Gerd Hoffmann <[email protected]> > > Cc: Igor Kulchytskyy <[email protected]> > > Cc: Jian J Wang <[email protected]> > > Cc: Jiewen Yao <[email protected]> > > Cc: Jordan Justen <[email protected]> > > Cc: Leif Lindholm <[email protected]> > > Cc: Liming Gao <[email protected]> > > Cc: Michael D Kinney <[email protected]> > > Cc: Nickle Wang <[email protected]> > > Cc: Pierre Gondois <[email protected]> > > Cc: Sami Mujawar <[email protected]> > > Cc: Sean Brogan <[email protected]> > > > > Michael Kubacki (7): > > RedfishPkg/PlatformHostInterfaceBmcUsbNicLib: Fix DEBUG macro > args > > pip-requirements.txt: Add regex > > SecurityPkg.ci.yaml: Add debug macro exception > > ArmVirtPkg.ci.yaml: Add debug macro exception > > DynamicTablesPkg.ci.yaml: Add debug macro exception > > OvmfPkg/PlatformCI: Disable DebugMacroCheck > > .pytool/Plugin: Add DebugMacroCheck > > > > > RedfishPkg/Library/PlatformHostInterfaceBmcUsbNicLib/PlatformHostInterfa > ceBmcUsbNicLib.c | 8 +- > > .pytool/Plugin/DebugMacroCheck/BuildPlugin/DebugMacroCheckBuildPl > ugin.py | 127 +++ > > .pytool/Plugin/DebugMacroCheck/BuildPlugin/DebugMacroCheck_plug_ > in.yaml | 11 + > > .pytool/Plugin/DebugMacroCheck/DebugMacroCheck.py > | 859 ++++++++++++++++++++ > > .pytool/Plugin/DebugMacroCheck/Readme.md > | 253 ++++++ > > .pytool/Plugin/DebugMacroCheck/tests/DebugMacroDataSet.py > | 674 +++++++++++++++ > > .pytool/Plugin/DebugMacroCheck/tests/MacroTest.py > | 131 +++ > > .pytool/Plugin/DebugMacroCheck/tests/__init__.py > | 0 > > .pytool/Plugin/DebugMacroCheck/tests/test_DebugMacroCheck.py > | 201 +++++ > > ArmVirtPkg/ArmVirtPkg.ci.yaml > | 8 + > > DynamicTablesPkg/DynamicTablesPkg.ci.yaml > | 8 + > > OvmfPkg/PlatformCI/PlatformBuildLib.py > | 1 + > > SecurityPkg/SecurityPkg.ci.yaml > | 9 + > > pip-requirements.txt > | 2 +- > > 14 files changed, 2287 insertions(+), 5 deletions(-) > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/BuildPlugin/DebugMacroCheckBuil > dPlugin.py > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/BuildPlugin/DebugMacroCheck_pl > ug_in.yaml > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/DebugMacroCheck.py > > create mode 100644 .pytool/Plugin/DebugMacroCheck/Readme.md > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/tests/DebugMacroDataSet.py > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/tests/MacroTest.py > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/tests/__init__.py > > create mode > 100644 .pytool/Plugin/DebugMacroCheck/tests/test_DebugMacroCheck.py > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#108247): https://edk2.groups.io/g/devel/message/108247 Mute This Topic: https://groups.io/mt/101142278/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
