Re: [PATCH] [RESEND] fscache: re-match MODULE_PARM_DESC() calls to module parameters

[Nicolas Schier <nicolas@fjasle.eu>]

[-- Attachment #1: Type: text/plain, Size: 2132 bytes --]

On Mon Sep  6 13:00:41 2021 Nicolas Schier wrote:
> Fix calls of MODULE_PARM_DESC() such that the first argument matches 
> the
> actual module parameter name.  This changes the 'parm' section in the
> output of `modinfo fscache` from:
> 
>     parm: defer_lookup:uint
>     parm: fscache_defer_lookup:Defer cookie lookup to background thread
>     parm: defer_create:uint
>     parm: fscache_defer_create:Defer cookie creation to background thread
>     parm: debug:uint
>     parm: fscache_debug:FS-Cache debugging mask
> 
> into:
> 
>     parm: defer_lookup:Defer cookie lookup to background thread (uint)
>     parm: defer_create:Defer cookie creation to background thread (uint)
>     parm: debug:FS-Cache debugging mask (uint)
> .
> 
> Signed-off-by: Nicolas Schier <nicolas@fjasle.eu>
> ---
>  fs/fscache/main.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> --
> Resend unmodified as list approval for linux-cachefs@r.c timed out.
> 
> diff --git a/fs/fscache/main.c b/fs/fscache/main.c
> index c1e6cc9091aa..ccb06dc0a6e9 100644
> --- a/fs/fscache/main.c
> +++ b/fs/fscache/main.c
> @@ -22,19 +22,19 @@ MODULE_LICENSE("GPL");
>  unsigned fscache_defer_lookup = 1;
>  module_param_named(defer_lookup, fscache_defer_lookup, uint,
>  		   S_IWUSR | S_IRUGO);
> -MODULE_PARM_DESC(fscache_defer_lookup,
> +MODULE_PARM_DESC(defer_lookup,
>  		 "Defer cookie lookup to background thread");
>  
>  unsigned fscache_defer_create = 1;
>  module_param_named(defer_create, fscache_defer_create, uint,
>  		   S_IWUSR | S_IRUGO);
> -MODULE_PARM_DESC(fscache_defer_create,
> +MODULE_PARM_DESC(defer_create,
>  		 "Defer cookie creation to background thread");
>  
>  unsigned fscache_debug;
>  module_param_named(debug, fscache_debug, uint,
>  		   S_IWUSR | S_IRUGO);
> -MODULE_PARM_DESC(fscache_debug,
> +MODULE_PARM_DESC(debug,
>  		 "FS-Cache debugging mask");
>  
>  struct kobject *fscache_root;
> -- 
> 2.30.1

David,  I got no feedback on this here, yet.  Shall I fix something or 
is there anything wrong with the patch?  

Kind regards,
Nicolas

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]