[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-11 Thread Bert van Leeuwen
> On 11 Nov 2016, at 11:29 AM, Thomas Monjalon > wrote: > > 2016-11-11 09:16, Alejandro Lucero: >> Thomas, >> >> We are wondering if you realize this patch fixes a bug with current ethdev >> code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. >> >> Maybe the commit message is

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-11 Thread Thomas Monjalon
2016-11-11 09:16, Alejandro Lucero: > Thomas, > > We are wondering if you realize this patch fixes a bug with current ethdev > code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. > > Maybe the commit message is giving the wrong impression and as you > commented, it should just focus

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-11 Thread Alejandro Lucero
On Fri, Nov 11, 2016 at 9:29 AM, Thomas Monjalon wrote: > 2016-11-11 09:16, Alejandro Lucero: > > Thomas, > > > > We are wondering if you realize this patch fixes a bug with current > ethdev > > code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. > > > > Maybe the commit message is

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-11 Thread Alejandro Lucero
Thomas, We are wondering if you realize this patch fixes a bug with current ethdev code as a device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS. Maybe the commit message is giving the wrong impression and as you commented, it should just focus on the bug it fixes and to leave for another

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-10 Thread Thomas Monjalon
2016-11-10 15:43, Alejandro Lucero: > On Thu, Nov 10, 2016 at 2:42 PM, Thomas Monjalon 6wind.com> > wrote: > > > 2016-11-10 14:00, Alejandro Lucero: > > > From: Bert van Leeuwen > > > > > > A device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS queues which > > > is used inside struct

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-10 Thread Alejandro Lucero
On Thu, Nov 10, 2016 at 4:01 PM, Thomas Monjalon wrote: > 2016-11-10 15:43, Alejandro Lucero: > > On Thu, Nov 10, 2016 at 2:42 PM, Thomas Monjalon < > thomas.monjalon at 6wind.com> > > wrote: > > > > > 2016-11-10 14:00, Alejandro Lucero: > > > > From: Bert van Leeuwen > > > > > > > > A device

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-10 Thread Alejandro Lucero
On Thu, Nov 10, 2016 at 2:42 PM, Thomas Monjalon wrote: > 2016-11-10 14:00, Alejandro Lucero: > > From: Bert van Leeuwen > > > > A device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS queues which > > is used inside struct rte_eth_stats. Ideally, DPDK should be built with > >

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-10 Thread Thomas Monjalon
2016-11-10 14:00, Alejandro Lucero: > From: Bert van Leeuwen > > A device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS queues which > is used inside struct rte_eth_stats. Ideally, DPDK should be built with > RTE_ETHDEV_QUEUE_STAT_CNTRS to the maximum number of queues a device > can support,

[dpdk-dev] [PATCH] ethdev: check number of queues less than RTE_ETHDEV_QUEUE_STAT_CNTRS

2016-11-10 Thread Alejandro Lucero
From: Bert van Leeuwen A device can have more than RTE_ETHDEV_QUEUE_STAT_CNTRS queues which is used inside struct rte_eth_stats. Ideally, DPDK should be built with RTE_ETHDEV_QUEUE_STAT_CNTRS to the maximum number of queues a device can support, 65536, as uint16_t