Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
From:   "NeilBrown" <neilb@suse.de>
To:     "Yue Haibing" <yuehaibing@huawei.com>
Cc:     trond.myklebust@hammerspace.com, anna@kernel.org,
        chuck.lever@oracle.com, jlayton@kernel.org, kolga@netapp.com,
        Dai.Ngo@oracle.com, tom@talpey.com, yuehaibing@huawei.com,
        kuba@kernel.org, linux-nfs@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [PATCH -next] SUNRPC: Remove unused declaration rpc_modcount()
In-reply-to: <20230809141426.41192-1-yuehaibing@huawei.com>
References: <20230809141426.41192-1-yuehaibing@huawei.com>
Date:   Thu, 10 Aug 2023 07:36:44 +1000
Message-id: <169161700457.32308.8657894998370155540@noble.neil.brown.name>
Precedence: bulk

On Thu, 10 Aug 2023, Yue Haibing wrote:
> These declarations are never implemented since the beginning of git history.
> Remove these, then merge the two #ifdef block for simplification.

For the historically minded, this was added in 2.1.79
https://git.kernel.org/pub/scm/linux/kernel/git/history/history.git/diff/net/=
sunrpc/stats.c?id=3Dae04feb38f319f0d389ea9e41d10986dba22b46d

and removed in 2.3.27.
https://git.kernel.org/pub/scm/linux/kernel/git/history/history.git/diff/net/=
sunrpc/stats.c?id=3D53022f15f8c0381a9b55bbe2893a5f9f6abda6f3

Reviewed-by: NeilBrown <neilb@suse.de>

Thanks,
NeilBRown

>=20
> Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
> ---
>  include/linux/sunrpc/stats.h | 23 +++++++----------------
>  1 file changed, 7 insertions(+), 16 deletions(-)
>=20
> diff --git a/include/linux/sunrpc/stats.h b/include/linux/sunrpc/stats.h
> index d94d4f410507..3ce1550d1beb 100644
> --- a/include/linux/sunrpc/stats.h
> +++ b/include/linux/sunrpc/stats.h
> @@ -43,22 +43,6 @@ struct net;
>  #ifdef CONFIG_PROC_FS
>  int			rpc_proc_init(struct net *);
>  void			rpc_proc_exit(struct net *);
> -#else
> -static inline int rpc_proc_init(struct net *net)
> -{
> -	return 0;
> -}
> -
> -static inline void rpc_proc_exit(struct net *net)
> -{
> -}
> -#endif
> -
> -#ifdef MODULE
> -void			rpc_modcount(struct inode *, int);
> -#endif
> -
> -#ifdef CONFIG_PROC_FS
>  struct proc_dir_entry *	rpc_proc_register(struct net *,struct rpc_stat *);
>  void			rpc_proc_unregister(struct net *,const char *);
>  void			rpc_proc_zero(const struct rpc_program *);
> @@ -69,7 +53,14 @@ void			svc_proc_unregister(struct net *, const char *);
>  void			svc_seq_show(struct seq_file *,
>  				     const struct svc_stat *);
>  #else
> +static inline int rpc_proc_init(struct net *net)
> +{
> +	return 0;
> +}
> =20
> +static inline void rpc_proc_exit(struct net *net)
> +{
> +}
>  static inline struct proc_dir_entry *rpc_proc_register(struct net *net, st=
ruct rpc_stat *s) { return NULL; }
>  static inline void rpc_proc_unregister(struct net *net, const char *p) {}
>  static inline void rpc_proc_zero(const struct rpc_program *p) {}
> --=20
> 2.34.1
>=20
>=20