2016-03-17 01:19, Xie, Huawei:
> On 3/16/2016 2:44 PM, Yuanhan Liu wrote:
> > We have to reset the virtio net hdr at virtio_enqueue_offload()
> > before, due to all mbufs share a single virtio_hdr structure:
> >
> > struct virtio_net_hdr_mrg_rxbuf virtio_hdr = {{0, }, 0};
> >
> > foreach (m
2016-03-17 11:37, Panu Matilainen:
> On 03/17/2016 01:22 AM, Sergio Gonzalez Monroy wrote:
> > The linker script is generated by simply finding all libraries in
> > RTE_OUTPUT/lib.
> >
> > The issue shows up when re-building the DPDK, hence already having a
> > linker script in that directory, resu
The link speed configuration is now done with bitmaps so 100G speed
requires only a new bit flag.
The actual link speed is a number so its size must be increased from
16-bit to 32-bit.
Signed-off-by: Marc Sune
Tested-by: Nelio Laranjeiro
Signed-off-by: Thomas Monjalon
---
app/test-pmd/cmdline.
From: Marc Sune
It is a helper for the bitmap configuration.
Signed-off-by: Marc Sune
Signed-off-by: Thomas Monjalon
---
lib/librte_ether/rte_ethdev.c | 31 +++
lib/librte_ether/rte_ethdev.h | 13 +
lib/librte_ether/rte_ether_version.m
From: Marc Sune
This patch redesigns the API to set the link speed/s configuration
of an ethernet port. Specifically:
- it allows to define a set of advertised speeds for
auto-negociation.
- it allows to disable link auto-negociation (single fixed speed).
- default: auto-negociate all supporte
From: Marc Sune
The speed capabilities of a device can be retrieved with
rte_eth_dev_info_get().
The new field speed_capa is initialized in the drivers without
taking care of device characteristics in this patch.
When the capabilities of a driver are accurate, the table in
overview.rst must be f
From: Marc Sune
The speed numbers ETH_LINK_SPEED_ are renamed ETH_SPEED_NUM_.
The prefix ETH_LINK_SPEED_ is kept for AUTONEG and will be used
for bit flags in next patch.
Signed-off-by: Marc Sune
---
app/test-pmd/cmdline.c| 10 +-
app/test/virtual_pmd.c
From: Marc Sune
The code for checking and parsing speed/duplex was duplicated.
The new function is also checking the speed/duplex combination.
Signed-off-by: Marc Sune
---
app/test-pmd/cmdline.c | 99 --
1 file changed, 47 insertions(+), 52 delet
From: Marc Sune
Some duplex values are replaced from 0 to half-duplex when link is down.
Some drivers are still using their own constants for duplex modes.
Signed-off-by: Marc Sune
---
drivers/net/e1000/em_ethdev.c | 2 +-
drivers/net/e1000/igb_ethdev.c | 2 +-
drivers/net/ixgbe/ixgb
Define and use ETH_LINK_UP and ETH_LINK_DOWN where appropriate.
Signed-off-by: Marc Sune
Signed-off-by: Thomas Monjalon
---
app/test-pipeline/init.c | 2 +-
app/test-pmd/testpmd.c | 2 +-
app/test/test_pmd_perf.c
There are still too few tests and reviews, especially for
autonegotiation with Intel devices (patch #6).
I would not be surprised to see some bugs in this rework.
The capabilities must be adapted per device. It can be
improved in a separate patch.
It will be integrated in 16.04-rc2.
Please test a
Issuing a zero objects dequeue with a single consumer has no effect.
Doing so with multiple consumers, can get more than one thread to succeed
the compare-and-set operation and observe starvation or even deadlock in
the while loop that checks for preceding dequeues. The problematic piece
of code w
Sure, I'm sending it again with your suggestions.
Lazaros.
On Wed, Mar 16, 2016 at 1:21 PM, Bruce Richardson
wrote:
> On Tue, Mar 15, 2016 at 06:58:45PM +0200, Lazaros Koromilas wrote:
>> Issuing a zero objects dequeue with a single consumer has no effect.
>> Doing so with multiple consumers, ca
CC Maryam and Olivier who had discussions about imissed and other stats:
http://dpdk.org/ml/archives/dev/2015-August/022905.html
http://dpdk.org/ml/archives/dev/2015-September/023351.html
http://dpdk.org/ml/archives/dev/2015-September/023612.html
2016-03-10 16:03, Igor Ryzh
On Thursday, March 03/10/16, 2016 at 16:03:30 +0300, Igor Ryzhov wrote:
> Comment for "ierrors" counter says that it counts erroneous received packets.
> But for some reason "imissed" counter is added to "ierrors" counter in most
> drivers. It is a mistake, because missed packets are obviously no
On 14/03/16 17:54, Saurabh Mishra wrote:
> Hi,
>
> We are planning to support virtio, vmxnet3, ixgbe, i40e, bxn2x and SR-IOV
> on some of them with DPDK.
>
> We have seen that even if we give correct number of mbufs given the number
> hugepages reserved, rte_eth_tx_queue_setup() may still fail wi
On 10/03/16 07:51, Wu, Jingjing wrote:
> Hi, Zoltan
>
>> -Original Message-
>> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss
>> Sent: Wednesday, March 2, 2016 3:19 AM
>> To: dev at dpdk.org
>> Subject: [dpdk-dev] ixgbe TX function selection
>>
>> Hi,
>>
>> I've notic
Hi Lazaros,
On Thu, Mar 17, 2016 at 4:49 PM, Lazaros Koromilas
wrote:
> Issuing a zero objects dequeue with a single consumer has no effect.
> Doing so with multiple consumers, can get more than one thread to succeed
> the compare-and-set operation and observe starvation or even deadlock in
> th
RE fixing email addresses...
2016-03-17 16:39, Thomas Monjalon:
> 2016-02-25 13:48, Aaron Conole:
> > This series brings a number of code cleanups to allow building using gcc6,
> > with various legitimate warnings being fixed.
> >
> > In particular, patch 3 ("drivers/net/e1000: Fix missing bracke
2016-02-25 13:48, Aaron Conole:
> This series brings a number of code cleanups to allow building using gcc6,
> with various legitimate warnings being fixed.
>
> In particular, patch 3 ("drivers/net/e1000: Fix missing brackets") should be
> checked for correctness (it does not alter any behavior fr
From: Yaacov Hazan
VLAN insertion can be done in hardware when supported in Verbs. A software
fallback is provided otherwise. The software implementation is also used
when multi-packet send is enabled on a queue, as both features are mutually
exclusive.
Signed-off-by: Yaacov Hazan
Signed-off-by
From: Olga Shern
Environment variable MLX5_PMD_ENABLE_PADDING enables HW packet padding
in PCI bus transactions.
When packet size is cache aligned and CRC stripping is enabled, 4 fewer
bytes are written to the PCI bus. Enabling padding makes such packets
aligned again.
In cases where PCI bandwi
From: Olga Shern
Until now, CRC was always stripped by hardware. This feature can be
configured since MLNX_OFED >= 3.2.
Signed-off-by: Olga Shern
---
doc/guides/nics/mlx5.rst | 2 ++
doc/guides/rel_notes/release_16_04.rst | 6 ++
drivers/net/mlx5/Makefile | 5
From: Or Ami
Secondary processes are expected to use queues and other resources
allocated by the primary, however Verbs resources can only be shared
between processes when inherited through fork().
This limitation can be worked around for TX by configuring separate queues
from secondary processe
From: Or Ami
Burst functions are updated to make sure applications cannot attempt to
send/receive after link is brought down.
Signed-off-by: Or Ami
---
doc/guides/rel_notes/release_16_04.rst | 4 ++
drivers/net/mlx5/mlx5.c| 2 +
drivers/net/mlx5/mlx5.h| 2 +
This patchset adds to mlx5 a few features available in mlx4 (TX from
secondary processes) or provided by Verbs (support for HW packet padding,
TX VLAN insertion).
Release notes and documentation are updated accordingly.
Changes in v3:
- Removed compilation option for TX VLAN insertion, the method
2016-02-24 14:47, Ilya Maximets:
> Implementation of rte_vhost_enqueue_burst() based on lockless ring-buffer
> algorithm and contains almost all to be thread-safe, but it's not.
>
> This set adds required changes.
>
> First patch in set is a standalone patch that fixes many times discussed
> issu
Helin,
You have probably missed this (old) patch / bug report.
2015-08-28 16:08, Sergey Balabanov:
> Hi,
>
> Probably I missed something in understanding why the mac is not set on kni
> creation. Any comments would be highly appreciated.
>
> Thanks,
> Sergey
>
> On Friday 28 August 2015 16:06:
On Thu, Mar 17, 2016 at 10:19:24AM -0700, Stephen Hemminger wrote:
> > > A better patch would be to move the data structure into the
> > > code block used, and get rid of the useless else (rte_panic never
> > > returns);
> > > and fix the indentation, and use C99 initialization which should make
Signed-off-by: John Daley
---
drivers/net/enic/enic_rx.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/drivers/net/enic/enic_rx.c b/drivers/net/enic/enic_rx.c
index 817a891..232987a 100644
--- a/drivers/net/enic/enic_rx.c
+++ b/drivers/net/enic/enic_rx.c
@@ -266,7
If the packet_error bit in the completion descriptor is set, the
remainder of the descriptor and data are invalid. PKT_RX_MAC_ERR
was set in the mbuf->ol_flags if packet_error was set and used
later to indicate an error packet. But since PKT_RX_MAC_ERR is
defined as 0, mbuf flags and packet types a
In the receive path, the function to set mbuf ol_flags used the
mbuf packet_type before it was set.
Fixes: 947d860c821f ("enic: improve Rx performance")
Signed-off-by: John Daley
---
drivers/net/enic/enic_rx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/e
These patches fix up some bugs in the enic receive path.
John Daley (3):
enic: mbuf->ol_flags could be set incorrectly
enic: handle error packets properly
enic: small cleanup- remove a packet_error conditional
drivers/net/enic/enic_rx.c | 53 ++
>From Cunming:
> I'm trying to understand the motivation.
>
> I don't think you're going to gracefully exit intr thread but leave all
> other eal threads live. We don't have API to new launch intr thread again.
The doc comment added for rte_eal_intr_exit already explains this. According
to the
On 2016-03-17 15:42, Thomas Monjalon wrote:
> 2015-08-07 19:20, Jan Kiszka:
>> On VHOST_*_RESET_OWNER, we reinitialize the device but without telling
>> the application. That will cause crashes when it continues to invoke
>> vhost services on the device. Fix it by calling the destruction hook if
>>
From: Nelson Escobar
Add checks to make sure we don't try to allocate more tx or rx queues
than we support.
Signed-off-by: Nelson Escobar
Reviewed-by: John Daley
---
drivers/net/enic/enic_ethdev.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/drivers/net/enic/enic_ethdev
From: Nelson Escobar
Signed-off-by: Nelson Escobar
Acked-by: John Daley
---
drivers/net/enic/enic_main.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c
index cd7857f..2f79cf0 100644
--- a/drivers/net/enic/en
With LIBRTE_PMD_XENVIRT enabled testpmd is built in a way to ONLY work
in XEN environments.
It will surface as:
PMD: gntalloc: ioctl error
EAL: Error - exiting with code: 1
Cause: Creation of mbuf pool for socket 0 failed
With LIBRTE_PMD_XENVIRT enabled this now tries the xen style gran
From: Nelson Escobar
When the requested number of rx descriptors was less than the amount
configured on the vic, enic_alloc_rq() was incorrectly setting
enic->config.rq_desc_count to the lower value. This screwed up later
calls to enic_alloc_rq().
Signed-off-by: Nelson Escobar
Reviewed-by: Joh
Change maintainers for ENIC PMD.
Signed-off-by: John Daley
---
doc/guides/nics/enic.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/doc/guides/nics/enic.rst b/doc/guides/nics/enic.rst
index 2a228fd..e67c3db 100644
--- a/doc/guides/nics/enic.rst
+++ b/doc/guides/nics/enic.
2015-08-07 19:20, Jan Kiszka:
> On VHOST_*_RESET_OWNER, we reinitialize the device but without telling
> the application. That will cause crashes when it continues to invoke
> vhost services on the device. Fix it by calling the destruction hook if
> the device is still in use.
>
> Signed-off-by: J
I am setting up OVS inside a Linux container. This OVS is built using DPDK
library.
During the startup of ovs-vswitchd, it core dumped due to fail to mmap.
in eal_memory.c
virtaddr = mmap(vma_addr, hugepage_sz, PROT_READ | PROT_WRITE,
MAP_SHARED, fd, 0);
This call is m
This is a PMD for the Amazon ethernet ENA family.
The driver operates variety of ENA adapters through feature negotiation
with the adapter and upgradable commands set.
ENA driver handles PCI Physical and Virtual ENA functions.
Signed-off-by: Evgeny Schemeilin
Signed-off-by: Jan Medala
Signed-off
Implementation of platform specific code for ENA communication layer.
Signed-off-by: Evgeny Schemeilin
Signed-off-by: Jan Medala
Signed-off-by: Jakub Palider
---
drivers/net/ena/base/ena_plat_dpdk.h | 217 +++
1 file changed, 217 insertions(+)
create mode 10064
Low level common abstraction for ENA device communication.
Signed-off-by: Netanel Belgazal
Signed-off-by: Jan Medala
Signed-off-by: Jakub Palider
---
drivers/net/ena/base/ena_com.c | 2809 +++
drivers/net/ena/base/ena_com.h | 1052 +
Signed-off-by: Alexander Matushevsky
Signed-off-by: Jan Medala
Signed-off-by: Jakub Palider
---
MAINTAINERS | 8 ++
doc/guides/nics/ena.rst | 251 +++
doc/guides/nics/index.rst| 1 +
doc/guides/nics/overview.rst | 116 +++
v3:
Additional features for Amazon ENA:
* Low Latenycy Queue (LLQ) for Tx
* RSS
v4:
* Improved doc
* Improved style according to checkpatch script
* Fixed build problems on: i686, clang, +shared, +debug
v5:
* Removed 'cvos' environment code from ena Makefile
* Driver symbol version fixed to DPDK_16
2016-03-17 14:57 GMT+01:00 Thomas Monjalon :
> 2016-03-17 14:48, Jan M?dala:
> > >
> > > > #
> > > > +# Compile burst-oriented Amazon ENA PMD driver
> > > > +#
> > > > +CONFIG_RTE_LIBRTE_ENA_PMD=y
> > > > +CONFIG_RTE_LIBRTE_ENA_DEBUG_INIT=y
> > >
> > > Do you really want initialization debuggin t
Hi Stephen,
Please, could you turn it into a real patch with your sign-off?
Thanks
2016-02-14 12:22, Stephen Hemminger:
> A better patch would be to move the data structure into the
> code block used, and get rid of the useless else (rte_panic never returns);
> and fix the indentation, and use C9
Any news? a v5 could be part of the RC2.
2016-03-08 20:59, Thomas Monjalon:
> 2016-02-09 23:04, Liming Sun:
> > rte_vect.h was missing earlier thus LPM was disabled and l3fwd is
> > not able to compile. This commit implements the vector api and
> > enable LPM in the tilegx configuration by default
2016-03-17 14:48, Jan M?dala:
> >
> > > #
> > > +# Compile burst-oriented Amazon ENA PMD driver
> > > +#
> > > +CONFIG_RTE_LIBRTE_ENA_PMD=y
> > > +CONFIG_RTE_LIBRTE_ENA_DEBUG_INIT=y
> >
> > Do you really want initialization debuggin to be on by default? Normally,
> > we
> > keep all debug options
>
> > #
> > +# Compile burst-oriented Amazon ENA PMD driver
> > +#
> > +CONFIG_RTE_LIBRTE_ENA_PMD=y
> > +CONFIG_RTE_LIBRTE_ENA_DEBUG_INIT=y
>
> Do you really want initialization debuggin to be on by default? Normally,
> we
> keep all debug options disabled.
This is actually error logging, so it's
-- next part --
A non-text attachment was scrubbed...
Name: Document8.zip
Type: application/zip
Size: 4982 bytes
Desc: Document8.zip
URL:
<http://dpdk.org/ml/archives/dev/attachments/20160317/77884823/attachment-0001.zip>
Hi Lu, Helin, Greg,
Many thanks for your response, which is really quick. Now, If I want to
implement L2 bridging with Intel virtualization technologies, using 82599
controller, then Michael is my only hope, as getting the new kernel
versions and upstream support will take considerable amount of
On 03/17/2016 01:22 AM, Sergio Gonzalez Monroy wrote:
> The linker script is generated by simply finding all libraries in
> RTE_OUTPUT/lib.
>
> The issue shows up when re-building the DPDK, hence already having a
> linker script in that directory, resulting in the linker script
> including itself.
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhang, Roy Fan
> Sent: Monday, March 14, 2016 5:57 PM
> To: dev at dpdk.org; Kobylinski, MichalX
> Cc: Dumitrescu, Cristian ; Singh, Jasvinder
> ; thomas.monjalon at 6wind.com; Glynn,
> Michael J
> Subject: [dp
Could this patch be reviewed please.
Thanks,
Gowrishankar
On Monday 07 March 2016 07:43 PM, Gowrishankar wrote:
> From: Gowri Shankar
>
> For a secondary process address space to map hugepages from every segment of
> primary process, hugepage_file entries has to be mapped reversely from the
> li
On 16 March 2016 at 19:14, Bruce Richardson
wrote:
> On Wed, Mar 16, 2016 at 03:51:53PM +0800, Jianbo Liu wrote:
>> Hi Wenzhuo,
>>
>> On 16 March 2016 at 14:06, Lu, Wenzhuo wrote:
>> > HI Jianbo,
>> >
>> >
>> >> -Original Message-
>> >> From: dev [mailto:dev-bounces at dpdk.org] On Behal
On Thu, 17 Mar 2016 15:18:15 +0100
Thomas Monjalon wrote:
> Hi Stephen,
>
> Please, could you turn it into a real patch with your sign-off?
> Thanks
>
> 2016-02-14 12:22, Stephen Hemminger:
> > A better patch would be to move the data structure into the
> > code block used, and get rid of the u
Add a deprecation notice for coming changes in mempool for 16.07.
Signed-off-by: Olivier Matz
---
doc/guides/rel_notes/deprecation.rst | 8
1 file changed, 8 insertions(+)
diff --git a/doc/guides/rel_notes/deprecation.rst
b/doc/guides/rel_notes/deprecation.rst
index 252a096..3e8e327 1
From: Michal Kobylinski
Updated ip_pipeline app is using new changes from LPM library
(Increased number of next hops and added new config structure
for LPM IPv4).
Fixes: 7164439d017d ("lpm: add a new config structure for IPv4")
Signed-off-by: Michal Kobylinski
Acked-by: Cristian Dumitrescu
Hi Huawei,
thank you for the quick response and for the pointer to the 16.04-rc1
version. Nice!
I think it would be great also to have a sanity check on the gpa_to_vva().
Although nothing recent has hit it we had some problems in that area
in the past.
Regards,
Patrik
On 03/17/2016 02:35 AM,
On 3/16/2016 8:53 PM, Patrik Andersson R wrote:
> Hello,
>
> When taking a snapshot of a running VM instance, using OpenStack
> "nova image-create", I noticed that one OVS pmd-thread eventually
> failed in DPDK rte_vhost_dequeue_burst() with repeating log entries:
>
>compute-0-6 ovs-vswitchd[38
On 3/16/2016 2:44 PM, Yuanhan Liu wrote:
> We have to reset the virtio net hdr at virtio_enqueue_offload()
> before, due to all mbufs share a single virtio_hdr structure:
>
> struct virtio_net_hdr_mrg_rxbuf virtio_hdr = {{0, }, 0};
>
> foreach (mbuf) {
> virtio_enqueue_off
64 matches
Mail list logo
