Re: [PATCH] nfsd: mark CONFIG_NFSD_FAULT_INJECTION as deprecated

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 06, 2014 at 03:02:51PM -0400, Jeff Layton wrote:
> The fault injection code is shaky at best. If you have a lot of stateful
> objects, then you can end up overflowing the client's refcount. The code
> is not widely used and is starting to become a maintenance burden mark
> it DEPRECATED and document that we'll remove it in v3.19.

I doubt anyone will notice a config text change on upgrade.

If we think a deprecation warning's necessary then it would be more
useful to put it in a printk() that fires the first time somebody uses
one of these.

--b.

> 
> Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxxxxxxx>
> ---
>  fs/nfsd/Kconfig | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/fs/nfsd/Kconfig b/fs/nfsd/Kconfig
> index f994e750e0d1..aad734c0b48a 100644
> --- a/fs/nfsd/Kconfig
> +++ b/fs/nfsd/Kconfig
> @@ -98,11 +98,14 @@ config NFSD_V4_SECURITY_LABEL
>  	For now we recommend "Y" only for developers and testers.
>  
>  config NFSD_FAULT_INJECTION
> -	bool "NFS server manual fault injection"
> +	bool "NFS server manual fault injection (DEPRECATED)"
>  	depends on NFSD_V4 && DEBUG_KERNEL
>  	help
>  	  This option enables support for manually injecting faults
>  	  into the NFS server.  This is intended to be used for
>  	  testing error recovery on the NFS client.
>  
> +	  This feature should not be enabled on production systems
> +	  and will be removed in v3.19.
> +
>  	  If unsure, say N.
> -- 
> 1.9.3
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux