[dpdk-dev] [PATCH v3] af_packet: make the device detachable

2016-02-29 Thread Wojciech Żmuda
Hi Bernard, > Does making rte_pmd_af_packet_devinit local result in an ABI breakage? If someone uses it in their app, they'll be forced to change it. However, as this function is not intentionally public and there is API to create devices that finally calls rte_pmd_af_packet_devinit(), I'm not

[dpdk-dev] [PATCH v4 4/4] virtio: return 1 to tell the upper layer we don't take over this device

2016-02-29 Thread Santosh Shukla
On Fri, Feb 26, 2016 at 7:23 AM, Huawei Xie wrote: > v4 changes: > Rebase as io port map is moved to eal. > Only fall back to PORT IO when there isn't any kernel driver (including Pl. mention that fallback behaviour applicable to x86 arch only.. However this patch fixes one problem in non-x86

[dpdk-dev] [PATCH v2] virtio: Use cpuflag for vector api

2016-02-29 Thread Santosh Shukla
Check cpuflag macro before using vectored api. -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added cpuflag. - Also wrap other vectored freind api ie.. 1) virtqueue_enqueue_recv_refill_simple 2) virtio_rxq_vec_setup - removed VIRTIO_PMD=n from armv7/v8 config. todo: 1) Move

[dpdk-dev] [PATCH] Adding maintainers for Intel QAT PMD

2016-02-29 Thread Jain, Deepak K
On 05/02/16 16:36, Fiona Trahe wrote: > Signed-off-by: Fiona Trahe Acked-by: John Griffin Acked-by: Deepak Kumar Jain

[dpdk-dev] [PATCH] Adding maintainers for Intel QAT PMD

2016-02-29 Thread John Griffin
On 05/02/16 16:36, Fiona Trahe wrote: > Signed-off-by: Fiona Trahe Acked-by: John Griffin

[dpdk-dev] [PATCH 1/1] arm: set CONFIG_RTE_ARCH_STRICT_ALIGN=y for armv7 target

2016-02-29 Thread Jan Viktorin
On Mon, 29 Feb 2016 16:55:38 +0100 Jan Viktorin wrote: > On Mon, 29 Feb 2016 16:14:58 +0100 > Thomas Monjalon wrote: > > > 2015-12-09 16:16, Jan Viktorin: > > > This patch reduces number of warnings from 53 to 40. It removes the usual > > > false > > > positives utilizing unaligned_uint*_t

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Panu Matilainen
On 02/29/2016 05:27 PM, Thomas Monjalon wrote: > 2016-02-29 17:19, Panu Matilainen: >> On 02/29/2016 01:35 PM, Ferruh Yigit wrote: >>> On 2/29/2016 11:06 AM, Thomas Monjalon wrote: Hi, I totally agree with Avi's comments. This topic is really important for the future of DPDK.

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-02-29 Thread Santosh Shukla
On Mon, Feb 29, 2016 at 9:57 AM, Yuanhan Liu wrote: > On Fri, Feb 26, 2016 at 02:21:02PM +0530, Santosh Shukla wrote: >> Check cpuflag macro before using vectored api. >> -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added >> cpuflag. >> - Also wrap other vectored freind api

[dpdk-dev] [PATCH v5 00/11] Add API to get packet type info

2016-02-29 Thread Adrien Mazarguil
On Mon, Feb 29, 2016 at 04:54:19PM +, Ananyev, Konstantin wrote: > > > > -Original Message- > > From: Tan, Jianfeng > > Sent: Friday, February 26, 2016 7:34 AM > > To: dev at dpdk.org > > Cc: Zhang, Helin; Ananyev, Konstantin; nelio.laranjeiro at 6wind.com; > > adrien.mazarguil at

[dpdk-dev] [PATCH v4 0/2] cryptodev API changes

2016-02-29 Thread Trahe, Fiona
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Declan Doherty > Sent: Monday, February 29, 2016 4:52 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v4 0/2] cryptodev API changes > > This patch set separates the symmetric crypto operations from

[dpdk-dev] [PATCH v7] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-29 Thread Thomas Monjalon
2016-02-28 20:44, Huawei Xie: > v7 changes: > rte_pktmbuf_alloc_bulk isn't exported as API, so shouldn't be listed in > version map > > v6 changes: > reflect the changes in release notes and library version map file > revise our duff's code style a bit to make it more readable > > v5 changes:

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Panu Matilainen
On 02/29/2016 01:35 PM, Ferruh Yigit wrote: > On 2/29/2016 11:06 AM, Thomas Monjalon wrote: >> Hi, >> I totally agree with Avi's comments. >> This topic is really important for the future of DPDK. >> So I think we must give some time to continue the discussion >> and have netdev involved in the

[dpdk-dev] VIRTIO interface with DPDK in Guest VM not receiving packets

2016-02-29 Thread Thomas Monjalon
May I kindly ask you to remove this footer from your emails? Thanks > =-=-= > Notice: The information contained in this e-mail > message and/or attachments to it may contain > confidential or privileged information. If you are > not the intended recipient, any dissemination,

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-29 Thread Thomas Monjalon
2016-02-29 12:51, Panu Matilainen: > On 02/24/2016 03:23 PM, Ananyev, Konstantin wrote: > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen > >> On 02/23/2016 07:35 AM, Xie, Huawei wrote: > >>> On 2/22/2016 10:52 PM, Xie, Huawei wrote: > On 2/4/2016 1:24 AM, Olivier

[dpdk-dev] [PATCH v2 0/7] vhost rxtx refactor

2016-02-29 Thread Thomas Monjalon
Hi Yuanhan 2016-02-18 21:49, Yuanhan Liu: > Here is a patchset for refactoring vhost rxtx code, mainly for > improving readability. This series requires to be rebased. And maybe you could check also the series about numa_realloc. Thanks

[dpdk-dev] [PATCH 0/3 v2] ixgbe fixes

2016-02-29 Thread Ananyev, Konstantin
> -Original Message- > From: Iremonger, Bernard > Sent: Friday, February 26, 2016 2:49 PM > To: dev at dpdk.org > Cc: Ananyev, Konstantin; Zhang, Helin; Iremonger, Bernard > Subject: [PATCH 0/3 v2] ixgbe fixes > > This patch set implements the following: > Removes code which was

[dpdk-dev] [PATCH v3 2/4] kcp: add kernel control path kernel module

2016-02-29 Thread Stephen Hemminger
On Fri, 26 Feb 2016 14:10:39 + Ferruh Yigit wrote: > +#define KCP_ERR(args...) printk(KERN_ERR "KCP: " args) > +#define KCP_INFO(args...) printk(KERN_INFO "KCP: " args) > + > +#ifdef RTE_KCP_KO_DEBUG > +#define KCP_DBG(args...) printk(KERN_DEBUG "KCP: " args) > +#else > +#define

[dpdk-dev] [PATCH] vhost: broadcast RARP pkt by injecting it to receiving mbuf array

2016-02-29 Thread Thomas Monjalon
2016-02-22 22:36, Yuanhan Liu: > The wrong mac table lead all the packets to the VM go to the "ovsbr0" > in the end, which ends up with all packets being lost, until the guest > send a ARP quest (or reply) to refresh the mac learning table. > > Jianfeng then came up with a solution I have thought

[dpdk-dev] [PATCH 1/1] arm: set CONFIG_RTE_ARCH_STRICT_ALIGN=y for armv7 target

2016-02-29 Thread Jan Viktorin
On Mon, 29 Feb 2016 16:14:58 +0100 Thomas Monjalon wrote: > 2015-12-09 16:16, Jan Viktorin: > > This patch reduces number of warnings from 53 to 40. It removes the usual > > false > > positives utilizing unaligned_uint*_t data types. > > > > Signed-off-by: Jan Viktorin > > Applied, thanks

[dpdk-dev] [PATCH v5 00/11] Add API to get packet type info

2016-02-29 Thread Ananyev, Konstantin
> -Original Message- > From: Tan, Jianfeng > Sent: Friday, February 26, 2016 7:34 AM > To: dev at dpdk.org > Cc: Zhang, Helin; Ananyev, Konstantin; nelio.laranjeiro at 6wind.com; > adrien.mazarguil at 6wind.com; rahul.lakkireddy at chelsio.com; > Tan, Jianfeng > Subject: [PATCH v5

[dpdk-dev] [PATCH v4 2/2] cryptodev: change burst API to be crypto op oriented

2016-02-29 Thread Declan Doherty
This patch modifies the crypto burst enqueue/dequeue APIs to operate on bursts rte_crypto_op's rather than the current implementation which operates on rte_mbuf bursts, this simplifies the burst processing in the crypto PMDs and the use of crypto operations in general. The changes also continues

[dpdk-dev] [PATCH v4 1/2] cryptodev: API tidy and changes to support future extensions

2016-02-29 Thread Declan Doherty
From: Fiona Trahe This patch splits symmetric specific definitions and functions away from the common crypto APIs to facilitate the future extension and expansion of the cryptodev framework, in order to allow asymmetric crypto operations to be introduced at a later date,

[dpdk-dev] [PATCH v4 0/2] cryptodev API changes

2016-02-29 Thread Declan Doherty
This patch set separates the symmetric crypto operations from generic operations and then modifies the cryptodev burst API to accept bursts of rte_crypto_op rather than rte_mbufs. V4: - Fixes for issues introduced in __rte_crypto_op_raw_bulk_alloc in V3 patcheset. - Typo fix in cached attribute

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Thomas Monjalon
2016-02-29 17:19, Panu Matilainen: > On 02/29/2016 01:35 PM, Ferruh Yigit wrote: > > On 2/29/2016 11:06 AM, Thomas Monjalon wrote: > >> Hi, > >> I totally agree with Avi's comments. > >> This topic is really important for the future of DPDK. > >> So I think we must give some time to continue the

[dpdk-dev] [PATCH 1/1] arm: set CONFIG_RTE_ARCH_STRICT_ALIGN=y for armv7 target

2016-02-29 Thread Thomas Monjalon
2015-12-09 16:16, Jan Viktorin: > This patch reduces number of warnings from 53 to 40. It removes the usual > false > positives utilizing unaligned_uint*_t data types. > > Signed-off-by: Jan Viktorin Applied, thanks Jan, what is the problem with the other ARM alignment warnings? Can they be

[dpdk-dev] [PATCH v2 0/5] add dpdk packet capture support for tcpdump

2016-02-29 Thread Pattan, Reshma
Hi, > -Original Message- > From: Pavel Fedin [mailto:p.fedin at samsung.com] > Sent: Wednesday, February 24, 2016 3:05 PM > To: Pattan, Reshma > Cc: dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH v2 0/5] add dpdk packet capture support for > tcpdump > > Hello! > > > > 2. What if i

[dpdk-dev] [PATCH] log: add missing symbol

2016-02-29 Thread Thomas Monjalon
2016-01-27 10:35, Thomas Monjalon: > 2015-12-16 16:38, Stephen Hemminger: > > rte_get_log_type and rte_get_log_level functions has been avaliable > > for many versions. But they are missing from the shared library map > > and therefore do not get exported correctly. > > > > Signed-off-by: Stephen

[dpdk-dev] [PATCH v3 0/2] cryptodev API changes

2016-02-29 Thread Declan Doherty
On 26/02/16 17:30, Declan Doherty wrote: > This patch set separates the symmetric crypto operations from generic > operations > and then modifies the cryptodev burst API to accept bursts of rte_crypto_op > rather than rte_mbufs. > > V3: > - Addresses V2 comments > - Rebased for head > >

[dpdk-dev] [dpdk-dev,v2] Clean up rte_memcpy.h file

