Fix the following sparse warning:
net/sunrpc/xprtrdma/transport.c:71:14: warning: symbol 'xprt_rdma_slot_table_entries' was not declared. Should it be static?
Fixes: 86c4ccd9b92b ("xprtrdma: Eliminate struct rpcrdma_create_data_internal")
Signed-off-by: Chen Wandun <redacted>
---
net/sunrpc/xprtrdma/transport.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From: Chuck Lever <hidden> Date: 2020-02-10 16:09:34
Hi-
The Fixes: tag seems incorrect to me.
On Feb 10, 2020, at 2:39 AM, Chen Wandun [off-list ref] wrote:
Fix the following sparse warning:
net/sunrpc/xprtrdma/transport.c:71:14: warning: symbol 'xprt_rdma_slot_table_entries' was not declared. Should it be static?
Fixes: 86c4ccd9b92b ("xprtrdma: Eliminate struct rpcrdma_create_data_internal")
86c4ccd9b92b correctly makes xprt_rdma_slot_table_entries a
global variable. This later commit (in v5.6-rc1)
7581d90109ca ("xprtrdma: Refactor initialization of ep->rep_max_requests")
should have changed xprt_rdma_slot_table_entries back to a static.
I'm not sure what the call is on sparse warnings these days, but
it doesn't seem like this clean up should be backported to stable.
Should Fixes: be removed?