Re: [lng-odp] [API-NEXT PATCHv4 1/5] drv: adding driver registration interface (stub)

2016-12-18 Thread Yi He
Hi, Chistophe, When I'm trying to apply your patchset on the most recent api-next branch, on the [3/5] patch it pops, I understand it needs to install libconfig-dev, but the below problem seems need a rebase? yihe01@A010291:~/workspace/odp-linux$ git am ~/Downloads/API-NEXT-PATCHv4-3-5-linux-gen-

Re: [lng-odp] [API-NEXT PATCHv4 1/5] drv: adding driver registration interface (stub)

2016-12-18 Thread Christophe Milard
On 19 December 2016 at 07:04, Yi He wrote: > Hi, Christophe > > For [1/5] and [2/5], are these typo of odrdrv_? > Yes. typos. will fix > For [3/5] I failed to apply to the api-next, may need rebase. > The overall approach and code looks ok for me and once you update I'll > apply and test. > T

Re: [lng-odp] [API-NEXT PATCHv4 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

2016-12-18 Thread Christophe Milard
Should be odpdrv_* I was obviously blind when writting these comments :-) Will fix. On 19 December 2016 at 08:20, Forrest Shi wrote: > I'm a bit curious why not odp_drv_enum? > > On 19 December 2016 at 14:04, Yi He wrote: > >> >> >> On 8 December 2016 at 17:20, Christophe Milard < >> christoph

Re: [lng-odp] [API-NEXT PATCHv4 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

2016-12-18 Thread Forrest Shi
I'm a bit curious why not odp_drv_enum? On 19 December 2016 at 14:04, Yi He wrote: > > > On 8 December 2016 at 17:20, Christophe Milard < > christophe.mil...@linaro.org> wrote: > >> The linux implementation for the enumerator class registration function, >> enumerator instance registration funct

Re: [lng-odp] [PATCH] example: introducing multi-threaded traffic management case

2016-12-18 Thread Forrest Shi
Hi, this patch is a followup of http://patches.opendataplane.org/patch/7515/ to keep the original traffic management example structure and keep it small, this multi-thread version is going to make a new example. Thanks, Forrest On 19 December 2016 at 14:01, wrote: > From: Xuelin Shi > > int

[lng-odp] [PATCH] example: introducing multi-threaded traffic management case

2016-12-18 Thread forrest.shi
From: Xuelin Shi introduce a new example of traffic management with following features: - multiple TM threads: one TM thread for each pktio - for small system less than 4 cores, only one TM thread created. - receiving packets from multiple pktios other than generating packets - identifyin

Re: [lng-odp] [API-NEXT PATCHv4 2/5] linux-gen: adding enum, devio and driver registration interface (stub)

2016-12-18 Thread Yi He
On 8 December 2016 at 17:20, Christophe Milard wrote: > The linux implementation for the enumerator class registration function, > enumerator instance registration function, > devio and driver registration functions (stub) > > Signed-off-by: Christophe Milard > --- > include/odp_drv.h

Re: [lng-odp] [API-NEXT PATCHv4 1/5] drv: adding driver registration interface (stub)

2016-12-18 Thread Yi He
Hi, Christophe For [1/5] and [2/5], are these typo of odrdrv_? For [3/5] I failed to apply to the api-next, may need rebase. The overall approach and code looks ok for me and once you update I'll apply and test. Best Regards, Yi On 8 December 2016 at 17:20, Christophe Milard wrote: > The enume

[lng-odp] [PATCH] linux-gen: use odp_queue_t for the internal queue interface

2016-12-18 Thread Kevin Wang
From: "kevin.wang" To make the queue component to be modulized, the other components aren't allowed to access the detailed data structure of queue. The only way to access queue is from the internal queue interface by using odp_queue_t as a handler Signed-off-by: kevin.wang --- .../include/odp_