-----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:

quoted

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"

quoted

 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