2016-02-29 Thread Wang, Zhihong
> -Original Message- > From: Ravi Kerur [mailto:rkerur at gmail.com] > Sent: Saturday, February 27, 2016 10:06 PM > To: Wang, Zhihong > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev,v2] Clean up rte_memcpy.h file > > > > On Wed, Jan 27, 2016 at 8:18 PM, Zhihong Wang > wrote: > >

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Ferruh Yigit
On 2/29/2016 11:35 AM, Ferruh Yigit wrote: > On 2/29/2016 11:06 AM, Thomas Monjalon wrote: >> Hi, >> I totally agree with Avi's comments. >> This topic is really important for the future of DPDK. >> So I think we must give some time to continue the discussion >> and have netdev involved in the

[dpdk-dev] [PATCH v2 2/2] modify action handlers in test_pipeline and ip_pipeline

2016-02-29 Thread Jasvinder Singh
Changes are made to the ports and table action handlers defined in app/test_pipeline and ip_pipeline sample application. Signed-off-by: Jasvinder Singh Acked-by: Cristian Dumitrescu --- app/test-pipeline/pipeline_acl.c | 3 +- app/test-pipeline/pipeline_hash.c

[dpdk-dev] [PATCH v2 1/2] librte_pipeline: add support for packet redirection at action handlers

2016-02-29 Thread Jasvinder Singh
Currently, there is no mechanism that allows the pipeline ports (in/out) and table action handlers to override the default forwarding decision (as previously configured per input port or in the table entry). Therefore, new pipeline API functions have been added which allows action handlers to

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Ferruh Yigit
On 2/29/2016 11:39 AM, Avi Kivity wrote: > > > On 02/29/2016 01:27 PM, Ferruh Yigit wrote: >> On 2/29/2016 10:58 AM, Avi Kivity wrote: >>> >>> On 02/29/2016 12:43 PM, Ferruh Yigit wrote: On 2/29/2016 9:43 AM, Avi Kivity wrote: > On 02/28/2016 10:16 PM, Ferruh Yigit wrote: >> On

[dpdk-dev] Issue with configuring iproute using netdpcmd and running opendp

2016-02-29 Thread Mariappan Rajendran
Hi, I am trying to configure the iproute using netdpcmd(from dpdk-odp repository), but it is failing. Kindly help to resolve this issue. root at ICSCHELAP1003:/home/hadmin/Mari/dpdk-odp/netdp_cmd# ./build/netdpcmd EAL: Detected lcore 0 as core 0 on socket 0 EAL: Detected lcore 1 as core 1 on

[dpdk-dev] [PATCH] mk: add makefile extention support

2016-02-29 Thread Wiles, Keith
>2016-02-28 21:47, Wiles, Keith: >> >Hi, >> > >> >2016-02-09 11:35, Keith Wiles: >> >> Adding support to the build system to allow for Makefile.XXX >> >> extention to a subtree, which already has Makefiles. These >> >> Makefiles could be from the autotools and others places. Using >> >> the

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Avi Kivity
On 02/29/2016 01:27 PM, Ferruh Yigit wrote: > On 2/29/2016 10:58 AM, Avi Kivity wrote: >> >> On 02/29/2016 12:43 PM, Ferruh Yigit wrote: >>> On 2/29/2016 9:43 AM, Avi Kivity wrote: On 02/28/2016 10:16 PM, Ferruh Yigit wrote: > On 2/28/2016 3:34 PM, Avi Kivity wrote: >> On 01/27/2016

[dpdk-dev] [PATCH v5 01/11] ethdev: add API to query packet type filling info

2016-02-29 Thread Panu Matilainen
On 02/26/2016 09:34 AM, Jianfeng Tan wrote: > Add a new API rte_eth_dev_get_ptype_info to query whether/what packet > type can be filled by given pmd rx burst function. > > Signed-off-by: Jianfeng Tan > --- > lib/librte_ether/rte_ethdev.c | 26 ++ >

[dpdk-dev] [PATCH v2] doc/nic: add ixgbe statistics on read frequency

