All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: David Miller <davem@davemloft.net>
Cc: akpm@linux-foundation.org, dada1@cosmosbay.com,
	herbert@gondor.apana.org.au, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: include/linux/pcounter.h
Date: Wed, 27 Feb 2008 08:36:50 +0100	[thread overview]
Message-ID: <20080227073650.GC4638@elte.hu> (raw)
In-Reply-To: <20080226.133531.69481052.davem@davemloft.net>


* David Miller <davem@davemloft.net> wrote:

> > but the main and fundamental question still remains unanswered (more 
> > than 3 weeks after Andrew asked that question): why was this piece 
> > of general infrastructure merged via net.git and not submitted to 
> > lkml ever? The code touching -mm does _not_ count as "review".
> 
> I already stated this was a mistake and it won't happen again in the 
> future.

sorry - that bit of the thread didnt seem to make it to lkml. I just saw 
this incomplete discussion with a denial and with no resolution.

And you did the right thing anyway by thinking in terms of a generic 
piece of infrastructure instead of hiding it away into say 
include/net/pcounter.h (which nobody could have objected against).

I sometimes think that the forced isolation of subsystems (rather 
strongly enforced both by -mm and by linux-next) and their hiding away 
on non-lkml lists will eventually hurt the core kernel because less and 
less people will be willing to go the trouble of doing proper 
cross-subsystem development. That results in duplicated or specialistic 
infrastructure, increased code size and longer term, ultimately less 
performance. (by the time we notice _that_ it will probably be too late 
to do anything about it)

	Ingo

  reply	other threads:[~2008-02-27  7:37 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-04  9:44 include/linux/pcounter.h Andrew Morton
2008-02-05  0:20 ` include/linux/pcounter.h David Miller
2008-02-05  0:43   ` include/linux/pcounter.h Andrew Morton
2008-02-16  3:37 ` include/linux/pcounter.h Andrew Morton
2008-02-16 10:07   ` include/linux/pcounter.h Eric Dumazet
2008-02-16 10:50     ` include/linux/pcounter.h Andrew Morton
2008-02-16 12:03       ` include/linux/pcounter.h Eric Dumazet
2008-02-16 19:26         ` include/linux/pcounter.h Andrew Morton
2008-02-16 19:39           ` include/linux/pcounter.h Arjan van de Ven
2008-02-17  5:54           ` include/linux/pcounter.h David Miller
2008-02-26  9:24             ` include/linux/pcounter.h Ingo Molnar
2008-02-26 21:35               ` include/linux/pcounter.h David Miller
2008-02-27  7:36                 ` Ingo Molnar [this message]
2008-02-28  1:27                   ` include/linux/pcounter.h Arnaldo Carvalho de Melo

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=20080227073650.GC4638@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@linux-foundation.org \
    --cc=dada1@cosmosbay.com \
    --cc=davem@davemloft.net \
    --cc=herbert@gondor.apana.org.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.