From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 1ADB4740035 for ; Sun, 29 Oct 2023 18:29:51 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=wSugw9eIYKjsqVRpqA28+b69Lx2Uu+DG6SbROdqEgzY=; c=relaxed/simple; d=groups.io; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding; s=20140610; t=1698604190; v=1; b=K2kkuE+r+pKQrlNVgKiWa0rjIgfBA1ma1LTPOSv/8+PWPG8Pb5j3YniYnaPzQPLvh5ZT2LgD alZvzQ72aapY8VDDcTIRI5nkm4uPZ/Oitlg/vewuue1v6ntu/r0aL6gO1fjcNwKAKMojCLIFbUP SjDmUPg511Trul6VYtXCwAYs= X-Received: by 127.0.0.2 with SMTP id AtcTYY7687511xrgIKbv3jao; Sun, 29 Oct 2023 11:29:50 -0700 X-Received: from mail-vs1-f50.google.com (mail-vs1-f50.google.com [209.85.217.50]) by mx.groups.io with SMTP id smtpd.web11.77939.1698604190084951274 for ; Sun, 29 Oct 2023 11:29:50 -0700 X-Received: by mail-vs1-f50.google.com with SMTP id ada2fe7eead31-457bda4cf32so1464787137.3 for ; Sun, 29 Oct 2023 11:29:49 -0700 (PDT) X-Gm-Message-State: YIinAz4hjhwEnOH67j7Fw3Vcx7686176AA= X-Google-Smtp-Source: AGHT+IEgWUExCK+H8yPx8rm1+4jSabwUKQNp0eOaCuBSleyRRjwvlbrUeC82dTN5cfT7evq91or1NcORgm35MRqw5ys= X-Received: by 2002:a67:e013:0:b0:45a:b852:9d6d with SMTP id c19-20020a67e013000000b0045ab8529d6dmr6708741vsl.13.1698604188883; Sun, 29 Oct 2023 11:29:48 -0700 (PDT) MIME-Version: 1.0 References: <20231028192330.1031-1-michael.d.kinney@intel.com> <99615ab9-f669-5ac8-fafd-f154e8af5da8@redhat.com> In-Reply-To: From: "Pedro Falcato" Date: Sun, 29 Oct 2023 18:29:37 +0000 Message-ID: Subject: Re: [edk2-devel] [Patch 1/1] Maintainers.txt: Update based on active community members To: devel@edk2.groups.io, jiewen.yao@intel.com Cc: "jejb@linux.ibm.com" , Laszlo Ersek , "Kinney, Michael D" , Andrew Fish , Leif Lindholm , "Warkentin, Andrei" , "West, Catharine" , "Bi, Dandan" , Daniel Schaefer , David Woodhouse , "De, Debkumar" , "Dong, Eric" , "Jiang, Guomin" , "Wu, Hao A" , "Wang, Jian J" , "Justen, Jordan L" , Julien Grall , Peter Grehan , "Zhang, Qi1" , "Ng, Ray Han Lim" , Stefan Berger , "Hou, Wenxing" , "Lu, Xiaoyu1" Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,pedro.falcato@gmail.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=K2kkuE+r; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io On Sun, Oct 29, 2023 at 4:25=E2=80=AFPM Yao, Jiewen = wrote: > > OK. Maintainer should do code review. I have no doubt on that. > > My confusion is about "reviewer" role. What is criteria and what is respo= nsibility? > > Are you saying that "reviewer" just means that someone raised the hand an= d said: "I want to be notified", and there is no expectation that he/she wo= uld review the patch? > > I would like to understand more on how that works and what that means. The Linux development process, as I understand it (it may be a bit imprecise, AFAIK lots of it isn't written): Maintainers are the primary caretakers of the code. They'll review and merge patches (in linux, they usually add their own Signed-off-by, they don't do Reviewed-by). Sometimes, they'll post a patch on the mailing list, and if there's no poor feedback, they just merge it to their trees, unilaterally (for Linux, Linus usually pulls from maintainer trees, and if he doesn't like something he *will* tell you). Reviewers are people that want to be CC'd and want to review patches, but they're not expected to always do so. There's of course an expectation that reviewers are relatively competent in the area they're reviewing. There's an expectation that you will help out and participate in code review from time to time. As an example: SLAB ALLOCATOR M: Christoph Lameter M: Pekka Enberg M: David Rientjes M: Joonsoo Kim M: Andrew Morton M: Vlastimil Babka R: Roman Gushchin R: Hyeonggon Yoo <42.hyeyoo@gmail.com> L: linux-mm@kvack.org S: Maintained T: git git://git.kernel.org/pub/scm/linux/kernel/git/vbabka/slab.git F: include/linux/sl?b*.h F: mm/sl?b* For a patch for mm/slab.c, you CC all M's and R's. Maintainers are the one with the 'power' to merge it, and should give you feedback. Reviewers sometimes help out (but are secondary in the patch review role), but they cannot merge patches. You only merge a patch if there's an understanding that most people and stakeholders are ok with it; for example, you may want feedback from people that are not M's and R's. If everyone is ok with your patch, a maintainer will apply it (in this case, it's vbabka's tree so he usually takes care of it), and it will eventually trickle up to Linus (who manages the 'master' git tree) who gives the final seal of approval when pulling changes. For a smaller example, we can look at EFI, which has a sole maintainer (Ard) and no reviewers; this is ok (EFI is a lot less central to the kernel than SLAB, there are a lot less patches), but stakeholders in the various changes should still test and review. This is a nice rough description of the whole development process: https://docs.kernel.org/process/2.Process.html Note that EDK2 is considerably smaller than the kernel in scope and patch volume, so it probably doesn't make much sense to be as distributed as Linux. PS: It's worth noting that the Linux equivalent to GetMaintainers.py takes into account the git blame of the files you're touching, terms you mention in the commit message; that is, it will automatically pick up people that have touched the code before or are responsible for features you're interacting with. --=20 Pedro -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110273): https://edk2.groups.io/g/devel/message/110273 Mute This Topic: https://groups.io/mt/102245264/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-