net/bnxt: add back dependency to virt kmods

Message ID 20220112134152.3850-1-geoffrey.le_gourrierec@6wind.com (mailing list archive)
State Accepted, archived
Delegated to: Ajit Khaparde
Headers
Series net/bnxt: add back dependency to virt kmods |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

Geoffrey Le Gourriérec Jan. 12, 2022, 1:41 p.m. UTC
  During a large refactoring sweep for 21.11, a previous commit
removed the dependency the bnxt driver had on Linux virtual
bus drivers, such as vfio-pci. This breaks port detection.

This patch adds the kmod dependency back as it was.

Fixes: 295968d17407 ("ethdev: add namespace")
Signed-off-by: Geoffrey Le Gourriérec <geoffrey.le_gourrierec@6wind.com>
---
 drivers/net/bnxt/bnxt_ethdev.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Ajit Khaparde Jan. 14, 2022, 5:46 p.m. UTC | #1
On Wed, Jan 12, 2022 at 5:42 AM Geoffrey Le Gourriérec
<geoffrey.le_gourrierec@6wind.com> wrote:
>
> During a large refactoring sweep for 21.11, a previous commit
> removed the dependency the bnxt driver had on Linux virtual
> bus drivers, such as vfio-pci. This breaks port detection.
>
> This patch adds the kmod dependency back as it was.
>
> Fixes: 295968d17407 ("ethdev: add namespace")
> Signed-off-by: Geoffrey Le Gourriérec <geoffrey.le_gourrierec@6wind.com>

Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
Thanks. Patch applied to dpdk-next-net-brcm.

>
> ---
>  drivers/net/bnxt/bnxt_ethdev.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
> index f79f33ab4e17..3ca6b7be7b6a 100644
> --- a/drivers/net/bnxt/bnxt_ethdev.c
> +++ b/drivers/net/bnxt/bnxt_ethdev.c
> @@ -6302,4 +6302,5 @@ bool is_bnxt_supported(struct rte_eth_dev *dev)
>  RTE_LOG_REGISTER_SUFFIX(bnxt_logtype_driver, driver, NOTICE);
>  RTE_PMD_REGISTER_PCI(net_bnxt, bnxt_rte_pmd);
>  RTE_PMD_REGISTER_PCI_TABLE(net_bnxt, bnxt_pci_id_map);
> +RTE_PMD_REGISTER_KMOD_DEP(net_bnxt, "* igb_uio | uio_pci_generic | vfio-pci");
>
> --
> 2.30.2
>
  

Patch

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index f79f33ab4e17..3ca6b7be7b6a 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -6302,4 +6302,5 @@  bool is_bnxt_supported(struct rte_eth_dev *dev)
 RTE_LOG_REGISTER_SUFFIX(bnxt_logtype_driver, driver, NOTICE);
 RTE_PMD_REGISTER_PCI(net_bnxt, bnxt_rte_pmd);
 RTE_PMD_REGISTER_PCI_TABLE(net_bnxt, bnxt_pci_id_map);
+RTE_PMD_REGISTER_KMOD_DEP(net_bnxt, "* igb_uio | uio_pci_generic | vfio-pci");