From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) by mx.groups.io with SMTP id smtpd.web11.14522.1594123002740172018 for ; Tue, 07 Jul 2020 04:56:43 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@nuviainc-com.20150623.gappssmtp.com header.s=20150623 header.b=HDCUl9SU; spf=pass (domain: nuviainc.com, ip: 209.85.128.54, mailfrom: leif@nuviainc.com) Received: by mail-wm1-f54.google.com with SMTP id l17so42985966wmj.0 for ; Tue, 07 Jul 2020 04:56:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nuviainc-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=xeIBTBCibDfHqdRZiNExCwOnfQQ1HG1CIKAg/TSG1ek=; b=HDCUl9SU/n8lK5WiuEVBN5seeLv9RrSd3I5Ego5RiDjxtF93boGA1pKZPMXIOXlAOq PTroEoKetTulHttT6PMLEEVvy1xehckO4RZ+I+cZ+GDWrt0Iwu/AwHyrL61Qhl6Qg0NO /r3AlEpUNtZbwXb70y+paQ4PRDsEmrEmm4mgcm6OScYFjPbUQXSqPAFkQyfys6r2eSmt yfPoOhIEvgEYZ3O3acsI9VRtMcPOHLDOfIf04zyNXwR7/09e7kPBb/T9cOI8QR/Pi+bg qwlxzBSEhDzrMbCH6edzeTuNwSYR/1vynlBFmLecMnd8mY0po59elrwKOTAZyk7yH1UI xKbQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=xeIBTBCibDfHqdRZiNExCwOnfQQ1HG1CIKAg/TSG1ek=; b=O3SQyB3GSPdJVOT3F8XOuFIJal0nxa+E6bdTKFd4DeEUESZ8fXDsZocPrp8eiJtX4a y1Ks8yO3bXF8Zq27qgbnITz0lct+wB0skJDGsS2jbzZN07sSqaPeJvd9fe5VwVoscbzD 2APZH2tw6HmslP6nk/dJEd8kRkVBBz67TjqZf3uicyNZYDR1HA4IIOh2NzgNDtZOH72s aHZZTXUTbxc1VfQXBwsnvEeI70Qz2f2uaaO5Qqn0fYuPb400J+B8Wvv7UE6UHYYP0gzI RvaLmZvf8DDCLPiT5ET3mWIv8i7XDCgvQDIl54PHGp7FsTM3MUfVm1vyjRd4/cl4tSQx cooQ== X-Gm-Message-State: AOAM531oaiYG0yy9OOj+Df4z4AuizN9bdnLJATt04PGbbXJTbNchjFYb YHCkhA0RPVeZD1CINupdYZk37g== X-Google-Smtp-Source: ABdhPJzMTxImn4Yuao1koqgFPjyeC8qHqPrJ6X8Y9MhcgWWAzzD7YtPYuoEWGGmWdtqzLTqyaa0GNg== X-Received: by 2002:a1c:2095:: with SMTP id g143mr3663507wmg.78.1594123001239; Tue, 07 Jul 2020 04:56:41 -0700 (PDT) Return-Path: Received: from vanye ([2001:470:1f09:12f0:b26e:bfff:fea9:f1b8]) by smtp.gmail.com with ESMTPSA id l132sm805488wmf.6.2020.07.07.04.56.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Jul 2020 04:56:40 -0700 (PDT) Date: Tue, 7 Jul 2020 12:56:38 +0100 From: "Leif Lindholm" To: "Zhang, Shenglei" Cc: "devel@edk2.groups.io" , "Feng, Bob C" , Bret Barkelew , "Kinney, Michael D" , "Gao, Liming" , Sean Brogan Subject: Re: [edk2-devel] [PATCH v7 00/16] Add a plugin to check Ecc issues for edk2 on open ci Message-ID: <20200707115638.GB12303@vanye> References: <20200706084846.12748-1-shenglei.zhang@intel.com> <20200706085508.GA12303@vanye> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Jul 07, 2020 at 02:24:14 +0000, Zhang, Shenglei wrote: > Comments below. > > > -----Original Message----- > > From: devel@edk2.groups.io On Behalf Of Leif > > Lindholm > > Sent: Monday, July 6, 2020 4:55 PM > > To: devel@edk2.groups.io; Zhang, Shenglei > > Cc: Feng, Bob C ; Bret Barkelew > > ; Kinney, Michael D > > ; Gao, Liming ; Sean > > Brogan > > Subject: Re: [edk2-devel] [PATCH v7 00/16] Add a plugin to check Ecc issues > > for edk2 on open ci > > > > On Mon, Jul 06, 2020 at 16:48:30 +0800, Zhang, Shenglei wrote: > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2606 > > > As planed we will enable Ecc check for edk2 on open ci. And they are > > > ready now. I appreciate receiving feedback and comments if someone > > > find errors or false positive issues. > > > > > > I created a pipline of EccCheck for my forked edk2. Welcome everyone to > > > create pull request to test the quality of this plugin. > > > My forked tree: https://github.com/shenglei10/edk2 > > > > > > And I also created some test cases for ECC plugin. Below are test cases. > > > https://github.com/shenglei10/edk2/tree/ECC > > > Results can be view in below azure server. > > > > > https://dev.azure.com/shengleizhang/shengleizhang/_build?definitionId=12 > > &_a=summary > > > > > > Patches > > > 1/16: It's a lib necessary for py3 to run Ecc on azure servers. > > > > > > 2/16: EccCheck.py is a plugin to report Ecc issues for commits. It can be run > > > on azure servers for open ci, or a local virtual environment. > > > > > > 3/16~16/16: We consider some cases that will report out Ecc issues but they > > won't > > > be fixed, like submodule and industry standard related things. So we > > > add two configuration fields "Exception" and "IgnoreFiles" for people > > > to use. These patches add configuration in yaml files for Ecc check. > > > > > > Cc: Bob Feng > > > Cc: Bret Barkelew > > > Cc: Michael D Kinney > > > Cc: Liming Gao > > > Cc: Sean Brogan > > > > > > v2: Update 1/17, fix the bug that the script can't hanlde multiple commits. > > > > > > v3: Update 1/17, set the only workalbe workspace is edk2 root directory. > > > Update 2/17, designate the version of antlr4 is 4.7.1. > > > Add 4/17~17/17. > > > > > > v4. Update 1/17, remove the function EdksetupRebuild(), instead add > > > function SetupEnvironment(). Update variables' format and type hints > > > to pass flake8 and mypy. > > > > > > v5. Conver the former method to plugin solution, to align with > > > other check points on open ci. > > > > > > v6. The 1/16 patch is missed in v5 series. Now add it in v6. > > > > > > v7. Fix a bug that Ecc plugin can not be run correctly under Linux OS. > > > > What is the bug? Where is it fixed? > > V6 plugin pipeline link: > https://dev.azure.com/shengleizhang/shengleizhang/_build/results?buildId=282&view=logs&j=9b8d87a5-bd93-5358-8ffa-588e312c4376&t=282a77bb-da29-5801-6299-2bbe15d5aabb > v7 plugin pipeline link: > https://dev.azure.com/shengleizhang/shengleizhang/_build/results?buildId=316&view=logs&j=5e3952f2-9c8a-5394-82c1-a803bcab3ca3&t=7ea9e85e-5c3d-5fcf-865e-8449f16cc745 > > For the same pull request, v6 plugin can't report out the Ecc issues under Linux OS. Right. So *what* was the bug and *where* (which patch or patches) was it fixed? This is information that should (in an abbreviated form) be provided in the cover letter. "Fix a bug" is data, not information. The same is true for "can not be run correctly". And neither tells me which patch I should look at. If this has been covered in a specific discussion thread, a link to the thread *and* mentioning which patch(es) it affects is sufficient. Regards, Leif > > Thanks, > Shenglei > > > > > / > > Leif > > > > > Shenglei Zhang (16): > > > pip-requirements.txt: Add Ecc required lib > > > .pytool/Plugin: Add a plugin EccCheck > > > MdeModulePkg/MdeModulePkg.ci.yaml: Add configuration for Ecc check > > > ArmVirtPkg/ArmVirtPkg.ci.yaml: Add configuration for Ecc check > > > CryptoPkg/CryptoPkg.ci.yaml: Add configuration for Ecc check > > > EmulatorPkg/EmulatorPkg.ci.yaml: Add configuration for Ecc check > > > FatPkg/FatPkg.ci.yaml: Add configuration for Ecc check > > > FmpDevicePkg/FmpDevicePkg.ci.yaml: Add configuration for Ecc check > > > MdePkg/MdePkg.ci.yaml: Add configuration for Ecc check > > > NetworkPkg/NetworkPkg.ci.yaml: Add configuration for Ecc check > > > OvmfPkg/OvmfPkg.ci.yaml: Add configuration for Ecc check > > > PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml: Add configuration for Ecc check > > > SecurityPkg/SecurityPkg.ci.yaml: Add configuration for Ecc check > > > ShellPkg/ShellPkg.ci.yaml: Add configuration for Ecc check > > > UefiCpuPkg/UefiCpuPkg.ci.yaml: Add configuration for Ecc check > > > UnitTestFrameworkPkg: Add configuration for Ecc check in yaml file > > > > > > .pytool/Plugin/EccCheck/EccCheck.py | 267 ++++++++++++++++++ > > > .pytool/Plugin/EccCheck/EccCheck_plug_in.yaml | 11 + > > > .pytool/Plugin/EccCheck/Readme.md | 15 + > > > ArmVirtPkg/ArmVirtPkg.ci.yaml | 11 + > > > CryptoPkg/CryptoPkg.ci.yaml | 11 + > > > EmulatorPkg/EmulatorPkg.ci.yaml | 11 + > > > FatPkg/FatPkg.ci.yaml | 11 + > > > FmpDevicePkg/FmpDevicePkg.ci.yaml | 11 + > > > MdeModulePkg/MdeModulePkg.ci.yaml | 11 + > > > MdePkg/MdePkg.ci.yaml | 11 + > > > NetworkPkg/NetworkPkg.ci.yaml | 11 + > > > OvmfPkg/OvmfPkg.ci.yaml | 11 + > > > PcAtChipsetPkg/PcAtChipsetPkg.ci.yaml | 11 + > > > SecurityPkg/SecurityPkg.ci.yaml | 11 + > > > ShellPkg/ShellPkg.ci.yaml | 11 + > > > UefiCpuPkg/UefiCpuPkg.ci.yaml | 11 + > > > .../UnitTestFrameworkPkg.ci.yaml | 10 + > > > pip-requirements.txt | 1 + > > > 18 files changed, 447 insertions(+) > > > create mode 100644 .pytool/Plugin/EccCheck/EccCheck.py > > > create mode 100644 .pytool/Plugin/EccCheck/EccCheck_plug_in.yaml > > > create mode 100644 .pytool/Plugin/EccCheck/Readme.md > > > > > > -- > > > 2.18.0.windows.1 > > > > > > > > > > > > > > > > >