On Tue, May 5, 2020 at 5:50 PM Jerin Jacob <jerinjac...@gmail.com> wrote: > > > > > > Removing this special case could break x86 applications running with > > > legacy virtio. > > > > > > > > > On the plus side, we have been announcing for some time in virtio: > > > RTE_PMD_REGISTER_KMOD_DEP(net_virtio, "* igb_uio | uio_pci_generic | > > > vfio-pci"); > > > > What is to conclude? > > # The In-tree virtio driver uses ""* igb_uio | uio_pci_generic | > > vfio-pci"" driver as backend and it does not need RTE_KDRV_NONE? > > OR > > # The in-tree, legacy virtio(const struct virtio_pci_ops legacy_op) > > can work without any kernel driver in the backend. So RTE_KDRV_NONE > > need? > > Ping. What is the conclusion? If it is former then this patch is valid.
I am fine with dropping the legacy part, but I wanted to hear from Maxime at least. -- David Marchand