From: Leon Romanovsky <leon@kernel.org>
To: Abhijit Gangurde <abhijit.gangurde@amd.com>
Cc: brett.creeley@amd.com, davem@davemloft.net, edumazet@google.com,
kuba@kernel.org, pabeni@redhat.com, corbet@lwn.net, jgg@ziepe.ca,
andrew+netdev@lunn.ch, sln@onemain.com, allen.hubbe@amd.com,
nikhil.agarwal@amd.com, linux-rdma@vger.kernel.org,
netdev@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v6 00/14] Introduce AMD Pensando RDMA driver
Date: Thu, 18 Sep 2025 21:07:50 +0300 [thread overview]
Message-ID: <20250918180750.GA135135@unreal> (raw)
In-Reply-To: <20250903061606.4139957-1-abhijit.gangurde@amd.com>
On Wed, Sep 03, 2025 at 11:45:52AM +0530, Abhijit Gangurde wrote:
> This patchset introduces an RDMA driver for the AMD Pensando adapter.
> An AMD Pensando Ethernet device with RDMA capabilities extends its
> functionality through an auxiliary device.
>
> The first 6 patches of the series modify the ionic Ethernet driver
> to support the RDMA driver. The ionic RDMA driver implementation is
> split into the remaining 8 patches.
>
> The user-mode of the driver is being reviewed at:
> https://github.com/linux-rdma/rdma-core/pull/1620
<...>
> Abhijit Gangurde (14):
> net: ionic: Create an auxiliary device for rdma driver
> net: ionic: Update LIF identity with additional RDMA capabilities
> net: ionic: Export the APIs from net driver to support device commands
> net: ionic: Provide RDMA reset support for the RDMA driver
> net: ionic: Provide interrupt allocation support for the RDMA driver
> net: ionic: Provide doorbell and CMB region information
> RDMA: Add IONIC to rdma_driver_id definition
> RDMA/ionic: Register auxiliary module for ionic ethernet adapter
> RDMA/ionic: Create device queues to support admin operations
> RDMA/ionic: Register device ops for control path
> RDMA/ionic: Register device ops for datapath
> RDMA/ionic: Register device ops for miscellaneous functionality
> RDMA/ionic: Implement device stats ops
> RDMA/ionic: Add Makefile/Kconfig to kernel build environment
This series generates CI warnings:
1. In my local CI
➜ kernel git:(rdma-next) yo ci
e81ec02df1e47 (HEAD -> rdma-next) RDMA: Use %pe format specifier for error pointers
In file included from ./include/linux/string.h:382,
from ./include/linux/bitmap.h:13,
from ./include/linux/cpumask.h:12,
from ./arch/x86/include/asm/paravirt.h:21,
from ./arch/x86/include/asm/cpuid/api.h:57,
from ./arch/x86/include/asm/processor.h:19,
from ./arch/x86/include/asm/timex.h:5,
from ./include/linux/timex.h:67,
from ./include/linux/time32.h:13,
from ./include/linux/time.h:60,
from ./include/linux/stat.h:19,
from ./include/linux/module.h:13,
from drivers/infiniband/hw/ionic/ionic_controlpath.c:4:
In function ‘fortify_memcpy_chk’,
inlined from ‘ionic_set_ah_attr.isra’ at drivers/infiniband/hw/ionic/ionic_controlpath.c:609:3:
./include/linux/fortify-string.h:580:25: error: call to ‘__read_overflow2_field’ declared with attribute warning: detected read beyond size of field (2nd parameter); maybe use struct_group()? [-Werror=attribute-warning]
580 | __read_overflow2_field(q_size_field, size);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[6]: *** [scripts/Makefile.build:287: drivers/infiniband/hw/ionic/ionic_controlpath.o] Error 1
make[5]: *** [scripts/Makefile.build:556: drivers/infiniband/hw/ionic] Error 2
make[5]: *** Waiting for unfinished jobs....
make[4]: *** [scripts/Makefile.build:556: drivers/infiniband/hw] Error 2
make[3]: *** [scripts/Makefile.build:556: drivers/infiniband] Error 2
make[2]: *** [scripts/Makefile.build:556: drivers] Error 2
make[1]: *** [/tmp/tmp53nb1nwr/Makefile:2011: .] Error 2
make: *** [Makefile:248: __sub-make] Error 2
2. From kbuild
Unverified Error/Warning (likely false positive, kindly check if interested):
ERROR: modpost: "__xchg_called_with_bad_pointer" [drivers/infiniband/hw/ionic/ionic_rdma.ko] undefined!
Error/Warning ids grouped by kconfigs:
recent_errors
`-- sparc-allmodconfig
`-- ERROR:__xchg_called_with_bad_pointer-drivers-infiniband-hw-ionic-ionic_rdma.ko-undefined
prev parent reply other threads:[~2025-09-18 18:07 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-03 6:15 [PATCH v6 00/14] Introduce AMD Pensando RDMA driver Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 01/14] net: ionic: Create an auxiliary device for rdma driver Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 02/14] net: ionic: Update LIF identity with additional RDMA capabilities Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 03/14] net: ionic: Export the APIs from net driver to support device commands Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 04/14] net: ionic: Provide RDMA reset support for the RDMA driver Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 05/14] net: ionic: Provide interrupt allocation " Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 06/14] net: ionic: Provide doorbell and CMB region information Abhijit Gangurde
2025-09-03 6:15 ` [PATCH v6 07/14] RDMA: Add IONIC to rdma_driver_id definition Abhijit Gangurde
2025-09-03 6:16 ` [PATCH v6 08/14] RDMA/ionic: Register auxiliary module for ionic ethernet adapter Abhijit Gangurde
2025-09-03 6:16 ` [PATCH v6 09/14] RDMA/ionic: Create device queues to support admin operations Abhijit Gangurde
2025-09-03 6:16 ` [PATCH v6 10/14] RDMA/ionic: Register device ops for control path Abhijit Gangurde
2025-09-09 14:07 ` [External] : " ALOK TIWARI
2025-09-10 10:22 ` Leon Romanovsky
2025-09-03 6:16 ` [PATCH v6 11/14] RDMA/ionic: Register device ops for datapath Abhijit Gangurde
2025-09-03 6:16 ` [PATCH v6 12/14] RDMA/ionic: Register device ops for miscellaneous functionality Abhijit Gangurde
2025-09-03 6:16 ` [PATCH v6 13/14] RDMA/ionic: Implement device stats ops Abhijit Gangurde
2025-09-03 6:16 ` [PATCH v6 14/14] RDMA/ionic: Add Makefile/Kconfig to kernel build environment Abhijit Gangurde
2025-09-09 12:08 ` [PATCH v6 00/14] Introduce AMD Pensando RDMA driver Leon Romanovsky
2025-09-18 18:07 ` Leon Romanovsky [this message]
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=20250918180750.GA135135@unreal \
--to=leon@kernel.org \
--cc=abhijit.gangurde@amd.com \
--cc=allen.hubbe@amd.com \
--cc=andrew+netdev@lunn.ch \
--cc=brett.creeley@amd.com \
--cc=corbet@lwn.net \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=jgg@ziepe.ca \
--cc=kuba@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nikhil.agarwal@amd.com \
--cc=pabeni@redhat.com \
--cc=sln@onemain.com \
/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.