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

RDMA/cxgb3: Don't expose DMA addresses 



DMA addresses like all other kernel addresses should be printed with
special %p* formatter. It is needed to allow control of exposure of such
information through a dedicated knob.

Signed-off-by: default avatarLeon Romanovsky <leonro@mellanox.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent d34d37d5

drivers/infiniband/hw/cxgb3/cxio_hal.c

+3 −3
Original line number Diff line number Diff line
@@ -565,9 +565,9 @@ static int cxio_hal_init_ctrl_qp(struct cxio_rdev *rdev_p) 
	wqe->sge_cmd = cpu_to_be64(sge_cmd);

	wqe->ctx1 = cpu_to_be64(ctx1);

	wqe->ctx0 = cpu_to_be64(ctx0);

	pr_debug("CtrlQP dma_addr 0x%llx workq %p size %d\n",

		 (unsigned long long)rdev_p->ctrl_qp.dma_addr,

		 rdev_p->ctrl_qp.workq, 1 << T3_CTRL_QP_SIZE_LOG2);

	pr_debug("CtrlQP dma_addr %pad workq %p size %d\n",

		 &rdev_p->ctrl_qp.dma_addr, rdev_p->ctrl_qp.workq,

		 1 << T3_CTRL_QP_SIZE_LOG2);

	skb->priority = CPL_PRIORITY_CONTROL;

	return iwch_cxgb3_ofld_send(rdev_p->t3cdev_p, skb);

err:

drivers/infiniband/hw/cxgb3/iwch_provider.c

+7 −6
Original line number Diff line number Diff line
@@ -205,9 +205,9 @@ static struct ib_cq *iwch_create_cq(struct ib_device *ibdev, 
		}

		insert_mmap(ucontext, mm);

	}

	pr_debug("created cqid 0x%0x chp %p size 0x%0x, dma_addr 0x%0llx\n",

	pr_debug("created cqid 0x%0x chp %p size 0x%0x, dma_addr %pad\n",

		 chp->cq.cqid, chp, (1 << chp->cq.size_log2),

		 (unsigned long long)chp->cq.dma_addr);

		 &chp->cq.dma_addr);

	return &chp->ibcq;

}
@@ -919,10 +919,11 @@ static struct ib_qp *iwch_create_qp(struct ib_pd *pd, 
		insert_mmap(ucontext, mm2);

	}

	qhp->ibqp.qp_num = qhp->wq.qpid;

	pr_debug("%s sq_num_entries %d, rq_num_entries %d qpid 0x%0x qhp %p dma_addr 0x%llx size %d rq_addr 0x%x\n",

	pr_debug(

		"%s sq_num_entries %d, rq_num_entries %d qpid 0x%0x qhp %p dma_addr %pad size %d rq_addr 0x%x\n",

		__func__, qhp->attr.sq_num_entries, qhp->attr.rq_num_entries,

		 qhp->wq.qpid, qhp, (unsigned long long)qhp->wq.dma_addr,

		 1 << qhp->wq.size_log2, qhp->wq.rq_addr);

		qhp->wq.qpid, qhp, &qhp->wq.dma_addr, 1 << qhp->wq.size_log2,

		qhp->wq.rq_addr);

	return &qhp->ibqp;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE