[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread Jan Viktorin
On Wed, 10 Feb 2016 12:38:20 +0100 David Marchand wrote: > On Wed, Feb 10, 2016 at 11:20 AM, Jan Viktorin > wrote: > > On Wed, 10 Feb 2016 10:27:14 +0100 > > David Marchand wrote: > >> #define RTE_EAL_PCI_REGISTER(name, d)\ > >> void pciinitfn_ ##name(void);\ > >> void

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 11:20 AM, Jan Viktorin wrote: > On Wed, 10 Feb 2016 10:27:14 +0100 > David Marchand wrote: >> #define RTE_EAL_PCI_REGISTER(name, d)\ >> void pciinitfn_ ##name(void);\ >> void __attribute__((constructor, used)) pciinitfn_ ##name(void)\ >> {\ >>

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread Jan Viktorin
On Wed, 10 Feb 2016 10:27:14 +0100 David Marchand wrote: > On Wed, Feb 10, 2016 at 9:51 AM, David Marchand > wrote: > > On Tue, Feb 9, 2016 at 6:05 PM, Jan Viktorin > > wrote: > >> What about introducing a macro for this? > >> > >> RTE_REGISTER_PCI_DRIVER(rte_qad_pmd); > > > > Yes. > >

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread David Marchand
On Wed, Feb 10, 2016 at 9:51 AM, David Marchand wrote: > On Tue, Feb 9, 2016 at 6:05 PM, Jan Viktorin > wrote: >> What about introducing a macro for this? >> >> RTE_REGISTER_PCI_DRIVER(rte_qad_pmd); > > Yes. The only problem here, is that rte_qad_pmd is a crypto structure (same problem with

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-10 Thread David Marchand
On Tue, Feb 9, 2016 at 6:05 PM, Jan Viktorin wrote: > Maybe, a better subject? > > drivers: init pdev drivers in constructors Why not, I will try to find a best one, and if I can't, I will go with this. > On Fri, 29 Jan 2016 15:08:30 +0100 > David Marchand wrote: >> -static int >>

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-02-09 Thread Jan Viktorin
Maybe, a better subject? drivers: init pdev drivers in constructors On Fri, 29 Jan 2016 15:08:30 +0100 David Marchand wrote: > Now that pdev drivers have been converted to pci drivers, there is nothing > left in their init functions that can't go in a constructor. > pdev / vdev drivers init

[dpdk-dev] [PATCH 3/9] drivers: no more pdev drivers

2016-01-29 Thread David Marchand
Now that pdev drivers have been converted to pci drivers, there is nothing left in their init functions that can't go in a constructor. pdev / vdev drivers init order is changed by this commit, but I can't see why we would need to preserve it. Signed-off-by: David Marchand ---