Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754640Ab2HGNfI (ORCPT ); Tue, 7 Aug 2012 09:35:08 -0400 Received: from rcsinet15.oracle.com ([148.87.113.117]:24622 "EHLO rcsinet15.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751910Ab2HGNfG convert rfc822-to-8bit (ORCPT ); Tue, 7 Aug 2012 09:35:06 -0400 MIME-Version: 1.0 Message-ID: Date: Tue, 7 Aug 2012 06:34:30 -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: acsinet22.oracle.com [141.146.126.238] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2039 Lines: 44 > From: Pekka Enberg [mailto:penberg@kernel.org] > Subject: Re: [RFC/PATCH] zcache/ramster rewrite and promotion > > On Mon, Aug 6, 2012 at 7:10 PM, Dan Magenheimer > wrote: > > 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? > > There's no reason we can't have subdirectories. That said, I really > don't see the point of having a separate directory called 'tmem'. It > might make sense to have mm/zcache and/or mm/ramster but I suspect > you can just fold the core code in mm/zcache.c and mm/ramster.c by > slimming down the weird Solaris-like 'tmem' abstractions. I'm not sure I understand... what is Solaris-like about tmem? And what would you slim down? While I agree one can often glom three separate 1000-line .c files into a single 3000-line .c file, I recently spent some time moving the other direction to, I thought, improve readability. Do kernel developers have a preference for huge .c files rather than smaller logically-separated moderate-sized files in a subdirectory? Thanks, 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/