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

2016-07-06 Thread Lu, Wenzhuo
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>

[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

[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,

[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 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

[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

[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 + >

[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 >>

[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

[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 >> --- >>

[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

[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

[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

[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 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

[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/'

[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

[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] 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

[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] 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 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

[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

[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:

[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 ---

[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 ---

[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

[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] 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

[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

[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

[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 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 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 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 +

[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 >>

[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

[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

[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 > >>

[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

[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

[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

[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

[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
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

[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' ;

[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

[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

[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" |

[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

[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: > >

[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

[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 ---

[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 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 ---

[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 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

[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(-)

[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

[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

[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 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

[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 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

[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 ---

[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

[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

[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 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

[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

[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 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

[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

[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

[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

[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