From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51663) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XSM0C-0006uM-Dh for qemu-devel@nongnu.org; Fri, 12 Sep 2014 04:14:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XSM07-0003aa-BV for qemu-devel@nongnu.org; Fri, 12 Sep 2014 04:14:44 -0400 Received: from mx1.redhat.com ([209.132.183.28]:8395) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XSM07-0003Zo-3z for qemu-devel@nongnu.org; Fri, 12 Sep 2014 04:14:39 -0400 Date: Fri, 12 Sep 2014 10:14:01 +0200 From: Kevin Wolf Message-ID: <20140912081401.GA5076@noname.redhat.com> References: <20140911201821.GA26751@stefanha-thinkpad.redhat.com> <20140911212243.GA16487@irqsave.net> <8761gt2vcn.fsf@blackfin.pond.sub.org> <33183CC9F5247A488A2544077AF1902086DC9EBE@SZXEMA503-MBS.china.huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <33183CC9F5247A488A2544077AF1902086DC9EBE@SZXEMA503-MBS.china.huawei.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] New requirement for getting block layer patches merged List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Gonglei (Arei)" Cc: =?iso-8859-1?Q?Beno=EEt?= Canet , Markus Armbruster , Stefan Hajnoczi , "qemu-devel@nongnu.org" Am 12.09.2014 um 09:02 hat Gonglei (Arei) geschrieben: > Hi, >=20 > > Subject: Re: [Qemu-devel] New requirement for getting block layer pat= ches > > merged > >=20 > > Beno=EEt Canet writes: > >=20 > > >> EOF > > >> --- > > >> If you have feedback or questions, let us know. The process can b= e > > >> tweaked as time goes on so we can continue to improve. > > > > > > Great mail. > >=20 > > Yup. Let's see how it works out. > >=20 >=20 > Yes. I can't agree more with you. >=20 > Recently I posted some patch series, but I can't get maintainer's feedb= ack in time. > That make me feel soulless TBH. I know maintainers are very busy usuall= y. They=20 > need to develop their own code and also need review the contributors' c= ode. > If some other peoples can spread the load of patch review, that's a gre= at thing IMHO. This is what Stefan's mail was actually for in some way: Letting you know that you should get a Reviewed-by first. At least for me, to be honest, this isn't a truly new process. I haven't been consistently requiring a Reviewed-by, but when I see someone else discuss a patch series and I don't have much time, I may scan the discussion to chime in if there is something fundamentally wrong, but otherwise let the author and the reviewer sort it out and wait until the discussion has settled. If I don't see a discussion, I might wait a few days for one. I'll probably keep reviewing paches without an R-b when they are simple or in my area of expertise (like qcow2), like any other reviewer should. The point is just that when I don't, before you ping us maintainers about a patch, try to get a good review from some other contributor. > > > Now we need a wiki entry describing the process. > > > Also we need something reminding who is the maintainer of the curre= nt > > week. > >=20 > > Usually obvious from the "applied to my tree" messages. You should CC both of us anyway (the patch might not be merged in the same week), so it doesn't matter that much who'll be handling it. Kevin