RDMA/rxe: Remove the unnecessary variable
authorZhu Yanjun <yanjun.zhu@linux.dev>
Tue, 7 Dec 2021 19:40:57 +0000 (14:40 -0500)
committerJason Gunthorpe <jgg@nvidia.com>
Tue, 7 Dec 2021 17:56:22 +0000 (13:56 -0400)
The variable pkey is assigned from a macro. Then this variable is passed
to a function bth_init directly, and pkey is not used again. So remove it
and use the macro directly.

Fixes: 76251e15ea73 ("RDMA/rxe: Remove pkey table")
Link: https://lore.kernel.org/r/20211207194057.713289-1-yanjun.zhu@linux.dev
Signed-off-by: Zhu Yanjun <yanjun.zhu@linux.dev>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/sw/rxe/rxe_req.c

index c8d674d..5eb8905 100644 (file)
@@ -369,7 +369,6 @@ static struct sk_buff *init_req_packet(struct rxe_qp *qp,
        int                     pad = (-payload) & 0x3;
        int                     paylen;
        int                     solicited;
-       u16                     pkey;
        u32                     qp_num;
        int                     ack_req;
 
@@ -401,8 +400,6 @@ static struct sk_buff *init_req_packet(struct rxe_qp *qp,
                        (pkt->mask & (RXE_WRITE_MASK | RXE_IMMDT_MASK)) ==
                        (RXE_WRITE_MASK | RXE_IMMDT_MASK));
 
-       pkey = IB_DEFAULT_PKEY_FULL;
-
        qp_num = (pkt->mask & RXE_DETH_MASK) ? ibwr->wr.ud.remote_qpn :
                                         qp->attr.dest_qp_num;
 
@@ -411,7 +408,7 @@ static struct sk_buff *init_req_packet(struct rxe_qp *qp,
        if (ack_req)
                qp->req.noack_pkts = 0;
 
-       bth_init(pkt, pkt->opcode, solicited, 0, pad, pkey, qp_num,
+       bth_init(pkt, pkt->opcode, solicited, 0, pad, IB_DEFAULT_PKEY_FULL, qp_num,
                 ack_req, pkt->psn);
 
        /* init optional headers */