From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:58815) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S78rb-0007Lu-Kw for qemu-devel@nongnu.org; Mon, 12 Mar 2012 13:16:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S78rV-0003V8-0b for qemu-devel@nongnu.org; Mon, 12 Mar 2012 13:16:51 -0400 Received: from mail-pz0-f45.google.com ([209.85.210.45]:59124) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S78rU-0003Uh-RD for qemu-devel@nongnu.org; Mon, 12 Mar 2012 13:16:44 -0400 Received: by dadp14 with SMTP id p14so6643079dad.4 for ; Mon, 12 Mar 2012 10:16:42 -0700 (PDT) Message-ID: <4F5E2F77.1060008@codemonkey.ws> Date: Mon, 12 Mar 2012 12:16:39 -0500 From: Anthony Liguori MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] We need more reviewers/maintainers!! List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefano Stabellini Cc: qemu-devel@nongnu.org On 03/12/2012 12:06 PM, Stefano Stabellini wrote: > Hi all, > I don't mean to steer any controversy or start any flame wars here, but > rather I want to point out a problem in the QEMU Community that is > preventing us and other people from having a good experience working > upstream with QEMU. Call it constructive criticism. > > Patches are being posted to the list that don't get any reviews at all. > Other patches get reviewed the first time, then once they are reposted > they don't get any other reviews or acked-by or reviewed-by. In all fairness, QEMU continues to grow year-to-year both in terms of total commits and number of contributors. The area that we struggle with is infrequent contributors that contribute non-trivial things and are write-only contributors. In this case, I really think the problem is expecting to be a write-only contributor. Part of participating in a community is not only pushing your own patches for acceptance but also reviewing other people's patches and participating in the discussion. If everyone only sends patches and doesn't review patches, then we'll never make progress. So I'd strongly suggest trying to spend some time reviewing other people's work. Right now, there are at least four different efforts around migration yet I don't see any of the people reviewing the other efforts. I think this is really the main problem. Regards, Anthony Liguori > > As a whole it takes biblical times to get through the QEMU review > process. I wonder how any commercial company with deadlines would be able > to cope with them. Even the Xen Community, that is far from a commercial > company, is having difficulties with them and now upstream QEMU is at > risk of missing the 4.2 release target. > > > We need more people reviewing patches. And we need more maintainers. > > > Anthony Liguori is still the maintainer for many areas within QEMU, and > he is clearly too busy for that. We need more people helping him review > patches for source files like savevm.c and vl.c. > > I believe in leading by example, so Anthony Perard and I will try to > review more patch series, even outside Xen support in QEMU, starting > from now. > I hope more people will start to do the same to the point that it will > get natural to add more names and email addresses to the MAINTAINERS > file. > > I hope that other people will recognize that this is a problem and be > willing to step up to find a solution. > > Thanks, > > Stefano