[PATCH V2 01/10] i40iw: Optimize setting fragments

Tatyana Nikolova
2016-12-09T17:54:55Z

From: Mustafa Ismail <mustafa.ismail-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

Small optimizations replace subtract and multiply with add.

Signed-off-by: Mustafa Ismail <mustafa.ismail-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
providers/i40iw/i40iw_uk.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/providers/i40iw/i40iw_uk.c b/providers/i40iw/i40iw_uk.c
index aff3de6..4d353b5 100644
--- a/providers/i40iw/i40iw_uk.c
+++ b/providers/i40iw/i40iw_uk.c
@@ -291,9 +291,9 @@ static enum i40iw_status_code i40iw_rdma_write(struct i40iw_qp_uk *qp,

i40iw_set_fragment(wqe, I40IW_BYTE_0, op_info->lo_sg_list);

- for (i = 1; i < op_info->num_lo_sges; i++) {
- byte_off = I40IW_BYTE_32 + (i - 1) * 16;
+ for (i = 1, byte_off = I40IW_BYTE_32; i < op_info->num_lo_sges; i++) {
i40iw_set_fragment(wqe, byte_off, &op_info->lo_sg_list[i]);
+ byte_off += 16;
}

i40iw_wmb(); /* make sure WQE is populated before valid bit is set */
@@ -404,9 +404,9 @@ static enum i40iw_status_code i40iw_send(struct i40iw_qp_uk *qp,

i40iw_set_fragment(wqe, I40IW_BYTE_0, op_info->sg_list);

- for (i = 1; i < op_info->num_sges; i++) {
- byte_off = I40IW_BYTE_32 + (i - 1) * 16;
+ for (i = 1, byte_off = I40IW_BYTE_32; i < op_info->num_sges; i++) {
i40iw_set_fragment(wqe, byte_off, &op_info->sg_list[i]);
+ byte_off += 16;
}

i40iw_wmb(); /* make sure WQE is populated before valid bit is set */
@@ -692,9 +692,9 @@ static enum i40iw_status_code i40iw_post_receive(struct i40iw_qp_uk *qp,

i40iw_set_fragment(wqe, I40IW_BYTE_0, info->sg_list);

- for (i = 1; i < info->num_sges; i++) {
- byte_off = I40IW_BYTE_32 + (i - 1) * 16;
+ for (i = 1, byte_off = I40IW_BYTE_32; i < info->num_sges; i++) {
i40iw_set_fragment(wqe, byte_off, &info->sg_list[i]);
+ byte_off += 16;
}

i40iw_wmb(); /* make sure WQE is populated before valid bit is set */
--
1.8.5.2