Problem:
When using the TSO + VXLAN feature in i40e, the outer UDP length fields in
the multiple UDP segments which are TSOed by the i40e will have a
wrong value.
Fix this problem by adding the tunnel type field in the i40e descriptor
which was missed before.
Fixes: 77b8301733c3 ("i40e: VXLAN Tx
On 7/18/2016 6:50 PM, Thomas Monjalon wrote:
> 2016-07-18 13:57, Sergio Gonzalez Monroy:
>> On 18/07/2016 13:41, Akhil Goyal wrote:
>>> In Ipsec-secgw application, while adding the outer IP header,
>>> it seems that the application does not update the checksum value
>>> for outbound packets. This
On 7/18/2016 6:27 PM, Sergio Gonzalez Monroy wrote:
> Hi,
>
> On 18/07/2016 13:41, Akhil Goyal wrote:
>> Hi,
>>
>> In Ipsec-secgw application, while adding the outer IP header, it seems
>> that the application does not update the checksum value for outbound
>> packets. This result in incorrect
Hi Chao,
I just tried building dpdk master with these patches in ubuntu 16.04 le
guest
and I could able to.
test at ubuntu:~/DPDK$ make install T=ppc_64-power8-linuxapp-gcc -j16
.
INSTALL-MAP test.map
INSTALL-APP test
Build complete [ppc_64-power8-linuxapp-gcc]
Installation cannot
After vring reservation, only the first bytes of the vring were
cleared.
This patch fixes this to clear the real size fo the vring.
Signed-off-by: Maxime Coquelin
---
Note: I found this bug while doing some code review, it is not a fix for
a problem I encountered.
---
Hi Helin,
On Mon, Jul 18, 2016 at 5:15 PM, Zhang, Helin wrote:
> Hi Ceara
>
> Could you help to let me know your firmware version?
# ethtool -i p7p1 | grep firmware
firmware-version: f4.40.35115 a1.4 n4.53 e2021
> And could you help to try with the standard DPDK example application, such as
>
Sorry, there was a typo in Huawei's mail, I copy-pasted it from the
other thread "[dpdk-dev] virtio PMD issue".
On 07/18/2016 06:06 PM, Olivier Matz wrote:
> Hi,
>
> On 16.07-rc3, when I start testpmd with a virtio driver in a VM,
> requesting options that are not implemented (rx checksum), it
Hi,
On 16.07-rc3, when I start testpmd with a virtio driver in a VM,
requesting options that are not implemented (rx checksum), it crashes:
./build/app/testpmd -l 0,1 --log-level 8 -- --total-num-mbufs=16384 -i
--port-topology=chained --disable-hw-vlan-filter --disable-hw-vlan-strip
2016-07-18 14:12, Ido Barnea:
> In Trex, we do include ixgbe_type.h from c++ code because we do some
> register writes ourselves to support some functionality not exposed in
> the DPDK API.
I agree we should find a way to expose some driver-specific headers for
very specific features which will
The sed syntax of '0,/regexp/' is GNU specific and fails with
non GNU sed in FreeBSD.
To solve the issue we can use awk instead to remove duplicates.
Fixes: b2063f104db7 ("mk: filter duplicate configuration entries")
Signed-off-by: Sergio Gonzalez Monroy
---
v2:
- Use temp var instead of temp
Signed-off-by: Shreyansh Jain
---
doc/guides/prog_guide/mempool_lib.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/doc/guides/prog_guide/mempool_lib.rst
b/doc/guides/prog_guide/mempool_lib.rst
index 5946675..ffdc109 100644
--- a/doc/guides/prog_guide/mempool_lib.rst
+++
On Mon, Jul 18, 2016 at 01:26:09PM +, Chandran, Sugesh wrote:
> Hi Adrien,
> Thank you for getting back on this.
> Please find my comments below.
Hi Sugesh,
Same for me, removed again the parts we agree on.
[...]
> > > > > > > [Sugesh] Another concern is the cost and time of installing
> >
Gowrishankar,
I still get this error with the v3 patches. My OS is Ubuntu 16.04 LTS
ppc64le. GCC version: Ubuntu/IBM 5.3.1-14ubuntu2
Can you help to check again?
In file included from
/root/test/sub/dpdk/lib/librte_acl/acl_run_altivec.c:34:0:
/root/test/sub/dpdk/lib/librte_acl/acl_run_altivec.h:
2016-07-18 14:57, Sergio Gonzalez Monroy:
> On 18/07/2016 14:53, Akhil Goyal wrote:
> > On 7/18/2016 6:50 PM, Thomas Monjalon wrote:
> >> 2016-07-18 13:57, Sergio Gonzalez Monroy:
> >>> On 18/07/2016 13:41, Akhil Goyal wrote:
> In Ipsec-secgw application, while adding the outer IP header,
>
2016-07-18 15:54, Christian Ehrhardt:
> Hi Sergio,
> you might have seen that I had a similar version with awk in v2 IIRC. I
> also had the secondary tmp file just like you now.
> So, since it is so close to my old submission I wont object :-)
>
> Back then the discussion went for reduced build
Hi,
DPDK is a C code and do not support C++ compilation.
Only the exposed headers have some C++ support requirements.
> drivers/net/ixgbe/base/ixgbe_osdep.h | 2 ++
Patching this file for C++ will be probably rejected, sorry.
I'm interested to understand what is your need?
Hi Sergio,
you might have seen that I had a similar version with awk in v2 IIRC. I
also had the secondary tmp file just like you now.
So, since it is so close to my old submission I wont object :-)
Back then the discussion went for reduced build time dependencies and
avoiding a second temp file,
On 18/07/2016 15:07, Thomas Monjalon wrote:
> 2016-07-18 15:54, Christian Ehrhardt:
>> Hi Sergio,
>> you might have seen that I had a similar version with awk in v2 IIRC. I
>> also had the secondary tmp file just like you now.
>> So, since it is so close to my old submission I wont object :-)
>>
2016-07-18 14:15, Sergio Gonzalez Monroy:
> On 18/07/2016 12:49, Olivier Matz wrote:
> > On 07/18/2016 01:33 PM, Sergio Gonzalez Monroy wrote:
> >> On 12/07/2016 12:30, Olivier MATZ wrote:
> >>> I think we cannot expect that memory is initialized at 0 when using
> >>> Xen dom0. If I add the
This patch adds an image of the Live Migration of a VM using vhost_user
on the host, test configuration.
Signed-off-by: Bernard Iremonger
---
doc/guides/howto/img/lm_vhost_user.svg| 644 ++
doc/guides/howto/lm_virtio_vhost_user.rst | 4 +
2 files changed, 648
This patch describes the procedure to be be followed to perform
Live Migration of a VM with Virtio PMD running on a host which
is running the vhost_user sample application (vhost-switch).
It includes sample host and VM scripts used in the procedure.
Signed-off-by: Bernard Iremonger
---
This patchset describes the procedure to Live migrate a VM with
Virtio PMD's with the vhost_user sample application (vhost-switch)
running on the host.
This patchset depends on the following patch:
http://dpdk.org/dev/patchwork/patch/14871
Changes in v3:
added links in rst file
updated rst file
2016-07-18 14:11, Sergio Gonzalez Monroy:
> The sed syntax of '0,/regexp/' is GNU specific and fails with
> non GNU sed in FreeBSD.
>
> To solve the issue we can use awk instead to remove duplicates.
Christian, an opinion please?
> Fixes: b2063f104db7 ("mk: filter duplicate configuration
On 18/07/2016 15:09, Thomas Monjalon wrote:
> 2016-07-18 14:57, Sergio Gonzalez Monroy:
>> On 18/07/2016 14:53, Akhil Goyal wrote:
>>> On 7/18/2016 6:50 PM, Thomas Monjalon wrote:
2016-07-18 13:57, Sergio Gonzalez Monroy:
> On 18/07/2016 13:41, Akhil Goyal wrote:
>> In Ipsec-secgw
2016-07-18 13:57, Sergio Gonzalez Monroy:
> On 18/07/2016 13:41, Akhil Goyal wrote:
> > In Ipsec-secgw application, while adding the outer IP header,
> > it seems that the application does not update the checksum value
> > for outbound packets. This result in incorrect ip->checksum in
> > the
Hi Ceara
Could you help to let me know your firmware version?
And could you help to try with the standard DPDK example application, such as
testpmd, to see if there is the same issue?
Basically we always set the same size for both rx and tx buffer, like the
default one of 2048 for a lot of
On 18/07/2016 14:54, Christian Ehrhardt wrote:
> Hi Sergio,
> you might have seen that I had a similar version with awk in v2 IIRC.
> I also had the secondary tmp file just like you now.
> So, since it is so close to my old submission I wont object :-)
>
> Back then the discussion went for
On 18/07/2016 14:49, Akhil Goyal wrote:
> On 7/18/2016 6:27 PM, Sergio Gonzalez Monroy wrote:
>> Hi,
>>
>> On 18/07/2016 13:41, Akhil Goyal wrote:
>>> Hi,
>>>
>>> In Ipsec-secgw application, while adding the outer IP header, it seems
>>> that the application does not update the checksum value for
On 18/07/2016 14:53, Akhil Goyal wrote:
> On 7/18/2016 6:50 PM, Thomas Monjalon wrote:
>> 2016-07-18 13:57, Sergio Gonzalez Monroy:
>>> On 18/07/2016 13:41, Akhil Goyal wrote:
In Ipsec-secgw application, while adding the outer IP header,
it seems that the application does not update the
On 18/07/2016 14:25, Thomas Monjalon wrote:
> 2016-07-18 14:11, Sergio Gonzalez Monroy:
>> The sed syntax of '0,/regexp/' is GNU specific and fails with
>> non GNU sed in FreeBSD.
>>
>> To solve the issue we can use awk instead to remove duplicates.
> Christian, an opinion please?
Sorry, forgot
When parsing the parameters for virtual device initialization,
rte_kvargs structure was being freed only if there was an error,
not when parsing was successful.
Coverity issue: 124568
Fixes: f3e764fa2fb7 ("cryptodev: uninline parameter parsing")
Signed-off-by: Pablo de Lara
Acked-by: Reshma
Hi Yong,
rte_intr_dp_is_en() returns true when rte_intr_efd_enable() (the way to
enable data-path interrupt) sets a number of event fds.
In this case, "intr_conf.rxq=1" configuration causes "nb_efd=1". The
value comes from RTE_MIN($nb_efd, 1) from data-path, but not from link
event.
Per link
On 18/07/2016 12:49, Olivier Matz wrote:
> Hi Sergio,
>
> On 07/18/2016 01:33 PM, Sergio Gonzalez Monroy wrote:
>> On 12/07/2016 12:30, Olivier MATZ wrote:
>>> On 07/12/2016 11:22 AM, Xu, HuilongX wrote:
Actually, dozens of h files contain this code:
#ifdef __cplusplus
extern "C" {
#endif
Less common in drivers, but also exists.
In Trex, we do include ixgbe_type.h from c++ code because we do some register
writes ourselves to support some functionality not exposed in the DPDK API.
We can
The sed syntax of '0,/regexp/' is GNU specific and fails with
non GNU sed in FreeBSD.
To solve the issue we can use awk instead to remove duplicates.
Fixes: b2063f104db7 ("mk: filter duplicate configuration entries")
Signed-off-by: Sergio Gonzalez Monroy
---
mk/rte.sdkconfig.mk | 8 +++-
Hi,
On 18/07/2016 13:41, Akhil Goyal wrote:
> Hi,
>
> In Ipsec-secgw application, while adding the outer IP header, it seems that
> the application does not update the checksum value for outbound packets. This
> result in incorrect ip->checksum in the encrypted packet.
>
> Please let me know if
Hi Sergio,
On 07/18/2016 01:33 PM, Sergio Gonzalez Monroy wrote:
> On 12/07/2016 12:30, Olivier MATZ wrote:
>> On 07/12/2016 11:22 AM, Xu, HuilongX wrote:
>>> /examples/multi_process/client_server_mp/mp_server/mp_server/x86_64-native-linuxapp-gcc/mp_server
>>>
>>> -c f -n 4 --xen-dom0 -- -p 0x3
When parsing the parameters for virtual device initialization,
rte_kvargs structure was being freed only if there was an error,
not when parsing was successful.
Coverity issue: 124568
Fixes: f3e764fa2fb7 ("cryptodev: uninline parameter parsing")
Signed-off-by: Pablo de Lara
---
Changes in v2:
Signed-off-by: Ido Barnea
---
drivers/net/ixgbe/base/ixgbe_osdep.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/ixgbe/base/ixgbe_osdep.h
b/drivers/net/ixgbe/base/ixgbe_osdep.h
index 31cc1be..06d1ee1 100644
--- a/drivers/net/ixgbe/base/ixgbe_osdep.h
+++
Hi Adrien,
Thank you for getting back on this.
Please find my comments below.
Regards
_Sugesh
> -Original Message-
> From: Adrien Mazarguil [mailto:adrien.mazarguil at 6wind.com]
> Sent: Friday, July 15, 2016 4:04 PM
> To: Chandran, Sugesh
> Cc: dev at dpdk.org; Thomas Monjalon ;
>
Hi Andrey,
Regards
_Sugesh
> -Original Message-
> From: Chilikin, Andrey
> Sent: Friday, July 15, 2016 11:02 AM
> To: Chandran, Sugesh ; 'Adrien Mazarguil'
>
> Cc: dev at dpdk.org; Thomas Monjalon ;
> Zhang, Helin ; Wu, Jingjing
> ; Rasesh Mody ; Ajit
> Khaparde ; Rahul Lakkireddy
> ;
When parsing the parameters for virtual device initialization,
rte_kvargs structure was being freed only if there was an error,
not when parsing was successful.
Coverity issue: 124568
Fixes: f3e764fa2fb7 ("cryptodev: uninline parameter parsing")
Signed-off-by: Pablo de Lara
---
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara
> Sent: Monday, July 18, 2016 1:47 PM
> To: dev at dpdk.org
> Cc: Doherty, Declan ; Pattan, Reshma
> ; De Lara Guarch, Pablo
>
> Subject: [dpdk-dev] [PATCH v2] crypto: fix memory leak
>
> ...
>
> -Original Message-
> From: De Lara Guarch, Pablo
> Sent: Monday, July 18, 2016 1:47 PM
> To: dev at dpdk.org
> Cc: Doherty, Declan ; Pattan, Reshma
> ; De Lara Guarch, Pablo
>
> Subject: [PATCH v2] crypto: fix memory leak
>
> When parsing the parameters for virtual device
On Fri, Jul 15, 2016 at 10:03:02PM +0100, Bruce Richardson wrote:
> On Wed, Jul 13, 2016 at 03:02:37PM +0200, Adrien Mazarguil wrote:
> > DPDK uses GNU C language extensions in most of its code base. This is fine
> > for internal source files whose compilation flags are controlled by DPDK,
> >
> -Original Message-
> From: Pattan, Reshma
> Sent: Monday, July 18, 2016 1:32 PM
> To: De Lara Guarch, Pablo
> Cc: Doherty, Declan; dev at dpdk.org
> Subject: RE: [dpdk-dev] [PATCH] crypto: fix memory leak
>
>
>
> > -Original Message-
> > From: dev [mailto:dev-bounces at
Hi,
In Ipsec-secgw application, while adding the outer IP header, it seems that the
application does not update the checksum value for outbound packets. This
result in incorrect ip->checksum in the encrypted packet.
Please let me know if the checksum value is updated somewhere else or not.
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara
> Sent: Monday, July 18, 2016 12:20 PM
> To: dev at dpdk.org
> Cc: De Lara Guarch, Pablo
> Subject: [dpdk-dev] [PATCH] examples/performance-thread: add missing
> braces
>
> pthread_detach()
Hi,
On 12/07/2016 12:30, Olivier MATZ wrote:
> Hi Huilong,
>
>
> On 07/12/2016 11:22 AM, Xu, HuilongX wrote:
>> Hi all,
>>
>> I run mutli procee C/S model example failed on xen dom0. Does anyone
>> give me some suggest how to debug it?
>>
>> Thanks a lot
>>
>> test environment:
>>
>>OS:
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara
> Sent: Monday, July 18, 2016 1:01 PM
> To: dev at dpdk.org
> Cc: Doherty, Declan ; De Lara Guarch, Pablo
>
> Subject: [dpdk-dev] [PATCH] crypto: fix memory leak
>
> When parsing the parameters
pthread_detach() function was returning 0 even when not calling
lthread_detach(), due to missing braces in conditional
(extra indentation was applied, giving a hint this is the correct fix).
Fixes: 433ba6228f9a ("examples/performance-thread: add pthread_shim app")
Signed-off-by: Pablo de Lara
You have to use a patched valgrind.
Search the mailing list archives for some further details of the patches.
Matthew.
On Wed, Jul 13, 2016 at 09:53:46AM +, Eelco Chaudron wrote:
> Hi All,
>
> Has some one successfully ran DPDK 16.04 with Valgrind (latest SVN copy)?
> I???m still getting
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Shreyansh Jain
> Sent: Monday, July 18, 2016 12:33 PM
> To: olivier.matz at 6wind.com
> Cc: Hunt, David ; dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] doc: Fix incorrect mempool ops register macro
> name
>
>
There is an error when linking static EAL library with an application:
eal_alarm.c:(.text+0xd7): undefined reference to `clock_gettime'
eal_alarm.c:(.text+0x20f): undefined reference to `clock_gettime'
eal_timer.c:(.text+0x108): undefined reference to `clock_gettime'
eal_timer.c:(.text+0x146):
rte_ring_create expects the size of the ring to
be a power of 2. REFCNT_RING_SIZE value is not
power of 2 in-case if RTE_MAX_LCORE == 96.
Fix it by aligning the size to next power of 2 value.
Fixes: af75078f ("first public release")
Signed-off-by: Jerin Jacob
---
app/test/test_mbuf.c | 2 +-
1
In TM, the read size should be lesser than the write size to improve
performance.
This enables the TM ports to push maximum packets to the output port.
This fix changes the burst_read value from 64 to 24 in default_tm_params.
Signed-off-by: Chokkalingam, SankarX
Acked-by: Cristian Dumitrescu
Hi Yong,
I think the interrupt management should be simpler.
If you want to invest some time to rework this API, you
are very welcome.
2016-07-18 14:19, Liang, Cunming:
> Hi Yong,
>
> rte_intr_dp_is_en() returns true when rte_intr_efd_enable() (the way to
> enable data-path interrupt) sets a
This patch adds an image of the Live Migration for
virtio and sriov test configuration.
Signed-off-by: Bernard Iremonger
---
doc/guides/howto/lm_bond_virtio_sriov.rst | 4
1 file changed, 4 insertions(+)
diff --git a/doc/guides/howto/lm_bond_virtio_sriov.rst
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
---
This pachset set describes the procedure to Live migrate
a VM with Virtio and VF PMD's using the bonding PMD.
Changes in v4:
rename image file and patch
added links to rst file
updated rst file in line with comments
Changes in v3:
rename directory from how_to to howto
move to after FAQ in the
IP Pipeline application with the configuration for Flow Classification IPV6 did
not instantiate.
Parse error in section "PIPELINE1": entry "dma_src_mask" too long
The dma_src_mask check in pipeline_passthrough_parse_args() is wrong.
This fix increases the length of dma_src_mask by 1 for NULL
On Thu, Jul 07, 2016 at 08:24:43PM +0800, Ananyev, Konstantin wrote:
>
>
> > -Original Message-
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, Konstantin
> > Sent: Thursday, July 07, 2016 11:51 AM
> > To: Tao, Zhe; dev at dpdk.org
> > Cc: Tao, Zhe; Wu, Jingjing
> >
Error while executing action flow bulk command
pipeline> p 1 action flow bulk ./config/action.txt
Command "action flow bulk" failed
pipeline>
The flow action entries are added successfully.
But the return value is not computed correctly.
Due to this, the error message appears on CLI.
The return
Signed-off-by: Ido Barnea
---
drivers/net/i40e/i40e_ethdev.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
index 3f9f05e..84c86aa 100644
--- a/drivers/net/i40e/i40e_ethdev.c
+++
Hi Bernard,
Thank you for your answer.
However, to do this, I have to have some communication protocol to the VM's
application in order for it to do this sequence and acknowledge that it is now
safe to proceed with detaching the device.
This implies some kind of integration from the host side,
Hi Kevin,
On Sat, Jul 16, 2016 at 07:38:50PM +0300, Kevin Wilson wrote:
> Hello,
>
> I had tried to build the connectx-3 driver with latest git tree and
> it fails with
> ...
> ...
> == Build drivers/net/mlx4
> CC mlx4.o
> /work/down/temp/dpdk/drivers/net/mlx4/mlx4.c: In function
On Fri, Jul 15, 2016 at 10:34:40AM +, Ananyev, Konstantin wrote:
> Hi Jerin,
>
> > >
> > >
> > > > The CPU also
> > > > knows already the value that will be written to cons.tail and that
> > > > value does not depend on the previous read either. The CPU does not
> > > > know we are planning
Hi John
> -Original Message-
> From: Mcnamara, John
> Sent: Sunday, July 17, 2016 7:13 PM
> To: Iremonger, Bernard ; dev at dpdk.org
> Cc: Liu, Yong ; Xu, Qian Q ;
> yuanhan.liu at linux.intel.com
> Subject: RE: [PATCH v2 1/2] doc: live migration of VM with vhost_user on
> host
>
> >
Hi, Ido
Got your opinion. If you can also fix the 40G, it will be better. You can just
send a v2 patch, and I can ack it. Or I can also send the v2 patch for you.
Thanks
Jingjing
> -Original Message-
> From: Ido Barnea (ibarnea) [mailto:ibarnea at cisco.com]
> Sent: Monday, July 18,
Hi Jingjing,
The existing code was obviously wrong.
For the XL710 it returned: ETH_LINK_SPEED_1G |ETH_LINK_SPEED_10G
And for the X710: ETH_LINK_SPEED_10G |ETH_LINK_SPEED_40G
I did a minimal fix, and swapped the lines.
Any fix that will return the correct values is welcome.
For my purpose your
When timer_cb resets another running timer on the same lcore,
the list of expired timers is chained to the pending-list.
This commit prevents a running timer from being reset
by not its own timer_cb.
Signed-off-by: Hiroyuki Mikita
---
lib/librte_timer/rte_timer.c | 12 ++--
1 file
When timer_set_running_state() fails in rte_timer_manage(),
the failed timer is put back on pending-list.
In this case, another core tries to reset or stop the timer.
It does not need to be on pending-list
Signed-off-by: Hiroyuki Mikita
---
lib/librte_timer/rte_timer.c | 5 ++---
1 file
Hi, Barnea
Thanks for the patch. But for 40G link, I think it should be
dev_info->speed_capa = ETH_LINK_SPEED_40G.
Thanks
Jingjing
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ido Barnea (ibarnea)
> Sent: Sunday, July 17, 2016 11:43 PM
> To: dev at
This commit fixes incorrect pending-list manipulation
when getting list of expired timers in rte_timer_manage().
When timer_get_prev_entries() sets pending_head on prev,
the pending-list is broken.
The next of pending_head always becomes NULL.
In this depth level, it is not need to manipulate the
74 matches
Mail list logo