Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932266Ab2HFQLh (ORCPT ); Mon, 6 Aug 2012 12:11:37 -0400 Received: from rcsinet15.oracle.com ([148.87.113.117]:50178 "EHLO rcsinet15.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932221Ab2HFQLg convert rfc822-to-8bit (ORCPT ); Mon, 6 Aug 2012 12:11:36 -0400 MIME-Version: 1.0 Message-ID: Date: Mon, 6 Aug 2012 09:10:26 -0700 (PDT) From: Dan Magenheimer To: Pekka Enberg Cc: Seth Jennings , Konrad Wilk , Minchan Kim , Nitin Gupta , Andrew Morton , Robert Jennings , Greg Kroah-Hartman , devel@driverdev.osuosl.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: RE: [RFC/PATCH] zcache/ramster rewrite and promotion References: In-Reply-To: X-Priority: 3 X-Mailer: Oracle Beehive Extensions for Outlook 2.0.1.7 (607090) [OL 12.0.6661.5003 (x86)] Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8BIT X-Source-IP: ucsinet21.oracle.com [156.151.31.93] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1679 Lines: 40 > From: Pekka Enberg [mailto:penberg@kernel.org] > Subject: Re: [RFC/PATCH] zcache/ramster rewrite and promotion > > On Mon, Aug 6, 2012 at 5:07 PM, Dan Magenheimer > wrote: > > I'm OK with placing it wherever kernel developers want to put > > it, as long as the reason is not NIMBY-ness. [1] My preference > > is to keep all the parts together, at least for the review phase, > > but if there is a consensus that it belongs someplace else, > > I will be happy to move it. > > I'd go for core code in mm/zcache.c and mm/ramster.c, and move the > clustering code under net/ramster or drivers/ramster. Hi Pekka -- Thanks for the quick feedback! Hmmm.. there's also zbud.c and tmem.c which are critical components of both zcache and ramster. And there are header files as well which will need to either be in mm/ or somewhere in include/linux/ Is there a reason or rule that mm/ can't have subdirectories? Since zcache has at least three .c files plus ramster.c, and since mm/frontswap.c and mm/cleancache.c are the foundation on which all of these are built, I was thinking grouping all six (plus headers) in the same mm/tmem/ subdirectory was a good way to keep mm/ from continuing to get more cluttered... not counting new zcache and ramster files, there are now 74 .c files in mm/! (Personally, I think a directory has too many files in it if "ls" doesn't fit in a 25x80 window.) Thoughts? Dan -- 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/