From: sashiko-bot@kernel.org
To: "Abhijit Gangurde" <abhijit.gangurde@amd.com>
Cc: linux-rdma@vger.kernel.org
Subject: Re: [for-next 3/4] RDMA/ionic: allocate a PD-scoped DMA MR
Date: Sun, 08 Nov 2026 15:04:14 +0000 [thread overview]
Message-ID: <sashiko-outbox-203825@kernel.org> (raw)
In-Reply-To: <20261107150249.1576835-4-abhijit.gangurde@amd.com>
Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [High] RDMA/ionic: build failure in ionic_fill_lif_cfg() due to
missing rdma.cap
--
commit 71ee46b9e5e60e185b9553d17b2722a569d100ad
Author: Gregor Dick <gregor.dick@amd.com>
RDMA/ionic: allocate a PD-scoped DMA MR
This commit allocates a PD-scoped DMA MR via CREATE_MR with
IONIC_MRF_DMA_MR when the firmware advertises IONIC_GEN_CAP_DMA_MR.
[ ... ]
> diff --git a/drivers/infiniband/hw/ionic/ionic_lif_cfg.c b/drivers/infiniband/hw/ionic/ionic_lif_cfg.c
> index 66cee57442e7a..dcf54856603d4 100644
> --- a/drivers/infiniband/hw/ionic/ionic_lif_cfg.c
> +++ b/drivers/infiniband/hw/ionic/ionic_lif_cfg.c
> @@ -95,6 +95,7 @@ void ionic_fill_lif_cfg(struct ionic_lif *lif, struct ionic_lif_cfg *cfg)
> cfg->rq_expdb =
> !!(lif->qtype_info[IONIC_QTYPE_RXQ].features & IONIC_QIDENT_F_EXPDB);
> cfg->alloc_qid_cap = ident->rdma.alloc_qid_cap;
> + cfg->cap = le64_to_cpu(ident->rdma.cap);
> }
[Severity: High]
Does this break the build when CONFIG_INFINIBAND_IONIC is enabled?
The rdma struct inside union ionic_lif_identity in
drivers/net/ethernet/pensando/ionic/ionic_if.h has no cap member.
--
Sashiko AI review · https://sashiko.dev/#/patchset/20261107150249.1576835-4-abhijit.gangurde@amd.com?part=1
next prev parent reply other threads:[~2026-11-08 15:04 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-07 15:02 [for-next 0/4] RDMA/ionic: add DMA MR and v2 atomic WQE Abhijit Gangurde
2026-11-07 15:02 ` [for-next 1/4] net: ionic: add an RDMA capability word to the LIF identity Abhijit Gangurde
2026-11-07 15:02 ` [for-next 2/4] RDMA/ionic: preserve atomic operands in the send WQE Abhijit Gangurde
2026-11-07 15:02 ` [for-next 3/4] RDMA/ionic: allocate a PD-scoped DMA MR Abhijit Gangurde
2026-11-08 15:04 ` sashiko-bot [this message]
2026-11-07 15:02 ` [for-next 4/4] RDMA/ionic: use the v2 atomic send WQE layout Abhijit Gangurde
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=sashiko-outbox-203825@kernel.org \
--to=sashiko-bot@kernel.org \
--cc=abhijit.gangurde@amd.com \
--cc=linux-rdma@vger.kernel.org \
--cc=sashiko-reviews@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.