Hi, On 5/5/20 6:16 PM, David Marchand wrote: > 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. > >
IIUC, it means that with Jerin patch, Virtio Legacy devices support will be dropped as they won't be probed anymore? Maxime
