Re: [TRIVIAL][PATCH]:/proc/fs/nfsd/

[Raj]

Neil Brown wrote:

> On Friday March 5, obelix123@xxxxxxxxxxxx wrote:
>  
>
> > Hi,
> >
> > Kernel Version: 2.6.3
> > Even if NFSD is not selected, the proc entry /proc/fs/nfsd is getting 
> > created.
> >    
>
> Is it a problem??
>  
Theoritically no, but behavior wise , yes.

>  
>
> > The following patch fixes it.
> >    
>
> Does it need fixing??
>
> If you remove this, then people who compile a kernel without nfsd
> support, and then later decide to compile an nfsd module and load it,
> will not be able to mount the nfsd filesystem at the right place.
>  

I guess choosing nfsd either builtin or as a module will cause a rebuild 
of some components of the main kernel and hence a
reboot is anyway need. Pls correct me if i am wrong.

>  
>
> > Pls apply.
> >
> > /Raj
> > --- linux-2.6.3/fs/proc/root.c	2004-02-19 09:52:32.000000000 +0530
> > +++ linux-2.6.3-fixed/fs/proc/root.c	2004-03-05 13:48:28.448516568 +0530
> > @@ -65,7 +65,11 @@ void __init proc_root_init(void)
> > #endif
> > 	proc_root_fs = proc_mkdir("fs", 0);
> > 	proc_root_driver = proc_mkdir("driver", 0);
> > +
> > +#if defined(CONFIG_NFSD) || defined(CONFIG_NFSD_MODULE)
> > 	proc_mkdir("fs/nfsd", 0); /* somewhere for the nfsd filesystem to be mounted */
> > +#endif
> > +
> > #if defined(CONFIG_SUN_OPENPROMFS) || defined(CONFIG_SUN_OPENPROMFS_MODULE)
> > 	/* just give it a mountpoint */
> > 	proc_mkdir("openprom", 0);
> >    
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
>  


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/