[dpdk-dev] [PATCHv8 0/6] Implement pmd hardware support exports

2016-07-06 Thread Thomas Monjalon
2016-06-30 08:45, Remy Horton: > On 17/06/2016 19:46, Neil Horman wrote: > > Hey all- > > So heres attempt number 2 at a method for exporting PMD hardware support > > information. As we discussed previously, the consensus seems to be that pmd > > information should be: > [..] > > Signed-off-by

[dpdk-dev] Question on rte_ring_dequeue_bulk()

2016-07-06 Thread Yeddula, Avinash
I apologies, I should have checked it more carefully. * if behavior = RTE_RING_QUEUE_VARIABLE * - n: Actual number of objects dequeued. This function returns the "n" I was looking for. Thanks -Avinash From: Yeddula, Avinash Sent: Wednesday, July 06, 2016 6:39 PM To: dev at dpdk.org Subject:

[dpdk-dev] Question on rte_ring_dequeue_bulk()

2016-07-06 Thread Yeddula, Avinash
Hi All, * @param r * A pointer to the ring structure. * @param obj_table * A pointer to a table of void * pointers (objects) that will be filled. * @param n * The number of objects to dequeue from the ring to the obj_table. By looking at the code I understood that, if the ring has less than

[dpdk-dev] [PATCH v2] Memory leak when adding/removing vhost_user ports

2016-07-06 Thread Yuanhan Liu
On Wed, Jul 06, 2016 at 02:24:57PM +0200, Christian Ehrhardt wrote: > Hi, > while checking for dpdk 16.07 what backports are accepted in the meantime so I > can drop them I found this particular discussion has been silently forgotten > by > all of us. Not really. As later I found that my patch wa

[dpdk-dev] [PATCH] vhost: fix segfault on bad descriptor address.

2016-07-06 Thread Yuanhan Liu
On Wed, Jul 06, 2016 at 02:19:12PM +0300, Ilya Maximets wrote: > On 01.07.2016 10:35, Yuanhan Liu wrote: > > Hi, > > > > Sorry for the long delay. > > > > On Fri, May 20, 2016 at 03:50:04PM +0300, Ilya Maximets wrote: > >> In current implementation guest application can reinitialize vrings > >> b

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

2016-07-06 Thread Thomas Monjalon
2016-07-05 12:44, Adrien Mazarguil: > This patchset affects exported headers only, compilation problems are > addressed as follows: I prefer postponing the headers changes to the very first days of the 16.11 cycle.

[dpdk-dev] [PATCH v2 1/2] doc: live migration of VM with Virtio and VF

2016-07-06 Thread Thomas Monjalon
2016-07-06 17:01, Bernard Iremonger: > @@ -45,3 +45,4 @@ DPDK documentation > faq/index > rel_notes/index > contributing/index > + how_to/index 2 personal opinions: - I feel a directory "howto" (without underscore) would be nice. - The position in the index could be just after the fa

[dpdk-dev] [PATCH v2 0/2] doc: live migration procedure

2016-07-06 Thread Thomas Monjalon
Hi Bernard, 2016-07-06 17:01, Bernard Iremonger: > This pachset set describes the procedure to Live migrate > a VM with Virtio and VF PMD's using the bonding PMD. [...] > doc/guides/how_to/img/lm_overview.svg | 666 +++ > doc/guides/how_to/index.rst|

[dpdk-dev] [PATCH v9 0/7] export PMD infos

2016-07-06 Thread Thomas Monjalon
2016-07-05 16:08, Neil Horman: > On Mon, Jul 04, 2016 at 10:10:37PM +0200, Thomas Monjalon wrote: > > 2016-07-04 12:41, Neil Horman: > > > On Mon, Jul 04, 2016 at 03:10:14PM +0200, Thomas Monjalon wrote: > > > > Hi Neil, > > > > > > > > I don't really understand why you don't accept I contribute t

[dpdk-dev] [PATCH] net/enic: remove useless assert macro

2016-07-06 Thread Thomas Monjalon
The macro ENIC_ASSERT does the same thing as RTE_ASSERT, thus it can be removed. Signed-off-by: Thomas Monjalon --- drivers/net/enic/enic.h | 12 drivers/net/enic/enic_rxtx.c | 2 +- 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/drivers/net/enic/enic.h b/drive

[dpdk-dev] [PATCH] app/test: remove useless clock estimation

2016-07-06 Thread Thomas Monjalon
The function get_machclk_freq() in RED test can be replaced by rte_get_timer_hz() which already estimates the clock frequency at EAL init. Signed-off-by: Thomas Monjalon --- app/test/test_red.c | 42 +- 1 file changed, 1 insertion(+), 41 deletions(-) diff

