Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751218AbZL1Blr (ORCPT ); Sun, 27 Dec 2009 20:41:47 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751139AbZL1Blr (ORCPT ); Sun, 27 Dec 2009 20:41:47 -0500 Received: from one.firstfloor.org ([213.235.205.2]:53637 "EHLO one.firstfloor.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751126AbZL1Blq (ORCPT ); Sun, 27 Dec 2009 20:41:46 -0500 Date: Mon, 28 Dec 2009 02:41:41 +0100 From: Andi Kleen To: Roland Dreier Cc: Stefani Seibold , Andi Kleen , linux-kernel@vger.kernel.org, akpm@osdl.org Subject: Re: [PATCH] [0/6] kfifo fixes/improvements Message-ID: <20091228014141.GE2399@basil.fritz.box> References: <200912271003.631128760@firstfloor.org> <1261949800.25298.18.camel@wall-e> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1077 Lines: 27 On Sun, Dec 27, 2009 at 04:12:06PM -0800, Roland Dreier wrote: > > > I am not happy to see you to take over my project. Especial as most of > > your fixes are part of my new macro based implementation. Have a look at > > http://patchwork.kernel.org/patch/69093/ > > I don't really understand. You spent a lot of time getting the kfifo > stuff merged, and now you want to merge (quoting from that patch above) > "a complete reimplementation of the new kfifo API"? > > What happened here? Couldn't you have done the reimplementation before > merging? I guess the reimplementation came too late (happens sometimes) And I agree that making kfifos record oriented makes sense. Still now that the old one is in we have to fix it at least until there are no users left. -Andi -- ak@linux.intel.com -- Speaking for myself only. -- 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/