Commit 3fe6d228 authored by Zhu Yanjun's avatar Zhu Yanjun Committed by Jason Gunthorpe
Browse files

RDMA/rxe: Remove the unnecessary variable 

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: 76251e15 ("RDMA/rxe: Remove pkey table")
Link: https://lore.kernel.org/r/20211207194057.713289-1-yanjun.zhu@linux.dev


Signed-off-by: default avatarZhu Yanjun <yanjun.zhu@linux.dev>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 0a0575a1

drivers/infiniband/sw/rxe/rxe_req.c

+1 −4
Original line number Diff line number Diff line
@@ -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 */