2016-02-29 Thread Kerlin, MarcinX
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Harry van Haaren > Sent: Monday, February 29, 2016 1:17 PM > To: Mcnamara, John > Cc: dev at dpdk.org > Subject: [dpdk-dev] [PATCH v2] doc/nic: add ixgbe statistics on read > frequency > > This patch adds a

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Avi Kivity
On 02/29/2016 12:43 PM, Ferruh Yigit wrote: > On 2/29/2016 9:43 AM, Avi Kivity wrote: >> On 02/28/2016 10:16 PM, Ferruh Yigit wrote: >>> On 2/28/2016 3:34 PM, Avi Kivity wrote: On 01/27/2016 06:24 PM, Ferruh Yigit wrote: > This kernel module is based on KNI module, but this one is

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-29 Thread Panu Matilainen
On 02/24/2016 03:23 PM, Ananyev, Konstantin wrote: > Hi Panu, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen >> Sent: Wednesday, February 24, 2016 12:12 PM >> To: Xie, Huawei; Olivier MATZ; dev at dpdk.org >> Cc: dprovan at bivio.net >>

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-02-29 Thread Yuanhan Liu
On Fri, Feb 26, 2016 at 02:21:02PM +0530, Santosh Shukla wrote: > Check cpuflag macro before using vectored api. > -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added > cpuflag. > - Also wrap other vectored freind api ie.. > 1) virtqueue_enqueue_recv_refill_simple > 2)

[dpdk-dev] [PATCH v2] doc/nic: add ixgbe statistics on read frequency