[dpdk-dev] [PATCH v3] mempool: replace c memcpy code semantics with optimized rte_memcpy

2016-07-06 Thread Ferruh Yigit
On 7/5/2016 3:09 PM, Ferruh Yigit wrote: > On 7/5/2016 2:13 PM, Jerin Jacob wrote: >> On Tue, Jul 05, 2016 at 07:32:46PM +0800, Yuanhan Liu wrote: >>> On Tue, Jul 05, 2016 at 09:43:03AM +0100, Ferruh Yigit wrote: On 6/30/2016 6:28 PM, Thomas Monjalon wrote: > 2016-06-30 17:46, Jerin Jacob:

[dpdk-dev] [PATCH v2 2/2] doc: add live migration overview image

2016-07-06 Thread Bernard Iremonger
This patch adds an image of the Live Migration test configuration. Signed-off-by: Bernard Iremonger --- doc/guides/how_to/img/lm_overview.svg | 666 + doc/guides/how_to/lm_bond_virtio_sriov.rst | 3 + 2 files changed, 669 insertions(+) create mode 100644 doc/g

[dpdk-dev] [PATCH v2 1/2] doc: live migration of VM with Virtio and VF

2016-07-06 Thread Bernard Iremonger
This patch describes the procedure to be be followed to perform Live Migration of a VM with Virtio and VF PMD's using the bonding PMD. It includes sample host and VM scripts used in the procedure, and a sample switch configuration. Signed-off-by: Bernard Iremonger --- doc/guides/how_to/index.rs

[dpdk-dev] [PATCH v2 0/2] doc: live migration procedure

2016-07-06 Thread Bernard Iremonger
This pachset set describes the procedure to Live migrate a VM with Virtio and VF PMD's using the bonding PMD. Changes in v2: change primary port before removing slave port remove unused variables from QEMU scripts identify NIC's in bridge setup scripts Bernard Iremonger (2): doc: live migration

[dpdk-dev] [PATCH 2/2] doc: add pdump shared library version to release notes

2016-07-06 Thread Reshma Pattan
Added missing librte_pdump shared library version to release notes. Signed-off-by: Reshma Pattan --- doc/guides/rel_notes/release_16_07.rst | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/guides/rel_notes/release_16_07.rst b/doc/guides/rel_notes/release_16_07.rst index 569f562..d758cd5

[dpdk-dev] [PATCH 1/2] doc: add pcap dependency note to pdump tool guide

2016-07-06 Thread Reshma Pattan
Added a missing note about dependencies on libpcap and CONFIG_RTE_LIBRTE_PMD_PCAP flag that pdump tool has. Signed-off-by: Reshma Pattan --- doc/guides/sample_app_ug/pdump.rst | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/doc/guides/sample_app_ug/pdump.rst b/do

[dpdk-dev] [PATCH 0/2] update packet capture framework doc

2016-07-06 Thread Reshma Pattan
This patch set contains 1) Release notes update for adding pdump library version to shared library section. 2) Added missing note about pdump tool dependency on libpcap and CONFIG_RTE_LIBRTE_PMD_PCAP. Reshma Pattan (2): doc: add pcap dependency note to pdump tool guide doc: add pdump shared

[dpdk-dev] [PATCH 2/2] doc: add pdump shared library version to release notes

2016-07-06 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Reshma Pattan > Sent: Wednesday, July 6, 2016 4:58 PM > To: dev at dpdk.org > Cc: Pattan, Reshma > Subject: [dpdk-dev] [PATCH 2/2] doc: add pdump shared library version to > release notes > > Added missing libr

[dpdk-dev] [PATCH 1/2] doc: add pcap dependency note to pdump tool guide

2016-07-06 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Reshma Pattan > Sent: Wednesday, July 6, 2016 4:58 PM > To: dev at dpdk.org > Cc: Pattan, Reshma > Subject: [dpdk-dev] [PATCH 1/2] doc: add pcap dependency note to pdump > tool guide > > Added a missing note ab

[dpdk-dev] [PATCH 3/3] examples: fix dependencies on hash library

2016-07-06 Thread Thomas Monjalon
The multi_process example do not need rte_hash. But these examples cannot compile if rte_hash is not available: - ipsec-secgw (was already protected - no change) - ipv4_multicast - l3fwd-power - l3fwd-vf - tep_termination - ip_pipeline The ip_pipeline example is not disabled because its dependenci

[dpdk-dev] [PATCH 2/3] table: remove unneeded dependency on hash library

