Re: [PATCH 1 1/3] SUNRPC: Remove unused svc_rqst::rq_lock field

[Jeff Layton <jlayton@kernel.org>]

On Sun, 2022-10-16 at 11:46 -0400, Chuck Lever wrote:
> Clean up after commit 22700f3c6df5 ("SUNRPC: Improve ordering of
> transport processing").
> 
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
> ---
>  include/linux/sunrpc/svc.h |    1 -
>  net/sunrpc/svc.c           |    1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/include/linux/sunrpc/svc.h b/include/linux/sunrpc/svc.h
> index daecb009c05b..3b59eb9cf884 100644
> --- a/include/linux/sunrpc/svc.h
> +++ b/include/linux/sunrpc/svc.h
> @@ -311,7 +311,6 @@ struct svc_rqst {
>  	struct auth_domain *	rq_gssclient;	/* "gss/"-style peer info */
>  	struct svc_cacherep *	rq_cacherep;	/* cache info */
>  	struct task_struct	*rq_task;	/* service thread */
> -	spinlock_t		rq_lock;	/* per-request lock */
>  	struct net		*rq_bc_net;	/* pointer to backchannel's
>  						 * net namespace
>  						 */
> diff --git a/net/sunrpc/svc.c b/net/sunrpc/svc.c
> index 7c9a0d0b1230..d2bb1d04c524 100644
> --- a/net/sunrpc/svc.c
> +++ b/net/sunrpc/svc.c
> @@ -638,7 +638,6 @@ svc_rqst_alloc(struct svc_serv *serv, struct svc_pool *pool, int node)
>  		return rqstp;
>  
>  	__set_bit(RQ_BUSY, &rqstp->rq_flags);
> -	spin_lock_init(&rqstp->rq_lock);
>  	rqstp->rq_server = serv;
>  	rqstp->rq_pool = pool;
>  
> 
> 

Reviewed-by: Jeff Layton <jlayton@kernel.org>