From: Jack Wang <jinpu.w...@cloud.ionos.com>

[ Upstream commit 6f5d1b3016d650f351e65c645a5eee5394547dd0 ]

Fix up wr_avail accounting. if wr_cnt is 0, then we do SIGNAL for first
wr, in completion we add queue_depth back, which is not right in the
sense of tracking for available wr.

So fix it by init wr_cnt to 1.

Fixes: 9cb837480424 ("RDMA/rtrs: server: main functionality")
Link: 
https://lore.kernel.org/r/20201217141915.56989-19-jinpu.w...@cloud.ionos.com
Signed-off-by: Jack Wang <jinpu.w...@cloud.ionos.com>
Signed-off-by: Gioh Kim <gi-oh....@cloud.ionos.com>
Signed-off-by: Jason Gunthorpe <j...@nvidia.com>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 drivers/infiniband/ulp/rtrs/rtrs-srv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/infiniband/ulp/rtrs/rtrs-srv.c 
b/drivers/infiniband/ulp/rtrs/rtrs-srv.c
index aac710764b44f..f3f4b640b0970 100644
--- a/drivers/infiniband/ulp/rtrs/rtrs-srv.c
+++ b/drivers/infiniband/ulp/rtrs/rtrs-srv.c
@@ -1620,7 +1620,7 @@ static int create_con(struct rtrs_srv_sess *sess,
        con->c.cm_id = cm_id;
        con->c.sess = &sess->s;
        con->c.cid = cid;
-       atomic_set(&con->wr_cnt, 0);
+       atomic_set(&con->wr_cnt, 1);
 
        if (con->c.cid == 0) {
                /*
-- 
2.27.0



Reply via email to