From: Olaf Kirch <okir@suse.de>
Subject: Re: [PATCH 3/4] Add /proc/sys/fs/nfs sysctls to nfsd module
Date: Sat, 5 Aug 2006 15:15:16 +0200
Message-ID: <20060805131516.GA9703@suse.de>
References: <20060803110538.GA17173@suse.de>
	<76bd70e30608030952p15e1500cy191b5ef19241b58e@mail.gmail.com>
	<1154680981.21040.2415.camel@hole.melbourne.sgi.com>
	<17619.55834.927259.225840@cse.unsw.edu.au>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Cc: Chuck Lever <chucklever@gmail.com>,
	Linux NFS Mailing List <nfs@lists.sourceforge.net>,
	Greg Banks <gnb@melbourne.sgi.com>
To: Neil Brown <neilb@suse.de>
In-Reply-To: <17619.55834.927259.225840@cse.unsw.edu.au>
Sender: nfs-bounces@lists.sourceforge.net
Errors-To: nfs-bounces@lists.sourceforge.net

On Sat, Aug 05, 2006 at 09:36:58AM +1000, Neil Brown wrote:
> So of the three options - the one that I find it hardest to justify
> if /proc/sys, which is what the original patch uses - sad.

I have no problem with fixing up the patch to use any other mechanism.
But if we do this, I think we should move all tunables. Having them
in multiple places just sucks. This includes /proc/sys/sunrpc/*_debug
(which ended up in /proc/sys/sunrpc because a the time I wrote this code,
there was no way a module could create a sysctl directory).

Olaf
-- 
Olaf Kirch   |  --- o --- Nous sommes du soleil we love when we play
okir@suse.de |    / | \   sol.dhoop.naytheet.ah kin.ir.samse.qurax

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs