[dpdk-dev] [PATCH v2] pipeline: add statistics for librte_pipeline ports and tables

2015-05-20 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, May 20, 2015 11:02 PM > To: Stephen Hemminger; Dumitrescu, Cristian > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] pipeline: add statistics for > librte_pipeline > ports and

[dpdk-dev] [PATCH v2] pipeline: add statistics for librte_pipeline ports and tables

2015-05-20 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, May 20, 2015 3:45 PM > To: Dumitrescu, Cristian > Cc: Wodkowski, PawelX; dev at dpdk.org; Jastrzebski, MichalX K > Subject: Re: [dpdk-dev] [PATCH v2] pipeline: add statistics for > librt

[dpdk-dev] [RFC PATCHv2 0/2] pktdev as wrapper type

2015-05-20 Thread Marc Sune
On 20/05/15 12:28, Neil Horman wrote: > On Wed, May 20, 2015 at 12:05:00PM +0200, Marc Sune wrote: >> >> On 20/05/15 10:31, Thomas Monjalon wrote: >>> 2015-05-19 12:31, Bruce Richardson: On Mon, May 11, 2015 at 05:29:39PM +0100, Bruce Richardson wrote: > Hi all, > > after a small

[dpdk-dev] [PATCH v2 06/19] enic: move enic PMD to drivers/net directory

2015-05-20 Thread Thomas Monjalon
2015-05-20 16:45, Sujith Sankar: > On 20/05/15 9:44 pm, "Thomas Monjalon" wrote: > >2015-05-20 16:04, Richardson, Bruce: > >> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > >> > I think that vnic/ should be renamed to base/ > >> > >> Yes, I was wondering about that. However, this w

[dpdk-dev] traffic no longer readable...

2015-05-20 Thread Thomas Monjalon
2015-05-20 11:59, Jeff Weeks: > forgot to include the list... > > -- Forwarded Message -- > > Subject: Re: [dpdk-dev] traffic no longer readable... > Date: Wednesday, May 20, 2015, 11:56:47 AM > From: Jeff Weeks > To: Thomas Monjalon > > On Wednesday, May 20, 2015 05:33:20 PM

[dpdk-dev] [PATCH v2 06/19] enic: move enic PMD to drivers/net directory

2015-05-20 Thread Thomas Monjalon
2015-05-20 16:04, Richardson, Bruce: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2015-05-15 16:56, Bruce Richardson: > > > move enic PMD to drivers/net directory > > > > > > Signed-off-by: Bruce Richardson > > > --- > > > drivers/net/Makefile |2 +

[dpdk-dev] [PATCH v2 06/19] enic: move enic PMD to drivers/net directory

2015-05-20 Thread Thomas Monjalon
2015-05-15 16:56, Bruce Richardson: > move enic PMD to drivers/net directory > > Signed-off-by: Bruce Richardson > --- > drivers/net/Makefile |2 +- > drivers/net/enic/LICENSE | 27 + > drivers/net/enic/Makefile| 71 ++ > dr

[dpdk-dev] traffic no longer readable...

2015-05-20 Thread Thomas Monjalon
2015-05-20 11:11, Jeff Weeks: > Hello all, > > I've been developing with dpdk, using virtio, and I'm noticing that I can > often get into a state whereby I no longer receive any packets. > > The call to rte_eth_rx_burst always returns 0, indicating the rx queue is > empty. > > How is this queu

[dpdk-dev] [PATCH v2 02/19] drivers: create drivers and drivers/net directory

2015-05-20 Thread Thomas Monjalon
2015-05-15 16:56, Bruce Richardson: > Add a new top-level "drivers" directory to which all PMDs will be moved > for easier maintenance of both lib folder and drivers themselves. This > new directory is a dependency of all the apps in the app folder, so > the makefiles for each app are updated. Why

[dpdk-dev] [PATCH v2 06/19] enic: move enic PMD to drivers/net directory

2015-05-20 Thread Sujith Sankar (ssujith)
On 20/05/15 9:44 pm, "Thomas Monjalon" wrote: >2015-05-20 16:04, Richardson, Bruce: >> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] >> > 2015-05-15 16:56, Bruce Richardson: >> > > move enic PMD to drivers/net directory >> > > >> > > Signed-off-by: Bruce Richardson >> > > --- >>

[dpdk-dev] [PATCH v2] pipeline: add statistics for librte_pipeline ports and tables

