15/01/2018 11:52, Guo, Jia: > On 1/15/2018 7:24 AM, Thomas Monjalon wrote: > > 11/01/2018 15:05, Jeff Guo: > >> +enum rte_dev_subsystem { > >> + RTE_DEV_SUBSYSTEM_UIO, > >> + RTE_DEV_SUBSYSTEM_VFIO, > >> + RTE_DEV_SUBSYSTEM_PCI, > >> + RTE_DEV_SUBSYSTEM_MAX > >> +}; > > > > I don't think PCI and UIO/VFIO should be described at the same level. > > Can you re-use the enum rte_kernel_driver? > > rte_kernel_driver might be not qualify for that use, since that is the event > sumsystem, it include pci/uio/vfio, such strings to identify each subsystem. > i will modify it to be rte_dev_event_subsystem.
I don't understand this classification. A device can be both PCI and VFIO.