Re: [dpdk-dev] [PATCH 7/8] ethdev: break ethernet driver and pci_driver connection

2017-01-10 Thread Stephen Hemminger
On Tue, 10 Jan 2017 17:11:15 +0100 Jan Blunck wrote: > On Sat, Jan 7, 2017 at 7:17 PM, Stephen Hemminger > wrote: > > There are multiple buses and device types now. Therefore it no longer > > makes sense that PCI driver information is part of the Ethernet driver > > structure. > > The Etherne

Re: [dpdk-dev] [PATCH 7/8] ethdev: break ethernet driver and pci_driver connection

2017-01-10 Thread Ferruh Yigit
On 1/10/2017 1:59 PM, Ferruh Yigit wrote: > On 1/7/2017 6:17 PM, Stephen Hemminger wrote: >> There are multiple buses and device types now. Therefore it no longer >> makes sense that PCI driver information is part of the Ethernet driver >> structure. >> >> This patch removes pci_driver from eth_dri

Re: [dpdk-dev] [PATCH 7/8] ethdev: break ethernet driver and pci_driver connection

2017-01-10 Thread Jan Blunck
On Sat, Jan 7, 2017 at 7:17 PM, Stephen Hemminger wrote: > There are multiple buses and device types now. Therefore it no longer > makes sense that PCI driver information is part of the Ethernet driver > structure. The Ethernet driver itself doesn't over alot of value from an abstraction point of

Re: [dpdk-dev] [PATCH 7/8] ethdev: break ethernet driver and pci_driver connection

2017-01-10 Thread Ferruh Yigit
On 1/7/2017 6:17 PM, Stephen Hemminger wrote: > There are multiple buses and device types now. Therefore it no longer > makes sense that PCI driver information is part of the Ethernet driver > structure. > > This patch removes pci_driver from eth_driver and introduces a > new combined structure fo

[dpdk-dev] [PATCH 7/8] ethdev: break ethernet driver and pci_driver connection

2017-01-07 Thread Stephen Hemminger
There are multiple buses and device types now. Therefore it no longer makes sense that PCI driver information is part of the Ethernet driver structure. This patch removes pci_driver from eth_driver and introduces a new combined structure for use in all existing PMD's. The rationale is that althoug