[dpdk-dev] [PATCH] net/virtio-user: fix error run close(0)

2020-12-08 Thread 17826875952
From: Jiawei Zhu When i < VIRTIO_MAX_VIRTQUEUES and j == i, dev->callfds[i] and dev->kickfds[i] are default 0. So it will close(0), close the standard input (stdin). Fixes: e6e7ad8b3024 ("net/virtio-user: move eventfd open/close into init/uninit") Cc: sta...@dpdk.org Signed-off-by: Jiawei Zhu

[dpdk-dev] [PATCH] net/iavf: fix negative GTP-U flow rules create successfully

2020-12-08 Thread Murphy Yang
Currently, when use 'flow' command to create a negative GTP-U rule, it will be created successfully. The list shows the impacted outer and inner 'ipv4' GTP-U patterns with 'ipv4' or 'gtpu' type: - iavf_pattern_eth_ipv4_gtpu_ipv4_udp - iavf_pattern_eth_ipv4_gtpu_eh_ipv4_udp - iavf_pattern_eth_ip

[dpdk-dev] [PATCH v1 12/12] net/thunderx: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 11/12] net/sfc: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports VLAN tag. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixes: ff6

[dpdk-dev] [PATCH v1 10/12] net/qede: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 09/12] net/octeontx2: fix the jumbo frame flag condition for mtu

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 08/12] net/octeontx: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 07/12] net/ipn3ke: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 06/12] net/ice: fix the jumbo frame flag condition

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 05/12] net/iavf: fix the jumbo frame flag condition

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 04/12] net/i40e: fix the jumbo frame flag condition

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 03/12] net/hns3: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports dual VLAN tags. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixe

[dpdk-dev] [PATCH v1 02/12] net/e1000: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports VLAN tag. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixes: ef9

[dpdk-dev] [PATCH v1 01/12] net/dpaa2: fix the jumbo frame flag condition for mtu set

2020-12-08 Thread Steve Yang
The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but the Ether overhead is larger than 18 when it supports VLAN tag. That will cause the jumbo flag rx offload is wrong when MTU size is 'RTE_ETHER_MTU'. This fix will change the boundary condition with 'RTE_ETHER_MTU'. Fixes: e16

[dpdk-dev] [PATCH v1 00/12] fix rx packets dropped issue

2020-12-08 Thread Steve Yang
The jumbo frame used the 'RTE_ETHER_MAX_LEN' as boundary condition, this fix will change the boundary condition with 'RTE_ETHER_MTU'. When the MTU(1500) set, the frame type of rx packet will be different if used different overhead, it will cause the consistency issue, and the normal packet will be

Re: [dpdk-dev] [PATCH 12/17] net/bnxt: remove function declaration

2020-12-08 Thread Ajit Khaparde
On Tue, Dec 8, 2020 at 1:42 PM Stephen Hemminger wrote: > > These patches have a signature which is not compatible with submissions > to open source mailing list. If we follow the exact legal wording > then it conflicts with the meaning of Signed-Off-By. Yikes!! Let me a fresh set without that leg

Re: [dpdk-dev] [PATCH] net/bnxt: disable receive end-of-packet padding

2020-12-08 Thread Ajit Khaparde
On Thu, Dec 3, 2020 at 6:16 AM Lance Richardson wrote: > > Testing has shown that the packet forwarding rate for packet sizes > that are not a multiple of the cache line size is reduced when the > DMA size is padded to a multiple of the cache line size. Improve > performance for these packet sizes

Re: [dpdk-dev] [PATCH] net/bnxt: fix memory leak when mapping failure

2020-12-08 Thread Ajit Khaparde
On Mon, Nov 30, 2020 at 5:00 PM wangyunjian wrote: > > From: Yunjian Wang > > We allocated memory for the 'buf' when sending message to HWRM, > but we don't free it when mapping the address to IO address > fails. It will lead to memory leak. > > Fixes: 19e6af01bb36 ("net/bnxt: support get/set EEP

[dpdk-dev] [Bug 578] i40evf_check_api_version(): PF/VF API version mismatch:(0.0)-(1.1) with multiple DPDK instances using different VFs with the same PF

