On 1/15/2018 7:29 PM, Thomas Monjalon wrote:
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.
yes , i think that might be a little strange, but what i saw in the uevent message is that , the item of subsystem info from kernel side is pci sometimes, but some time is uio, i don't know if it is good to defferentiy them by "subsystem " or "driver" or other. let me think about it more.

Reply via email to