[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-04 Thread Damien Millescamps
On 06/03/2013 07:26 PM, Stephen Hemminger wrote: > There is nothing that says a PMD could load a second module, but why not put > that special case code in the the driver. Or make it a list of strings. There is nothing that says anything about PMD and loading a single module. However, the Getting S

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Antti Kantee
On 03.06.2013 18:29, Thomas Monjalon wrote: >> Is breaking the API a bad thing in this context? IMHO the >> initialization APIs need work before they're general enough and >> perpetually supporting the current ones seems like an unnecessary >> burden. I'm trying to understand the general guidelin

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Thomas Monjalon
03/06/2013 17:41, Stephen Hemminger : > On Mon, 03 Jun 2013 10:58:01 +0200 > Damien Millescamps wrote: > > > > > > --- a/lib/librte_eal/common/include/rte_pci.h 2013-05-29 > > > 08:45:38.0 -0700 +++ > > > b/lib/librte_eal/common/include/rte_pci.h 2013-05-29 > > > 09:02:50.0 -0

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Thomas Monjalon
03/06/2013 18:08, Antti Kantee : > On 03.06.2013 10:58, Damien Millescamps wrote: > >> -/** Device needs igb_uio kernel module */ > >> -#define RTE_PCI_DRV_NEED_IGB_UIO 0x0001 > >> > >> /** Device driver must be registered several times until failure */ > >> > >> -#define RTE_PCI_DRV_MULTIPLE 0

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Antti Kantee
On 03.06.2013 10:58, Damien Millescamps wrote: >> }; >> >> -/** Device needs igb_uio kernel module */ >> -#define RTE_PCI_DRV_NEED_IGB_UIO 0x0001 >> /** Device driver must be registered several times until failure */ >> -#define RTE_PCI_DRV_MULTIPLE 0x0002 >> +#define RTE_PCI_DRV_MULTIPLE 0x000

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Damien Millescamps
comments inlined. On 05/30/2013 07:12 PM, Stephen Hemminger wrote: > An additional change is that failure to detect module should not be fatal, > but an error like other EAL setup problems, allowing the application > to recover. > > Signed-off-by: Stephen Hemminger > > --- > app/chkincs/test_pci

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Stephen Hemminger
On Mon, 3 Jun 2013 18:36:52 +0200 Thomas Monjalon wrote: > 03/06/2013 17:41, Stephen Hemminger : > > On Mon, 03 Jun 2013 10:58:01 +0200 > > Damien Millescamps wrote: > > > > > > > > --- a/lib/librte_eal/common/include/rte_pci.h 2013-05-29 > > > > 08:45:38.0 -0700 +++ > > > > b/lib/lib

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Stephen Hemminger
On Mon, 3 Jun 2013 18:29:02 +0200 Thomas Monjalon wrote: > 03/06/2013 18:08, Antti Kantee : > > On 03.06.2013 10:58, Damien Millescamps wrote: > > >> -/** Device needs igb_uio kernel module */ > > >> -#define RTE_PCI_DRV_NEED_IGB_UIO 0x0001 > > >> > > >> /** Device driver must be registered se

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-06-03 Thread Stephen Hemminger
On Mon, 03 Jun 2013 10:58:01 +0200 Damien Millescamps wrote: > comments inlined. > > On 05/30/2013 07:12 PM, Stephen Hemminger wrote: > > An additional change is that failure to detect module should not be fatal, > > but an error like other EAL setup problems, allowing the application > > to rec

[dpdk-dev] [PATCH 4/7] eal: support different modules

2013-05-30 Thread Stephen Hemminger
An embedded and charset-unspecified text was scrubbed... Name: module-name.patch URL: