Commit ffc59e32 authored by Leon Romanovsky's avatar Leon Romanovsky Committed by Jason Gunthorpe
Browse files

RDMA/bnxt_re: Remove unusable nq variable 

Remove nq variable from bnxt_re_create_srq() and bnxt_re_destroy_srq()
as it generates the following compilation warnings:

>> drivers/infiniband/hw/bnxt_re/ib_verbs.c:1777:24: warning: variable
'nq' set but not used [-Wunused-but-set-variable]
    1777 |         struct bnxt_qplib_nq *nq = NULL;
         |                               ^
   drivers/infiniband/hw/bnxt_re/ib_verbs.c:1828:24: warning: variable
'nq' set but not used [-Wunused-but-set-variable]
    1828 |         struct bnxt_qplib_nq *nq = NULL;
         |                               ^
   2 warnings generated.

Fixes: 6b395d31 ("RDMA/bnxt_re: Fix budget handling of notification queue")
Link: https://patch.msgid.link/r/8a4343e217d7d1c0a5a786b785c4ac57cb72a2a0.1744288299.git.leonro@nvidia.com


Reported-by: default avatarkernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202504091055.CzgXnk4C-lkp@intel.com/


Signed-off-by: default avatarLeon Romanovsky <leonro@nvidia.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 9a0e6f15

drivers/infiniband/hw/bnxt_re/ib_verbs.c

+0 −5
Original line number Diff line number Diff line
@@ -1774,10 +1774,7 @@ int bnxt_re_destroy_srq(struct ib_srq *ib_srq, struct ib_udata *udata) 
					       ib_srq);

	struct bnxt_re_dev *rdev = srq->rdev;

	struct bnxt_qplib_srq *qplib_srq = &srq->qplib_srq;

	struct bnxt_qplib_nq *nq = NULL;



	if (qplib_srq->cq)

		nq = qplib_srq->cq->nq;

	if (rdev->chip_ctx->modes.toggle_bits & BNXT_QPLIB_SRQ_TOGGLE_BIT) {

		free_page((unsigned long)srq->uctx_srq_page);

		hash_del(&srq->hash_entry);
@@ -1825,7 +1822,6 @@ int bnxt_re_create_srq(struct ib_srq *ib_srq, 
		       struct ib_udata *udata)

{

	struct bnxt_qplib_dev_attr *dev_attr;

	struct bnxt_qplib_nq *nq = NULL;

	struct bnxt_re_ucontext *uctx;

	struct bnxt_re_dev *rdev;

	struct bnxt_re_srq *srq;
@@ -1871,7 +1867,6 @@ int bnxt_re_create_srq(struct ib_srq *ib_srq, 
	srq->qplib_srq.eventq_hw_ring_id = rdev->nqr->nq[0].ring_id;

	srq->qplib_srq.sg_info.pgsize = PAGE_SIZE;

	srq->qplib_srq.sg_info.pgshft = PAGE_SHIFT;

	nq = &rdev->nqr->nq[0];



	if (udata) {

		rc = bnxt_re_init_user_srq(rdev, pd, srq, udata);