[net-next] xprtrdma: Remove unused function declaration rpcrdma_bc_post_recv()

Message ID 20230729123152.35132-1-yuehaibing@huawei.com
State New
Headers
Series [net-next] xprtrdma: Remove unused function declaration rpcrdma_bc_post_recv() |

Commit Message

Yue Haibing July 29, 2023, 12:31 p.m. UTC
  rpcrdma_bc_post_recv() is never implemented since introduction in
commit f531a5dbc451 ("xprtrdma: Pre-allocate backward rpc_rqst and send/receive buffers").

Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
---
 net/sunrpc/xprtrdma/xprt_rdma.h | 1 -
 1 file changed, 1 deletion(-)
  

Comments

Chuck Lever July 30, 2023, 4:01 p.m. UTC | #1
On Sat, Jul 29, 2023 at 08:31:52PM +0800, Yue Haibing wrote:
> rpcrdma_bc_post_recv() is never implemented since introduction in
> commit f531a5dbc451 ("xprtrdma: Pre-allocate backward rpc_rqst and send/receive buffers").
> 
> Signed-off-by: Yue Haibing <yuehaibing@huawei.com>

Reviewed-by: Chuck Lever <chuck.lever@oracle.com>

Anna, can you take this one?


> ---
>  net/sunrpc/xprtrdma/xprt_rdma.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/net/sunrpc/xprtrdma/xprt_rdma.h b/net/sunrpc/xprtrdma/xprt_rdma.h
> index 5e5ff6784ef5..da409450dfc0 100644
> --- a/net/sunrpc/xprtrdma/xprt_rdma.h
> +++ b/net/sunrpc/xprtrdma/xprt_rdma.h
> @@ -593,7 +593,6 @@ void xprt_rdma_cleanup(void);
>  int xprt_rdma_bc_setup(struct rpc_xprt *, unsigned int);
>  size_t xprt_rdma_bc_maxpayload(struct rpc_xprt *);
>  unsigned int xprt_rdma_bc_max_slots(struct rpc_xprt *);
> -int rpcrdma_bc_post_recv(struct rpcrdma_xprt *, unsigned int);
>  void rpcrdma_bc_receive_call(struct rpcrdma_xprt *, struct rpcrdma_rep *);
>  int xprt_rdma_bc_send_reply(struct rpc_rqst *rqst);
>  void xprt_rdma_bc_free_rqst(struct rpc_rqst *);
> -- 
> 2.34.1
>
  
Anna Schumaker Aug. 1, 2023, 7:45 p.m. UTC | #2
On Sun, Jul 30, 2023 at 12:01 PM Chuck Lever <chuck.lever@oracle.com> wrote:
>
> On Sat, Jul 29, 2023 at 08:31:52PM +0800, Yue Haibing wrote:
> > rpcrdma_bc_post_recv() is never implemented since introduction in
> > commit f531a5dbc451 ("xprtrdma: Pre-allocate backward rpc_rqst and send/receive buffers").
> >
> > Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
>
> Reviewed-by: Chuck Lever <chuck.lever@oracle.com>
>
> Anna, can you take this one?

Yep! Applying it now so it doesn't get lost!

Anna

>
>
> > ---
> >  net/sunrpc/xprtrdma/xprt_rdma.h | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/net/sunrpc/xprtrdma/xprt_rdma.h b/net/sunrpc/xprtrdma/xprt_rdma.h
> > index 5e5ff6784ef5..da409450dfc0 100644
> > --- a/net/sunrpc/xprtrdma/xprt_rdma.h
> > +++ b/net/sunrpc/xprtrdma/xprt_rdma.h
> > @@ -593,7 +593,6 @@ void xprt_rdma_cleanup(void);
> >  int xprt_rdma_bc_setup(struct rpc_xprt *, unsigned int);
> >  size_t xprt_rdma_bc_maxpayload(struct rpc_xprt *);
> >  unsigned int xprt_rdma_bc_max_slots(struct rpc_xprt *);
> > -int rpcrdma_bc_post_recv(struct rpcrdma_xprt *, unsigned int);
> >  void rpcrdma_bc_receive_call(struct rpcrdma_xprt *, struct rpcrdma_rep *);
> >  int xprt_rdma_bc_send_reply(struct rpc_rqst *rqst);
> >  void xprt_rdma_bc_free_rqst(struct rpc_rqst *);
> > --
> > 2.34.1
> >
>
> --
> Chuck Lever
  

Patch

diff --git a/net/sunrpc/xprtrdma/xprt_rdma.h b/net/sunrpc/xprtrdma/xprt_rdma.h
index 5e5ff6784ef5..da409450dfc0 100644
--- a/net/sunrpc/xprtrdma/xprt_rdma.h
+++ b/net/sunrpc/xprtrdma/xprt_rdma.h
@@ -593,7 +593,6 @@  void xprt_rdma_cleanup(void);
 int xprt_rdma_bc_setup(struct rpc_xprt *, unsigned int);
 size_t xprt_rdma_bc_maxpayload(struct rpc_xprt *);
 unsigned int xprt_rdma_bc_max_slots(struct rpc_xprt *);
-int rpcrdma_bc_post_recv(struct rpcrdma_xprt *, unsigned int);
 void rpcrdma_bc_receive_call(struct rpcrdma_xprt *, struct rpcrdma_rep *);
 int xprt_rdma_bc_send_reply(struct rpc_rqst *rqst);
 void xprt_rdma_bc_free_rqst(struct rpc_rqst *);