2020-12-08 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=578 beilei.x...@intel.com changed: What|Removed |Added CC||beilei.x...@intel.com Status|U

Re: [dpdk-dev] Faulty VF initialization during DPDK startup when multiple DPDK instances use different VFs with the same PF

2020-12-08 Thread Xing, Beilei
> -Original Message- > From: Juraj Linkeš > Sent: Tuesday, December 8, 2020 5:27 PM > To: Xing, Beilei ; David Marchand > ; Guo, Jia > Cc: dev@dpdk.org; Kinsella, Ray ; Andrew > Yourtchenko (ayourtch) ; Yigit, Ferruh > > Subject: RE: [dpdk-dev] Faulty VF initialization during DPDK star

Re: [dpdk-dev] [PATCH 2/3] net/i40e: add changes to support i40e PMD on windows

2020-12-08 Thread Kadam, Pallavi
On 12/6/2020 7:49 AM, Thomas Monjalon wrote: 05/12/2020 02:10, Pallavi Kadam: --- a/drivers/net/meson.build +++ b/drivers/net/meson.build @@ -1,9 +1,6 @@ # SPDX-License-Identifier: BSD-3-Clause # Copyright(c) 2017 Intel Corporation -if is_windows - subdir_done() -endif drive

[dpdk-dev] [PATCH 17/17] net/bnxt: modify ring index logic

2020-12-08 Thread Ajit Khaparde
Change the ring logic so that the index increments unbounded and mask it only when needed. Modify the existing macros so that the index is not masked. Add a new macro RING_IDX() to mask it only when needed. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_cpr.h | 10 ++- driv

[dpdk-dev] [PATCH 16/17] net/bnxt: add missing comments

2020-12-08 Thread Ajit Khaparde
Add and update some missing comments in the code. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h | 6 ++ drivers/net/bnxt/tf_ulp/ulp_template_db_act.c | 5 + drivers/net/bnxt/tf_ulp/ulp_template_db_class.c | 6 ++ 3 files changed, 17 insertions(+)

[dpdk-dev] [PATCH 14/17] net/bnxt: fix PF resource query

2020-12-08 Thread Ajit Khaparde
From: Somnath Kotur This cmd should be called by every driver after 'hwrm_func_cfg' to get the actual number of resources allocated by the HWRM. The values returned in the cmd are the max values for that PF. Also, now that the max values for the PF are computed in probe itself, no need to invoke

[dpdk-dev] [PATCH 15/17] net/bnxt: changes to indentation and coding style

2020-12-08 Thread Ajit Khaparde
Indentation and coding style fixes. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h| 4 ++-- drivers/net/bnxt/bnxt_ethdev.c | 9 - drivers/net/bnxt/bnxt_reps.c | 2 +- drivers/net/bnxt/bnxt_ring.c | 3 +-- 4 files changed, 8 insertions(+), 10 deletions(-) diff --git

[dpdk-dev] [PATCH 13/17] net/bnxt: fix vnic RSS configure function

2020-12-08 Thread Ajit Khaparde
From: Kalesh AP 1. moved invalid VNIC id check to the beginning of the function. 2. removed a duplicate check which avoids unnecessary code indentation. Fixes: 49d0709b257fc ("net/bnxt: delete and flush L2 filters cleanly") Cc: sta...@dpdk.org Signed-off-by: Kalesh AP Reviewed-by: Somnath Kotu

[dpdk-dev] [PATCH 12/17] net/bnxt: remove function declaration

2020-12-08 Thread Ajit Khaparde
Remove an orphan function declaration. It has not been defined yet and hence not used anywhere. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_hwrm.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_hwrm.h b/drivers/net/bnxt/bnxt_hwrm.h index 23ca6ab51..4b329077

[dpdk-dev] [PATCH 11/17] net/bnxt: use the right function to free mbuf

2020-12-08 Thread Ajit Khaparde
mbufs are being allocated using rte_mbuf_raw_alloc(). Use corresponding rte_mbuf_raw_free() to free mbuf. Signed-off-by: Somnath Kotur Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_reps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/bnxt/bnxt_reps.c

[dpdk-dev] [PATCH 09/17] net/bnxt: fix max rings computation