2016-07-06 Thread Thomas Monjalon
Fixes: 8aa327214ceb ("table: hash") Signed-off-by: Thomas Monjalon --- lib/librte_table/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/librte_table/Makefile b/lib/librte_table/Makefile index 7f02af3..7a8a3f3 100644 --- a/lib/librte_table/Makefile +++ b/lib/librte_table/Makefile

[dpdk-dev] [PATCH 1/3] net/bnx2x: remove unneeded dependency on hash library

2016-07-06 Thread Thomas Monjalon
Fixes: 9fb557035d90 ("bnx2x: enable PMD build") Signed-off-by: Thomas Monjalon --- drivers/net/bnx2x/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/bnx2x/Makefile b/drivers/net/bnx2x/Makefile index c2ddd8d..ab69680 100644 --- a/drivers/net/bnx2x/Makefile

[dpdk-dev] [PATCH 0/3] fix dependencies on librte_hash

2016-07-06 Thread Thomas Monjalon
Thomas Monjalon (3): net/bnx2x: remove unneeded dependency on hash library table: remove unneeded dependency on hash library examples: fix dependencies on hash library drivers/net/bnx2x/Makefile | 2 +- examples/Makefile|

[dpdk-dev] [PATCH v6 0/4] support reset of VF link

2016-07-06 Thread Luca Boccassi
On Wed, 2016-07-06 at 00:45 +, Lu, Wenzhuo wrote: > Hi Luca, > > > -Original Message- > > From: Luca Boccassi [mailto:lboccass at Brocade.com] > > Sent: Tuesday, July 5, 2016 5:53 PM > > To: Lu, Wenzhuo > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v6 0/4] support reset of

[dpdk-dev] [PATCH] net/enic: fix checking for sufficient resources

2016-07-06 Thread Nelson Escobar
The enic PMD was using the same variables in the enic structure to track two different things. Initially rq_count, wq_count, cq_count, and intr_count were set to the values obtained from the VIC adapters as the maximum resources allocated on the VIC, then in enic_set_vnic_res(), they were set to t

[dpdk-dev] [PATCH v2] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
From: Nelio Laranjeiro A driver patch under net should start with "net/" or if a patch touch multiple drivers, it should only start with "net:". The same apply for crypto. A patch touching all drivers (net + crypto) should start with "drivers:". Longer prefixes like "net/mlx:" (for mlx4/mlx5) or

[dpdk-dev] about rx checksum flags

2016-07-06 Thread Olivier MATZ
Hi Sugesh On 07/06/2016 02:52 PM, Chandran, Sugesh wrote: > Hi Olivier, > > Just to confirm , is this rx checksum patch already submitted in the DPDK ML? > We would like to use these flags to speed up the tunneling in OVS. No it is not submitted yet. I plan to send it in the coming days, it will

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
2016-07-06 13:59, N?lio Laranjeiro: > > +# check headline prefix when touching only drivers/, e.g. net/ > > +bad=$(for commit in $commits ; do > > + headline=$(git log --format='%s' -1 $commit) > > + files=$(git diff-tree --no-commit-id --name-only -r $commit) > > + [ -z "$(echo "$files" | gr

[dpdk-dev] [PATCH 09/18] mbuf: support Mpls in software packet type parser

2016-07-06 Thread Liang, Cunming
Hi Olivier, On 7/5/2016 11:41 PM, Olivier Matz wrote: > Add a new RTE_PTYPE_L2_ETHER_MPLS packet type, and its support in > rte_pktmbuf_get_ptype(). > > Signed-off-by: Didier Pallard > Signed-off-by: Olivier Matz > --- > lib/librte_mbuf/rte_mbuf_ptype.c | 25 + > lib/l

[dpdk-dev] [PATCH v3 1/3] kasumi: add new KASUMI PMD

