When TSO is enabled, Verbs layer aggregates the TSO
inline size with the txq inline size for the Tx creation,
while the PMD takes the maximum among them.
Fixing it by adjusting the max inline parameter before
passing to to Verbs.
Fixes: 3f13f8c23a7c ("net/mlx5: support hardware TSO")
Signed-off-
Hi Harish,
> -Original Message-
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Patil, Harish
> Sent: Wednesday, May 03, 2017 7:34 AM
> To: Horton, Remy; De Lara Guarch, Pablo
> Cc: dev@dpdk.org
> Subject: [dpdk-dev] app/testpmd: compilation error with bitrate/latency
> config flags
Hi,
Seeing compilation errors with DPDK 17.05-rc3 when I happened to disable
bitrate/latency configs during our internal testing.
CONFIG_RTE_LIBRTE_BITRATE=n and
CONFIG_RTE_LIBRTE_LATENCY_STATS=n.
/home/root1/hpatil/17.05-rc3/dpdk/app/test-pmd/testpmd.c: In function
?main?:
/home/root1/hpatil/17.0
This patch adds two note in doc build_dpdk, suggests DPDK user
don't use ``vfio-pci`` and ``uio_pci_generic`` module to create
virtual functions(VFs).
Signed-off-by: Qiming Yang
---
v2 changes:
* use better expression.
---
---
doc/guides/linux_gsg/build_dpdk.rst | 8
1 file changed, 8 i
We find that VF receive address register is not set
if MAC address is assigned by PF. This patch fixes it.
Fixes: d82170d27918 ("igb: add VF support")
Signed-off-by: Qiming Yang
---
drivers/net/e1000/igb_ethdev.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/d
VF default MAC address be added in PF Mac address list
instead of VF MAC address list, makes VF can't receive
packets. This patch fixes this issue.
Fixes: be2d648a2dd3 ("igb: add PF support")
Signed-off-by: Qiming Yang
---
drivers/net/e1000/igb_pf.c | 4
1 file changed, 4 insertions(+)
di
Hi all,
Here is the meeting notes for the last DPDK technical board meeting held on
2017-04-27.
Please note that meetings are open to all to attend. Any topics to be referred
to the tech board for discussion at that meeting should be emailed to
techbo...@dpdk.org. Normally, it's a bi-weekly me
Add more explanation about how to disable MDD on
kernel PF.
Signed-off-by: Wenzhuo Lu
---
doc/guides/nics/ixgbe.rst | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
index 130765b..6773fe5 100644
--- a/doc/guides/ni
Hi Ashwin,
On 2 May 2017 at 19:47, Sekhar, Ashwin wrote:
> Hi Jianbo,
>
> I tested your neon changes on thunderx. I am seeing a performance
> regression of ~10% for LPM case and ~20% for EM case with your changes.
> Did you see improvement on any arm64 platform with these changes. If
> yes, how m
On Tue, May 02, 2017 at 08:44:23PM +0800, Wei Dai wrote:
> Some customers find adding MAC addr to VF sometimes can fail,
> but it is still stored in dev->data->mac_addrs[ ]. So this
> can lead to some errors that assumes the non-zero entry in
> dev->data->mac_addrs[ ] is valid.
> Following acknowle
Hi Igor,
From: Igor Ryzhov [mailto:iryz...@nfware.com]
Sent: Tuesday, May 2, 2017 9:33 PM
To: Lu, Wenzhuo
Cc: dev@dpdk.org; sta...@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] net/ixgbe: fix default MAC setting
Hello Wenzhuo,
How about also delete meaningless "ixgbe_remove_rar(dev, 0);"?
[Wenzhuo] T
Hi,
> -Original Message-
> From: Dai, Wei
> Sent: Tuesday, May 2, 2017 8:44 PM
> To: Lu, Wenzhuo; tho...@monjalon.net; harish.pa...@cavium.com;
> rasesh.m...@cavium.com; stephen.h...@broadcom.com;
> ajit.khapa...@broadcom.com; Zhang, Helin; Ananyev, Konstantin; Wu,
> Jingjing; Chen, Jing D
Correct CONFIG_RTE_LIBRTE_BNX2X_PMD config file option and add a note
about external zlib dependency for loading the firmware image.
Signed-off-by: Rasesh Mody
---
doc/guides/nics/bnx2x.rst |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/doc/guides/nics/bnx2x.rst b/d
Correct CONFIG_RTE_LIBRTE_BNX2X_PMD config file option and add a note
about external zlib dependency for loading the firmware image.
Signed-off-by: Rasesh Mody
---
doc/guides/nics/bnx2x.rst |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/doc/guides/nics/bnx2x.rst b/d
Signed-off-by: Cristian Dumitrescu
---
doc/guides/rel_notes/deprecation.rst | 6 ++
1 file changed, 6 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst
b/doc/guides/rel_notes/deprecation.rst
index a3e7c72..ae937f5 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/gui
Some comments inline with [HvH] prefix
> -Original Message-
> From: Jerin Jacob [mailto:jerin.ja...@caviumnetworks.com]
> Sent: Tuesday, May 2, 2017 5:01 PM
> To: Eads, Gage
> Cc: dev@dpdk.org; Richardson, Bruce ; Van Haaren,
> Harry
> ; hemant.agra...@nxp.com; nipun.gu...@nxp.com;
> Va
-Original Message-
> Date: Fri, 21 Apr 2017 22:31:52 +
> From: "Eads, Gage"
> To: Jerin Jacob , "dev@dpdk.org"
>
> CC: "Richardson, Bruce" , "Van Haaren, Harry"
> , "hemant.agra...@nxp.com"
> , "nipun.gu...@nxp.com" ,
> "Vangati, Narender"
> Subject: RE: [RFC] [dpdk-dev] [PATCH]
Hello Wenzhuo,
How about also delete meaningless "ixgbe_remove_rar(dev, 0);"?
Best regards,
Igor
On Tue, May 2, 2017 at 11:34 AM, Wenzhuo Lu wrote:
> Pool 0 is not PF, it's VF 0. So the MAC is set for VF 0
> but not PF.
> The code introduced a weird issue. In the scenario PF + VF,
> when only
CC'ing Jingjing.
> -Original Message-
> From: De Lara Guarch, Pablo
> Sent: Tuesday, May 02, 2017 2:11 PM
> To: jingj...@intel.com; Pattan, Reshma
> Cc: dev@dpdk.org; De Lara Guarch, Pablo
> Subject: [PATCH] app/testpmd: disable latency stats by default
>
> Disable latency stats gathering
Disable latency stats gathering by default,
so there is not performance degradation if user
is not interested in them.
Signed-off-by: Pablo de Lara
---
app/test-pmd/testpmd.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c
02/05/2017 14:53, Mcnamara, John:
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> > 02/05/2017 13:54, Yong Wang:
> > > -python tools/dpdk-devbind.py -b vfio-pci 00:03.0
> > > +python usertools/dpdk-devbind.py -b vfio-pci 00:03.0
> >
> > There is another occurence in doc/guid
> -Original Message-
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> Sent: Tuesday, May 2, 2017 1:39 PM
> To: Yong Wang
> Cc: dev@dpdk.org; Mcnamara, John
> Subject: Re: [dpdk-dev] [PATCH] doc: fix directory err in virtio guide
>
> 02/05/2017 13:54, Yong Wang:
> > -pytho
This patch is added to introduce a testpmd command which
is used to add more than one MAC addresses one time.
This command can simplify the test for the change where
the type of return value of adding MAC address.
Normally a MAC address may fails to be added only after
many MAC addresses have been
Some customers find adding MAC addr to VF sometimes can fail,
but it is still stored in dev->data->mac_addrs[ ]. So this
can lead to some errors that assumes the non-zero entry in
dev->data->mac_addrs[ ] is valid.
Following acknowledgements are from specific NIC PMD
maintainer for their managing pa
Current ethdev always stores MAC address even it fails to be added.
Other function may regard the failed MAC address valid and lead to
some errors. So There is a need to check if the addr is added
successfully or not and discard it if it fails.
In 3rd patch, add a command "add_more_mac_addr port_i
Add following lines in section of API change in release note.
If a MAC address fails to be added without this change, it is still
stored and may be regarded as a valid one. This may lead to errors
in application. The type of return value of eth_mac_addr_add_t in
rte_ethdev.h is changed. Any specif
Update doc to the new dpdk-devbind.py usertools path.
Fixes: 58a2551a160f ("doc: introduce PVP reference benchmark")
Signed-off-by: John McNamara
---
doc/guides/howto/pvp_reference_benchmark.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/doc/guides/howto/pvp_reference_b
02/05/2017 13:54, Yong Wang:
> -python tools/dpdk-devbind.py -b vfio-pci 00:03.0
> +python usertools/dpdk-devbind.py -b vfio-pci 00:03.0
There is another occurence in doc/guides/howto/pvp_reference_benchmark.rst
> -Original Message-
> From: Yong Wang [mailto:wang.yon...@zte.com.cn]
> Sent: Tuesday, May 2, 2017 12:54 PM
> To: Mcnamara, John
> Cc: dev@dpdk.org; Yong Wang
> Subject: [PATCH] doc: fix directory err in virtio guide
>
> Signed-off-by: Yong Wang
Acked-by: John McNamara
> -Original Message-
> From: Shreyansh Jain [mailto:shreyansh.j...@nxp.com]
> Sent: Tuesday, May 2, 2017 12:45 PM
> To: tho...@monjalon.net; Mcnamara, John
> Cc: dev@dpdk.org; Shreyansh Jain
> Subject: [PATCH] doc: fix incorrect indexing
>
> Because of extra space before each list item
On Tue, May 02, 2017 at 11:18:06AM +0200, Thomas Monjalon wrote:
02/05/2017 09:35, Jan Blunck:
Am 25.04.2017 11:06 schrieb "Gaëtan Rivet" :
Hi Ferruh,
On Fri, Apr 21, 2017 at 03:59:24PM +0100, Ferruh Yigit wrote:
> On 4/18/2017 1:17 PM, Gaetan Rivet wrote:
>
>> This new API allows reacting t
Signed-off-by: Yong Wang
---
doc/guides/nics/virtio.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/doc/guides/nics/virtio.rst b/doc/guides/nics/virtio.rst
index e1a80dc..91bedea 100644
--- a/doc/guides/nics/virtio.rst
+++ b/doc/guides/nics/virtio.rst
@@ -321,7 +321,7 @@ T
Hi Jianbo,
I tested your neon changes on thunderx. I am seeing a performance
regression of ~10% for LPM case and ~20% for EM case with your changes.
Did you see improvement on any arm64 platform with these changes. If
yes, how much was the improvement?
FYI, I had also tried vectorizing the l3fwd
Because of extra space before each list item, indexing numbers
generated by Sphinx were same.
Signed-off-by: Shreyansh Jain
---
**
Current documentation has same numbering for top 3 list items, but
thereafter numbering is correct even with space. Not sure what is
wrong - but, removing space fi
Hi,
Please find comments inline.
On Tue, 2017-05-02 at 15:14 +0800, Jianbo Liu wrote:
> Use ARM NEON intrinsics to accelerate l3 fowarding.
>
> Signed-off-by: Jianbo Liu
> ---
> examples/l3fwd/l3fwd.h | 4 -
> examples/l3fwd/l3fwd_em.c | 4 +-
> examples/l3fwd/l3fwd_em
> -Original Message-
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> Sent: Monday, May 01, 2017 9:22 PM
> To: Patil, Harish; Horton, Remy
> Cc: dev@dpdk.org; De Lara Guarch, Pablo; Wu, Jingjing
> Subject: Re: [dpdk-dev] [PATCH v2] app/testpmd: add bitrate stats option
>
> 01/05/20
From: Nidhia Varghese [mailto:nidhiavarghes...@gmail.com]
Sent: Tuesday, May 2, 2017 10:29 AM
To: Singh, Jasvinder
Cc: dev@dpdk.org; us...@dpdk.org
Subject: Re: [dpdk-users] Adding multiple fields as key in ip_pipeline
application
Hi,
Can we do the append/prepend (of the port id with vlan id)
Add two parameters to testpmd:
--print-event
--mask-event
To enable or disable to printing of events. This display is configured
on a per-event basis. By default, all except VF_MBOX are displayed.
Fixes: 76ad4a2d82d4 ("app/testpmd: add generic event handler")
Cc: "Lu, Wenzhuo"
Signed-off
> -Original Message-
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Pablo de Lara
> Sent: Friday, April 28, 2017 7:07 PM
> To: dev@dpdk.org
> Cc: Doherty, Declan ; akhil.go...@nxp.com;
> hemant.agra...@nxp.com; zbigniew.bo...@caviumnetworks.com;
> jerin.ja...@caviumnetworks.com; D
02/05/2017 09:03, Gaetan Rivet:
> * ``--bitrate-stats=N``
>
> Set the logical core N to perform bitrate calculation.
> +
> +* ``--print-event
> ``
> +
> + Enable printing the occurrence of the designated event.
> +
> +* ``--mask-event
> ``
> +
> + Disable printing the occurrence of
On Tue, 2017-05-02 at 15:14 +0800, Jianbo Liu wrote:
> The l3fwd_em_sse.h is enabled by NO_HASH_LOOKUP_MULTI.
> Renaming it because it's only for single hash lookup,
> and doesn't include any x86 SSE instructions.
>
> Signed-off-by: Jianbo Liu
> ---
> examples/l3fwd/l3fwd_em.c
02/05/2017 03:58, Pei, Yulong:
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> > 26/04/2017 06:31, Yulong Pei:
> > > Display pci device bus info when show port info.
> > >
> > > Signed-off-by: Yulong Pei
> > > ---
> > > --- a/app/test-pmd/config.c
> > > +++ b/app/test-pmd/config.c
> > > @
Hi,
Can we do the append/prepend (of the port id with vlan id) through config
file?
Thanks,
Nidhia
On Fri, Apr 28, 2017 at 7:24 PM, Singh, Jasvinder wrote:
> Hi Nidhia,
>
>
> I am developing an application in which I need to use both incoming port
> and vlan as the key for the flow table. Port
02/05/2017 09:35, Jan Blunck:
> Am 25.04.2017 11:06 schrieb "Gaëtan Rivet" :
>
> Hi Ferruh,
>
>
> On Fri, Apr 21, 2017 at 03:59:24PM +0100, Ferruh Yigit wrote:
>
> > On 4/18/2017 1:17 PM, Gaetan Rivet wrote:
> >
> >> This new API allows reacting to a device removal.
> >> A device removal is the
This problem is caused by a missing set of E1000_RXCSUM_CRCOFL
in eth_igb_rx_init(), it should be set to enable SCTP packet
L4 checksum.If it is not set, the printf message in cksum fwd
about L4 SCTP cksum flag is error.
Fixes: af75078fece3 ("first public release")
Signed-off-by: Wei Zhao
---
Ch
This problem is caused by a missing set of E1000_RXCSUM_CRCOFL
in eth_igb_rx_init(), it should be set to enable SCTP packet
L4 checksum.If it is not set, the printf message in cksum fwd
about L4 SCTP cksum flag is error.
Fixes: af75078fece3 ("first public release")
Signed-off-by: Wei Zhao
---
Ch
Hi, wenzhuo
> -Original Message-
> From: Lu, Wenzhuo
> Sent: Tuesday, May 2, 2017 4:14 PM
> To: Zhao1, Wei ; dev@dpdk.org
> Subject: RE: [PATCH] net/e1000: fix checksum valid flags error
>
> Hi Wei,
>
> > -Original Message-
> > From: Zhao1, Wei
> > Sent: Tuesday, May 2, 2017 10:4
Pool 0 is not PF, it's VF 0. So the MAC is set for VF 0
but not PF.
The code introduced a weird issue. In the scenario PF + VF,
when only starting PF, the default PF MAC address is working.
But after starting a VF, the default PF MAC address becomes
the VF's address.
Use the pool which is not occu
Hi Wei,
> -Original Message-
> From: Zhao1, Wei
> Sent: Tuesday, May 2, 2017 10:46 AM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo; Zhao1, Wei
> Subject: [PATCH] net/e1000: fix checksum valid flags error
>
> This problem is caused by a missing set of E1000_RXCSUM_CRCOFL in
> eth_igb_rx_init(), it
On Tue, 2017-05-02 at 15:59 +0800, Jianbo Liu wrote:
> On 2 May 2017 at 14:41, Jerin Jacob
> wrote:
> >
> > -Original Message-
> > >
> > > Date: Mon, 1 May 2017 22:59:53 -0700
> > > From: Ashwin Sekhar T K
> > > To: byron.mar...@intel.com, pablo.de.lara.gua...@intel.com,
> > > jerin.j
On 2 May 2017 at 14:41, Jerin Jacob wrote:
> -Original Message-
>> Date: Mon, 1 May 2017 22:59:53 -0700
>> From: Ashwin Sekhar T K
>> To: byron.mar...@intel.com, pablo.de.lara.gua...@intel.com,
>> jerin.ja...@caviumnetworks.com, jianbo@linaro.org
>> Cc: dev@dpdk.org, Ashwin Sekhar T
Am 25.04.2017 11:06 schrieb "Gaëtan Rivet" :
Hi Ferruh,
On Fri, Apr 21, 2017 at 03:59:24PM +0100, Ferruh Yigit wrote:
> On 4/18/2017 1:17 PM, Gaetan Rivet wrote:
>
>> This new API allows reacting to a device removal.
>> A device removal is the sudden disappearance of a device from its
>> bus.
>
Hi Akhil,
> -Original Message-
> From: Akhil Goyal [mailto:akhil.go...@nxp.com]
> Sent: Wednesday, April 26, 2017 12:02 PM
> To: De Lara Guarch, Pablo; dev@dpdk.org
> Cc: Doherty, Declan; hemant.agra...@nxp.com
> Subject: Re: [PATCH] test/test: improve dequeue logic for crypto operation
>
Keep x86 related code in l3fwd_sse.h, and move common code to
l3fwd_common.h, which will be used by other Archs.
Signed-off-by: Jianbo Liu
---
examples/l3fwd/l3fwd_common.h | 293 ++
examples/l3fwd/l3fwd_sse.h| 255 +---
Use ARM NEON intrinsics to accelerate l3 fowarding.
Signed-off-by: Jianbo Liu
---
examples/l3fwd/l3fwd.h | 4 -
examples/l3fwd/l3fwd_em.c | 4 +-
examples/l3fwd/l3fwd_em_hlm.h | 5 +
examples/l3fwd/l3fwd_em_hlm_neon.h | 74 +++
examples/l3fwd/l3fwd_em_sin
Signed-off-by: Jianbo Liu
Some common code can be used by other ARCHs, move to l3fwd_lpm.c
---
examples/l3fwd/l3fwd_lpm.c | 83 ++
examples/l3fwd/l3fwd_lpm.h | 26 +
examples/l3fwd/l3fwd_lpm_sse.h | 66 -
The l3fwd_em_sse.h is enabled by NO_HASH_LOOKUP_MULTI.
Renaming it because it's only for single hash lookup,
and doesn't include any x86 SSE instructions.
Signed-off-by: Jianbo Liu
---
examples/l3fwd/l3fwd_em.c| 2 +-
examples/l3fwd/{l3fwd_em_sse.h => l3fwd_em_single.
Extract common code from l3fwd_em_hlm_sse.h, and add to the new file
l3fwd_em_hlm.h.
Signed-off-by: Jianbo Liu
---
examples/l3fwd/l3fwd_em.c | 2 +-
examples/l3fwd/l3fwd_em_hlm.h | 302 ++
examples/l3fwd/l3fwd_em_hlm_sse.h | 280 +
Add two parameters to testpmd:
--print-event
--mask-event
To enable or disable to printing of events. This display is configured
on a per-event basis. By default, all except VF_MBOX are displayed.
Fixes: 76ad4a2d82d4 ("app/testpmd: add generic event handler")
Cc: "Lu, Wenzhuo"
Signed-off
59 matches
Mail list logo