* [PATCH rdma-next] RDMA/mana_ib: check cqe length for kernel CQs
@ 2025-10-23 10:03 Konstantin Taranov
2025-10-23 20:48 ` Long Li
0 siblings, 1 reply; 2+ messages in thread
From: Konstantin Taranov @ 2025-10-23 10:03 UTC (permalink / raw)
To: kotaranov, shirazsaleem, longli, jgg, leon; +Cc: linux-rdma, linux-kernel
From: Konstantin Taranov <kotaranov@microsoft.com>
Check queue size during kernel CQ creation to prevent overflow of u32.
Fixes: bec127e45d9f ("RDMA/mana_ib: create kernel-level CQs")
Signed-off-by: Konstantin Taranov <kotaranov@microsoft.com>
---
drivers/infiniband/hw/mana/cq.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/infiniband/hw/mana/cq.c b/drivers/infiniband/hw/mana/cq.c
index 1becc8779..7600412b0 100644
--- a/drivers/infiniband/hw/mana/cq.c
+++ b/drivers/infiniband/hw/mana/cq.c
@@ -56,6 +56,10 @@ int mana_ib_create_cq(struct ib_cq *ibcq, const struct ib_cq_init_attr *attr,
doorbell = mana_ucontext->doorbell;
} else {
is_rnic_cq = true;
+ if (attr->cqe > U32_MAX / COMP_ENTRY_SIZE / 2 + 1) {
+ ibdev_dbg(ibdev, "CQE %d exceeding limit\n", attr->cqe);
+ return -EINVAL;
+ }
buf_size = MANA_PAGE_ALIGN(roundup_pow_of_two(attr->cqe * COMP_ENTRY_SIZE));
cq->cqe = buf_size / COMP_ENTRY_SIZE;
err = mana_ib_create_kernel_queue(mdev, buf_size, GDMA_CQ, &cq->queue);
--
2.43.0
^ permalink raw reply related [flat|nested] 2+ messages in thread* RE: [PATCH rdma-next] RDMA/mana_ib: check cqe length for kernel CQs
2025-10-23 10:03 [PATCH rdma-next] RDMA/mana_ib: check cqe length for kernel CQs Konstantin Taranov
@ 2025-10-23 20:48 ` Long Li
0 siblings, 0 replies; 2+ messages in thread
From: Long Li @ 2025-10-23 20:48 UTC (permalink / raw)
To: Konstantin Taranov, Konstantin Taranov, Shiraz Saleem,
jgg@ziepe.ca, leon@kernel.org
Cc: linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org
> Subject: [PATCH rdma-next] RDMA/mana_ib: check cqe length for kernel CQs
>
> From: Konstantin Taranov <kotaranov@microsoft.com>
>
> Check queue size during kernel CQ creation to prevent overflow of u32.
>
> Fixes: bec127e45d9f ("RDMA/mana_ib: create kernel-level CQs")
> Signed-off-by: Konstantin Taranov <kotaranov@microsoft.com>
Reviewed-by: Long Li <longli@microsoft.com>
> ---
> drivers/infiniband/hw/mana/cq.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/infiniband/hw/mana/cq.c
> b/drivers/infiniband/hw/mana/cq.c index 1becc8779..7600412b0 100644
> --- a/drivers/infiniband/hw/mana/cq.c
> +++ b/drivers/infiniband/hw/mana/cq.c
> @@ -56,6 +56,10 @@ int mana_ib_create_cq(struct ib_cq *ibcq, const struct
> ib_cq_init_attr *attr,
> doorbell = mana_ucontext->doorbell;
> } else {
> is_rnic_cq = true;
> + if (attr->cqe > U32_MAX / COMP_ENTRY_SIZE / 2 + 1) {
> + ibdev_dbg(ibdev, "CQE %d exceeding limit\n", attr-
> >cqe);
> + return -EINVAL;
> + }
> buf_size = MANA_PAGE_ALIGN(roundup_pow_of_two(attr-
> >cqe * COMP_ENTRY_SIZE));
> cq->cqe = buf_size / COMP_ENTRY_SIZE;
> err = mana_ib_create_kernel_queue(mdev, buf_size,
> GDMA_CQ, &cq->queue);
> --
> 2.43.0
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-10-23 20:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-23 10:03 [PATCH rdma-next] RDMA/mana_ib: check cqe length for kernel CQs Konstantin Taranov
2025-10-23 20:48 ` Long Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).