From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=B5EeUQI0; spf=pass (domain: linaro.org, ip: 209.85.221.52, mailfrom: leif.lindholm@linaro.org) Received: from mail-wr1-f52.google.com (mail-wr1-f52.google.com [209.85.221.52]) by groups.io with SMTP; Tue, 11 Jun 2019 02:51:55 -0700 Received: by mail-wr1-f52.google.com with SMTP id n4so12290150wrs.3 for ; Tue, 11 Jun 2019 02:51:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=1sI3qTrW587SyEWcXospxKPOJdkmCVcsYoI3Ft0ijpM=; b=B5EeUQI0uhd1qSMYOFxRfdvhbIZ0DQDyH59EK76PIQ7aDQL0HP3ok7PtjZFOrvCo7h YZXtCbdjyHYiAV2d8jtAWP1I78BOnAH7D/jHkq7EcqyqNSl72hSG9xn5u/zhaC7Xpuhl mTZGvXsjOIrK8yD3I660659sdVdtWHpqFLXAHMVY7f3Hmkr3BJWjGD+znwE3ssutbDrz da7qeRxxcCY2Vt7MOuhuPBe6h8nt0yh2NZ1USIKFisxZtgpQmFf3OuzU9sERxhUKGqCK Hm3RiE+prB/af8iA21gVRn6EmSt/rfBxQiWpcMx0wiRGHKh4DCY5TMloZjy9+vXbxiEV +ooQ== 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=1sI3qTrW587SyEWcXospxKPOJdkmCVcsYoI3Ft0ijpM=; b=CkBHEtdnsJrkzZghsgDlw3VhMorwPcJSmr1hWMNOkOcUbz9jX5L5Xw8V4f4OUOlRAe sTnAocDAHlPoCVnGhZ/Qh7PGvwKR/yTe0WF+5XL/mbyUtLPXaENtnUOONCMJPXG0jSZe dv9LNydOFBSVHkVYKjyZJ2w/HKwBkv7F2l5tcXeVlD/qskhQBomz2Wux1oop/IHiwEp2 pUQkfaHtUIj+EqQWBQ0m1iCQagKj2N1rcz4rz5Pi4uxQ0K0jR/MdMj/jtH8pbSDXs2h9 vGASRa8P/9Oc3rpNyOuzzpywXStedAuSjEI02Z06REWvmwT5ClY0PxeBZBSIUBy/rv+L k2lQ== X-Gm-Message-State: APjAAAXh2oSnB9UCWC2Pm4b9uewMJR5uohyGYqnbS/VLvrAS+C1tSiSE EGkbET0cRt01D5EvQXMhpM1gWA== X-Google-Smtp-Source: APXvYqzBDOPf+rarlFX38Yjnot2JP/NiNn+aS2OLyeJ74Y0niElUfNYUZW/OSLYqMRSQL4nCrGC7ag== X-Received: by 2002:a5d:5152:: with SMTP id u18mr16018456wrt.319.1560246712762; Tue, 11 Jun 2019 02:51:52 -0700 (PDT) Return-Path: Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id 5sm2051682wmg.7.2019.06.11.02.51.51 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 11 Jun 2019 02:51:51 -0700 (PDT) Date: Tue, 11 Jun 2019 10:51:50 +0100 From: "Leif Lindholm" To: "Wu, Hao A" Cc: "devel@edk2.groups.io" , "lersek@redhat.com" , "Kinney, Michael D" , Andrew Fish , "Justen, Jordan L" , Ard Biesheuvel , "Gao, Liming" , "Yao, Jiewen" , "Zeng, Star" Subject: Re: [RFC] Fine-grained review ownership for MdeModulePkg Message-ID: <20190611095150.l7vmqyn3ln4gmvyq@bivouac.eciton.net> References: MIME-Version: 1.0 In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Wu Hao, I am working on a proposal (and proof-of-concept) machine readable Maintainers.txt format, and a GetMaintainer.py script, based on the format used by QEMU. One of the key features of that format is the F: tag, which specifes a filesytem path (with wildcard support). While this is just an RFC in progress (to be sent out later in week), if you could take that into account, and figure out which paths correspond to which areas of responsibility, that would make a test run of the script much easier. For my part, I would suggest adding (not just for MdeModulePkg) F: */ARM/* F: */AARCH64/* as maintained by myself and Ard. Best Regards, Leif On Mon, Jun 10, 2019 at 08:06:23AM +0000, Wu, Hao A wrote: > Hello all, > > Driven by the issue raised and discussions with thread: > https://edk2.groups.io/g/devel/topic/31907735#42076 > > This RFC proposal is raised to work out a fine-grained review ownership > for MdeModulePkg. > > The goal is to update the 'MdeModulePkg' section in file Maintainers.txt > to resemble the format like the 'OvmfPkg' section. Several reviewers will > be listed to take the ownership to review patches for a specific feature > in the package. > > In order to get the list of reviewers, a 2-step process will be adopted: > 1. Work out a list that contains major features in MdeModulePkg. > 2. Call for owners for every item in the list. > > For 1, a coarse feature list is: > Acpi > Bds > Capsule > Compress > Console > Core (PeiCore, DxeCore, PiSmmCore, etc. Mainly content in MdeModulePkg/Core/ folder) > Device (peripheral & storage) > Disk > EBC > HII > MemoryManagement (include memory test, memory allocation, memory profile, etc.) > PCD > PCI bus > Reset > S3 (including LockBox) > SMBIOS > SMM > Status Code > UI > Variable > Misc (all the components that do not fit in the above items) > > Once we think the granularity of the above list is fine and there is no > significant feature missing, I will update such list to include the > modules that belong to a specific feature. So adjustments can be done to > work out the final feature list. > > The proposal here is far from mature, so any feedback/suggestion will be > appreciated. > > > Best Regards, > Hao Wu > >