> -----Original Message----- > From: [email protected] <[email protected]> On Behalf Of Sami Mujawar > Sent: Wednesday, November 24, 2021 2:55 AM > To: Kinney, Michael D <[email protected]>; [email protected] > Cc: Ard Biesheuvel <[email protected]>; Yao, Jiewen > <[email protected]>; Supreeth Venkatesh > <[email protected]>; Sean Brogan <[email protected]>; Bret > Barkelew <[email protected]>; Liming > Gao <[email protected]>; Michael Kubacki > <[email protected]>; nd <[email protected]> > Subject: Re: [edk2-devel] [Patch 10/12] StandaloneMmPkg: Update YAML to > ignore specific ECC files/errors > > Hi Mike, > > > On 23/11/2021 09:21 PM, Michael D Kinney wrote: > > + "EccCheck": { > > + ## Exception sample looks like below: > > + ## "ExceptionList": [ > > + ## "<ErrorID>", "<KeyWord>" > > + ## ] > > + "ExceptionList": [ > > + "4002", "HobConstructor" > > + ], > > I noticed there are other ECC errors with ID 4002 in StandaloneMmPkg. > > e.g. > > edk2\StandaloneMmPkg\Library\StandaloneMmCoreHobLib\Arm\StandaloneMmCoreHobLib.c(24): > [4002]Function header doesn't exist Function [CreateHob] has NO comment > immediately preceding it. > edk2\StandaloneMmPkg\Library\StandaloneMmCoreHobLib\Arm\StandaloneMmCoreHobLibInternal.c(24): > [4002]Function header doesn't exist Function [HobConstructor] has NO > comment immediately preceding it. > edk2\StandaloneMmPkg\Library\StandaloneMmHobLib\StandaloneMmHobLib.c(247): > [4002]Function header doesn't exist Function [CreateHob] has NO comment > immediately preceding it. > edk2\StandaloneMmPkg\Library\StandaloneMmPeCoffExtraActionLib\AArch64\StandaloneMmPeCoffExtraActionLib.c(26): > [4002]Function header doesn't exist Function [UpdatePeCoffPermissions] > has NO comment immediately preceding it. > > I am trying to undestand how the ExceptionList works. Would Keyword = > HobConstructor ignore the other errors too?
Good question. EccCheck in EDK II CI only checks lines of files that have been added or changed. So if these lines were not modified by uncrustify, then they would not block the commit, even if they have ECC issues. > > We will look into getting these issues fixed. However, to unblock this > patch can proceed. Thank you. Yes. That is the intention here. Temporarily use YAML file to get EccCheck to pass, then the work with package owners to update code so these exceptions can be removed. > > Reviewed-by: Sami Mujawar <[email protected]> > > Regards, > > Sami Mujawar > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#84063): https://edk2.groups.io/g/devel/message/84063 Mute This Topic: https://groups.io/mt/87268744/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
