Date: Mon, 4 Jan 2010 13:57:54 -0800
From: Andrew Morton <akpm@linux-foundation.org>
To: Andi Kleen <andi@firstfloor.org>
Cc: Stefani Seibold <stefani@seibold.net>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] [4/6] kfifo: add kfifo_out_peek
Message-Id: <20100104135754.07d4e442.akpm@linux-foundation.org>
In-Reply-To: <20091227234137.GD2399@basil.fritz.box>
References: <200912271003.631128760@firstfloor.org>
	<20091227210314.ABA04B17C3@basil.firstfloor.org>
	<1261950589.25298.32.camel@wall-e>
	<20091227234137.GD2399@basil.fritz.box>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1678
Lines: 41

On Mon, 28 Dec 2009 00:41:37 +0100
Andi Kleen <andi@firstfloor.org> wrote:

> On Sun, Dec 27, 2009 at 10:49:49PM +0100, Stefani Seibold wrote:
> > Am Sonntag, den 27.12.2009, 22:03 +0100 schrieb Andi Kleen:
> > > In some upcoming code it's useful to peek into a FIFO without
> > > permanentely removing data. This patch implements a new 
> > > kfifo_out_peek() to do this. 
> > > 
> > 
> > Yes this function can be useful. I will implement it in the macro
> > version of the kfifo.
> 
> Well that doesn't help me -- I need a working kfifo on a 2.6.33
> base. Or do you want to abandon the old interface?
> 
> I must say I'm a little surprised by the general tone of  
> your comments -- first you submit these patches all the time and when they 
> finally get reviewed and merged and people start to use them and make them 
> production ready you state it's all abandoned.

Yup, the first priority is to get the 2.6.33 code fixed up.  It is
unwise to assume that we'll be merging some other patch some time in
the future.

Macros are unpopular, for good reasons.  But the case for a
template-based container such as this is a good one.  However I worry
about the code bloat whcih the macro version might add.  We worry about
all this later on.

> 
> Anyways Andrew could you please merge these fixes? Thanks.

I shall.  Stefani, pelase review those patches for 2.6.33 suitability. 
Let's not worry about a big rewrite at this time.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/