2020-12-08 Thread Ajit Khaparde
From: Sriharsha Basavapatna The current max_rings computation does not take into account the case when max_nq_rings is <= num_async_cpr. This results in a wrong value like 0, when max_nq_rings is 1. Fix this by subtracting num_async_cpr only when max_cp_rings > num_async_cpr. Apart from this, th

[dpdk-dev] [PATCH 08/17] net/bnxt: fix format specifier for unsigned int

2020-12-08 Thread Ajit Khaparde
&device requires the %u format specifer not the %d specifier, as &device is unsigned. Fixes: a46bbb57605b ("net/bnxt: update multi device design") Cc: sta...@dpdk.org Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/tf_core/tf_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[dpdk-dev] [PATCH 10/17] net/bnxt: support for 236 queues in NS3

2020-12-08 Thread Ajit Khaparde
From: Rahul Gupta For NS3, increased CONFIG_RTE_ETHDEV_QUEUE_STAT_CNTRS to support 236 counters and increased MAX_STINGRAY_RINGS in driver to support more rings. Signed-off-by: Rahul Gupta Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h | 3 +-- 1 file changed, 1 insertion(+), 2 deleti

Re: [dpdk-dev] [PATCH 2/3] net/i40e: add changes to support i40e PMD on windows

2020-12-08 Thread Kadam, Pallavi
On 12/7/2020 1:14 AM, Bruce Richardson wrote: On Sun, Dec 06, 2020 at 04:49:40PM +0100, Thomas Monjalon wrote: 05/12/2020 02:10, Pallavi Kadam: --- a/drivers/net/meson.build +++ b/drivers/net/meson.build @@ -1,9 +1,6 @@ # SPDX-License-Identifier: BSD-3-Clause # Copyright(c) 2017 Intel Cor

[dpdk-dev] [PATCH 07/17] net/bnxt: fix cleanup on mutex init failure

2020-12-08 Thread Ajit Khaparde
In case mutex init fails during initialization, start cleanup and fail the initialization process. Fixes: a73b8e939f10 ("net/bnxt: fix race between start and interrupt handler") Cc: sta...@dpdk.org Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_ethdev.c | 4 +++- 1 file changed, 3 inser

[dpdk-dev] [PATCH 05/17] net/bnxt: remove references to Thor

2020-12-08 Thread Ajit Khaparde
Refactor code to remove references to Thor. Instead use P5 as in phase 5 of development cycle since it is applicable to boards other than Thor as well. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h| 20 +++--- drivers/net/bnxt/bnxt_ethdev.c | 43 +++

[dpdk-dev] [PATCH 06/17] net/bnxt: fix to return error when fw command fails

2020-12-08 Thread Ajit Khaparde
From: Kalesh AP In bnxt_reta_update_op() and bnxt_rss_hash_update_op(), driver does not propagate the error back to the application when the fw command fails. Fixes: 378ab645bb0b ("net/bnxt: fix RSS RETA indirection table ops") Fixes: fcc0aa1edc10 ("net/bnxt: add RSS hash configuration") Cc: sta

[dpdk-dev] [PATCH 03/17] net/bnxt: remove unused field

2020-12-08 Thread Ajit Khaparde
Remove flow_xstat from bnxt structure. Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h index 9bd4f1da9..f13d586d4 100644 --- a/drivers/net/bnxt/bnxt.h +++ b/drivers/net/bnxt/bnxt.h @@ -8

[dpdk-dev] [PATCH 04/17] net/bnxt: release hwrm lock in the error case

2020-12-08 Thread Ajit Khaparde
From: Kalesh AP In __bnxt_hwrm_func_qcaps, when memory allocations fails driver is not releasing the hwrm lock. This patch fixes it by calling hwrm_unlock in that error case. Fixes: b7778e8a1c00 ("net/bnxt: refactor to properly allocate resources for PF/VF") Cc: sta...@dpdk.org Signed-off-by:

[dpdk-dev] [PATCH 01/17] net/bnxt: fix RX rings in RSS redirection table

2020-12-08 Thread Ajit Khaparde
From: Samik Gupta This commit introduces a limit on the number of RX rings included in the RSS redirection table to a value no larger than the size supported by Thor as defined by BNXT_RSS_TBL_SIZE_THOR. Fixes: d819382543f3 ("net/bnxt: add RSS redirection table operations") Cc: sta...@dpdk.org