2015-05-20 Thread Thomas Monjalon
Please Cristian, do not top post. I'm replacing your comment in the right context. 2015-05-20 13:57, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Thanks for the detailed explanation. > > > > You are raising a trade-off problem about > > feature/maintenan

[dpdk-dev] [PATCH 1/2] enic: silence log message

2015-05-20 Thread Sujith Sankar (ssujith)
On 19/05/15 3:38 pm, "Bruce Richardson" wrote: >On Wed, Apr 08, 2015 at 10:37:49AM -0700, Stephen Hemminger wrote: >> From: Stephen Hemminger >> >> Silence is normal. drivers should speak only when spoken to and not >> be chatty. >> >> Signed-off-by: Stephen Hemminger > >Acked-by: Bruce Ric

[dpdk-dev] [PATCH v2 06/19] enic: move enic PMD to drivers/net directory

2015-05-20 Thread Richardson, Bruce
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, May 20, 2015 4:56 PM > To: Richardson, Bruce > Cc: dev at dpdk.org; Sujith Sankar > Subject: Re: [dpdk-dev] [PATCH v2 06/19] enic: move enic PMD to > drivers/net directory > > 2015-05-15

[dpdk-dev] [PATCH v2 02/19] drivers: create drivers and drivers/net directory

2015-05-20 Thread Richardson, Bruce
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Wednesday, May 20, 2015 4:05 PM > To: Richardson, Bruce > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 02/19] drivers: create drivers and > drivers/net directory > > 2015-05-15 16:56, Bruc

[dpdk-dev] [RFC PATCHv2 0/2] pktdev as wrapper type

2015-05-20 Thread Neil Horman
On Wed, May 20, 2015 at 07:01:02PM +0200, Marc Sune wrote: > > > On 20/05/15 12:28, Neil Horman wrote: > >On Wed, May 20, 2015 at 12:05:00PM +0200, Marc Sune wrote: > >> > >>On 20/05/15 10:31, Thomas Monjalon wrote: > >>>2015-05-19 12:31, Bruce Richardson: > On Mon, May 11, 2015 at 05:29:39PM

[dpdk-dev] DPDK ACL bug? pkt matches the wrong ACL rule.

2015-05-20 Thread Ananyev, Konstantin
Hi Zi, > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zi Hu > Sent: Friday, May 15, 2015 1:27 AM > To: dev at dpdk.org > Subject: [dpdk-dev] DPDK ACL bug? pkt matches the wrong ACL rule. > > Hi, there, > > I recently noticed that sometimes packets are matched with the wrong ACL > rul

[dpdk-dev] [PATCH] eal_common_options: Allow combining -m and --no-huge

2015-05-20 Thread Simon Kagstrom
Needed to run as non-root but with higher memory allocations. Signed-off-by: Simon Kagstrom Signed-off-by: Johan Faltstrom --- lib/librte_eal/common/eal_common_options.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/common/eal_common_options.c b/lib/lib

[dpdk-dev] [PATCH v2] pipeline: add statistics for librte_pipeline ports and tables

2015-05-20 Thread Dumitrescu, Cristian
Hi Thomas, Can you please describe what solution 2 on your list (build-time log level) consists of? I see log level useful for printing messages when an event takes place, but this is not what these stats patches are about. We want to poll for those counters on demand: if the build-time flag i

[dpdk-dev] bond: mode 4 promiscuous mode

