DPDK patches and discussions
 help / color / mirror / Atom feed
From: Stephen Hemminger <stephen@networkplumber.org>
To: Bruce Richardson <bruce.richardson@intel.com>
Cc: Thomas Monjalon <thomas.monjalon@6wind.com>,
	dev@dpdk.org, techboard@dpdk.org
Subject: Re: [dpdk-dev] [dpdk-techboard] decision process and DPDK scope
Date: Thu, 9 Feb 2017 14:49:05 -0800	[thread overview]
Message-ID: <20170209144905.6dc0db5f@xeon-e3> (raw)
In-Reply-To: <20170209122047.GA327480@bricha3-MOBL3.ger.corp.intel.com>

On Thu, 9 Feb 2017 12:20:47 +0000
Bruce Richardson <bruce.richardson@intel.com> wrote:

> > I think we can use this case to avoid seeing it again in the future.
> > I suggest that the technical board should check whether every new proposed
> > features are explained, discussed and approved enough in the community.
> > If needed, the technical board meeting minutes will give some lights to
> > the threads which require more attention.
> > Before adding a new library or adding a major API, there should be
> > some strong reviews which include discussing the DPDK scope.
> >   
> 
> The bigger question here is the default position of the DPDK community -
> default accept, or default reject. Your statements above all are very
> much keeping in the style of default reject i.e. every patch or change
> suggested is assumed to be unfit for acceptance unless reviewed in
> detail to prove beyond doubt otherwise.
> 
> I believe that we should change this default position, as I think that
> reject by default is hurting the community and will continue to do so.
> 
> NOTE: I am not suggesting that we allow all code in with zero review,
> but I am suggesting that if something has been reviewed and acked by at
> least one reviewer it should be autom

I agree but in a more assertive manner. The maintainer should be the default
and active reviewer of all submissions. Like other projects the maintainers job
is to review and accept (or provide constructive feedback). Otherwise the
job could just by done by some manager.

But recently, I have changed my mind. The current DPDK project model is not
scaling well. After hearing some of the arguments in favor of a multiple
committer model (see "Maintainers Don't Scale" )
https://kernel-recipes.org/en/2016/talks/maintainers-dont-scale/

And comments on lwn:
https://lwn.net/Articles/703005/

  reply	other threads:[~2017-02-09 22:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-09 11:11 [dpdk-dev] " Thomas Monjalon
2017-02-09 11:54 ` O'Driscoll, Tim
2017-02-09 13:23   ` Thomas Monjalon
2017-02-09 12:20 ` [dpdk-dev] [dpdk-techboard] " Bruce Richardson
2017-02-09 22:49   ` Stephen Hemminger [this message]
2017-02-10 15:54     ` Bruce Richardson
2017-02-10 17:23       ` Thomas Monjalon
2017-02-13 10:34         ` Bruce Richardson
2017-02-13 15:21     ` Mcnamara, John
2017-02-13 15:58       ` Wiles, Keith

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170209144905.6dc0db5f@xeon-e3 \
    --to=stephen@networkplumber.org \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=techboard@dpdk.org \
    --cc=thomas.monjalon@6wind.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).