[dpdk-dev] [PATCH 02/17] net/bnxt: fix VNIC config on Rx queue stop

2020-12-08 Thread Ajit Khaparde
From: Samik Gupta This commit reconfigures a vnic's default ring if the current default ring is stopped by the application. It picks the lowest numbered ring that is currently active to be the new default, and issues the hwrm_vnic_cfg command to update the configuration. Applies to adapters that

[dpdk-dev] [PATCH 00/17] fixes and refactoring changes for bnxt

2020-12-08 Thread Ajit Khaparde
Some fixes and refactoring changes for bnxt PMD. Ajit Khaparde (9): net/bnxt: remove unused field net/bnxt: remove references to Thor net/bnxt: fix cleanup on mutex init failure net/bnxt: fix format specifier for unsigned int net/bnxt: use the right function to free mbuf net/bnxt: remo

Re: [dpdk-dev] [PATCH 2/3] net/i40e: add changes to support i40e PMD on windows

2020-12-08 Thread Kadam, Pallavi
On 12/5/2020 5:52 AM, Dmitry Kozlyuk wrote: On Fri, 4 Dec 2020 17:10:19 -0800, Pallavi Kadam wrote: You could drop "add changes" and "i40e PMD" from subject line, as any commit changes something and topic is "net/i40e" already. Adding build changes to compile i40e PMD on windows. This is r

Re: [dpdk-dev] [PATCH v2 0/2] net/mlx5: hash list optimization

2020-12-08 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Suanming Mou > Sent: Thursday, December 3, 2020 4:19 AM > To: Slava Ovsiienko ; Matan Azrad > > Cc: Raslan Darawsheh ; dev@dpdk.org > Subject: [PATCH v2 0/2] net/mlx5: hash list optimization > > This patch set contains the hash list optimization for the

Re: [dpdk-dev] [PATCH 12/17] net/bnxt: remove function declaration

2020-12-08 Thread Stephen Hemminger
These patches have a signature which is not compatible with submissions to open source mailing list. If we follow the exact legal wording then it conflicts with the meaning of Signed-Off-By. > -- > This electronic communication and the information and any files transmitted > with it, or attached

Re: [dpdk-dev] [PATCH v2 1/2] common/sfc_efx/base: remove warnings about inline specifiers

2020-12-08 Thread Ferruh Yigit
On 12/1/2020 7:24 AM, Ivan Malov wrote: Windows build of the current libefx rejects these specifiers. They're unneeded anyway; the compiler should decide inlining. Fixes: 34285fd0891d ("common/sfc_efx/base: add match spec validate API") Cc: sta...@dpdk.org Signed-off-by: Ivan Malov Reviewed-by

Re: [dpdk-dev] [PATCH 2/2] net/netvsc: fix ethdev get_supported_ptypes

2020-12-08 Thread Kevin Traynor
On 08/12/2020 15:53, Ferruh Yigit wrote: > On 11/23/2020 10:19 PM, Stephen Hemminger wrote: >> When doing rte_eth_dev_get_supported_ptypes on netvsc device >> the values reported are incorrect if VF is not present. >> >> If no VF is present the device uses rte_net_get_ptype() >> which can return a

[dpdk-dev] [PATCH] rib: fix undefined behavior

2020-12-08 Thread Vladimir Medvedkin
According to GCC documentation for __builtin_clz: Returns the number of leading 0-bits in x, starting at the most significant bit position. If x is 0, the result is undefined. __builtin_clz will be called with 0 if the existing prefix address matches the one we want to insert. Fixes: 5a5793a5ffa2

Re: [dpdk-dev] [PATCH 2/2] net/netvsc: fix ethdev get_supported_ptypes

2020-12-08 Thread Ferruh Yigit
On 11/23/2020 10:19 PM, Stephen Hemminger wrote: When doing rte_eth_dev_get_supported_ptypes on netvsc device the values reported are incorrect if VF is not present. If no VF is present the device uses rte_net_get_ptype() which can return a wide range of values. Use the same table as TAP device

Re: [dpdk-dev] [PATCH 1/1] devtools: avoid installing static binaries

