Hi Matej, On 08/30/2016 10:40 AM, Matej Vido wrote: > On 26.08.2016 15:20, Olivier Matz wrote: > >> Add a new macro DRIVER_REGISTER_KMOD_DEP() that allows a driver to >> declare the list of kernel modules required to run properly. >> >> Today, most PCI drivers require uio/vfio. >> >> Signed-off-by: Olivier Matz <olivier.matz at 6wind.com> >> --- > [..] >> >> diff --git a/drivers/net/szedata2/rte_eth_szedata2.c >> b/drivers/net/szedata2/rte_eth_szedata2.c >> index 483d789..409e71f 100644 >> --- a/drivers/net/szedata2/rte_eth_szedata2.c >> +++ b/drivers/net/szedata2/rte_eth_szedata2.c >> @@ -1602,3 +1602,5 @@ static struct rte_driver rte_szedata2_driver = { >> PMD_REGISTER_DRIVER(rte_szedata2_driver, RTE_SZEDATA2_DRIVER_NAME); >> DRIVER_REGISTER_PCI_TABLE(RTE_SZEDATA2_DRIVER_NAME, >> rte_szedata2_pci_id_table); >> +DRIVER_REGISTER_KMOD_DEP(RTE_SZEDATA2_DRIVER_NAME, >> + "uio,igb_uio:uio,uio_pci_generic:vfio,vfio-pci"); > Hi Olivier, > > szedata2 doesn't require uio/vfio modules. Instead the following lines > could be used: > > +DRIVER_REGISTER_KMOD_DEP(RTE_SZEDATA2_DRIVER_NAME, > + "combo6core,combov3,szedata2,szedata2_cv3"); >
ok, I will update it for next revision, thanks ! Olivier