2015-05-20 Thread Declan Doherty
On 15/05/15 14:36, Andriy Berestovskyy wrote: > Hey guys, > Can we in function bond_mode_8023ad_activate_slave() try to add to the > slave bond and LACP multicast MACs first? And then we would fall back > into promiscuous mode if the adding has failed. > > In other words: > > if (rte_eth_dev_mac_ad

[dpdk-dev] [PATCH] rte_reorder: Allow sequence numbers > 0 as starting point

2015-05-20 Thread Simon Kagstrom
We use sequence numbers from a generator which has potentially started long before the receiver. Therefore, the first number will typically be > 0. The rte_reorder code will not work in this case, since the packet is seen as outside of the buffer. The patch instead records the first sequence numbe

[dpdk-dev] [RFC PATCHv2 0/2] pktdev as wrapper type

2015-05-20 Thread Marc Sune
On 20/05/15 10:31, Thomas Monjalon wrote: > 2015-05-19 12:31, Bruce Richardson: >> On Mon, May 11, 2015 at 05:29:39PM +0100, Bruce Richardson wrote: >>> Hi all, >>> >>> after a small amount of offline discussion with Marc Sune, here is an >>> alternative proposal for a higher-level interface - ak

[dpdk-dev] Technical Steering Committee (TSC)

2015-05-20 Thread Dave Neary
Hi, On 05/14/2015 01:55 PM, O'Driscoll, Tim wrote: > At Tuesday's Beyond DPDK 2.0 call, one topic we discussed was decision making > and whether we need a Technical Steering Committee (TSC). As a follow-up to > that discussion, I'd like to propose that we create a TSC for DPDK to guide > the lo

[dpdk-dev] Fwd: Re: traffic no longer readable...

2015-05-20 Thread Jeff Weeks
forgot to include the list... -- Forwarded Message -- Subject: Re: [dpdk-dev] traffic no longer readable... Date: Wednesday, May 20, 2015, 11:56:47 AM From: Jeff Weeks To: Thomas Monjalon On Wednesday, May 20, 2015 05:33:20 PM you wrote: > 2015-05-20 11:11, Jeff Weeks: > > He

[dpdk-dev] How do you setup a VM in Promiscuous Mode using PCI Pass-Through (SR-IOV)?

2015-05-20 Thread Gonzalez Monroy, Sergio
On 19/05/2015 20:23, Assaad, Sami (Sami) wrote: > Hello Michael, > > I've updated the kernel and QEMU. Here are the packages I'm using: > > --> CentOS 7 - 3.10.0-229.4.2.el7.x86_64 > - qemu-kvm-1.5.3-86.el7_1.2.x86_64 > - libvirt-1.2.8-16.el7_1.3.x86_64 > - virt-manager-1.1.0-12.el7.

[dpdk-dev] traffic no longer readable...

2015-05-20 Thread Jeff Weeks
Hello all, I've been developing with dpdk, using virtio, and I'm noticing that I can often get into a state whereby I no longer receive any packets. The call to rte_eth_rx_burst always returns 0, indicating the rx queue is empty. How is this queue filled? I see there is a dpdk thread, which w

[dpdk-dev] [PATCH v2] pipeline: add statistics for librte_pipeline ports and tables

2015-05-20 Thread Stephen Hemminger
On Wed, 20 May 2015 16:44:35 +0200 Thomas Monjalon wrote: > Please Cristian, do not top post. > I'm replacing your comment in the right context. > > 2015-05-20 13:57, Dumitrescu, Cristian: > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > Thanks for the detailed explanation.

[dpdk-dev] [PATCH 1/2] enic: silence log message

2015-05-20 Thread Stephen Hemminger
Other drivers are quiet, why does this one need to be noisy On Wed, May 20, 2015 at 9:30 AM, Sujith Sankar (ssujith) wrote: > > > On 19/05/15 3:38 pm, "Bruce Richardson" > wrote: > > >On Wed, Apr 08, 2015 at 10:37:49AM -0700, Stephen Hemminger wrote: > >> From: Stephen Hemminger > >> > >> Sile

[dpdk-dev] [PATCH v2] mempool: limit cache_size

2015-05-20 Thread Thomas Monjalon
2015-05-20 10:32, Olivier MATZ: > On 05/18/2015 05:35 PM, Zoltan Kiss wrote: > > Otherwise cache_flushthresh can be bigger than n, and > > a consumer can starve others by keeping every element > > either in use or in the cache. > > > > Signed-off-by: Zoltan Kiss > > Acked-by: Olivier Matz Appli

[dpdk-dev] [PATCH v2] mempool: limit cache_size

2015-05-20 Thread Olivier MATZ
On 05/18/2015 05:35 PM, Zoltan Kiss wrote: > Otherwise cache_flushthresh can be bigger than n, and > a consumer can starve others by keeping every element > either in use or in the cache. > > Signed-off-by: Zoltan Kiss Acked-by: Olivier Matz > --- > v2: use macro for calculation, with proper c

[dpdk-dev] [RFC PATCHv2 0/2] pktdev as wrapper type

2015-05-20 Thread Thomas Monjalon
2015-05-19 12:31, Bruce Richardson: > On Mon, May 11, 2015 at 05:29:39PM +0100, Bruce Richardson wrote: > > Hi all, > > > > after a small amount of offline discussion with Marc Sune, here is an > > alternative proposal for a higher-level interface - aka pktdev - to allow a > > common Rx/Tx API acr

[dpdk-dev] DPDK ACL bug? pkt matches the wrong ACL rule.

2015-05-20 Thread Zi Hu
Hi, Konstantin, The patch does fix the bug with the my test rule/trace file. I will do a little bit more test later to verify that. thanks a lot. -Zi On Wed, May 20, 2015 at 7:28 AM, Ananyev, Konstantin < konstantin.ananyev at intel.com> wrote: > > Hi Zi, > > > From: dev [mailto:dev-bounces at

[dpdk-dev] [PATCH] ixgbe: reset hardware stat when initialize

2015-05-20 Thread Qiu, Michael
Any comments about this patch? Thanks, Michael On 5/8/2015 4:14 PM, Michael Qiu wrote: > From: Michael Qiu > > When initialize the hardware, the stat should be reset. > Otherwise when detach then attach port, the stat will not > be re-init to zero. > > Signed-off-by: Michael Qiu > --- > lib/li

[dpdk-dev] How do you setup a VM in Promiscuous Mode using PCI Pass-Through (SR-IOV)?

2015-05-20 Thread Qiu, Michael
25 What is RMRR? 26 - 27

[dpdk-dev] How do you setup a VM in Promiscuous Mode using PCI Pass-Through (SR-IOV)?

2015-05-20 Thread Qiu, Michael
You should refer this patch about VFIO in kernel https://lkml.org/lkml/2014/6/13/394 Seems your device is unable to do pci pass through using vfio. Could you try pci-stub driver? Thanks, Michael On 5/20/2015 3:24 AM, Assaad, Sami (Sami) wrote: > Hello Michael, > > I've updated the kernel and QE

[dpdk-dev] [PATCH / RFC] kni: Add set_rx_mode callback to handle multicast groups

2015-05-20 Thread Simon Kågström
Ping? // Simon On 2015-05-07 15:17, Simon Kagstrom wrote: > This is needed to add / remove interfaces in multicast groups via the > ip tool. > > The callback does nothing - the same as the kernel tun.c. > > Signed-off-by: Simon Kagstrom > --- > Marked RFC since I'm by no means an expert on thi

[dpdk-dev] [RFC PATCHv2 0/2] pktdev as wrapper type

2015-05-20 Thread Neil Horman
On Wed, May 20, 2015 at 12:05:00PM +0200, Marc Sune wrote: > > > On 20/05/15 10:31, Thomas Monjalon wrote: > >2015-05-19 12:31, Bruce Richardson: > >>On Mon, May 11, 2015 at 05:29:39PM +0100, Bruce Richardson wrote: > >>>Hi all, > >>> > >>>after a small amount of offline discussion with Marc Sune

[dpdk-dev] [PATCH] virtio: Fix enqueue/dequeue can't handle chained vring descriptors.

2015-05-20 Thread Xie, Huawei
On 5/18/2015 9:23 PM, Ouyang, Changchun wrote: > Hi Huawei, > >> -Original Message- >> From: Xie, Huawei >> Sent: Monday, May 18, 2015 5:39 PM >> To: Ouyang, Changchun; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH] virtio: Fix enqueue/dequeue can't handle >> chained vring descriptors.

[dpdk-dev] [PATCH v2] pipeline: add statistics for librte_pipeline ports and tables

2015-05-20 Thread Thomas Monjalon
Hi Cristian, Thanks for the detailed explanation. You are raising a trade-off problem about feature/maintenance/performance. I think we must choose among 4 solutions: 1/ always enabled 2/ build-time log level 3/ run-time option 4/ build-time option It's good to ha

[dpdk-dev] [PATCH v2 1/3] port: added WRITER_APPROACH == 1 implementation to ring port

2015-05-20 Thread Thomas Monjalon
Hi Cristian, 2015-05-19 22:49, Dumitrescu, Cristian: > HI Thomas and David, > > We can remove one of the code branches if you guys feel strongly about it. We don't feel strongly as we are not authors nor testers of this new code. But we want to avoid the maintenance nightmare of #ifdef. > The r

[dpdk-dev] [RFC PATCHv2 0/2] pktdev as wrapper type

2015-05-20 Thread Wiles, Keith
It looks fine to me. On 5/19/15, 7:31 AM, "Richardson, Bruce" wrote: >On Mon, May 11, 2015 at 05:29:39PM +0100, Bruce Richardson wrote: >> Hi all, >> >> after a small amount of offline discussion with Marc Sune, here is an >> alternative proposal for a higher-level interface - aka pktdev - to >