Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Wed, 13 Feb 2002 16:27:13 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Wed, 13 Feb 2002 16:26:54 -0500 Received: from pat.uio.no ([129.240.130.16]:16292 "EHLO pat.uio.no") by vger.kernel.org with ESMTP id ; Wed, 13 Feb 2002 16:26:47 -0500 To: Craig Christophel Cc: linux-kernel@vger.kernel.org, Keith Owens Subject: Re: [PATCH] -- filesystems.c::sys_nfsservctl In-Reply-To: <20020213205144Z282414-24962+32@thor.valueweb.net> From: Trond Myklebust Date: 13 Feb 2002 22:26:35 +0100 In-Reply-To: <20020213205144Z282414-24962+32@thor.valueweb.net> Message-ID: Lines: 16 User-Agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.1 (Cuyahoga Valley) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org >>>>> " " == Craig Christophel writes: > Ok guys get ready to flame me.... > The attached patch removes the lock/unlock in this function. > Now I am 80% > sure of this one, but would like a word from the kmod > maintainer about whether request_module needs the BKL or not. > do_nfsservctl already takes the BKL inside the function so as > long as request_module is safe this pair can be removed -- > effectively making do_nfsservctl responsible for it's own > locking scheme. What would remain to protect 'nfsd_linkage' if you removed the BKL? Cheers, Trond - 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/