2016-02-29 Thread Harry van Haaren
This patch adds a note to the ixgbe PMD guide, stating the minimum time that statistics must be polled from the hardware in order to avoid register values becoming saturated and "sticking" to the max value. Reported-by: Jerry Zhang Tested-by: Marcin Kerlin Signed-off-by: Harry van Haaren ---

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Stephen Hemminger
On Wed, 27 Jan 2016 16:24:07 + Ferruh Yigit wrote: > +static int > +kcp_ioctl_release(unsigned int ioctl_num, unsigned long ioctl_param) > +{ > + int ret = -EINVAL; > + struct kcp_dev *dev; > + struct kcp_dev *n; > + char name[RTE_KCP_NAMESIZE]; > + unsigned int instance

[dpdk-dev] ACL memory allocation failures

2016-02-29 Thread Rapelly, Varun
Thanks Konstantin. Few more questions in line: > > Previous allocation error was coming with 1024 huge pages of 2 MB size. > > After increasing the huge pages to 2048, I was able to add another > ~140 rules [IPv4 rule data--> with src, dst IP address & port, next header ] > more, ie., 950

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Thomas Monjalon
Hi, I totally agree with Avi's comments. This topic is really important for the future of DPDK. So I think we must give some time to continue the discussion and have netdev involved in the choices done. As a consequence, these series should not be merged in the release 16.04. Thanks for continuing

[dpdk-dev] [PATCH v3] examples/l3fwd: exact-match rework

2016-02-29 Thread Thomas Monjalon
2016-02-29 11:33, Tomasz Kulasek: > Current implementation of Exact-Match uses different execution path than > for LPM. Unifying them allows to reuse big part of LPM code and sightly > increase performance of Exact-Match. > > Main changes: > - > * Packet classification stage is

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Avi Kivity
On 02/28/2016 10:16 PM, Ferruh Yigit wrote: > On 2/28/2016 3:34 PM, Avi Kivity wrote: >> On 01/27/2016 06:24 PM, Ferruh Yigit wrote: >>> This kernel module is based on KNI module, but this one is stripped >>> version of it and only for control messages, no data transfer >>> functionality provided.

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Ferruh Yigit
On 2/29/2016 11:06 AM, Thomas Monjalon wrote: > Hi, > I totally agree with Avi's comments. > This topic is really important for the future of DPDK. > So I think we must give some time to continue the discussion > and have netdev involved in the choices done. > As a consequence, these series should

[dpdk-dev] [PATCH v3] examples/l3fwd: exact-match rework

2016-02-29 Thread Tomasz Kulasek
Current implementation of Exact-Match uses different execution path than for LPM. Unifying them allows to reuse big part of LPM code and sightly increase performance of Exact-Match. Main changes: - * Packet classification stage is separated from the rest of path for both LPM and EM.

[dpdk-dev] [PATCH] doc/nic: add ixgbe statistics on read frequency

2016-02-29 Thread Harry van Haaren
This patch adds a note to the ixgbe PMD guide, stating the minimum time that statistics must be polled from the hardware in order to avoid register values becoming saturated and "sticking" to the max value. Signed-off-by: Harry van Haaren --- doc/guides/nics/ixgbe.rst | 24

[dpdk-dev] [PATCH v6] cfgfile: support looking up sections by index

2016-02-29 Thread Thomas Monjalon
> > This is useful when sections have duplicate names. > > > > Signed-off-by: Rich Lane > > --- > > v5->v6: > > - Reordered sectionname argument in comment. > > Acked-by: Cristian Dumitrescu > > Thanks, Rich! Applied, thanks

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Ferruh Yigit
On 2/29/2016 10:58 AM, Avi Kivity wrote: > > > On 02/29/2016 12:43 PM, Ferruh Yigit wrote: >> On 2/29/2016 9:43 AM, Avi Kivity wrote: >>> On 02/28/2016 10:16 PM, Ferruh Yigit wrote: On 2/28/2016 3:34 PM, Avi Kivity wrote: > On 01/27/2016 06:24 PM, Ferruh Yigit wrote: >> This kernel

[dpdk-dev] [PATCH v3 1/1] jobstats: added function abort for job

2016-02-29 Thread Thomas Monjalon
2016-02-16 13:19, Zhang, Roy Fan: > > On 12/02/2016 16:04, Marcin Kerlin wrote: > > This patch adds new function rte_jobstats_abort. It marks *job* as finished > > and > > time of this work will be add to management time instead of execution time. > > This > > function should be used instead of

[dpdk-dev] [PATCH 1/6] mempool: add external mempool manager support

2016-02-29 Thread Hunt, David
On 2/19/2016 1:30 PM, Olivier MATZ wrote: > Hi David, > > On 02/16/2016 03:48 PM, David Hunt wrote: >> Adds the new rte_mempool_create_ext api and callback mechanism for >> external mempool handlers >> >> Modifies the existing rte_mempool_create to set up the handler_idx to >> the relevant

[dpdk-dev] [PATCH 2/6] mempool: add stack (lifo) based external mempool handler

2016-02-29 Thread Hunt, David
On 2/19/2016 1:31 PM, Olivier MATZ wrote: > Hi David, > > On 02/16/2016 03:48 PM, David Hunt wrote: >> adds a simple stack based mempool handler >> >> Signed-off-by: David Hunt >> --- >> lib/librte_mempool/Makefile| 2 +- >> lib/librte_mempool/rte_mempool.c | 4 +- >>

[dpdk-dev] [PATCH 0/6] external mempool manager

2016-02-29 Thread Hunt, David
On 2/19/2016 1:25 PM, Olivier MATZ wrote: > Hi, > > On 02/16/2016 03:48 PM, David Hunt wrote: >> Hi list. >> >> Here's the v2 version of a proposed patch for an external mempool manager > Just to notice the "v2" is missing in the title, it would help > to have it for next versions of the series.

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Ferruh Yigit
On 2/29/2016 9:43 AM, Avi Kivity wrote: > On 02/28/2016 10:16 PM, Ferruh Yigit wrote: >> On 2/28/2016 3:34 PM, Avi Kivity wrote: >>> On 01/27/2016 06:24 PM, Ferruh Yigit wrote: This kernel module is based on KNI module, but this one is stripped version of it and only for control

[dpdk-dev] [PATCH v2] I217 and I218 changes

2016-02-29 Thread Ravi Kerur
v2: Incorporate Wenzhou's comments Compiled and tested (via testpmd) on Ubuntu 14.04 on target x86_64-native-linuxapp-gcc Compiled for target x86_64-native-linuxapp-clang v1: Modified driver and eal code to recognize and support I217 and

[dpdk-dev] [PATCH v3 3/3] keepalive: add rte_keepalive_xstats_get()

2016-02-29 Thread Thomas Monjalon
Hi, There is a compilation error for 32-bit arch: 2016-02-22 11:26, Harry van Haaren: > + for (i = 0; i < nstats; i++) > + printf("%s\t%lu\n", xstats[i].name, xstats[i].value); examples/l2fwd-keepalive/main.c:206:10: error: format ?%lu? expects argument of type ?long

[dpdk-dev] [PATCH v4 2/4] eal: set kdrv to RTE_KDRV_NONE if kernel driver isn't managing the device

2016-02-29 Thread David Marchand
On Mon, Feb 29, 2016 at 10:00 AM, Xie, Huawei wrote: > On 2/29/2016 4:47 PM, David Marchand wrote: >> On Fri, Feb 26, 2016 at 2:53 AM, Huawei Xie wrote: >>> v4 changes: >>> reword the commit message. When we mention kernel driver, emphasizes >>> that it includes UIO/VFIO. >> Annotations should

[dpdk-dev] [PATCH v4 3/4] eal: call pci_ioport_map when kernel driver isn't managing the device

2016-02-29 Thread David Marchand
On Fri, Feb 26, 2016 at 2:53 AM, Huawei Xie wrote: > Call rte_eal_pci_ioport_map only if driver type is RTE_KDRV_NONE, which > means kernel driver(including UIO/VFIO) isn't managing the device. I suppose you meant 'Call pci_ioport_map when the pci device is not bound to a kernel driver'. If you

[dpdk-dev] [PATCH] eal: make resource initialization more robust

2016-02-29 Thread Tan, Jianfeng
Hi Thomas, On 2/29/2016 5:12 AM, Thomas Monjalon wrote: > Hi, > > 2016-01-29 19:22, Jianfeng Tan: >> Current issue: DPDK is not that friendly to container environment, which >> caused by that it pre-alloc resource like cores and hugepages. But there >> are this or that resource limitations, for

[dpdk-dev] [PATCH v4 1/4] eal: make the comment more accurate

2016-02-29 Thread David Marchand
On Fri, Feb 26, 2016 at 2:53 AM, Huawei Xie wrote: > positive return of rte_eal_pci_probe_one_driver means the driver doesn't > support > the device. > > Signed-off-by: Huawei Xie > Acked-by: Yuanhan Liu Acked-by: David Marchand -- David Marchand

[dpdk-dev] [PATCH v4 2/4] eal: set kdrv to RTE_KDRV_NONE if kernel driver isn't managing the device

2016-02-29 Thread David Marchand
On Fri, Feb 26, 2016 at 2:53 AM, Huawei Xie wrote: > v4 changes: > reword the commit message. When we mention kernel driver, emphasizes > that it includes UIO/VFIO. Annotations should not be part of the commitlog itself. > Use RTE_KDRV_NONE to indicate that kernel driver(including UIO/VFIO) >

[dpdk-dev] [PATCH v3 0/4] Use common Linux tools to control DPDK ports

2016-02-29 Thread Remy Horton
On 26/02/2016 14:10, Ferruh Yigit wrote: > Ferruh Yigit (4): >lib/librte_ethtool: move librte_ethtool form examples to lib folder >kcp: add kernel control path kernel module >rte_ctrl_if: add control interface library >examples/ethtool: add control interface support to the

[dpdk-dev] [PATCH] mk: add makefile extention support

2016-02-29 Thread Thomas Monjalon
2016-02-28 21:47, Wiles, Keith: > >Hi, > > > >2016-02-09 11:35, Keith Wiles: > >> Adding support to the build system to allow for Makefile.XXX > >> extention to a subtree, which already has Makefiles. These > >> Makefiles could be from the autotools and others places. Using > >> the Makefile

[dpdk-dev] [PATCH v4 2/4] eal: set kdrv to RTE_KDRV_NONE if kernel driver isn't managing the device

2016-02-29 Thread Xie, Huawei
On 2/29/2016 4:47 PM, David Marchand wrote: > On Fri, Feb 26, 2016 at 2:53 AM, Huawei Xie wrote: >> v4 changes: >> reword the commit message. When we mention kernel driver, emphasizes >> that it includes UIO/VFIO. > Annotations should not be part of the commitlog itself. Do you mean that

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Jay Rolette
On Mon, Feb 29, 2016 at 5:06 AM, Thomas Monjalon wrote: > Hi, > I totally agree with Avi's comments. > This topic is really important for the future of DPDK. > So I think we must give some time to continue the discussion > and have netdev involved in the choices done. > As a consequence, these

[dpdk-dev] [PATCH] i40e: remove redundant compiler warning disablers

2016-02-29 Thread Pei, Yulong
This patch caused build error with i686-native-linuxapp-gcc (gcc version is 4.8.3) > > i686-native-linuxapp-gcc compile error info: > > > > INSTALL-LIB librte_pmd_vmxnet3_uio.a > > /root/dpdk/drivers/net/i40e/base/i40e_common.c: In function > > ?i40e_aq_set_lldp_mib?: > >

[dpdk-dev] [PATCH v1] virtio: Use cpuflag for vector api

2016-02-29 Thread Xie, Huawei
On 2/29/2016 12:26 PM, Yuanhan Liu wrote: > On Fri, Feb 26, 2016 at 02:21:02PM +0530, Santosh Shukla wrote: >> Check cpuflag macro before using vectored api. >> -virtio_recv_pkts_vec() uses _sse3__ simd instruction for now so added >> cpuflag. >> - Also wrap other vectored freind api ie.. >> 1)

