diff mbox series

[rdma-next] irdma: Fix unused variable total_size warning

Message ID 20210707211455.2076-1-tatyana.e.nikolova@intel.com (mailing list archive)
State Accepted
Delegated to: Jason Gunthorpe
Headers show
Series [rdma-next] irdma: Fix unused variable total_size warning | expand

Commit Message

Nikolova, Tatyana E July 7, 2021, 9:14 p.m. UTC
Fix the following unused variable warning:
>> drivers/infiniband/hw/irdma/uk.c:934:6: warning: variable 'total_size'
set but not used [-Wunused-but-set-variable]
           u32 total_size = 0, wqe_idx, i, byte_off;

Link: https://lkml.org/lkml/2021/7/1/726
Reported-by: kernel test robot <lkp@intel.com>
Fixes: 551c46edc769 ("RDMA/irdma: Add user/kernel shared libraries")
Signed-off-by: Mustafa Ismail <mustafa.ismail@intel.com>
Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
---
 drivers/infiniband/hw/irdma/uk.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Leon Romanovsky July 8, 2021, 2:30 p.m. UTC | #1
On Wed, Jul 07, 2021 at 02:14:55PM -0700, Tatyana Nikolova wrote:
> Fix the following unused variable warning:
> >> drivers/infiniband/hw/irdma/uk.c:934:6: warning: variable 'total_size'
> set but not used [-Wunused-but-set-variable]
>            u32 total_size = 0, wqe_idx, i, byte_off;
> 
> Link: https://lkml.org/lkml/2021/7/1/726
> Reported-by: kernel test robot <lkp@intel.com>
> Fixes: 551c46edc769 ("RDMA/irdma: Add user/kernel shared libraries")
> Signed-off-by: Mustafa Ismail <mustafa.ismail@intel.com>
> Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
> ---
>  drivers/infiniband/hw/irdma/uk.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 

Thanks,
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
Jason Gunthorpe July 12, 2021, 6:11 p.m. UTC | #2
On Wed, Jul 07, 2021 at 02:14:55PM -0700, Tatyana Nikolova wrote:
> Fix the following unused variable warning:
> >> drivers/infiniband/hw/irdma/uk.c:934:6: warning: variable 'total_size'
> set but not used [-Wunused-but-set-variable]
>            u32 total_size = 0, wqe_idx, i, byte_off;
> 
> Link: https://lkml.org/lkml/2021/7/1/726
> Reported-by: kernel test robot <lkp@intel.com>
> Fixes: 551c46edc769 ("RDMA/irdma: Add user/kernel shared libraries")
> Signed-off-by: Mustafa Ismail <mustafa.ismail@intel.com>
> Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
> Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
> ---
>  drivers/infiniband/hw/irdma/uk.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)

Applied to for-rc, thanks

Jason
diff mbox series

Patch

diff --git a/drivers/infiniband/hw/irdma/uk.c b/drivers/infiniband/hw/irdma/uk.c
index a6d52c20091c..5fb92de1f015 100644
--- a/drivers/infiniband/hw/irdma/uk.c
+++ b/drivers/infiniband/hw/irdma/uk.c
@@ -931,7 +931,7 @@  enum irdma_status_code irdma_uk_mw_bind(struct irdma_qp_uk *qp,
 enum irdma_status_code irdma_uk_post_receive(struct irdma_qp_uk *qp,
 					     struct irdma_post_rq_info *info)
 {
-	u32 total_size = 0, wqe_idx, i, byte_off;
+	u32 wqe_idx, i, byte_off;
 	u32 addl_frag_cnt;
 	__le64 *wqe;
 	u64 hdr;
@@ -939,9 +939,6 @@  enum irdma_status_code irdma_uk_post_receive(struct irdma_qp_uk *qp,
 	if (qp->max_rq_frag_cnt < info->num_sges)
 		return IRDMA_ERR_INVALID_FRAG_COUNT;
 
-	for (i = 0; i < info->num_sges; i++)
-		total_size += info->sg_list[i].len;
-
 	wqe = irdma_qp_get_next_recv_wqe(qp, &wqe_idx);
 	if (!wqe)
 		return IRDMA_ERR_QP_TOOMANY_WRS_POSTED;