RDMA/rtrs: server: Use already dereferenced rtrs_sess structure
authorMd Haris Iqbal <haris.phnx@gmail.com>
Fri, 22 May 2020 08:28:33 +0000 (08:28 +0000)
committerJason Gunthorpe <jgg@mellanox.com>
Fri, 22 May 2020 18:50:22 +0000 (15:50 -0300)
The rtrs_sess structure has already been extracted above from the
rtrs_srv_sess structure. Use that to avoid redundant dereferencing.

Fixes: 9cb837480424 ("RDMA/rtrs: server: main functionality")
Link: https://lore.kernel.org/r/20200522082833.1480551-1-haris.phnx@gmail.com
Signed-off-by: Md Haris Iqbal <haris.phnx@gmail.com>
Acked-by: Danil Kipnis <danil.kipnis@cloud.ionos.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/ulp/rtrs/rtrs-srv.c

index 1fc6ece..5ef8988 100644 (file)
@@ -1822,13 +1822,13 @@ static int rtrs_rdma_connect(struct rdma_cm_id *cm_id,
                /*
                 * Sanity checks
                 */
-               if (con_num != sess->s.con_num || cid >= sess->s.con_num) {
+               if (con_num != s->con_num || cid >= s->con_num) {
                        rtrs_err(s, "Incorrect request: %d, %d\n",
                                  cid, con_num);
                        mutex_unlock(&srv->paths_mutex);
                        goto reject_w_econnreset;
                }
-               if (sess->s.con[cid]) {
+               if (s->con[cid]) {
                        rtrs_err(s, "Connection already exists: %d\n",
                                  cid);
                        mutex_unlock(&srv->paths_mutex);