Skip to content

Commit 52e3b42

Browse files
oulijunjgunthorpe
authored andcommitted
RDMA/hns: Filter for zero length of sge in hip08 kernel mode
When the length of sge is zero, the driver need to filter it Signed-off-by: Lijun Ou <[email protected]> Signed-off-by: Jason Gunthorpe <[email protected]>
1 parent ace1c54 commit 52e3b42

File tree

1 file changed

+20
-9
lines changed

1 file changed

+20
-9
lines changed

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

Lines changed: 20 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -230,26 +230,37 @@ static int hns_roce_v2_post_send(struct ib_qp *ibqp, struct ib_send_wr *wr,
230230
V2_RC_SEND_WQE_BYTE_4_INLINE_S, 1);
231231
} else {
232232
if (wr->num_sge <= 2) {
233-
for (i = 0; i < wr->num_sge; i++)
234-
set_data_seg_v2(dseg + i,
235-
wr->sg_list + i);
233+
for (i = 0; i < wr->num_sge; i++) {
234+
if (likely(wr->sg_list[i].length)) {
235+
set_data_seg_v2(dseg,
236+
wr->sg_list + i);
237+
dseg++;
238+
}
239+
}
236240
} else {
237241
roce_set_field(rc_sq_wqe->byte_20,
238242
V2_RC_SEND_WQE_BYTE_20_MSG_START_SGE_IDX_M,
239243
V2_RC_SEND_WQE_BYTE_20_MSG_START_SGE_IDX_S,
240244
sge_ind & (qp->sge.sge_cnt - 1));
241245

242-
for (i = 0; i < 2; i++)
243-
set_data_seg_v2(dseg + i,
244-
wr->sg_list + i);
246+
for (i = 0; i < 2; i++) {
247+
if (likely(wr->sg_list[i].length)) {
248+
set_data_seg_v2(dseg,
249+
wr->sg_list + i);
250+
dseg++;
251+
}
252+
}
245253

246254
dseg = get_send_extend_sge(qp,
247255
sge_ind & (qp->sge.sge_cnt - 1));
248256

249257
for (i = 0; i < wr->num_sge - 2; i++) {
250-
set_data_seg_v2(dseg + i,
251-
wr->sg_list + 2 + i);
252-
sge_ind++;
258+
if (likely(wr->sg_list[i + 2].length)) {
259+
set_data_seg_v2(dseg,
260+
wr->sg_list + 2 + i);
261+
dseg++;
262+
sge_ind++;
263+
}
253264
}
254265
}
255266

0 commit comments

Comments
 (0)