* [PATCH] irdma: Add ice and irdma to kernel-boot rules
@ 2021-08-06 17:58 Tatyana Nikolova
2021-08-06 18:28 ` Jason Gunthorpe
0 siblings, 1 reply; 5+ messages in thread
From: Tatyana Nikolova @ 2021-08-06 17:58 UTC (permalink / raw)
To: jgg, dledford, leon; +Cc: linux-rdma, Tatyana Nikolova
Add ice and irdma to kernel-boot rules so that
these devices are recognized as iWARP and RoCE capable.
Otherwise the port mapper service which is only relevant
for iWARP devices may not start automatically after boot.
Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
---
kernel-boot/rdma-description.rules | 2 ++
kernel-boot/rdma-hw-modules.rules | 1 +
2 files changed, 3 insertions(+)
diff --git a/kernel-boot/rdma-description.rules b/kernel-boot/rdma-description.rules
index 48a7ced..f2f7b38 100644
--- a/kernel-boot/rdma-description.rules
+++ b/kernel-boot/rdma-description.rules
@@ -24,11 +24,13 @@ DRIVERS=="hfi1", ENV{ID_RDMA_OPA}="1"
# Hardware that supports iWarp
DRIVERS=="cxgb4", ENV{ID_RDMA_IWARP}="1"
DRIVERS=="i40e", ENV{ID_RDMA_IWARP}="1"
+DRIVERS=="ice", ENV{ID_RDMA_IWARP}="1"
# Hardware that supports RoCE
DRIVERS=="be2net", ENV{ID_RDMA_ROCE}="1"
DRIVERS=="bnxt_en", ENV{ID_RDMA_ROCE}="1"
DRIVERS=="hns", ENV{ID_RDMA_ROCE}="1"
+DRIVERS=="ice", ENV{ID_RDMA_ROCE}="1"
DRIVERS=="mlx4_core", ENV{ID_RDMA_ROCE}="1"
DRIVERS=="mlx5_core", ENV{ID_RDMA_ROCE}="1"
DRIVERS=="qede", ENV{ID_RDMA_ROCE}="1"
diff --git a/kernel-boot/rdma-hw-modules.rules b/kernel-boot/rdma-hw-modules.rules
index 95eaf72..040deb3 100644
--- a/kernel-boot/rdma-hw-modules.rules
+++ b/kernel-boot/rdma-hw-modules.rules
@@ -12,6 +12,7 @@ ENV{ID_NET_DRIVER}=="bnxt_en", RUN{builtin}+="kmod load bnxt_re"
ENV{ID_NET_DRIVER}=="cxgb4", RUN{builtin}+="kmod load iw_cxgb4"
ENV{ID_NET_DRIVER}=="hns", RUN{builtin}+="kmod load hns_roce"
ENV{ID_NET_DRIVER}=="i40e", RUN{builtin}+="kmod load i40iw"
+ENV{ID_NET_DRIVER}=="ice", RUN{builtin}+="kmod load irdma"
ENV{ID_NET_DRIVER}=="mlx4_en", RUN{builtin}+="kmod load mlx4_ib"
ENV{ID_NET_DRIVER}=="mlx5_core", RUN{builtin}+="kmod load mlx5_ib"
ENV{ID_NET_DRIVER}=="qede", RUN{builtin}+="kmod load qedr"
--
1.8.3.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] irdma: Add ice and irdma to kernel-boot rules
2021-08-06 17:58 [PATCH] irdma: Add ice and irdma to kernel-boot rules Tatyana Nikolova
@ 2021-08-06 18:28 ` Jason Gunthorpe
2021-08-12 23:33 ` Nikolova, Tatyana E
0 siblings, 1 reply; 5+ messages in thread
From: Jason Gunthorpe @ 2021-08-06 18:28 UTC (permalink / raw)
To: Tatyana Nikolova; +Cc: dledford, leon, linux-rdma
On Fri, Aug 06, 2021 at 12:58:08PM -0500, Tatyana Nikolova wrote:
> Add ice and irdma to kernel-boot rules so that
> these devices are recognized as iWARP and RoCE capable.
>
> Otherwise the port mapper service which is only relevant
> for iWARP devices may not start automatically after boot.
>
> Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
> kernel-boot/rdma-description.rules | 2 ++
> kernel-boot/rdma-hw-modules.rules | 1 +
> 2 files changed, 3 insertions(+)
>
> diff --git a/kernel-boot/rdma-description.rules b/kernel-boot/rdma-description.rules
> index 48a7ced..f2f7b38 100644
> +++ b/kernel-boot/rdma-description.rules
> @@ -24,11 +24,13 @@ DRIVERS=="hfi1", ENV{ID_RDMA_OPA}="1"
> # Hardware that supports iWarp
> DRIVERS=="cxgb4", ENV{ID_RDMA_IWARP}="1"
> DRIVERS=="i40e", ENV{ID_RDMA_IWARP}="1"
> +DRIVERS=="ice", ENV{ID_RDMA_IWARP}="1"
>
> # Hardware that supports RoCE
> DRIVERS=="be2net", ENV{ID_RDMA_ROCE}="1"
> DRIVERS=="bnxt_en", ENV{ID_RDMA_ROCE}="1"
> DRIVERS=="hns", ENV{ID_RDMA_ROCE}="1"
> +DRIVERS=="ice", ENV{ID_RDMA_ROCE}="1"
> DRIVERS=="mlx4_core", ENV{ID_RDMA_ROCE}="1"
> DRIVERS=="mlx5_core", ENV{ID_RDMA_ROCE}="1"
> DRIVERS=="qede", ENV{ID_RDMA_ROCE}="1"
> diff --git a/kernel-boot/rdma-hw-modules.rules b/kernel-boot/rdma-hw-modules.rules
> index 95eaf72..040deb3 100644
> +++ b/kernel-boot/rdma-hw-modules.rules
> @@ -12,6 +12,7 @@ ENV{ID_NET_DRIVER}=="bnxt_en", RUN{builtin}+="kmod load bnxt_re"
> ENV{ID_NET_DRIVER}=="cxgb4", RUN{builtin}+="kmod load iw_cxgb4"
> ENV{ID_NET_DRIVER}=="hns", RUN{builtin}+="kmod load hns_roce"
> ENV{ID_NET_DRIVER}=="i40e", RUN{builtin}+="kmod load i40iw"
> +ENV{ID_NET_DRIVER}=="ice", RUN{builtin}+="kmod load irdma"
This should not be needed, right? The auxbux stuff triggers
proper module autoloading?
Jason
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] irdma: Add ice and irdma to kernel-boot rules
2021-08-06 18:28 ` Jason Gunthorpe
@ 2021-08-12 23:33 ` Nikolova, Tatyana E
2021-08-18 16:45 ` Jason Gunthorpe
0 siblings, 1 reply; 5+ messages in thread
From: Nikolova, Tatyana E @ 2021-08-12 23:33 UTC (permalink / raw)
To: Jason Gunthorpe; +Cc: dledford, leon, linux-rdma
> -----Original Message-----
> From: Jason Gunthorpe <jgg@nvidia.com>
> Sent: Friday, August 6, 2021 1:29 PM
> To: Nikolova, Tatyana E <tatyana.e.nikolova@intel.com>
> Cc: dledford@redhat.com; leon@kernel.org; linux-rdma@vger.kernel.org
> Subject: Re: [PATCH] irdma: Add ice and irdma to kernel-boot rules
>
> On Fri, Aug 06, 2021 at 12:58:08PM -0500, Tatyana Nikolova wrote:
> > Add ice and irdma to kernel-boot rules so that these devices are
> > recognized as iWARP and RoCE capable.
> >
> > Otherwise the port mapper service which is only relevant for iWARP
> > devices may not start automatically after boot.
> >
> > Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
> > kernel-boot/rdma-description.rules | 2 ++
> > kernel-boot/rdma-hw-modules.rules | 1 +
> > 2 files changed, 3 insertions(+)
> >
> > diff --git a/kernel-boot/rdma-description.rules
> > b/kernel-boot/rdma-description.rules
> > index 48a7ced..f2f7b38 100644
> > +++ b/kernel-boot/rdma-description.rules
> > @@ -24,11 +24,13 @@ DRIVERS=="hfi1", ENV{ID_RDMA_OPA}="1"
> > # Hardware that supports iWarp
> > DRIVERS=="cxgb4", ENV{ID_RDMA_IWARP}="1"
> > DRIVERS=="i40e", ENV{ID_RDMA_IWARP}="1"
> > +DRIVERS=="ice", ENV{ID_RDMA_IWARP}="1"
> >
> > # Hardware that supports RoCE
> > DRIVERS=="be2net", ENV{ID_RDMA_ROCE}="1"
> > DRIVERS=="bnxt_en", ENV{ID_RDMA_ROCE}="1"
> > DRIVERS=="hns", ENV{ID_RDMA_ROCE}="1"
> > +DRIVERS=="ice", ENV{ID_RDMA_ROCE}="1"
> > DRIVERS=="mlx4_core", ENV{ID_RDMA_ROCE}="1"
> > DRIVERS=="mlx5_core", ENV{ID_RDMA_ROCE}="1"
> > DRIVERS=="qede", ENV{ID_RDMA_ROCE}="1"
> > diff --git a/kernel-boot/rdma-hw-modules.rules
> > b/kernel-boot/rdma-hw-modules.rules
> > index 95eaf72..040deb3 100644
> > +++ b/kernel-boot/rdma-hw-modules.rules
> > @@ -12,6 +12,7 @@ ENV{ID_NET_DRIVER}=="bnxt_en",
> RUN{builtin}+="kmod load bnxt_re"
> > ENV{ID_NET_DRIVER}=="cxgb4", RUN{builtin}+="kmod load iw_cxgb4"
> > ENV{ID_NET_DRIVER}=="hns", RUN{builtin}+="kmod load hns_roce"
> > ENV{ID_NET_DRIVER}=="i40e", RUN{builtin}+="kmod load i40iw"
> > +ENV{ID_NET_DRIVER}=="ice", RUN{builtin}+="kmod load irdma"
>
> This should not be needed, right? The auxbux stuff triggers proper module
> autoloading?
Hi Jason,
Our module depends on the auxbus, but we don't know how the auxbus could trigger loading of irdma. Could you please explain?
Thank you,
Tatyana
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] irdma: Add ice and irdma to kernel-boot rules
2021-08-12 23:33 ` Nikolova, Tatyana E
@ 2021-08-18 16:45 ` Jason Gunthorpe
2021-08-20 21:46 ` Nikolova, Tatyana E
0 siblings, 1 reply; 5+ messages in thread
From: Jason Gunthorpe @ 2021-08-18 16:45 UTC (permalink / raw)
To: Nikolova, Tatyana E; +Cc: dledford, leon, linux-rdma
On Thu, Aug 12, 2021 at 11:33:04PM +0000, Nikolova, Tatyana E wrote:
> > > @@ -12,6 +12,7 @@ ENV{ID_NET_DRIVER}=="bnxt_en",
> > RUN{builtin}+="kmod load bnxt_re"
> > > ENV{ID_NET_DRIVER}=="cxgb4", RUN{builtin}+="kmod load iw_cxgb4"
> > > ENV{ID_NET_DRIVER}=="hns", RUN{builtin}+="kmod load hns_roce"
> > > ENV{ID_NET_DRIVER}=="i40e", RUN{builtin}+="kmod load i40iw"
> > > +ENV{ID_NET_DRIVER}=="ice", RUN{builtin}+="kmod load irdma"
> >
> > This should not be needed, right? The auxbux stuff triggers proper module
> > autoloading?
>
> Hi Jason,
>
> Our module depends on the auxbus, but we don't know how the auxbus
> could trigger loading of irdma. Could you please explain?
It should simply happen automatically once the aux device is
created. If it doesn't something is missing in the kernel.
It works the same way as any loading a module for any other struct
device, the aux device exposes a modalias and the userspace matches it
to the modules.alias file and then loads the specified module.
Jason
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] irdma: Add ice and irdma to kernel-boot rules
2021-08-18 16:45 ` Jason Gunthorpe
@ 2021-08-20 21:46 ` Nikolova, Tatyana E
0 siblings, 0 replies; 5+ messages in thread
From: Nikolova, Tatyana E @ 2021-08-20 21:46 UTC (permalink / raw)
To: Jason Gunthorpe; +Cc: dledford, leon, linux-rdma
> -----Original Message-----
> From: Jason Gunthorpe <jgg@nvidia.com>
> Sent: Wednesday, August 18, 2021 11:46 AM
> To: Nikolova, Tatyana E <tatyana.e.nikolova@intel.com>
> Cc: dledford@redhat.com; leon@kernel.org; linux-rdma@vger.kernel.org
> Subject: Re: [PATCH] irdma: Add ice and irdma to kernel-boot rules
>
> On Thu, Aug 12, 2021 at 11:33:04PM +0000, Nikolova, Tatyana E wrote:
> > > > @@ -12,6 +12,7 @@ ENV{ID_NET_DRIVER}=="bnxt_en",
> > > RUN{builtin}+="kmod load bnxt_re"
> > > > ENV{ID_NET_DRIVER}=="cxgb4", RUN{builtin}+="kmod load iw_cxgb4"
> > > > ENV{ID_NET_DRIVER}=="hns", RUN{builtin}+="kmod load hns_roce"
> > > > ENV{ID_NET_DRIVER}=="i40e", RUN{builtin}+="kmod load i40iw"
> > > > +ENV{ID_NET_DRIVER}=="ice", RUN{builtin}+="kmod load irdma"
> > >
> > > This should not be needed, right? The auxbux stuff triggers proper
> > > module autoloading?
> >
> > Hi Jason,
> >
> > Our module depends on the auxbus, but we don't know how the auxbus
> > could trigger loading of irdma. Could you please explain?
>
> It should simply happen automatically once the aux device is created. If it
> doesn't something is missing in the kernel.
>
> It works the same way as any loading a module for any other struct device,
> the aux device exposes a modalias and the userspace matches it to the
> modules.alias file and then loads the specified module.
Thank you for the explanation. The modalias is exposed by the aux device as you described.
Tatyana
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-08-20 21:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-06 17:58 [PATCH] irdma: Add ice and irdma to kernel-boot rules Tatyana Nikolova
2021-08-06 18:28 ` Jason Gunthorpe
2021-08-12 23:33 ` Nikolova, Tatyana E
2021-08-18 16:45 ` Jason Gunthorpe
2021-08-20 21:46 ` Nikolova, Tatyana E
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).