2016-07-06 Thread Thomas Monjalon
2016-07-06 12:26, Ferruh Yigit: > On 6/20/2016 3:40 PM, Pablo de Lara wrote: > > enum rte_cryptodev_type { > > RTE_CRYPTODEV_NULL_PMD = 1, /**< Null crypto PMD */ > > RTE_CRYPTODEV_AESNI_GCM_PMD,/**< AES-NI GCM PMD */ > > RTE_CRYPTODEV_AESNI_MB_PMD, /**< AES-NI multi buffer

[dpdk-dev] [PATCH] cryptodev: move new cryptodev type to bottom of enum

2016-07-06 Thread Pablo de Lara
New cryptodev type for the new KASUMI PMD was added in the cryptodev type enum, but not at the end of it, causing an ABI breakage. Fixes: 2773c86d061a ("crypto/kasumi: add driver for KASUMI library") Signed-off-by: Pablo de Lara Reported-by: Ferruh Yigit --- lib/librte_cryptodev/rte_cryptodev.

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-06 Thread Liang, Cunming
Hi Olivier, On 7/5/2016 11:41 PM, Olivier Matz wrote: > Introduce the function rte_pktmbuf_get_ptype() that parses a > mbuf and returns its packet type. For now, the following packet > types are parsed: > L2: Ether > L3: IPv4, IPv6 > L4: TCP, UDP, SCTP > > The goal here is to provide a

[dpdk-dev] [PATCH v2] Memory leak when adding/removing vhost_user ports

2016-07-06 Thread Christian Ehrhardt
Now let us really get to this old discussion. Once again - sorry for mixing this up :-/ Found too much at once today. ## Ok - refocus ## Back then in http://dpdk.org/dev/patchwork/patch/12103/ http://dpdk.org/dev/patchwork/patch/12118/ We came up with a nice solution for the leak. But it was nev

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
2016-07-06 12:09, Bruce Richardson: > On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > > From: Nelio Laranjeiro > > +# check headline prefix when touching only drivers/, e.g. net/ > > +bad=$(for commit in $commits ; do > > + headline=$(git log --format='%s' -1 $commit) > > +

[dpdk-dev] [PATCH v2] Memory leak when adding/removing vhost_user ports

2016-07-06 Thread Christian Ehrhardt
This is the series this really belongs to http://dpdk.org/dev/patchwork/patch/11581/ Message ID <1458292380-9258-1-git-send-email-patrik.r.andersson at ericsson.com > Should I wait for a v2 to point the patch at the right ID or do you prefer a fixed resubmit right away? Christian Ehrhardt Softwar

[dpdk-dev] [PATCH v2] Memory leak when adding/removing vhost_user ports

2016-07-06 Thread Christian Ehrhardt
Sorry, please ignore the two links, the cover letter has - they belong to a different issue I have to bring up again. Everything else still applies. Christian Ehrhardt Software Engineer, Ubuntu Server Canonical Ltd On Wed, Jul 6, 2016 at 2:24 PM, Christian Ehrhardt < christian.ehrhardt at canonic

[dpdk-dev] [PATCH v2] vhost_user: avoid crash when exeeding file descriptors

2016-07-06 Thread Christian Ehrhardt
*update in v2* - refreshing for DPDK 16.07 - Close fd on vserver->listenfd as suggested in discussion Original From: From: Patrik Andersson Protect against DPDK crash when allocation of listen fd >= 1023. For events on fd:s >1023, the current implementation will trigger an abort due to access ou

[dpdk-dev] [PATCH v2] Memory leak when adding/removing vhost_user ports

2016-07-06 Thread Christian Ehrhardt
Hi, while checking for dpdk 16.07 what backports are accepted in the meantime so I can drop them I found this particular discussion has been silently forgotten by all of us. Back then we had the patch and discussion first in http://dpdk.org/dev/patchwork/patch/12103/ and then http://dpdk.org/dev/p

[dpdk-dev] [PATCH] vhost: fix segfault on bad descriptor address.

2016-07-06 Thread Ilya Maximets
On 01.07.2016 10:35, Yuanhan Liu wrote: > Hi, > > Sorry for the long delay. > > On Fri, May 20, 2016 at 03:50:04PM +0300, Ilya Maximets wrote: >> In current implementation guest application can reinitialize vrings >> by executing start after stop. In the same time host application >> can still po

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-06 Thread Olivier MATZ
Hi Andrey, On 07/06/2016 01:59 PM, Chilikin, Andrey wrote: > Hi Oliver, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier MATZ >> Sent: Wednesday, July 6, 2016 8:43 AM >> To: Liang, Cunming ; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH 05/18]

[dpdk-dev] [PATCH 01/18] doc: add template for release notes 16.11

2016-07-06 Thread Olivier MATZ
Hi John, On 07/06/2016 01:48 PM, Mcnamara, John wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz >> Sent: Tuesday, July 5, 2016 4:42 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH 01/18] doc: add template for release notes >> 16.

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Nélio Laranjeiro
Thomas, On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > From: Nelio Laranjeiro > > A driver patch under net should start with "net/" or if > a patch touch multiple drivers, it should only start with "net:". > The same apply for crypto. > A patch touching all drivers (net + cry

[dpdk-dev] [PATCH] librte_pmd_bond: fix exported symbol versioning

2016-07-06 Thread Christian Ehrhardt
The older versions of rte_eth_bond_8023ad_conf_get and rte_eth_bond_8023ad_setup were available in the old way since 2.0 - at least according to the map file. But versioning in the code was set to 16.04. That breaks compatibility checks for 2.0 on that library. For example with the dpdk abi check

[dpdk-dev] [PATCH v3 1/3] kasumi: add new KASUMI PMD

2016-07-06 Thread De Lara Guarch, Pablo
> -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, July 06, 2016 12:26 PM > To: De Lara Guarch, Pablo; dev at dpdk.org > Cc: Doherty, Declan; Jain, Deepak K > Subject: Re: [dpdk-dev] [PATCH v3 1/3] kasumi: add new KASUMI PMD > > On 6/20/2016 3:40 PM, Pablo de Lara wrote: > > A

[dpdk-dev] about rx checksum flags

2016-07-06 Thread Chandran, Sugesh
Hi Olivier, Just to confirm , is this rx checksum patch already submitted in the DPDK ML? We would like to use these flags to speed up the tunneling in OVS. Regards _Sugesh > -Original Message- > From: Chandran, Sugesh > Sent: Friday, June 10, 2016 5:16 PM > To: 'Olivier Matz' ; Anany

[dpdk-dev] [PATCH v4] eal/linuxapp: fix resource leak

2016-07-06 Thread Daniel Mrzyglod
Current code does not munmap 'hugepage' mapping (hugepage info file) on function exit, leaking resources. Coverity issue: 97920 Fixes: b6a468ad41d5 ("memory: add --socket-mem option") Signed-off-by: Daniel Mrzyglod --- lib/librte_eal/linuxapp/eal/eal_memory.c | 9 +++-- 1 file changed, 7 in

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Thomas Monjalon
From: Nelio Laranjeiro A driver patch under net should start with "net/" or if a patch touch multiple drivers, it should only start with "net:". The same apply for crypto. A patch touching all drivers (net + crypto) should start with "drivers:". Longer prefixes like "net/mlx:" (for mlx4/mlx5) or

[dpdk-dev] [PATCH 2/3] scripts: reduce line size of commit checks

2016-07-06 Thread Thomas Monjalon
Small cleanup to comply with the 80 chars limit. Signed-off-by: Thomas Monjalon --- scripts/check-git-log.sh | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh index a24f101..833aa39 100755 --- a/scripts/check-git

[dpdk-dev] [PATCH 1/3] scripts: check spacing after commit references

2016-07-06 Thread Thomas Monjalon
A blank line between "Fixes" and "Signed-off-by" helps to separate information in blocks. Signed-off-by: Thomas Monjalon --- scripts/check-git-log.sh | 6 ++ 1 file changed, 6 insertions(+) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh index c7fc0e4..a24f101 100755 --- a/

[dpdk-dev] [PATCH 0/3] more commits checks

2016-07-06 Thread Thomas Monjalon
The script check-git-log.sh helps to keep a consistent git history. Scrolling/parsing the history is easier when we are used to some rules. N?lio Laranjeiro (1): scripts: check headline of drivers commits Thomas Monjalon (2): scripts: check spacing after commit references scripts: reduce li

[dpdk-dev] [PATCH v3 1/3] kasumi: add new KASUMI PMD

2016-07-06 Thread Ferruh Yigit
On 6/20/2016 3:40 PM, Pablo de Lara wrote: > Added new SW PMD which makes use of the libsso_kasumi SW library, > which provides wireless algorithms KASUMI F8 and F9 > in software. > > This PMD supports cipher-only, hash-only and chained operations > ("cipher then hash" and "hash then cipher") of t

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-06 Thread Chilikin, Andrey
Hi Oliver, > -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Wednesday, July 6, 2016 1:09 PM > To: Chilikin, Andrey ; Liang, Cunming > ; dev at dpdk.org > Cc: Ananyev, Konstantin > Subject: Re: [dpdk-dev] [PATCH 05/18] mbuf: add function to get packet ty

[dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits

2016-07-06 Thread Bruce Richardson
On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > From: Nelio Laranjeiro > > A driver patch under net should start with "net/" or if > a patch touch multiple drivers, it should only start with "net:". > The same apply for crypto. > A patch touching all drivers (net + crypto) shou

[dpdk-dev] [PATCH v4] Pci: Add the class_id support

2016-07-06 Thread Ferruh Yigit
On 6/14/2016 3:52 PM, Thomas Monjalon wrote: > 2016-05-24 20:50, Ziye Yang: >> This patch is used to add the class_id (class_code, >> subclass_code, programming_interface) support for >> pci_device probe. With this patch, it will be >> flexible for users to probe a class of devices >> by class_id.

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-06 Thread Chilikin, Andrey
Hi Oliver, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier MATZ > Sent: Wednesday, July 6, 2016 8:43 AM > To: Liang, Cunming ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type > from data > > Hi Cunming, >

[dpdk-dev] [PATCH v2] doc: update GSG for i40e firmware version.

2016-07-06 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ian Stokes > Sent: Wednesday, July 6, 2016 11:30 AM > To: dev at dpdk.org > Cc: Stokes, Ian > Subject: [dpdk-dev] [PATCH v2] doc: update GSG for i40e firmware version. > > Update the 'High Performance of Small

[dpdk-dev] [PATCH v4] eal/linuxapp: fix resource leak

2016-07-06 Thread Sergio Gonzalez Monroy
On 06/07/2016 11:44, Daniel Mrzyglod wrote: > Current code does not munmap 'hugepage' mapping (hugepage info file) on > function exit, leaking resources. > > Coverity issue: 97920 > Fixes: b6a468ad41d5 ("memory: add --socket-mem option") > > Signed-off-by: Daniel Mrzyglod > --- Please next time r

[dpdk-dev] [PATCH 01/18] doc: add template for release notes 16.11

2016-07-06 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Olivier Matz > Sent: Tuesday, July 5, 2016 4:42 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 01/18] doc: add template for release notes > 16.11 Hi, This is useful but it is probably a little bit early

[dpdk-dev] [PATCH] scripts: add check for net driver names without net prefix

2016-07-06 Thread Thomas Monjalon
2016-07-06 10:25, Bruce Richardson: > On Wed, Jul 06, 2016 at 11:10:48AM +0200, Thomas Monjalon wrote: > > 2016-07-05 15:40, Bruce Richardson: > > > +# check headline label for net/ prefix if needed > > > +bad=$(echo "$headlines" | grep -E --color=always \ > > > + -e "^($(ls drivers/net | grep -v M

[dpdk-dev] [PATCH v2] doc: update GSG for i40e firmware version.

2016-07-06 Thread Ian Stokes
Update the 'High Performance of Small Packets on 40G NIC' section of the GSG as the firmware version referenced for a NIC using the i40e driver was firmware version 4.2.5 which is no longer validated. Instruct users to consult release notes for current validated firmware versions. Signed-off-by: I

[dpdk-dev] [PATCH] scripts: add check for net driver names without net prefix

2016-07-06 Thread Bruce Richardson
On Wed, Jul 06, 2016 at 11:46:23AM +0200, Thomas Monjalon wrote: > 2016-07-06 10:25, Bruce Richardson: > > On Wed, Jul 06, 2016 at 11:10:48AM +0200, Thomas Monjalon wrote: > > > 2016-07-05 15:40, Bruce Richardson: > > > > +# check headline label for net/ prefix if needed > > > > +bad=$(echo "$headl

[dpdk-dev] [PATCH v4] mk: filter duplicate configuration entries

2016-07-06 Thread Christian Ehrhardt
*updates in v4* - replace awk usage with sed - re-add the old loop to be able to get rid of awk - add more explanation to the header of the makefile section *updates in v3* - replace tac with sed '1!G;h;$!d' to avoid build time dependency *updates in v2* - move to .config target - fix usage order

[dpdk-dev] [PATCH v3 4/4] virtio: add neon support

2016-07-06 Thread Jianbo Liu
On 5 July 2016 at 20:49, Jerin Jacob wrote: > Added neon based Rx vector implementation. > Selection of the new handler based neon availability at runtime. > Updated the release notes and MAINTAINERS file. > > Signed-off-by: Jerin Jacob > --- > MAINTAINERS | 1

[dpdk-dev] [PATCH] scripts: add check for net driver names without net prefix

2016-07-06 Thread Thomas Monjalon
2016-07-05 15:40, Bruce Richardson: > +# check headline label for net/ prefix if needed > +bad=$(echo "$headlines" | grep -E --color=always \ > + -e "^($(ls drivers/net | grep -v Makefile | tr '\n' '|')):" \ > + | sed 's,^,\t,') > +[ -z "$bad" ] || printf "Headline missing 'net/' prefix:\n$

[dpdk-dev] [PATCH v3] mk: filter duplicate configuration entries

2016-07-06 Thread Christian Ehrhardt
Thanks Ferruh, I'm personally more an awk guy so that was my natural choice. But I in general agree, the less tools used the better for dependencies and stability. I checked your suggestion and works like a charm. I'll still follow Bruce guidance to add more explanation. v4 should show up any min

[dpdk-dev] [PATCH] doc: announce API change for virtual device initialization

2016-07-06 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of De Lara Guarch, > Pablo > Sent: Tuesday, July 05, 2016 3:55 PM > To: Thomas Monjalon; Yigit, Ferruh > Cc: dev at dpdk.org; Mcnamara, John > Subject: Re: [dpdk-dev] [PATCH] doc: announce API change for virtual d

[dpdk-dev] [PATCH v3 6/6] ena: fix for icc compiler

2016-07-06 Thread Jan Mędala
There are no obstacles. We first produced patches 1-5, then tested on different compilers so patch no. 6 was made. Thank you Bruce for support! I'm still learning DPDK's submission/upstream process so all comments and gained knowledge will easier future patches. Jan 2016-07-06 10:14 GMT+02:00

[dpdk-dev] [PATCH] examples/l2fwd-crypto: flush crypto dev buffers

2016-07-06 Thread Pablo de Lara
Crypto operations are enqueued in the crypto devices when the crypto device buffers are full (MAX_PKT_BURST), in order to be more efficient. The problem is that operations might be stuck in those buffers, if they never get full, and therefore, those operations will never be performed. Therefore,

[dpdk-dev] [PATCH v3] mk: filter duplicate configuration entries

2016-07-06 Thread Bruce Richardson
On Wed, Jul 06, 2016 at 09:57:01AM +0100, Ferruh Yigit wrote: > On 7/6/2016 9:12 AM, Bruce Richardson wrote: > > On Wed, Jul 06, 2016 at 07:37:45AM +0200, Christian Ehrhardt wrote: > >> *updates in v3* > >> - replace tac with sed '1!G;h;$!d' to avoid build time dependency > >> > >> *updates in v2*

[dpdk-dev] [PATCH] scripts: add check for net driver names without net prefix

2016-07-06 Thread Bruce Richardson
On Wed, Jul 06, 2016 at 11:10:48AM +0200, Thomas Monjalon wrote: > 2016-07-05 15:40, Bruce Richardson: > > +# check headline label for net/ prefix if needed > > +bad=$(echo "$headlines" | grep -E --color=always \ > > + -e "^($(ls drivers/net | grep -v Makefile | tr '\n' '|')):" \ > > + | sed 's

[dpdk-dev] [PATCH 09/18] mbuf: support Mpls in software packet type parser

2016-07-06 Thread Olivier MATZ
Hi Cunming, On 07/06/2016 09:08 AM, Liang, Cunming wrote: > Hi Olivier, > > On 7/5/2016 11:41 PM, Olivier Matz wrote: >> Add a new RTE_PTYPE_L2_ETHER_MPLS packet type, and its support in >> rte_pktmbuf_get_ptype(). >> >> Signed-off-by: Didier Pallard >> Signed-off-by: Olivier Matz >> --- >> l

[dpdk-dev] [PATCH v3] mk: filter duplicate configuration entries

2016-07-06 Thread Ferruh Yigit
On 7/6/2016 9:12 AM, Bruce Richardson wrote: > On Wed, Jul 06, 2016 at 07:37:45AM +0200, Christian Ehrhardt wrote: >> *updates in v3* >> - replace tac with sed '1!G;h;$!d' to avoid build time dependency >> >> *updates in v2* >> - move to .config target >> - fix usage order of tac >> - simplify inne

[dpdk-dev] [PATCH 05/18] mbuf: add function to get packet type from data

2016-07-06 Thread Olivier MATZ
Hi Cunming, On 07/06/2016 08:44 AM, Liang, Cunming wrote: > Hi Olivier, > > On 7/5/2016 11:41 PM, Olivier Matz wrote: >> Introduce the function rte_pktmbuf_get_ptype() that parses a >> mbuf and returns its packet type. For now, the following packet >> types are parsed: >> L2: Ether >> L3:

[dpdk-dev] Possible Bug: DPDK 16.04 Empty packet_type

2016-07-06 Thread Tan, Jianfeng
Hi, On 7/6/2016 3:54 AM, Craig, Chris A. wrote: > We recently upgraded our environment to the 16.04 version of DPDK on a 14.04 > installation of Ubuntu. After upgrading, we noticed our code was behaving > oddly. After further inspection, we discovered the `packet_type` attribute of > the mbuf

[dpdk-dev] [app/test/test_cryptodev.c] setup_test_string() - buffer overflow, segfault

2016-07-06 Thread Catalin Vasile
Hi, I was trying to run cryptodev_aesni_mb_autotest with the "test" app. I ran into an overflow which I think I?debugged. The initial symptom was at?rte_mempool_lookup() called?from?rte_cryptodev_sym_session_pool_create(), strncmp() went on a segfault, because "te"' 's members were NULL. I set a

[dpdk-dev] [PATCH v3 6/6] ena: fix for icc compiler

2016-07-06 Thread Bruce Richardson
On Tue, Jul 05, 2016 at 07:10:54PM +0200, Jan M?dala wrote: > Let me point it this way: > ??This patch fixes: [dpdk-dev,v3,1/6] ena: update of ENA communication layer > > Jan Since it fixes another patch in the same series, I think the two patches could actually be merged, rather than adding co

[dpdk-dev] [PATCH v3] mk: filter duplicate configuration entries

2016-07-06 Thread Bruce Richardson
On Wed, Jul 06, 2016 at 07:37:45AM +0200, Christian Ehrhardt wrote: > *updates in v3* > - replace tac with sed '1!G;h;$!d' to avoid build time dependency > > *updates in v2* > - move to .config target > - fix usage order of tac > - simplify inner section by only using awk (instead of awk+loop+bash

[dpdk-dev] [PATCH v3] mk: filter duplicate configuration entries

2016-07-06 Thread Ferruh Yigit
On 7/6/2016 6:37 AM, Christian Ehrhardt wrote: > *updates in v3* > - replace tac with sed '1!G;h;$!d' to avoid build time dependency > > *updates in v2* > - move to .config target > - fix usage order of tac > - simplify inner section by only using awk (instead of awk+loop+bash+sed) > > Due to the

[dpdk-dev] xstats performance

2016-07-06 Thread Remy Horton
On 05/07/2016 19:10, Rasesh Mody wrote: [..] >> For all the current drivers xstats_names[idx].id==idx so it would just >> involve removing the references to the id field and updating the >> documentation. Complication is coordinating with QLogic for the bnx2x & >> qede xstats patches. > > We could

[dpdk-dev] [PATCH v3] mk: filter duplicate configuration entries

2016-07-06 Thread Christian Ehrhardt
*updates in v3* - replace tac with sed '1!G;h;$!d' to avoid build time dependency *updates in v2* - move to .config target - fix usage order of tac - simplify inner section by only using awk (instead of awk+loop+bash+sed) Due to the hierarchy and the demand to keep the base config showing all opt

[dpdk-dev] [PATCH v2] mk: filter duplicate configuration entries

2016-07-06 Thread Christian Ehrhardt
Hi, I came up with something very similar when looking for tac replacements yesterday, but had no time to finish things. But your suggestion is even shorter - I had found "sed -n '1{h;T;};G;h;$p;' file" or "sed -n '1!G;h;$p'". That removes the tac dependency, which I agree is a good thing. To chai

[dpdk-dev] [PATCH v1] i40: fix the VXLAN TSO issue

2016-07-06 Thread Wu, Jingjing
> --- a/drivers/net/i40e/i40e_rxtx.c > +++ b/drivers/net/i40e/i40e_rxtx.c > @@ -801,6 +801,12 @@ i40e_txd_enable_checksum(uint64_t ol_flags, > union i40e_tx_offload tx_offload, > uint32_t *cd_tunneling) > { > + /* Tx pkts tunnel type*/ > + if ((o

[dpdk-dev] [PATCH v1] i40: fix the VXLAN TSO issue

2016-07-06 Thread Zhe Tao
Problem: when using the TSO + VXLAM feature in i40e, the outer UDP len will sometimes be a invalid value for the multiple UDP segements which are TSOed by the i40e for the inner TCP. Fix this problem by add the tunnel type field in the i40e descriptor which is missed before. Fixes: 77b8301733c3 (

[dpdk-dev] [PATCH v2 3/3] i40e: fix out-of-bounds access

2016-07-06 Thread Xing, Beilei
> -Original Message- > From: Richardson, Bruce > Sent: Tuesday, July 5, 2016 9:26 PM > To: Xing, Beilei > Cc: Wu, Jingjing ; Jastrzebski, MichalX K > ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 3/3] i40e: fix out-of-bounds access > > On Tue, Jul 05, 2016 at 02:10:05PM +0800, B

[dpdk-dev] [PATCH v6 0/4] support reset of VF link

2016-07-06 Thread Lu, Wenzhuo
; > You may notice this feature is postponed to 16.11. Would you like to wait > > for > the new version that will include an example? > > Hi, > > Unfortunately we need the VF reset working sooner than that, so one way or > the other I'll need to sort it out. Given I've got a use case where this is > happening, > if it can be helpful for you I'm more than happy to help as a guinea pig. If > you > could please give some guidance/guidelines with regards to which API to use to > sort the mbuf problem, I can try it out and give back some feedback. > > Thanks! I made a stupid mistake and deleted all my code. So, I have to take some time to rewrite it :( Attached the example I used to test the reset API. It's modified from the l2fwd example. So you can compare it with l2fwd to see what need to be added. Hopefully it can help :) > > -- > Kind regards, > Luca Boccassi -- next part -- An embedded and charset-unspecified text was scrubbed... Name: main.c URL: <http://dpdk.org/ml/archives/dev/attachments/20160706/1e1df3ba/attachment-0001.c>