Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753520Ab0KIOUu (ORCPT ); Tue, 9 Nov 2010 09:20:50 -0500 Received: from kroah.org ([198.145.64.141]:43202 "EHLO coco.kroah.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752156Ab0KIOUr (ORCPT ); Tue, 9 Nov 2010 09:20:47 -0500 Date: Tue, 9 Nov 2010 06:20:53 -0800 From: Greg KH To: Am??rico Wang Cc: Mike Waychison , simon.kagstrom@netinsight.net, davem@davemloft.net, nhorman@tuxdriver.com, Matt Mackall , adurbin@google.com, linux-kernel@vger.kernel.org, chavey@google.com, akpm@linux-foundation.org, linux-api@vger.kernel.org Subject: Re: [PATCH v2 12/23] netpoll: Introduce netpoll_target configs Message-ID: <20101109142053.GA3067@kroah.com> References: <20101108203120.22479.19708.stgit@crlf.mtv.corp.google.com> <20101108203246.22479.60118.stgit@crlf.mtv.corp.google.com> <20101109033024.GA5220@cr0.nay.redhat.com> <20101109042732.GC5220@cr0.nay.redhat.com> <20101109090645.GG5220@cr0.nay.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101109090645.GG5220@cr0.nay.redhat.com> 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: 1652 Lines: 42 On Tue, Nov 09, 2010 at 05:06:45PM +0800, Am??rico Wang wrote: > On Tue, Nov 09, 2010 at 12:34:24AM -0800, Mike Waychison wrote: > >On Mon, Nov 8, 2010 at 8:27 PM, Am??rico Wang wrote: > >> On Tue, Nov 09, 2010 at 11:30:24AM +0800, Am??rico Wang wrote: > >>> > >> .... > >>> > >>>So, either we need to de-modulize configfs or replace configfs API > >>>with sysfs API. Personally, I prefer the former one, I don't think > >>>configfs should be a module as long as it can provide API's > >>>for other subsystems, like debugfs. > >>> > >> > >> To clarify, I meant "as long as the API it provides can be used by > >> other core subsystems". > >> > > > >Ya, I see the problem with it being a tristate. > > > >Why not just make netconsole support being compiled in force configfs > >to be compiled in? Or does that just set bad precedent? > > That is what netconsole does now, and this is fine, since netconsole is > a module too, however, after you move that code into netpoll, then netpoll > will have a dependence on it, we will have problems. > > I think we can let NETPOLL_TARGET depend on CONFIGFS_FS=y, but I still > see no reason why CONFIGFS_FS should be a module. No, just have the NETPOLL_TARGET set CONFIG_FS to y, don't force the code to always be that way if the user isn't going to want that option. So as it originally was should have been fine. thanks, greg k-h -- 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/