2020-12-08 Thread Thomas Monjalon
08/12/2020 16:37, David Marchand: > On Mon, Dec 7, 2020 at 6:33 PM Thomas Monjalon wrote: > > > > When testing compilation and checking ABI compatibility, > > there is no real need of static binaries eating disks. > > The static linkage of applications are tested with GCC and Clang, > > plus some

Re: [dpdk-dev] [PATCH V1] app/testpmd: fix queue stats mapping configuration

2020-12-08 Thread Ferruh Yigit
On 12/2/2020 12:48 PM, Min Hu (Connor) wrote: From: Huisong Li Currently, the queue stats mapping has the following problems: 1) Many PMD drivers don't support queue stats mapping. But there is no failure message after executing the command "set stat_qmap rx 0 2 2". 2) Once queue mapping is s

Re: [dpdk-dev] [PATCH 1/1] devtools: avoid installing static binaries

2020-12-08 Thread David Marchand
On Mon, Dec 7, 2020 at 6:33 PM Thomas Monjalon wrote: > > When testing compilation and checking ABI compatibility, > there is no real need of static binaries eating disks. > The static linkage of applications are tested with GCC and Clang, > plus some examples are statically linked. > The after-in

Re: [dpdk-dev] [PATCH 1/1] devtools: adjust verbosity of ABI check

2020-12-08 Thread Thomas Monjalon
08/12/2020 16:22, Kinsella, Ray: > > On 07/12/2020 17:32, Thomas Monjalon wrote: > > The scripts gen-abi.sh and check-abi.sh are updated > > to print error messages to stderr so they are likely never ignored. > > > > When called from test-meson-builds.sh, the standard messages on stdout > > can b

Re: [dpdk-dev] [PATCH 1/1] devtools: adjust verbosity of ABI check

2020-12-08 Thread David Marchand
On Mon, Dec 7, 2020 at 6:33 PM Thomas Monjalon wrote: > diff --git a/devtools/check-abi.sh b/devtools/check-abi.sh > index ab6748cfbc..381db2cdd1 100755 > --- a/devtools/check-abi.sh > +++ b/devtools/check-abi.sh [snip] > @@ -46,23 +46,24 @@ for dump in $(find $refdir -name "*.dump"); do >

Re: [dpdk-dev] [PATCH 1/1] devtools: adjust verbosity of ABI check

2020-12-08 Thread Kinsella, Ray
On 07/12/2020 17:32, Thomas Monjalon wrote: > The scripts gen-abi.sh and check-abi.sh are updated > to print error messages to stderr so they are likely never ignored. > > When called from test-meson-builds.sh, the standard messages on stdout > can be more quiet depending on the verbosity setti

Re: [dpdk-dev] [PATCH] ci: hook to Github Actions

2020-12-08 Thread David Marchand
On Thu, Nov 26, 2020 at 6:01 PM Honnappa Nagarahalli wrote: > > > Is there any guarantee that GitHub actions will be free forever? > > > > There is no "forever". > I think we are spending our efforts on things that will not work for the > community in the long run (unless the project spends money

Re: [dpdk-dev] [PATCH][v4] net/af_xdp: optimize RX path by removing the unneeded allocation mbuf

2020-12-08 Thread Ferruh Yigit
On 11/25/2020 1:03 PM, Loftus, Ciara wrote: Subject: [dpdk-dev] [PATCH][v4] net/af_xdp: optimize RX path by removing the unneeded allocation mbuf when receive packets, the max bunch number of mbuf are allocated if hardware does not receive the max bunch number packets, it will free redundancy mb

Re: [dpdk-dev] [PATCH] net/mlx5: fix shared age action validation

2020-12-08 Thread Raslan Darawsheh
HI, > -Original Message- > From: dev On Behalf Of Dekel Peled > Sent: Tuesday, November 24, 2020 3:46 PM > To: Slava Ovsiienko ; Shahaf Shuler > ; Matan Azrad > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] net/mlx5: fix shared age action validation > > Previous patch

Re: [dpdk-dev] [PATCH] net/mlx5: fix Verbs memory allocation callback

