> On Nov 7, 2016, at 7:28 AM, Keren Hochman
> wrote:
>
> Hi,
> I need to run 2 process that uses dpdk on the same machine. One uses dpdk
> drivers, and the other just read from a pcap file. If I disable hugepages
> in the second process rte_mempool_create fails. What is the correct way to
>
A new DPDK release candidate is ready for testing:
http://dpdk.org/browse/dpdk/tag/?id=v16.11-rc3
It will be hopefully the last release candidate for 16.11.
As it is not so small, please test it very well in order to
secure the release planned to be out at the end of this week.
Thank you
> > L3fwd-power app needs vector mode to be disabled in order to work
> > properly. The app used to work previously, because it was using Rx scalar
> > function, but now it uses vector function.
> >
> > Vector mode needs to be disabled to make the app works, which has been
> > documented in
> > Add more information about VT-d/iommu settings for QAT PMD.
> > Remove limitation indicating QAT driver is not performance tuned.
> >
> > Signed-off-by: Fiona Trahe
> Acked-by: Arek Kusztal
Wrapped lines and applied.
> > multi-process support has been verified on non IA such as ARMv8.
> >
> > Signed-off-by: Jerin Jacob
>
> Acked-by: John McNamara
Applied, thanks
> > Fixes typos present in the documentation and code comments.
> >
> > Signed-off-by: Alain Leon
>
> Acked-by: John McNamara
Applied, thanks
2016-11-07 17:31, Ferruh Yigit:
> On 11/7/2016 5:25 PM, Fan Zhang wrote:
> > Coverity issue: 137871
> > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
> >
> > Signed-off-by: Fan Zhang
>
> Although checkpatch will complain about long lines, I believe it is
> better not
2016-11-07 15:59, Ferruh Yigit:
> On 11/7/2016 2:21 PM, Fan Zhang wrote:
> > Fixes: 0d547ed0 ("examples/ipsec-secgw: support configuration
> > file")
> > Coverity issue: 137875
> >
> > Signed-off-by: Fan Zhang
>
> Acked-by: Ferruh Yigit
Applied
2016-11-02 10:21, Morten Br?rup:
> > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> > > 2016-08-26 18:08, Wei Dai:
> > > > /**
> > > > * A structure used to retrieve statistics for an Ethernet port.
> > > > + * Not all statistics fields in struct rte_eth_stats are supported
> > >
2016-11-06 12:16, Wenzhuo Lu:
> After changing pkt[0] to pkt[], the example IP reassembly is not
> working.
> It's weird because this change is fine. There should be no
> difference between them.
> As a workaround, revert this change.
>
> Fixes: 347a1e037fd3 (lib: use C99 syntax for zero-size
2016-10-26 10:45, Yong Wang:
> During device reset/stop, vmxnet3 releases all mbufs in tx and
> rx cmd ring. For rx, we should go over all ring descriptors and
> free using rte_pktmbuf_free_seg() instead of rte_pktmbuf_free()
> as the metadata of the mbuf might not be properly initialized
>
2016-10-27 23:37, Rasesh Mody:
> From: Rasesh Mody
>
> Using GCC_VERSION to check gcc version and decide whether to include
> that compiler option.
>
> Fixes: ec94dbc57362 ("qede: add base driver")
No need for the above line.
It is fixing the commit below (which fixes above one).
> Fixes:
2016-10-28 22:49, Mody, Rasesh:
> > From: Stephen Hemminger
> > > ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y)
> > > -ifeq ($(shell gcc -Wno-unused-but-set-variable -Werror -E - < /dev/null
> > > > /dev/null 2>&1; echo $$?),0)
> > > +ifeq ($(shell test $(GCC_VERSION) -ge 44 && echo 1), 1)
> > >
2016-10-31 11:35, Rasesh Mody:
> From: Harish Patil
>
> Fix to advertise device's link speed capability based on NVM
> port configuration instead of returning driver supported speeds.
>
> Fixes: 95e67b479506 ("net/qede: add 100G link speed capability")
>
> Signed-off-by: Harish Patil
[...]
>
2016-11-07 09:12, Ajit Khaparde:
> Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing
> any unintentional havoc because of the usage of a signed variable.
>
> Coverity: 137874
>
> Signed-off-by: Ajit Khaparde
Series applied, thanks
As a regular contributor, please follow the
> > Nelio Laranjeiro (2):
> > doc: update mlx5 dependencies
> > doc: add mlx5 release notes
>
> Acked-by: Adrien Mazarguil
Applied with 2/2 v2, thanks
> > Fixes: 75ef62a94301 ("net/mlx5: fix link speed capability information")
> > Fixes: 188408719888 ("net/mlx5: fix support for newer link speeds")
> >
> > Signed-off-by: Nelio Laranjeiro
>
> Acked-by: John McNamara
Applied, thanks
2016-11-02 16:57, Adrien Mazarguil:
> On Wed, Nov 02, 2016 at 11:39:36AM +0100, Nelio Laranjeiro wrote:
> > Fill correctly the Mbuf Rx offloads.
> >
> > Nelio Laranjeiro (3):
> > net/mlx5: fix Rx checksum macros
> > net/mlx5: define explicit fields for Rx offloads
> > net/mlx: fix support
2016-11-07 17:12, Montorsi, Francesco:
> Hi all,
> if this was already raised, sorry for that.
> I noticed that the gmane archive for this mailing list is not working anymore:
>
> http://news.gmane.org/gmane.comp.networking.dpdk.devel
>
> reports "Page not found". Also I noticed that the
Hey,
You can try "site:" Google search operator, i.e. try to google:
site:dpdk.org/ml/archives/dev/
Regards,
Andriy
On Mon, Nov 7, 2016 at 6:12 PM, Montorsi, Francesco
wrote:
> Hi all,
> if this was already raised, sorry for that.
> I noticed that the gmane archive for this mailing list is
2016-10-28 14:19, Adrien Mazarguil:
> On Mon, Oct 24, 2016 at 11:10:59AM +0300, Raslan Darawsheh wrote:
> > When mbufs are smaller than MRU, multi-segment support must be enabled to
> > default set when not in promiscuous or allmulticast modes.
> >
> > Fixes: 9964b965ad69 ("net/mlx5: re-add Rx
2016-11-02 16:56, Adrien Mazarguil:
> On Tue, Nov 01, 2016 at 08:13:27AM +, Elad Persiko wrote:
> > Constraint alignment was not respected in Tx.
> >
> > Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path")
> >
> > Signed-off-by: Elad Persiko
>
> Acked-by: Adrien Mazarguil
"wrong use
> > Signed-off-by: John Daley
>
> Acked-by: John McNamara
Applied with few fixes
2016-11-01 11:08, John Daley:
> When the device was configured with an explicit maximum packet length,
> it would fail if the value was greater than MTU configured in CIMC/UCSM
> (plus L2 header length). It should have been compared against maximum
> allowed by the device.
>
> Fixes: bb34ffb848a0
2016-10-29 03:06, Jakub Palider:
> In some configurations there is mismatch between declared and actual
> descriptor count which under heavy load may lead to resource shortage.
>
> The first patch unifies the way head and tail indexes are handled and
> is crucial for compactness and succeeding
2016-11-04 17:10, Qiming Yang:
> If VF device is used as slave of a bond device, it will be polled
> periodically through alarm. Interrupt is involved here. And then
> VF will send I40E_VIRTCHNL_OP_GET_LINK_STAT message to
> PF to query the status. The response is handled by interrupt
> callback.
2016-11-04 19:08, Yuanhan Liu:
> From: Beilei Xing
>
> Turning off S-TAG identification will impact floating VEB,
> VFs can't communicate with each other.
> This patch fixes this issue by judging whether floating
> VEB is enabled, S-TAG identification will be turned off
> only when floating VEB
> > Removing stopping LLDP in firmware is a workaround for a known errata which
> > can cause Rx hang. But the changing will cause DCB configuration fails.
> > That is
> > because when LLDP is enabled, the return value of i40e_init_dcb is success.
> > But
> > following check just considered the
On 11/7/2016 5:25 PM, Fan Zhang wrote:
> Coverity issue: 137871
> Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
>
> Signed-off-by: Fan Zhang
> ---
Although checkpatch will complain about long lines, I believe it is
better not to wrap log messages.
Acked-by: Ferruh
Hi,
I need to run 2 process that uses dpdk on the same machine. One uses dpdk
drivers, and the other just read from a pcap file. If I disable hugepages
in the second process rte_mempool_create fails. What is the correct way to
handle this?
Thanks, Keren
Coverity issue: 137871
Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
Signed-off-by: Fan Zhang
---
examples/ipsec-secgw/parser.c | 116 +++---
1 file changed, 52 insertions(+), 64 deletions(-)
diff --git
On Sat, Nov 05, 2016 at 05:41:05PM +0800, Yuanhan Liu wrote:
> When queue number shrinks to 1 from X, the following code stops us
> sending the multiple queue ctrl message:
>
> if (nb_queues > 1) {
> if (virtio_set_multiple_queues(dev, nb_queues) != 0)
>
Hi all,
I'm using DPDK inside the OS of a VM that is SR-IOV-accelerated.
I noticed however that the "rte_ixgbevf_pmd" PMD does not report drops... I am
sending packets at TX side at 14Mpps; at the RX side I'm using "testpmd" and
it's reporting 'just' 12Mpps, but zero drops (also through
Hi all,
if this was already raised, sorry for that.
I noticed that the gmane archive for this mailing list is not working anymore:
http://news.gmane.org/gmane.comp.networking.dpdk.devel
reports "Page not found". Also I noticed that the gmane link on the dpdk.org
website has been removed.
> > Data Centre Bridge (DCB) configuration fails when SRIOV is enabled if nb_rxq
> > or nb_txq are greater than nb_q_per_pool.
> >
> > The failure occurs during configuration of the ixgbe PMD when it is
> > started, in
> > the ixgbe_check_mq_mode function.
> >
> > Fixes: 2a977b891f99
> > add information about new ixgbe PMD API.
> >
> > Signed-off-by: Bernard Iremonger
>
> Acked-by: John McNamara
Applied without patch 2 and squashed with release notes
2016-10-14 17:35, Bernard Iremonger:
> --- a/doc/guides/prog_guide/poll_mode_drv.rst
> +++ b/doc/guides/prog_guide/poll_mode_drv.rst
> @@ -356,3 +356,9 @@ Some additions in the metadata scheme are as follows:
> An example where queue numbers are used is as follows: ``tx_q7_bytes`` which
>
2016-11-06 11:55, Wenzhuo Lu:
> The VF TX registers are using PF's addresses by mistake,
> correct them.
>
> Fixes: 0198848a47f5 ("ixgbe: add access to specific device info")
>
> Reported-by: Xuekun Hu
> Signed-off-by: Wenzhuo Lu
Applied with other similar patches squashed, thanks
> > From the code, it looks like, hw->mac.autoneg, variable is used to switch
> > between calling either autoneg function or forcing speed/duplex function.
> > But
> > this variable is not modified in eth_em_start/eth_igb_start routines (it is
> > always
> > set to 1) even while forcing the link
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Robert Sanford
> Sent: Monday, August 1, 2016 22:43
> To: dev at dpdk.org
> Cc: Doherty, Declan ; De Lara Guarch, Pablo
> ; olivier.matz at 6wind.com
> Subject: [dpdk-dev] [PATCH 4/4] net/bonding: fix
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Robert Sanford
> Sent: Monday, August 1, 2016 22:43
> To: dev at dpdk.org
> Cc: Doherty, Declan ; De Lara Guarch, Pablo
> ; olivier.matz at 6wind.com
> Subject: [dpdk-dev] [PATCH 3/4] net/bonding: another fix
2016-11-02 11:14, Yuanhan Liu:
> Here is a small patchset of updating vhost programming and sample guide
> and comments, due to the removal of vhost-cuse.
Applied, thanks
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of linhaifeng
> Sent: Friday, November 4, 2016 04:30
> To: dev at dpdk.org
> Cc: Yigit, Ferruh ; Doherty, Declan
>
> Subject: [dpdk-dev] [PATCH] net/bonding: only handle lacp slow packets
>
> From: Linhaifeng
On 11/7/2016 2:21 PM, Fan Zhang wrote:
> Fixes: 0d547ed0 ("examples/ipsec-secgw: support configuration
> file")
> Coverity issue: 137875
>
> Signed-off-by: Fan Zhang
Acked-by: Ferruh Yigit
2016-11-05 17:40, Yuanhan Liu:
> This patchset fixes few issues related to virtio queue reconfigure:
> increase or shrink the queue number. The major issue and the reason
> behind is described with length details in patch 4 "net/virtio: allocate
> queue at init stage".
>
> Those bugs can not be
Hi,
> -Original Message-
> From: Pattan, Reshma
> Sent: Friday, November 4, 2016 4:43 PM
> To: Remy Horton
> Cc: dev at dpdk.org
> Subject: RE: [dpdk-dev] [PATCH v3 1/3] lib: add information metrics library
>
> Hi,
>
> Few comments below.
>
> > -Original Message-
> > From: dev
2016-11-05 17:40, Yuanhan Liu:
> -int virtio_dev_queue_setup(struct rte_eth_dev *dev,
> - int queue_type,
> - uint16_t queue_idx,
> - uint16_t vtpci_queue_idx,
> - uint16_t nb_desc,
> - unsigned int
Tested-by: Lei Yao
- Apply patch to v16.11-rc2
- Compile: Pass
- OS: Ubuntu16.04 4.4.0-45-generic
- GCC: 5.4.0
Most of the basic Virtio related test cases are tested with this patch. No
function issue found and no obvious performance drop. The following is the pass
case list:
TC1:?
2016-11-07 14:26, David Marchand:
> On Thu, Aug 4, 2016 at 1:50 PM, Igor Ryzhov wrote:
> > The rte_eal_pci_probe_one function could return false positive result if
> > no driver is found for the device.
> >
> > Signed-off-by: Igor Ryzhov
>
> Acked-by: David Marchand
Applied, thanks
2016-11-07 10:03, David Marchand:
> dev->driver should be set only if a driver did take the device.
>
> Signed-off-by: David Marchand
Applied, thanks
On Thu, Aug 4, 2016 at 1:50 PM, Igor Ryzhov wrote:
> The rte_eal_pci_probe_one function could return false positive result if
> no driver is found for the device.
>
> Signed-off-by: Igor Ryzhov
> ---
> lib/librte_eal/common/eal_common_pci.c | 2 +-
> 1 file changed, 1 insertion(+), 1
Coverity issue: 137871
Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
Signed-off-by: Fan Zhang
---
examples/ipsec-secgw/parser.c | 120 --
1 file changed, 58 insertions(+), 62 deletions(-)
diff --git
Fixes: 0d547ed0 ("examples/ipsec-secgw: support configuration
file")
Coverity issue: 137875
Signed-off-by: Fan Zhang
---
examples/ipsec-secgw/sa.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/examples/ipsec-secgw/sa.c b/examples/ipsec-secgw/sa.c
index
2016-11-02 14:05, Ferruh Yigit:
> On 11/2/2016 1:46 PM, Mauricio Vasquez B wrote:
> > Coverity detected this as an issue because internals->data will never be
> > NULL,
> > then the check is not necessary.
> >
> > Fixes: d082c0395bf6 ("ring: fix memory leak when detaching")
> > Coverity issue:
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Alain Leon
> Sent: Monday, November 7, 2016 3:20 AM
> To: dev at dpdk.org
> Cc: Alain Leon
> Subject: [dpdk-dev] [PATCH] doc: fix typos
>
> Fixes typos present in the documentation and code comments.
>
>
Library is designed to calculate latency stats and report them to the
application when queried. Library measures minimum, average, maximum
latencies and jitter in nano seconds.
Current implementation supports global latency stats, i.e. per application
stats.
Added new field to mbuf struct to mark
Hi Olivier,
> -Original Message-
> From: Olivier Matz [mailto:olivier.matz at 6wind.com]
> Sent: Friday, October 14, 2016 5:41 PM
> > On 9/22/2016 6:42 PM, Hemant Agrawal wrote:
> >> Hi Olivier
> >>
> >> On 9/19/2016 7:27 PM, Olivier Matz wrote:
> >>> Hi Hemant,
> >>>
> >>> On 09/16/2016
dev->driver should be set only if a driver did take the device.
Signed-off-by: David Marchand
---
lib/librte_eal/common/eal_common_pci.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/lib/librte_eal/common/eal_common_pci.c
b/lib/librte_eal/common/eal_common_pci.c
Hello Thomas,
On Sun, Nov 6, 2016 at 10:59 PM, Thomas Monjalon
wrote:
> 2016-10-25 14:50, Ben Walker:
>> If the user asks to probe multiple times, the probe
>> callback should only be called on devices that don't have
>> a driver already loaded.
>>
>> This is useful if a driver is registered
Prevent the arithmetic in bnxt_alloc_vnic_attributes from causing
any unintentional havoc because of the usage of a signed variable.
Coverity: 137874
Signed-off-by: Ajit Khaparde
--
v1: attempt did not seem complete.
v2: simplify the fix by redoing the fix for Coverity 127557
---
The cascading switch statement in bnxt_hwrm.c is missing the FALLTHROUGH
comment. Adding that.
Coverity: 127552
Signed-off-by: Ajit Khaparde
Acked-by: Ferruh Yigit
---
drivers/net/bnxt/bnxt_hwrm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/bnxt/bnxt_hwrm.c
Some of the production parts will arrive after the 16.11 release.
Back off support for those devices. We will add these IDs again
at an appropriate time.
Signed-off-by: Ajit Khaparde
--
v2: Adding Signed-off-by which seems to have fallen off in the previous attempt.
---
As some users are still using xen as the hypervisor, I suggest to
continue support for xen in DPDK. And from 16.11, I will be the
maintainer of all xen-related files.
Signed-off-by: Jianfeng Tan
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index
Fixes typos present in the documentation and code comments.
Signed-off-by: Alain Leon
---
doc/guides/prog_guide/overview.rst | 2 +-
lib/librte_kni/rte_kni.h | 2 +-
lib/librte_mbuf/rte_mbuf.h | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git
2016-11-04 14:18, Ferruh Yigit:
> On 11/3/2016 12:12 PM, Fan Zhang wrote:
> > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
> > Coverity issue: 137854
> >
> > Signed-off-by: Fan Zhang
>
> Acked-by: Ferruh Yigit
>
>
> Minor nit, for all coverity fixes, defined
2016-11-04 14:16, Ferruh Yigit:
> On 11/3/2016 12:12 PM, Fan Zhang wrote:
> > Fixes: 0d547ed03717 ("examples/ipsec-secgw: support configuration file")
> > Coverity issue: 137855
> >
> > Signed-off-by: Fan Zhang
>
> Acked-by: Ferruh Yigit
Merged with previous patch and applied
> > This commit fixes problem with device hanging because of
> > wrong pointer values in snow3g performance test
> >
> > Fixes: 97fe6461c7cb ("app/test: add SNOW 3G performance test")
> >
> > Signed-off-by: Arek Kusztal
> > ---
> Acked-by: Fiona Trahe
Applied, thanks
> > Updated comments on API to clarify which parts of mbufs are
> > copied or changed by crypto operations.
> >
> > Signed-off-by: Fiona Trahe
>
> Acked-by: Pablo de Lara
Applied, thanks
68 matches
Mail list logo