Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-22 Thread Adrien Mazarguil
On Fri, May 19, 2017 at 12:11:53PM +0200, Thomas Monjalon wrote: > 19/05/2017 11:11, Gaëtan Rivet: > > On Fri, May 19, 2017 at 08:57:01AM +, Ananyev, Konstantin wrote: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > >> 18/05/2017 13:33, Ferruh Yigit: > > >> > On 5/17/2017 5:38 PM,

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-19 Thread Thomas Monjalon
19/05/2017 11:11, Gaëtan Rivet: > On Fri, May 19, 2017 at 08:57:01AM +, Ananyev, Konstantin wrote: > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > >> 18/05/2017 13:33, Ferruh Yigit: > >> > On 5/17/2017 5:38 PM, Gaëtan Rivet wrote: > >> > > The other is the expression of flows through a

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-19 Thread Ananyev, Konstantin
] > >> Sent: Thursday, May 18, 2017 9:32 PM > >> To: Yigit, Ferruh > >> Cc: dev@dpdk.org; Gaëtan Rivet ; Ananyev, > >> Konstantin ; Mcnamara, John > >> ; Tahhan, Maryam ; > >> adrien.mazarg...@6wind.com > >> Subject: Re: [dpdk-dev] [RF

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-19 Thread Gaëtan Rivet
; adrien.mazarg...@6wind.com Subject: Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library 18/05/2017 13:33, Ferruh Yigit: > On 5/17/2017 5:38 PM, Gaëtan Rivet wrote: > > The other is the expression of flows through a shared syntax. Using > > flags to propose

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-19 Thread Ananyev, Konstantin
> Subject: Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify > library > > 18/05/2017 13:33, Ferruh Yigit: > > On 5/17/2017 5:38 PM, Gaëtan Rivet wrote: > > > The other is the expression of flows through a shared syntax. Using > > > flags to propo

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-18 Thread Thomas Monjalon
18/05/2017 13:33, Ferruh Yigit: > On 5/17/2017 5:38 PM, Gaëtan Rivet wrote: > > The other is the expression of flows through a shared syntax. Using > > flags to propose presets can be simpler, but will probably not be flexible > > enough. rte_flow_items are a first-class citizen in DPDK and are > >

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-18 Thread Ferruh Yigit
On 5/17/2017 5:10 PM, Ananyev, Konstantin wrote: >>> Hi Ferruh, >>> Please see my comments/questions below. >>> Thanks >>> Konstantin >>> + +/** + * @file + * + * RTE Flow Classify Library + * + * This library provides flow record information with some measured >

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-18 Thread Ferruh Yigit
On 5/17/2017 5:38 PM, Gaëtan Rivet wrote: > Hi Ferruh, Hi Gaetan, > > On Wed, May 17, 2017 at 05:02:50PM +0100, Ferruh Yigit wrote: >> On 5/17/2017 3:54 PM, Ananyev, Konstantin wrote: >>> Hi Ferruh, >>> Please see my comments/questions below. >> >> Thanks for review. >> >>> Thanks >>> Konstantin

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-17 Thread Gaëtan Rivet
Hi Ferruh, On Wed, May 17, 2017 at 05:02:50PM +0100, Ferruh Yigit wrote: On 5/17/2017 3:54 PM, Ananyev, Konstantin wrote: Hi Ferruh, Please see my comments/questions below. Thanks for review. Thanks Konstantin <...> I think it was discussed already, but I still wonder why rte_flow_item

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-17 Thread Ananyev, Konstantin
> > On 5/17/2017 3:54 PM, Ananyev, Konstantin wrote: > > Hi Ferruh, > > Please see my comments/questions below. > > Thanks for review. > > > Thanks > > Konstantin > > <...> > > > I think it was discussed already, but I still wonder why rte_flow_item > > can't be used for that approach? > >

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-17 Thread Ananyev, Konstantin
> > Hi Ferruh, > > Please see my comments/questions below. > > Thanks > > Konstantin > > > >> + > >> +/** > >> + * @file > >> + * > >> + * RTE Flow Classify Library > >> + * > >> + * This library provides flow record information with some measured > >> properties. > >> + * > >> + * Application can

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-17 Thread Ferruh Yigit
On 5/17/2017 3:54 PM, Ananyev, Konstantin wrote: > Hi Ferruh, > Please see my comments/questions below. Thanks for review. > Thanks > Konstantin <...> > I think it was discussed already, but I still wonder why rte_flow_item can't > be used for that approach? Missed this one: Gaëtan also had

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-17 Thread Ferruh Yigit
On 5/17/2017 3:54 PM, Ananyev, Konstantin wrote: > Hi Ferruh, > Please see my comments/questions below. > Thanks > Konstantin > >> + >> +/** >> + * @file >> + * >> + * RTE Flow Classify Library >> + * >> + * This library provides flow record information with some measured >> properties. >> + * >>

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-17 Thread Ananyev, Konstantin
Hi Ferruh, Please see my comments/questions below. Thanks Konstantin > + > +/** > + * @file > + * > + * RTE Flow Classify Library > + * > + * This library provides flow record information with some measured > properties. > + * > + * Application can select variety of flow types based on various fl

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-16 Thread Thomas Monjalon
04/05/2017 13:35, Mcnamara, John: > > > -Original Message- > > From: Yigit, Ferruh > > Sent: Thursday, April 20, 2017 7:55 PM > > To: dev@dpdk.org > > Cc: Yigit, Ferruh ; Mcnamara, John > > ; Tahhan, Maryam > > Subject: [RFC 17.08] flow_classify: add librte_flow_classify library > > > C

Re: [dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-05-04 Thread Mcnamara, John
> -Original Message- > From: Yigit, Ferruh > Sent: Thursday, April 20, 2017 7:55 PM > To: dev@dpdk.org > Cc: Yigit, Ferruh ; Mcnamara, John > ; Tahhan, Maryam > Subject: [RFC 17.08] flow_classify: add librte_flow_classify library CCing techbo...@dpdk.org since we would like this RFC add

[dpdk-dev] [RFC 17.08] flow_classify: add librte_flow_classify library

2017-04-20 Thread Ferruh Yigit
Signed-off-by: Ferruh Yigit --- config/common_base | 5 + doc/api/doxy-api-index.md | 1 + doc/api/doxy-api.conf | 1 + doc/guides/rel_notes/release_17_05.rst | 1 + lib/Makefile