2020-12-08 Thread Raslan Darawsheh
Hi, > -Original Message- > From: Viacheslav Ovsiienko > Sent: Tuesday, November 24, 2020 12:27 PM > To: dev@dpdk.org > Cc: Raslan Darawsheh ; Matan Azrad > ; Ori Kam ; NBU-Contact-Thomas > Monjalon ; sta...@dpdk.org > Subject: [PATCH] net/mlx5: fix Verbs memory allocation callback > > The

Re: [dpdk-dev] [PATCH v1] net/axgbe: add support for device reset

2020-12-08 Thread Ferruh Yigit
On 11/20/2020 5:16 AM, Somalapuram, Amaranath wrote: [AMD Official Use Only - Internal Distribution Only] -Original Message- From: Sebastian, Selwin Sent: Thursday, November 19, 2020 6:52 PM To: dev@dpdk.org Cc: Somalapuram, Amaranath Subject: [PATCH v1] net/axgbe: add support for devi

Re: [dpdk-dev] [PATCH] net/bonding: Changes for accomodating jumbo frames in bond

2020-12-08 Thread Ferruh Yigit
On 11/26/2020 1:33 AM, Min Hu (Connor) wrote: It looks good to me, thanks. Converting to an explicit ack: Acked-by: Min Hu (Connor) Applied to dpdk-next-net/main, thanks.

Re: [dpdk-dev] [PATCH] net/e1000: fix incorrect return value if filter type not supported.

2020-12-08 Thread Wang, Haiyue
From: haz ben Sent: Tuesday, December 8, 2020 17:58 To: Wang, Haiyue Cc: ban, xiaozhen ; Guo, Jia ; dev@dpdk.org Subject: Re: [PATCH] net/e1000: fix incorrect return value if filter type not supported. my name is ban, xiaozhen. the commit message is "net/e1000: fix incorrect return value if fi

[dpdk-dev] Importing DATA into the application

2020-12-08 Thread Narcisa Ana Maria Vasile
Hi, While using the DPDK libs as DLLs, I've ran into some access violation errors. I believe they are caused by some incorrect imports. In Windows, accessing an imported variable is done either by using __declspec(dllimport), or by using an extra level of indirection[1]. Some examples of variab

Re: [dpdk-dev] [PATCH] net/e1000: fix incorrect return value if filter type not supported.

2020-12-08 Thread Wang, Haiyue
From: haz ben Sent: Tuesday, December 8, 2020 17:58 To: Wang, Haiyue Cc: ban, xiaozhen ; Guo, Jia ; dev@dpdk.org Subject: Re: [PATCH] net/e1000: fix incorrect return value if filter type not supported. my name is ban, xiaozhen. OK, ;-) the commit message is "net/e1000: fix incorrect return v

Re: [dpdk-dev] Importing DATA into the application

2020-12-08 Thread Dmitry Kozlyuk
On Tue, 08 Dec 2020 08:47:15 +0100, Thomas Monjalon wrote: > 08/12/2020 08:03, Narcisa Ana Maria Vasile: [...] > > However, there are a few issues to consider: > > * We cannot add __declspec(dllimport) when declaring per_lcore__lcore_id > > for example. We cannot have both __thread and __declspec

[dpdk-dev] Questions about vdpa patch--handle hardware error

2020-12-08 Thread Zhongjinming
Hi all, I find a patch about vdpa driver of dpdk --vdpa/mlx5: handle hardware error I want to know ,what is scenario of this patch? Our enviroment is dpdk19.11 +some patches from Mellanox ,and Connect6 DX. We tested vdpa by taking a long time of iperf between two VMs , but found that VMs can

[dpdk-dev] [PATCH v1 2/2] linux/kni: Added support for KNI multiple fifos

2020-12-08 Thread dheemanth
In order to improve performance, the KNI is made to support multiple fifos, So that multiple threads pinned to multiple cores can process packets in parallel. Signed-off-by: dheemanth --- app/test/test_kni.c | 4 +- drivers/net/kni/rte_eth_kni.c | 5 +- examples/kni/main.c

[dpdk-dev] [PATCH] net/e1000: fix incorrect return value if filter type not supported.

2020-12-08 Thread banxiaozhen
From: banxiaozhen Signed-off-by: banxiaozhen --- drivers/net/e1000/igb_ethdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e1000/igb_ethdev.c index 647aa8d99..8695426c2 100644 --- a/drivers/net/e1000/igb_ethdev.c +++ b/drivers/net/e1000/igb_

Re: [dpdk-dev] [PATCH] net/e1000: fix incorrect return value if filter type not supported.

2020-12-08 Thread haz ben
my name is ban, xiaozhen. the commit message is "net/e1000: fix incorrect return value if filter type not supported." does additional details should be add? Wang, Haiyue 于2020年12月8日周二 下午5:04写道: > > -Original Message- > > From: banxiaozhen benhaz1...@gmail.com > > > Sent: Tuesday, Decemb

Re: [dpdk-dev] [PATCH v2] maintainers: update for testpmd

2020-12-08 Thread Ferruh Yigit
On 12/2/2020 6:33 AM, beilei.x...@intel.com wrote: From: Beilei Xing Replace Beilei Xing with Xiaoyun Li. Signed-off-by: Beilei Xing Acked-by: Ferruh Yigit

Re: [dpdk-dev] [PATCH 1/1] devtools: avoid installing static binaries

2020-12-08 Thread Bruce Richardson
On Mon, Dec 07, 2020 at 07:12:32PM +0100, Thomas Monjalon wrote: > 07/12/2020 18:47, Bruce Richardson: > > On Mon, Dec 07, 2020 at 06:33:19PM +0100, Thomas Monjalon wrote: > > > When testing compilation and checking ABI compatibility, > > > there is no real need of static binaries eating disks. > >

Re: [dpdk-dev] Faulty VF initialization during DPDK startup when multiple DPDK instances use different VFs with the same PF

2020-12-08 Thread Juraj Linkeš
> -Original Message- > From: Xing, Beilei > Sent: Tuesday, December 8, 2020 8:14 AM > To: David Marchand ; Guo, Jia > > Cc: dev@dpdk.org; Kinsella, Ray ; Andrew Yourtchenko > (ayourtch) ; Juraj Linkeš ; > Yigit, Ferruh > Subject: RE: [dpdk-dev] Faulty VF initialization during DPDK star

Re: [dpdk-dev] [PATCH] net/e1000: fix incorrect return value if filter type not supported.

2020-12-08 Thread Wang, Haiyue
> -Original Message- > From: banxiaozhen benhaz1...@gmail.com > Sent: Tuesday, December 8, 2020 16:10 > To: Guo, Jia ; Wang, Haiyue > Cc: dev@dpdk.org; banxiaozhen > Subject: [PATCH] net/e1000: fix incorrect return value if filter type not > supported. > > From: banxiaozhen > > Signe

Re: [dpdk-dev] [PATCH v2] net/mlx5: fix flow descriptor allocation in Direct Verbs mode.

2020-12-08 Thread Xueming(Steven) Li
>-Original Message- >From: Gregory Etelson >Sent: Tuesday, December 8, 2020 4:17 PM >To: dev@dpdk.org >Cc: Gregory Etelson ; Matan Azrad >; Raslan Darawsheh ; Slava >Ovsiienko ; Shahaf Shuler ; >Bing Zhao ; Xueming(Steven) Li > >Subject: [PATCH v2] net/mlx5: fix flow descriptor allocati

[dpdk-dev] [PATCH v2] net/mlx5: fix flow descriptor allocation in Direct Verbs mode.

2020-12-08 Thread Gregory Etelson
Initialize flow descriptor tunnel member during flow creation. Prevent access to stale data and pointers when flow descriptor is reallocated after release. Fix flow index validation. Fixes: e7bfa3596a0a ("net/mlx5: separate the flow handle resource") Fixes: 8bb81f2649b1 ("net/mlx5: use thread spec

[dpdk-dev] [PATCH] net/e1000: fix incorrect return value if filter type not supported.

2020-12-08 Thread banxiaozhen
From: banxiaozhen Signed-off-by: banxiaozhen --- drivers/net/e1000/igb_ethdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/e1000/igb_ethdev.c b/drivers/net/e1000/igb_ethdev.c index 647aa8d99..8695426c2 100644 --- a/drivers/net/e1000/igb_ethdev.c +++ b/drivers/net/e1000/igb_