From: "J. Bruce Fields" <bfields@fieldses.org>
Subject: Re: [PATCH] nfsd: fail module init on reply cache init	failure
Date: Fri, 16 Nov 2007 11:01:19 -0500
Message-ID: <20071116160119.GH31958@fieldses.org>
References: <1195163823-24609-1-git-send-email-bfields@citi.umich.edu>
	<1195163823-24609-2-git-send-email-bfields@citi.umich.edu>
	<1195163823-24609-3-git-send-email-bfields@citi.umich.edu>
	<1195163823-24609-4-git-send-email-bfields@citi.umich.edu>
	<473DA941.4050101@redhat.com> <20071116153036.GG31958@fieldses.org>
	<473DB987.7090807@redhat.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Cc: Neil Brown <neilb@suse.de>, linux-nfs@vger.kernel.org,
	nfs@lists.sourceforge.net
To: Peter Staubach <staubach@redhat.com>
In-Reply-To: <473DB987.7090807@redhat.com>
Sender: nfs-bounces@lists.sourceforge.net
Errors-To: nfs-bounces@lists.sourceforge.net

On Fri, Nov 16, 2007 at 10:38:47AM -0500, Peter Staubach wrote:
> J. Bruce Fields wrote:
>> On Fri, Nov 16, 2007 at 09:29:21AM -0500, Peter Staubach wrote:
>>   
>>> J. Bruce Fields wrote:
>>>     
>>>> +	printk(KERN_ERR "nfsd: failed to allocate reply cache\n");
>>>>         
>>> I was thinking that it might be nice to have something which
>>> better explains what the ramifications of this failure might
>>> be.  This explains _precisely_ what happened, but not what
>>> will happen in the future.
>>>     
>>
>> The module will fail to load (or, I suppose, the kernel will fail to
>> boot?).  So the failure will be pretty obvious.
>>
>>   
>
> The module will fail to load, but I suspect that that won't cause
> the system to fail to boot.

OK, but I was thinking of the case where nfsd was built in.

> The only way to notice that the module
> didn't load is to run lsmod or some such and to look for the module.

Typical distro init scripts probably emit a pretty loud warning in this
case, don't they?

> The admin may notice that the NFS server fails to start, but I
> think that it would be nice to better connect this memory allocation
> failure to the NFS server not running.

Well, send in a patch if you'd like, but it should probably add a
printk() to the end of init_nfsd() rather than fooling with the message
here, so it can catch failures that occur elsewhere in the
initialization process.

--b.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs
_______________________________________________
Please note that nfs@lists.sourceforge.net is being discontinued.
Please subscribe to linux-nfs@vger.kernel.org instead.
    http://vger.kernel.org/vger-lists.html#linux-nfs