Hi, On 09/19/2016 10:17 AM, Bruce Richardson wrote: > On Fri, Sep 16, 2016 at 04:26:35PM +0200, David Marchand wrote: >> On Thu, Sep 15, 2016 at 4:22 PM, Olivier Matz <olivier.matz at 6wind.com> >> 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> >> >> Thanks Olivier, this looks good to me. >> >> Btw, I remember some issues with binding ixgbevf devices to >> uio_pci_generic, so advertising should be carefully checked for each >> driver. > > +1 here. In general, uio_pci_generic will not bind to VF devices as VF devices > don't provide legacy interrupts, which is required to use uio_pci_generic. > It's > not limited to just ixgbevf
Thanks for commenting, I'll remove uio_pci_generic from *vf drivers in next version. Olivier
