Commit cfcc048c authored by YueHaibing's avatar YueHaibing Committed by Jason Gunthorpe

IB/hfi1: Remove set but not used variables 'offset' and 'fspsn'

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/infiniband/hw/hfi1/tid_rdma.c: In function tid_rdma_rcv_error:
drivers/infiniband/hw/hfi1/tid_rdma.c:2029:7: warning: variable offset set but not used [-Wunused-but-set-variable]
drivers/infiniband/hw/hfi1/tid_rdma.c: In function hfi1_rc_rcv_tid_rdma_ack:
drivers/infiniband/hw/hfi1/tid_rdma.c:4555:35: warning: variable fspsn set but not used [-Wunused-but-set-variable]

'offset' is never used since introduction in
commit d0d564a1 ("IB/hfi1: Add functions to receive TID RDMA READ request")

'fspsn' is never used since introduciotn in
commit 9e93e967 ("IB/hfi1: Add a function to receive TID RDMA ACK packet")
Signed-off-by: default avatarYueHaibing <yuehaibing@huawei.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent 2a3d923f
...@@ -2026,7 +2026,6 @@ static int tid_rdma_rcv_error(struct hfi1_packet *packet, ...@@ -2026,7 +2026,6 @@ static int tid_rdma_rcv_error(struct hfi1_packet *packet,
trace_hfi1_tid_req_rcv_err(qp, 0, e->opcode, e->psn, e->lpsn, req); trace_hfi1_tid_req_rcv_err(qp, 0, e->opcode, e->psn, e->lpsn, req);
if (e->opcode == TID_OP(READ_REQ)) { if (e->opcode == TID_OP(READ_REQ)) {
struct ib_reth *reth; struct ib_reth *reth;
u32 offset;
u32 len; u32 len;
u32 rkey; u32 rkey;
u64 vaddr; u64 vaddr;
...@@ -2038,7 +2037,6 @@ static int tid_rdma_rcv_error(struct hfi1_packet *packet, ...@@ -2038,7 +2037,6 @@ static int tid_rdma_rcv_error(struct hfi1_packet *packet,
* The requester always restarts from the start of the original * The requester always restarts from the start of the original
* request. * request.
*/ */
offset = delta_psn(psn, e->psn) * qp->pmtu;
len = be32_to_cpu(reth->length); len = be32_to_cpu(reth->length);
if (psn != e->psn || len != req->total_len) if (psn != e->psn || len != req->total_len)
goto unlock; goto unlock;
...@@ -4552,7 +4550,7 @@ void hfi1_rc_rcv_tid_rdma_ack(struct hfi1_packet *packet) ...@@ -4552,7 +4550,7 @@ void hfi1_rc_rcv_tid_rdma_ack(struct hfi1_packet *packet)
struct rvt_swqe *wqe; struct rvt_swqe *wqe;
struct tid_rdma_request *req; struct tid_rdma_request *req;
struct tid_rdma_flow *flow; struct tid_rdma_flow *flow;
u32 aeth, psn, req_psn, ack_psn, fspsn, resync_psn, ack_kpsn; u32 aeth, psn, req_psn, ack_psn, resync_psn, ack_kpsn;
unsigned long flags; unsigned long flags;
u16 fidx; u16 fidx;
...@@ -4756,7 +4754,6 @@ void hfi1_rc_rcv_tid_rdma_ack(struct hfi1_packet *packet) ...@@ -4756,7 +4754,6 @@ void hfi1_rc_rcv_tid_rdma_ack(struct hfi1_packet *packet)
IB_AETH_CREDIT_MASK) { IB_AETH_CREDIT_MASK) {
case 0: /* PSN sequence error */ case 0: /* PSN sequence error */
flow = &req->flows[req->acked_tail]; flow = &req->flows[req->acked_tail];
fspsn = full_flow_psn(flow, flow->flow_state.spsn);
trace_hfi1_tid_flow_rcv_tid_ack(qp, req->acked_tail, trace_hfi1_tid_flow_rcv_tid_ack(qp, req->acked_tail,
flow); flow);
req->r_ack_psn = mask_psn(be32_to_cpu(ohdr->bth[2])); req->r_ack_psn = mask_psn(be32_to_cpu(ohdr->bth[2]));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment