[dpdk-dev] [PATCH v9] drivers/net:new PMD using tun/tap host interface

2016-11-29 Thread Wiles, Keith
> On Nov 29, 2016, at 3:36 PM, Aws Ismail wrote: > > I have verified that adding just a single tap device works with testpmd. But > as soon as I try more than one tap device, I would get a coredump, e.g.: > > root@?localhost:~# testpmd -c f -n 4 --socket-mem 512 > --vdev=net_tap?,iface=tap0?

[dpdk-dev] Adding API to force freeing consumed buffers in TX ring

2016-11-22 Thread Wiles, Keith
> On Nov 21, 2016, at 9:25 AM, Richardson, Bruce intel.com> wrote: > > On Mon, Nov 21, 2016 at 04:06:32PM +0100, Olivier Matz wrote: >> Hi, >> >> On 11/21/2016 03:33 PM, Wiles, Keith wrote: >>> >>>> On Nov 21, 2016, at 4:4

[dpdk-dev] [PATCH] doc: move testpmd guide with other tools

2016-11-10 Thread Wiles, Keith
> On Nov 10, 2016, at 5:02 PM, Thomas Monjalon > wrote: > > 2016-11-10 16:11, Mcnamara, John: >> I had a look at the html output before and after this patch and I don't >> quite agree with it. I see that you are trying to clean up and make the >> documentation more consistent but I don't

[dpdk-dev] disable hugepages

2016-11-10 Thread Wiles, Keith
> On Nov 10, 2016, at 6:32 AM, Keren Hochman > wrote: > > I tried using the following dpdk options: > --no-huge --vdev eth_pcap0 ,rx_pcap=/t1,tx_pcap=/t2 > *It's worked but the number of elements is limited, although the machine > has enough free memory. *rte_mempool_create is failed when I'm

[dpdk-dev] Running 2 process on the same machine

2016-11-07 Thread Wiles, Keith
> On Nov 7, 2016, at 7:28 AM, Keren Hochman > wrote: > > Hi, > I need to run 2 process that uses dpdk on the same machine. One uses dpdk > drivers, and the other just read from a pcap file. If I disable hugepages > in the second process rte_mempool_create fails. What is the correct way to >

[dpdk-dev] Unable to change source MAC address of packet

2016-10-27 Thread Wiles, Keith
> On Oct 27, 2016, at 6:33 AM, Padam Jeet Singh > wrote: > > Hi, > > I am crafting a packet in which the source MAC address as set in the Ethernet > header is different than the transmit port?s default MAC address. A packet > capture of the packets coming out of this port however comes with

[dpdk-dev] mbuf changes

2016-10-24 Thread Wiles, Keith
> On Oct 24, 2016, at 10:49 AM, Morten Br?rup > wrote: > > First of all: Thanks for a great DPDK Userspace 2016! > > > > Continuing the Userspace discussion about Olivier Matz?s proposed mbuf > changes... > > > > 1. > > Stephen Hemminger had a noteworthy general comment about keeping

[dpdk-dev] Project Governance and Linux Foundation

2016-10-21 Thread Wiles, Keith
Thanks Dave for your work and notes: Comment inline > On Oct 21, 2016, at 3:00 PM, Dave Neary wrote: > > Hi all, > > We had a great session yesterday on this topic, I took some notes - does > anyone who was there have any corrections, or anyone who was not have > any comments? > > Thanks, >

[dpdk-dev] [PATCH v5] drivers/net:new PMD using tun/tap host interface

