Re: [PATCH] nfsd: fail module init on reply cache init failure

[Peter Staubach <staubach@redhat.com>]

J. Bruce Fields wrote:
> 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.
>
>   

Ahh.  Sorry, didn't think about that.

This seems a little strong, doesn't it?  To cause the system
to fail to boot?

>> 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?
>
>   

Well, they note that the NFS service failed to start, yes.

>> 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.

Yes, I think that you have hit it on the head.  Do we care
whether any particular allocation failed or just that the
initialization failed?

    Thanx...

       ps

-------------------------------------------------------------------------
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