[dpdk-dev] ACL memory allocation failures

2016-02-29 Thread Rapelly, Varun
> > Thanks Konstantin. > > Previous allocation error was coming with 1024 huge pages of 2 MB size. > > After increasing the huge pages to 2048, I was able to add another > ~140 rules [IPv4 rule data--> with src, dst IP address & port, next header ] > more, ie., 950 rules were added. That's

[dpdk-dev] [PATCH v4 2/4] eal: set kdrv to RTE_KDRV_NONE if kernel driver isn't managing the device

2016-02-29 Thread Xie, Huawei
? 2/27/2016 1:47 AM, Xie, Huawei ??: > Use RTE_KDRV_NONE to indicate that kernel driver(including UIO/VFIO) > isn't manipulating the device. Thomas, could you kindly help change manipulating->managing? I have changed others per Panu's suggestion but missed this.

[dpdk-dev] [PATCH v3 00/18] fm10k: update shared code

2016-02-29 Thread Ding, HengX
Tested-by: Heng Ding -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Wang Xiao W Sent: Friday, February 19, 2016 7:07 PM To: Chen, Jing D Cc: dev at dpdk.org Subject: [dpdk-dev] [PATCH v3 00/18] fm10k: update shared code v3: * Fixed checkpatch.pl warning about

[dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding

2016-02-29 Thread Wang, Xiao W
> -Original Message- > From: Richardson, Bruce > Sent: Saturday, February 27, 2016 12:33 AM > To: David Marchand > Cc: Wang, Xiao W ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 1/3] fm10k: enable FTAG based forwarding > > On Fri, Feb 26, 2016 at 04:00:49PM +0100, David

[dpdk-dev] [PATCH v2] doc: Malicious Driver Detection not supported by ixgbe

2016-02-29 Thread Lu, Wenzhuo
Hi Bruce, > -Original Message- > From: Richardson, Bruce > Sent: Friday, February 26, 2016 10:41 PM > To: Lu, Wenzhuo > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] doc: Malicious Driver Detection not > supported by ixgbe > > On Fri, Feb 26, 2016 at 12:48:37PM +0800, Wenzhuo