2016-10-12 Thread Wiles, Keith
Regards, Keith > On Oct 12, 2016, at 1:19 PM, Wiles, Keith wrote: > > > Regards, > Keith > >> On Oct 12, 2016, at 9:56 AM, Yigit, Ferruh wrote: >> >>> + >>> +static void >>> +tap_stats_get(struct rte_eth_dev *dev, struct

[dpdk-dev] [PATCH v5] drivers/net:new PMD using tun/tap host interface

2016-10-12 Thread Wiles, Keith
Regards, Keith > On Oct 12, 2016, at 9:56 AM, Yigit, Ferruh wrote: > > On 10/11/2016 10:51 PM, Keith Wiles wrote: >> The rte_eth_tap.c PMD creates a device using TUN/TAP interfaces >> on the local host. The PMD allows for DPDK and the host to >> communicate using a raw device interface on the

[dpdk-dev] [PATCH v4] drivers/net:new PMD using tun/tap host interface

2016-10-11 Thread Wiles, Keith
Regards, Keith > On Oct 11, 2016, at 6:49 AM, Yigit, Ferruh wrote: > > On 10/4/2016 3:45 PM, Keith Wiles wrote: >> The rte_eth_tap.c PMD creates a device using TUN/TAP interfaces >> on the local host. The PMD allows for DPDK and the host to >> communicate using a raw device interface on the

[dpdk-dev] [PATCH v4] drivers/net:new PMD using tun/tap host interface

2016-10-11 Thread Wiles, Keith
Regards, Keith > On Oct 11, 2016, at 7:28 AM, Yigit, Ferruh wrote: > > On 10/4/2016 3:45 PM, Keith Wiles wrote: >> +/* >> + * Open a TAP interface device. >> + */ >> +static int >> +rte_pmd_tap_devinit(const char *name, const char *params) >> +{ >> +int ret = 0; >> +struct rte_kvargs

[dpdk-dev] [PATCH v4] drivers/net:new PMD using tun/tap host interface

2016-10-11 Thread Wiles, Keith
Regards, Keith > On Oct 11, 2016, at 6:30 AM, Micha? Miros?aw wrote: > > 2016-10-04 16:45 GMT+02:00, Keith Wiles : >> The rte_eth_tap.c PMD creates a device using TUN/TAP interfaces >> on the local host. The PMD allows for DPDK and the host to >> communicate using a raw device interface on the

[dpdk-dev] [PATCH] log: do not drop debug logs at compile time

2016-10-03 Thread Wiles, Keith
Regards, Keith > On Oct 3, 2016, at 10:37 AM, Olivier Matz wrote: > > > > On 10/03/2016 05:27 PM, Wiles, Keith wrote: >> >> Regards, >> Keith >> >>> On Oct 3, 2016, at 10:02 AM, Olivier Matz wrote: >>> >>> Hi Keith, &g

[dpdk-dev] [PATCH] log: do not drop debug logs at compile time

2016-10-03 Thread Wiles, Keith
Regards, Keith > On Oct 3, 2016, at 10:02 AM, Olivier Matz wrote: > > Hi Keith, > > On 09/30/2016 05:48 PM, Wiles, Keith wrote: >>> On Sep 30, 2016, at 4:33 AM, Thomas Monjalon >>> wrote: >>> >>> 2016-09-16 09:43, Olivier Matz:

[dpdk-dev] [PATCH 3/3] drivers/net:build support for new tap device driver

2016-09-16 Thread Wiles, Keith
Regards, Keith > On Sep 16, 2016, at 2:36 AM, Panu Matilainen wrote: > > On 09/15/2016 05:10 PM, Keith Wiles wrote: >> Signed-off-by: Keith Wiles >> --- >> config/common_linuxapp | 3 +++ >> drivers/net/Makefile | 1 + >> mk/rte.app.mk | 1 + >> 3 files changed, 5 insertions(+) >> >>

[dpdk-dev] [PATCH 2/3] docs:tun/tap PMD information

2016-09-15 Thread Wiles, Keith
Regards, Keith > On Sep 15, 2016, at 9:13 AM, Wiles, Keith wrote: > > self Nak - just noticed the copyright notices are wrong. My mistake I was looking at the wrong file the headers appear correct. > > Regards, > Keith > >> On Sep 15, 2016, at 9:10 AM, Keith

[dpdk-dev] [PATCH 2/3] docs:tun/tap PMD information

2016-09-15 Thread Wiles, Keith
self Nak - just noticed the copyright notices are wrong. Regards, Keith > On Sep 15, 2016, at 9:10 AM, Keith Wiles wrote: > > Signed-off-by: Keith Wiles > --- > doc/guides/nics/tap.rst | 84 + > 1 file changed, 84 insertions(+) > create mode

[dpdk-dev] Run 2 different testpmd from the same machine

2016-09-12 Thread Wiles, Keith
Regards, Keith > On Sep 12, 2016, at 6:18 AM, Keren Hochman > wrote: > > Hi, > I tried to run 2 instances of testpmd from the same machine but received a > message: Cannot get hugepage information when I tried to run the second > instance. Is there a way to disable hugepages or allow to

[dpdk-dev] rte_kni.ko with lo_mode=lo_mode_ring

2016-09-02 Thread Wiles, Keith
HI Gary, Regards, Keith > On Sep 2, 2016, at 8:39 AM, Mussar, Gary wrote: > > The pktgen docs state that the rte_kni.ko should be loaded with > lo_mode=lo_mode_ring however the source in dpdk master does not appear to > understand this value. This wasn't an issue in the past since the kni

[dpdk-dev] [PATCH] doc/guides: add info on how to enable QAT

2016-08-30 Thread Wiles, Keith
Regards, Keith > On Aug 30, 2016, at 8:46 AM, Thomas Monjalon > wrote: > > 2016-08-30 14:26, Eoin Breen: >> --- a/doc/guides/cryptodevs/qat.rst >> +++ b/doc/guides/cryptodevs/qat.rst >> @@ -78,6 +78,11 @@ Installation >> To use the DPDK QAT PMD an SRIOV-enabled QAT kernel driver is required.

[dpdk-dev] Huge ring allocation

2016-08-25 Thread Wiles, Keith
Regards, Keith > On Aug 25, 2016, at 1:05 AM, Gregory Etelson wrote: > > Hello, > > I have a DPDK process that creates pktmbuf pool with 2_000_000 objects > In CentOS 6.x x86-64, ring size of this pool is 16MB > 8 2MB hugepages required to hold such ring. Have you try to use 1G huge pages,

[dpdk-dev] help

2016-08-19 Thread Wiles, Keith
Regards, Keith > On Aug 19, 2016, at 1:09 PM, Stephen Hemminger networkplumber.org> wrote: > > On Fri, 19 Aug 2016 13:11:48 + > wrote: > >> Hi , >> >> I needs to enable SMP(symmetrical multiprocessing) in DPDK PMD NIC Port . >> >> Please let me know the command to get the same >> >>

[dpdk-dev] [RFC 0/4] Use Google Test as DPDK unit test framework

2016-08-04 Thread Wiles, Keith
> On Aug 4, 2016, at 2:47 PM, Jim Murphy wrote: > > Hi, > > We are looking at using our existing test environment for our DPDK > applications that will run on our build servers. Hughpages therefore is an > issue. What is involved in running DPDK without huge pages? Command line option

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-08-01 Thread Wiles, Keith
>> Neil >> > Sorry for the delayed response, I've been on vacation. > >> Your modified copy of make has no bearing on the topic we are taking about >> customers using dpdk in standard distros right? >> > !? I really don't know what you're saying here. My only reason for > commenting > on

[dpdk-dev] Application framework vs. library

2016-08-01 Thread Wiles, Keith
> On Aug 1, 2016, at 5:04 AM, Thomas Monjalon > wrote: > > Hi, > > Thanks for explaining your context. > Your concerns are known under the term "usability enhancements". > As they require some API changes, we won't make them in the release 16.11. > But we must work on it now to be able to

[dpdk-dev] dpdk-pktgen how to show more than 4 ports in one page?

2016-07-30 Thread Wiles, Keith
> On Jul 30, 2016, at 1:03 AM, linhaifeng wrote: > > hi > > I use 6 ports to send pkts in VM, but can only 4 ports work, how to enable > more ports to work? > In the help screen the command ?ppp [1-6]? is ports per page.

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-24 Thread Wiles, Keith
Sent from my iPhone > On Jul 21, 2016, at 1:34 PM, Neil Horman wrote: > >> On Thu, Jul 21, 2016 at 03:22:45PM +0000, Wiles, Keith wrote: >> >>> On Jul 21, 2016, at 10:06 AM, Neil Horman wrote: >>> >>> On Thu, Jul 21, 2016 at 02:09:19PM +, Wi

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-21 Thread Wiles, Keith
> On Jul 21, 2016, at 10:06 AM, Neil Horman wrote: > > On Thu, Jul 21, 2016 at 02:09:19PM +0000, Wiles, Keith wrote: >> >>> On Jul 21, 2016, at 8:54 AM, Neil Horman wrote: >>> >>> On Wed, Jul 20, 2016 at 10:32:28PM +, Wiles, Keith wrote: >

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-21 Thread Wiles, Keith
> On Jul 21, 2016, at 8:54 AM, Neil Horman wrote: > > On Wed, Jul 20, 2016 at 10:32:28PM +0000, Wiles, Keith wrote: >> >>> On Jul 20, 2016, at 3:16 PM, Neil Horman wrote: >>> >>> On Wed, Jul 20, 2016 at 07:47:32PM +, Wiles, Keith wrote: >

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-20 Thread Wiles, Keith
> On Jul 20, 2016, at 3:16 PM, Neil Horman wrote: > > On Wed, Jul 20, 2016 at 07:47:32PM +0000, Wiles, Keith wrote: >> >>> On Jul 20, 2016, at 12:48 PM, Neil Horman wrote: >>> >>> On Wed, Jul 20, 2016 at 07:40:49PM +0200, Thomas Monjalon wrote: >

[dpdk-dev] [PATCH] validate_abi: build faster by augmenting make with job count

2016-07-20 Thread Wiles, Keith
> On Jul 20, 2016, at 12:48 PM, Neil Horman wrote: > > On Wed, Jul 20, 2016 at 07:40:49PM +0200, Thomas Monjalon wrote: >> 2016-07-20 13:09, Neil Horman: >>> From: Neil Horman >>> >>> John Mcnamara and I were discussing enhacing the validate_abi script to >>> build >>> the dpdk tree faster

[dpdk-dev] [PATCH] rte_delay_us can be replaced with user function

2016-07-19 Thread Wiles, Keith
Hi Jozef, I have two quick comments inline. > On Jul 19, 2016, at 7:42 AM, jozmarti at cisco.com wrote: > > From: Jozef Martiniak > > when running single-core, some drivers tend to call rte_delay_us for a > long time, and that is causing packet drops. > Attached patch introduces 2 new

[dpdk-dev] [PATCH] spinlock:move constructor function out of header

2016-07-14 Thread Wiles, Keith
> On Jul 14, 2016, at 2:59 PM, Thomas Monjalon > wrote: > > Thanks Keith for continuing work. > > 2016-07-14 14:31, Keith Wiles: >> Moving the rte_rtm_init() constructor routine out of the >> header file and into a new rte_spinlock.c for all archs/platforms. >> Having constructor routines in

[dpdk-dev] pktgen wr_cksum error

2016-07-11 Thread Wiles, Keith
the line is using C99 option of ?for (int i = 0; i < X; i++) need to move the declare of the int i out to the function instead of in the for loop. I have fixed his already, but will make a special patch to update this problem when i can. > On Jul 11, 2016, at 9:09 AM, Posadas, Emerson >

[dpdk-dev] [PATCH v3 00/11] Fix build errors related to exported headers

2016-07-08 Thread Wiles, Keith
> On Jul 8, 2016, at 4:56 AM, Yigit, Ferruh wrote: > > On 7/8/2016 9:05 AM, Adrien Mazarguil wrote: >> On Thu, Jul 07, 2016 at 06:33:17PM +0000, Wiles, Keith wrote: >>> >>>> On Jul 7, 2016, at 10:49 AM, Adrien Mazarguil >>> 6wind.com> wrote

[dpdk-dev] [PATCH v3 00/11] Fix build errors related to exported headers

2016-07-07 Thread Wiles, Keith
> On Jul 7, 2016, at 10:49 AM, Adrien Mazarguil > wrote: > > DPDK uses GNU C language extensions in most of its code base. This is fine > for internal source files whose compilation flags are controlled by DPDK, > however user applications that use exported "public" headers may experience >

[dpdk-dev] Low packet generation rate of 526kpps using pktgen-dpdk from inside VM

2016-07-05 Thread Wiles, Keith
are used directly from the application (Pktgen), but in this case it must have its own set of mbufs. Regards Abhishek -Original Message- From: Wiles, Keith [mailto:keith.wi...@intel.com] Sent: Monday, July 04, 2016 8:03 PM To: Abhishek Mahajan ; dev at dpdk.org Cc: Addepalli, Srinivasa R ;

[dpdk-dev] Low packet generation rate of 526kpps using pktgen-dpdk from inside VM

2016-07-04 Thread Wiles, Keith
-Original Message- From: dev on behalf of Abhishek Mahajan Date: Monday, July 4, 2016 at 7:27 AM To: "dev at dpdk.org" Cc: "Addepalli, Srinivasa R" , "Shivastava, RakeshX" Subject: [dpdk-dev] Low packet generation rate of 526kpps using pktgen-dpdk from

[dpdk-dev] e1000: unused variable warnings with clang

2016-07-01 Thread Wiles, Keith
On 7/1/16, 12:16 PM, "dev on behalf of Wiles, Keith" wrote: Well, after make sure I had CCACHE off and having the RTE_SDK variable set correctly, it does appear we have a problem building DPDK/e1000 directory with CLANG on my Ubuntu 16.04 updated as of today. If someone could ple

[dpdk-dev] e1000: unused variable warnings with clang

2016-07-01 Thread Wiles, Keith
Well, after make sure I had CCACHE off and having the RTE_SDK variable set correctly, it does appear we have a problem building DPDK/e1000 directory with CLANG on my Ubuntu 16.04 updated as of today. If someone could please verify the if this is something in my system. Using

[dpdk-dev] e1000: unused variable warnings with clang

2016-07-01 Thread Wiles, Keith
Update: I pulled a new clone and now I do not see that problem, so please ignore. Regards, Keith

[dpdk-dev] e1000: unused variable warnings with clang

2016-07-01 Thread Wiles, Keith
I am seeing unused variable warnings/errors when building the current repo with the clang compiler builds in the e1000/base directory. Does anyone else see this problem? Regards, Keith

[dpdk-dev] [PATCH] mempool: rename functions with confusing names

2016-06-29 Thread Wiles, Keith
On 6/29/16, 11:00 AM, "dev on behalf of Bruce Richardson" wrote: >On Wed, Jun 29, 2016 at 05:55:27PM +0200, Thomas Monjalon wrote: >> 2016-06-29 14:55, Bruce Richardson: >> > The mempool_count and mempool_free_count behaved contrary to what their >> > names suggested. The free_count function

[dpdk-dev] FYI: Using ccache a linux compiler caching tool with DPDK

2016-06-27 Thread Wiles, Keith
FYI, Just to help document the issue here. Using ccache on Linux can improve your compile time from a few minutes to a few seconds depending the use case. On my machine Ubuntu 16.04 up to date as of 06/26/2016 using: sudo apt-get install ccache System Information: Model Name : Intel(R)

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

2016-06-27 Thread Wiles, Keith
On 6/27/16, 11:40 AM, "Richardson, Bruce" wrote: >On Mon, Jun 27, 2016 at 05:29:59PM +0100, Wiles, Keith wrote: >> On 6/27/16, 7:58 AM, "dev on behalf of Wiles, Keith" > dpdk.org on behalf of keith.wiles at intel.com> wrote: >> >> > >&g

[dpdk-dev] [PATCH v2 2/2] fix building with clang-3.8.0 compiler

2016-06-27 Thread Wiles, Keith
On 6/27/16, 7:58 AM, "dev on behalf of Wiles, Keith" wrote: > >On 6/27/16, 3:46 AM, "Richardson, Bruce" wrote: > >>On Sun, Jun 26, 2016 at 10:54:12AM -0500, Keith Wiles wrote: >>> Latest clang compiler 3.8.0 on latest update of Ubuntu >>> cr

[dpdk-dev] [PATCH] mbuf:rearrange mbuf to be more mbuf chain friendly

2016-06-25 Thread Wiles, Keith
On 6/25/16, 10:29 AM, "dev on behalf of Keith Wiles" wrote: >Move the next pointer to the first cacheline of the rte_mbuf structure >and move the offload values to the second cacheline to give better >performance to applications using chained mbufs. > >Enabled by a configuration option

[dpdk-dev] [PATCH] scripts: relax line length check for fixed commit

2016-06-25 Thread Wiles, Keith
On 6/24/16, 4:30 AM, "dev on behalf of Bruce Richardson" wrote: >On Fri, Jun 24, 2016 at 12:44:18AM +0200, Thomas Monjalon wrote: >> It is better to keep the line "Fixes:" longer than 75 characters >> than splitting. >> >> Signed-off-by: Thomas Monjalon > >Definite +1 Yes, +2 for me. >

[dpdk-dev] [PATCH v16 0/3] mempool: add mempool handler feature

2016-06-24 Thread Wiles, Keith
On 6/23/16, 11:22 PM, "dev on behalf of Thomas Monjalon" wrote: >> David Hunt (2): >> mempool: support mempool handler operations >> app/test: test mempool handler >> mbuf: make default mempool ops configurable at build > >Applied, thanks for the nice feature > >I'm sorry David, the

[dpdk-dev] pktgen counters not

2016-06-20 Thread Wiles, Keith
Most of the counters come from the NIC hardware, but I have to do the different sizes in software. If the packet was received then it will get counted as that works in my machine and many others. Unless I broke something in Pktgen it would mean to me that the packets are not getting received by

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-16 Thread Wiles, Keith
On 6/16/16, 3:16 PM, "dev on behalf of Wiles, Keith" wrote: > >On 6/16/16, 3:00 PM, "Take Ceara" wrote: > >>On Thu, Jun 16, 2016 at 9:33 PM, Wiles, Keith >>wrote: >>> On 6/16/16, 1:20 PM, "Take Ceara" wrote: >>> >>&g

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-16 Thread Wiles, Keith
On 6/16/16, 3:00 PM, "Take Ceara" wrote: >On Thu, Jun 16, 2016 at 9:33 PM, Wiles, Keith wrote: >> On 6/16/16, 1:20 PM, "Take Ceara" wrote: >> >>>On Thu, Jun 16, 2016 at 6:59 PM, Wiles, Keith >>>wrote: >>>> >>>>

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-16 Thread Wiles, Keith
On 6/16/16, 11:56 AM, "dev on behalf of Wiles, Keith" wrote: > >On 6/16/16, 11:20 AM, "Take Ceara" wrote: > >>On Thu, Jun 16, 2016 at 5:29 PM, Wiles, Keith >>wrote: >> >>> >>> Right now I do not know what the issue is

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-16 Thread Wiles, Keith
On 6/16/16, 11:20 AM, "Take Ceara" wrote: >On Thu, Jun 16, 2016 at 5:29 PM, Wiles, Keith wrote: > >> >> Right now I do not know what the issue is with the system. Could be too many >> Rx/Tx ring pairs per port and limiting the memory in the NICs, which is

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-16 Thread Wiles, Keith
On 6/16/16, 9:36 AM, "Take Ceara" wrote: >Hi Keith, > >On Tue, Jun 14, 2016 at 3:47 PM, Wiles, Keith wrote: >>>> Normally the limitation is in the hardware, basically how the PCI bus is >>>> connected to the CPUs (or sockets). How the PCI buse

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-14 Thread Wiles, Keith
On 6/14/16, 2:46 AM, "Take Ceara" wrote: >Hi Keith, > >On Mon, Jun 13, 2016 at 9:35 PM, Wiles, Keith wrote: >> >> On 6/13/16, 9:07 AM, "dev on behalf of Take Ceara" > on behalf of dumitru.ceara at gmail.com> wrote: >> >>>Hi, >

[dpdk-dev] Performance hit - NICs on different CPU sockets

2016-06-13 Thread Wiles, Keith
On 6/13/16, 9:07 AM, "dev on behalf of Take Ceara" wrote: >Hi, > >I'm reposting here as I didn't get any answers on the dpdk-users mailing list. > >We're working on a stateful traffic generator (www.warp17.net) using >DPDK and we would like to control two XL710 NICs (one on each socket) >to

[dpdk-dev] [PATCH v3 1/6] mk: sort drivers in static application link list

2016-06-10 Thread Wiles, Keith
On 6/10/16, 1:32 PM, "dev on behalf of Ferruh Yigit" wrote: >From: Thomas Monjalon > >Just a clean up to prepare next patches. One thing you have or will do with these patches is create a set of groups PMD, Core, LIB, ? and sort the items in each group. I was thinking it maybe more useful

[dpdk-dev] [PATCH] mk: generate internal library dependencies from DEPDIRS-y automatically

2016-06-07 Thread Wiles, Keith
On 6/7/16, 9:19 AM, "dev on behalf of Thomas Monjalon" wrote: >2016-06-07 15:07, Bruce Richardson: >> On Tue, Jun 07, 2016 at 03:00:45PM +0200, Thomas Monjalon wrote: >> > 2016-06-07 14:36, Christian Ehrhardt: >> > > But I still struggle to see how to fix the circular dependency between >> > >

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-03 Thread Wiles, Keith
On 6/3/16, 2:18 PM, "Neil Horman" wrote: >On Fri, Jun 03, 2016 at 07:07:50PM +, Wiles, Keith wrote: >> On 6/3/16, 2:00 PM, "dev on behalf of Wiles, Keith" > on behalf of keith.wiles at intel.com> wrote: >> >> >On 6/3/16, 1:52 PM, "A

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-03 Thread Wiles, Keith
On 6/3/16, 2:00 PM, "dev on behalf of Wiles, Keith" wrote: >On 6/3/16, 1:52 PM, "Arnon Warshavsky" mailto:arnon at >qwilt.com>> wrote: > > > >On Fri, Jun 3, 2016 at 9:38 PM, Neil Horman tuxdriver.com<mailto:nhorman at tuxdriver.com>> wr

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-03 Thread Wiles, Keith
On 6/3/16, 1:52 PM, "Arnon Warshavsky" mailto:arnon at qwilt.com>> wrote: On Fri, Jun 3, 2016 at 9:38 PM, Neil Horman mailto:nhorman at tuxdriver.com>> wrote: On Fri, Jun 03, 2016 at 06:29:13PM +, Wiles, Keith wrote: > > On 6/3/16, 12:44 PM, "Neil Horman&qu

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-03 Thread Wiles, Keith
On 6/3/16, 12:44 PM, "Neil Horman" wrote: >On Fri, Jun 03, 2016 at 04:04:14PM +, Wiles, Keith wrote: >> Sorry, I deleted all of the text as it was getting a bit long. >> >> Here are my thoughts as of now, which is a combination of many suggestions I >>

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-03 Thread Wiles, Keith
Regards, Keith On 6/3/16, 11:04 AM, "dev on behalf of Wiles, Keith" wrote: >Sorry, I deleted all of the text as it was getting a bit long. > >Here are my thoughts as of now, which is a combination of many suggestions I >read from everyone?s emails. I hope this is not

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-03 Thread Wiles, Keith
Sorry, I deleted all of the text as it was getting a bit long. Here are my thoughts as of now, which is a combination of many suggestions I read from everyone?s emails. I hope this is not too hard to understand. - Break out the current command line options out of the DPDK common code and move

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-02 Thread Wiles, Keith
On 6/2/16, 12:11 PM, "Neil Horman" wrote: > >1) The definition of a config structure that can be passed to rte_eal_init, >defining the configuration for that running process Having a configuration structure means we have to have an ABI change to that structure anytime we add or remove an

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-02 Thread Wiles, Keith
On 6/2/16, 12:11 PM, "Neil Horman" wrote: >On Thu, Jun 02, 2016 at 01:53:32PM +, Wiles, Keith wrote: >> >> On 6/2/16, 8:19 AM, "Thomas Monjalon" wrote: >> >> >2016-06-02 06:41, Neil Horman: >> >> I'm not sure why you're

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-02 Thread Wiles, Keith
On 6/2/16, 8:19 AM, "Thomas Monjalon" wrote: >2016-06-02 06:41, Neil Horman: >> I'm not sure why you're focusing no selecting a config file format at all. >> Why The reason is I am on now looking at formats is because I have been thinking about this issue for some time and already

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-01 Thread Wiles, Keith
On 6/1/16, 11:18 AM, "Richardson, Bruce" wrote: >On Wed, Jun 01, 2016 at 10:58:41AM -0500, Jay Rolette wrote: >> On Wed, Jun 1, 2016 at 10:00 AM, Wiles, Keith >> wrote: >> >> > Started from the link below, but did not want to highjack the thread. &

[dpdk-dev] [RFC] kernel paramters like DPDK CLI options

2016-06-01 Thread Wiles, Keith
Not to highjack this thread I created another one ? please have a look, thanks. http://dpdk.org/ml/archives/dev/2016-June/040079.html Regards, Keith -Original Message- From: dev on behalf of Thomas Monjalon Date: Wednesday, June 1, 2016 at 9:03 AM To:

[dpdk-dev] [RFC] Yet another option for DPDK options

2016-06-01 Thread Wiles, Keith
Started from the link below, but did not want to highjack the thread. http://dpdk.org/ml/archives/dev/2016-June/040021.html I was thinking about this problem from a user perspective and command line options are very difficult to manage specifically when you have a large number of options as we

[dpdk-dev] [dpdk-users] memory tracker in PMD level

2016-05-31 Thread Wiles, Keith
l stats, but that is about it. > >On Tue, May 31, 2016 at 8:29 PM, SAKTHIVEL ANAND S >wrote: >Ok thanks. > > >On Tue, May 31, 2016 at 8:19 PM, Wiles, Keith wrote: > >From: SAKTHIVEL ANAND S >Date: Tuesday, May 31, 2016 at 9:45 AM >To: Keith Wiles >Subject:

[dpdk-dev] about memzone name size issue

2016-05-31 Thread Wiles, Keith
>Hi all, >I find a issue on link_bonding unit test case. > >When I run model6 test case, will generate core dump error. >I debug it, find the error code in function: >rte_mempool_create_empty(const char *name, unsigned n, unsigned elt_size, >unsigned cache_size, unsigned

[dpdk-dev] rte_pmd_init_all is missing while upgrading from DPDK 1.6 to DPDK 1.7

2016-05-24 Thread Wiles, Keith
The PMD?s are now being inited in the EAL init call for the code and that API is not required anymore. If you are adding a new driver, make sure it is up to date with any changes in the driver code too. Regards, Keith >Hi, > >I have rte_pmd_init_all used in my code. While upgrading from DPDK

[dpdk-dev] [PATCH] Add rte_mempool_free

2016-05-16 Thread Wiles, Keith
>There is no inverse of rte_mempool_create, so this patch adds one. >The typical usage of rte_mempool_create is to create a pool at >initialization time and only to free it upon program exit, so an >rte_mempool_free function at first seems to be of little value. >However, it is very useful as a

[dpdk-dev] [RFC PATCH] tools:new tool for system info CPU, memory and huge pages

2016-05-16 Thread Wiles, Keith
>2016-05-16 14:11, Wiles, Keith: >> >2016-05-13 15:48, Wiles, Keith: >> >> I create this new tool to combine some information and use /sys/devices >> >> instead. What I was hoping was some of you could try this script and see >> >> if

[dpdk-dev] [RFC PATCH] tools:new tool for system info CPU, memory and huge pages

2016-05-16 Thread Wiles, Keith
>2016-05-13 15:48, Wiles, Keith: >> I create this new tool to combine some information and use /sys/devices >> instead. What I was hoping was some of you could try this script and see if >> it works correctly. Also I was hope to find out if this script is useful and >&

[dpdk-dev] [RFC PATCH] tools:new tool for system info CPU, memory and huge pages

2016-05-13 Thread Wiles, Keith
Sorry this one should have been an RFC instead of a patch. I create this new tool to combine some information and use /sys/devices instead. What I was hoping was some of you could try this script and see if it works correctly. Also I was hope to find out if this script is useful and what other

[dpdk-dev] [PATCH] mbuf: add helpers to prefetch mbuf

2016-05-09 Thread Wiles, Keith
>diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h >index 529debb..e3ee0b3 100644 >--- a/lib/librte_mbuf/rte_mbuf.h >+++ b/lib/librte_mbuf/rte_mbuf.h >@@ -842,6 +842,44 @@ struct rte_mbuf { > uint16_t timesync; > } __rte_cache_aligned; > >+/** >+ * Prefetch the first part

[dpdk-dev] Fwd: Rx Error when two pktgen sending packet to each other at the same time on Intel 82599ES 10G

2016-05-06 Thread Wiles, Keith
>Hi, > >I am doing experiments about packet classification algorithm and I found I >always get RX Error when the throughput is too high, so I did the following >experiments. > >There are two PC servers (A and B), each of them has a Intel 82599ES 10G >with two ports(1 and 2). And they are connected

[dpdk-dev] DPDK cpu layout script won't work on non x86

2016-05-06 Thread Wiles, Keith
>The format and layout of /proc/cpuinfo changes on different architectures, >therefore the DPDK cpu_layout.py tool doesn't work. > >Could you please change the tool to use sysfs >/sys/devices/system/cpu/cpuN/topology/ >instead? Then the script would be portable and use the same files as the DPDK

[dpdk-dev] Question about memzone failure problem.

2016-04-22 Thread Wiles, Keith
I was creating a rte_ring and got a message ?RING: Cannot reserve memory? I track it down to me trying create a ring using the same name as another ring. The question is I tracked down the problem and in the eal_common_memzone.c file I noticed the message for the problem was using DEBUG in the

[dpdk-dev] [PATCH] Rate as a decimal number

2016-04-21 Thread Wiles, Keith
>Currently Pktgen does not accept a decimal number for the rate. This patch >makes possible to set a decimal number as a rate. > >Signed-off-by: I?aki Murillo Arroyo >--- > app/cmd-functions.c| 71 +- > app/cmd-functions.h| 38 + > app/lpktgenlib.c |2 +- > app/pktgen-cmds.c

[dpdk-dev] Pktgen rate issue

2016-04-21 Thread Wiles, Keith
Pktgen and I can review the changes. > >Thank you, > > I?aki Murillo > >El 12/04/16 a las 14:55, Wiles, Keith escribi?: >>> Hello >>> >>> I have been using pktgen for a while and I released that there is no >>> possibility to set a rate betwe

[dpdk-dev] perfomance of rte_lpm rule subsystem

2016-04-20 Thread Wiles, Keith
>I just realizied that my patch could be confusing. I want to emphasize that it >contains two completly different and independent set of changes. One is new >rule subsystem and the other is 64 bit next hop. Maybe I should've prepared a >patch with only rule changes, but I wanted to discuss fist

[dpdk-dev] [PATCH v2 1/1] cmdline: add any multi string mode to token string

2016-04-15 Thread Wiles, Keith
>While parsing token string there may be several modes: >- fixed single string >- multi-choice single string >- any single string > >This patch add one more mode - any multi string. > >Signed-off-by: Piotr Azarewicz Does this patch also require updates to the documentation? > Regards, Keith

[dpdk-dev] [PATCH PktGen/PCAP] let multi tx queues with pcap work

2016-04-15 Thread Wiles, Keith
Thank you. >Currently, dpdk-pktgen will panic when assigning multi tx queues with pcap >packet, for example: >./app/app/build/pktgen -c f -n 2 -- -P -m "[0:1-3].0" -s 0:pcap/large.pcap > >This patch fix this bug by assigning qid to memory pool name. > >Signed-off-by: Zhouyi Zhou >--- >

[dpdk-dev] [PATCH 26/36] mempool: introduce a function to create an empty mempool

2016-04-15 Thread Wiles, Keith
> > >On 04/14/2016 05:57 PM, Wiles, Keith wrote: >>> Introduce a new function rte_mempool_create_empty() >>> that allocates a mempool that is not populated. >>> >>> The functions rte_mempool_create() and rte_mempool_xmem_create() >>> now

[dpdk-dev] [PATCH 10/36] mempool: use the list to iterate the mempool elements

2016-04-15 Thread Wiles, Keith
>Hi, > >On 04/14/2016 05:33 PM, Wiles, Keith wrote: >>> >>> static void >>> -txq_mp2mr_mbuf_check(void *arg, void *start, void *end, >>> -uint32_t index __rte_unused) >>> +txq_mp2mr_mbuf_check(struct rte_mempool *mp, voi

[dpdk-dev] [PATCH 26/36] mempool: introduce a function to create an empty mempool

2016-04-14 Thread Wiles, Keith
>Introduce a new function rte_mempool_create_empty() >that allocates a mempool that is not populated. > >The functions rte_mempool_create() and rte_mempool_xmem_create() >now make use of it, making their code much easier to read. >Currently, they are the only users of rte_mempool_create_empty()

[dpdk-dev] [PATCH 10/36] mempool: use the list to iterate the mempool elements

2016-04-14 Thread Wiles, Keith
> > static void >-txq_mp2mr_mbuf_check(void *arg, void *start, void *end, >- uint32_t index __rte_unused) >+txq_mp2mr_mbuf_check(struct rte_mempool *mp, void *arg, void *obj, >+ __rte_unused uint32_t index) I have seen this use of __rte_unused or attributes attached to

[dpdk-dev] [PATCH 02/36] mempool: replace elt_size by total_elt_size

2016-04-14 Thread Wiles, Keith
>In some mempool functions, we use the size of the elements as arguments or in >variables. There is a confusion between the size including or not including >the header and trailer. > >To avoid this confusion: >- update the API documentation >- rename the variables and argument names as "elt_size"

[dpdk-dev] [PATCH 01/36] mempool: fix comments and style

2016-04-14 Thread Wiles, Keith
>No functional change, just fix some comments and styling issues. >Also avoid to duplicate comments between rte_mempool_create() >and rte_mempool_xmem_create(). > >Signed-off-by: Olivier Matz Acked by: Keith Wiles >--- > lib/librte_mempool/rte_mempool.c | 17 +--- >

[dpdk-dev] [PATCH 00/36] mempool: rework memory allocation

2016-04-14 Thread Wiles, Keith
> > >On 04/14/2016 03:50 PM, Wiles, Keith wrote: >>> This series is a rework of mempool. For those who don't want to read >>> all the cover letter, here is a sumary: >>> >>> - it is not possible to allocate large mempools if there is not enoug

[dpdk-dev] [PATCH v5] mempool: reduce rte_mempool structure size

2016-04-14 Thread Wiles, Keith
>From: Keith Wiles > >The rte_mempool structure is changed, which will cause an ABI change >for this structure. Providing backward compat is not reasonable >here as this structure is used in multiple defines/inlines. > >Allow mempool cache support to be dynamic depending on if the >mempool being

[dpdk-dev] [PATCH v5] mempool: reduce rte_mempool structure size

2016-04-14 Thread Wiles, Keith
>From: Keith Wiles > >The rte_mempool structure is changed, which will cause an ABI change >for this structure. Providing backward compat is not reasonable >here as this structure is used in multiple defines/inlines. > >Allow mempool cache support to be dynamic depending on if the >mempool being

[dpdk-dev] memory allocation requirements

2016-04-13 Thread Wiles, Keith
>After looking at the patches for container support, it appears that >some changes are needed in the memory management: >http://thread.gmane.org/gmane.comp.networking.dpdk.devel/32786/focus=32788 > >I think it is time to collect what are the needs and expectations of >the DPDK memory allocator.

[dpdk-dev] Pktgen rate issue

2016-04-12 Thread Wiles, Keith
>Hello > >I have been using pktgen for a while and I released that there is no >possibility to set a rate between two whole numbers. > > I looked up the source code and I found out that the rate is stored in >a uint8_t. So, I made a quick-and-dirty change just to check if it is >possible to

[dpdk-dev] DPDK libraries not compiling from source

2016-04-11 Thread Wiles, Keith
>Hi all, > >I tried compiling DPDK from source using setup.sh script in tools >directory, but I get this error > >[32] Remove KNI module >[33] Remove hugepage mappings > >[34] Exit Script The way I compile DPDK is this way: # cd # export RTE_SDK=`pwd` # export

  1   2   3   4   >