Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-12 Thread Thomas Monjalon
11/07/2017 20:20, Dumitrescu, Cristian: > Hi Thomas, > > Fixed everything you asked on the next-tm repository, please resume the pull. > > I am working to send documentation as separate patch most likely next week. > > Changes: > 1. rte_ethdev.[hc]: removed unused function rte_eth_dev_tm_ops_ge

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-11 Thread Dumitrescu, Cristian
ll, Tim > ; Glynn, Michael J ; > Adrien Mazarguil > Subject: Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1 > > 10/07/2017 18:47, Dumitrescu, Cristian: > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > 10/07/2017 17:46, Dumitrescu, Cristian: &

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Thomas Monjalon
10/07/2017 18:47, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 10/07/2017 17:46, Dumitrescu, Cristian: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > 10/07/2017 15:21, Dumitrescu, Cristian: > > > > > From: Thomas Monjalon [mailto:tho...@monjalon.n

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Dumitrescu, Cristian
ll, Tim > ; Glynn, Michael J ; > Adrien Mazarguil > Subject: Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1 > > 10/07/2017 17:46, Dumitrescu, Cristian: > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > 10/07/2017 15:21, Dumitrescu, Cristian: &

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Thomas Monjalon
10/07/2017 17:46, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 10/07/2017 15:21, Dumitrescu, Cristian: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > 10/07/2017 12:55, Dumitrescu, Cristian: > > > > > From: Thomas Monjalon [mailto:tho...@monjalon.n

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Dumitrescu, Cristian
ll, Tim > ; Glynn, Michael J ; > Adrien Mazarguil > Subject: Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1 > > 10/07/2017 15:21, Dumitrescu, Cristian: > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > 10/07/2017 12:55, Dumitrescu, Cristian: &

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Thomas Monjalon
10/07/2017 15:21, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 10/07/2017 12:55, Dumitrescu, Cristian: > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > 2/ Some functions are exposed in the API to query the ops. > > > > It seems dangerous and useless

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Dumitrescu, Cristian
ll, Tim > ; Glynn, Michael J ; > Adrien Mazarguil > Subject: Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1 > > Hi, > > 10/07/2017 12:55, Dumitrescu, Cristian: > > Hi Thomas, > > > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > &

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Thomas Monjalon
Hi, 10/07/2017 12:55, Dumitrescu, Cristian: > Hi Thomas, > > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > > > Hi, > > > > 04/07/2017 17:38, Cristian Dumitrescu: > > > http://dpdk.org/git/next/dpdk-next-tm > > > > I'm sorry to not have considered this tree as a high priority. > > I

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Dumitrescu, Cristian
; Subject: Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1 > > Hi, > > 04/07/2017 17:38, Cristian Dumitrescu: > > http://dpdk.org/git/next/dpdk-next-tm > > I'm sorry to not have considered this tree as a high priority. > I think it may be integrated in RC2 bec

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Thomas Monjalon
10/07/2017 09:43, Adrien Mazarguil: > Hi Thomas, > > On Sun, Jul 09, 2017 at 10:01:31PM +0200, Thomas Monjalon wrote: > > 3/ The PMD interface file is referenced in the doxygen index: > > + [rte_tm_driver] (@ref rte_tm_driver.h), > > I see that rte_flow_driver.h is also referenced but it see

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-10 Thread Adrien Mazarguil
Hi Thomas, On Sun, Jul 09, 2017 at 10:01:31PM +0200, Thomas Monjalon wrote: > Hi, > > 04/07/2017 17:38, Cristian Dumitrescu: > > http://dpdk.org/git/next/dpdk-next-tm > > I'm sorry to not have considered this tree as a high priority. > I think it may be integrated in RC2 because it is a total

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-09 Thread Thomas Monjalon
Hi, 04/07/2017 17:38, Cristian Dumitrescu: > http://dpdk.org/git/next/dpdk-next-tm I'm sorry to not have considered this tree as a high priority. I think it may be integrated in RC2 because it is a totally new area and should not break any existing code. I prefer to wait because I have seen s

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-05 Thread Dumitrescu, Cristian
> I will check for pulling your tree in following days. > Please try to be available on IRC, in case I catch a last minute detail to > fix. > Thanks Thank you, Thomas. No issues, I will present on IRC and reply promptly on any emails. Regards, Cristian

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-04 Thread Thomas Monjalon
04/07/2017 18:52, Dumitrescu, Cristian: > > Hi Cristian, > > > > > Dumitrescu, Cristian (2): > > > ethdev: add traffic management ops get API > > > ethdev: add traffic management API > > > > The original request was to split this huge patch. > > It is too messy to bring a whole new AP

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-04 Thread Dumitrescu, Cristian
rruh > > Subject: Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1 > > Hi Cristian, > > > Dumitrescu, Cristian (2): > > ethdev: add traffic management ops get API > > ethdev: add traffic management API > > The original request was to split this hu

Re: [dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-04 Thread Thomas Monjalon
Hi Cristian, > Dumitrescu, Cristian (2): > ethdev: add traffic management ops get API > ethdev: add traffic management API The original request was to split this huge patch. It is too messy to bring a whole new API area in one patch. We have nothing to refer in case of bug, and it is

[dpdk-dev] [pull-request] next-tm 17.08 pre-rc1

2017-07-04 Thread Cristian Dumitrescu
The following changes since commit a566400e8b73ec646e0cc6dd0bc44def8535fb98: net: implement CRC for ARM64 NEON (2017-07-04 15:58:45 +0200) are available in the git repository at: http://dpdk.org/git/next/dpdk-next-tm for you to fetch changes up to dcb4c3f881e29e1062a3e298e60474912e8a4ece: