[dpdk-dev] [PATCH 5/5] librte_ethdev: add to use apistats

2020-12-03 Thread Hideyuki Yamashita
This patch modifies to use apistats by librte_ethdev. Signed-off-by: Hideyuki Yamashita --- lib/librte_ethdev/meson.build| 6 ++- lib/librte_ethdev/rte_apistats.c | 64 lib/librte_ethdev/rte_apistats.h | 64 lib/librte_ethdev

[dpdk-dev] [PATCH 4/5] docs: add description of apistats parameter into proc-info

2020-12-03 Thread Hideyuki Yamashita
This patch modifies document of proc-info to introduce "--apistats" parameter. Signed-off-by: Hideyuki Yamashita --- doc/guides/tools/proc_info.rst | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/doc/guides/tools/proc_info.rst b/doc/guides/tools/proc_info.rst index

[dpdk-dev] [PATCH 3/5] app/test-pmd: add to use apistats

2020-12-03 Thread Hideyuki Yamashita
This patch modifies to use apistats by testpmd app. - change on testpmd.c to call apistats functions to accumlate stats info Signed-off-by: Hideyuki Yamashita --- app/test-pmd/testpmd.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index

[dpdk-dev] [PATCH 2/5] app/proc-info: add to use apistats

2020-12-03 Thread Hideyuki Yamashita
This patch modifies to use apistats by proc-info app. - change on main.c to call apistats functions - change on Makefile to include experimental API Signed-off-by: Hideyuki Yamashita --- app/proc-info/main.c | 46 1 file changed, 46 insertions(+) dif

[dpdk-dev] [PATCH 1/5] maintainers: update maintainers file for apistats

2020-12-03 Thread Hideyuki Yamashita
This patch adds maintainer of rte_apistats.c and rte_apistats.h. Signed-off-by: Hideyuki Yamashita --- MAINTAINERS | 3 +++ 1 file changed, 3 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index eafe9f8..dba2acf 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -489,6 +489,9 @@ F: drivers/raw

[dpdk-dev] [PATCH 0/5] add apistats function

2020-12-03 Thread Hideyuki Yamashita
In general, DPDK application consumes CPU usage because it polls incoming packets using rx_burst API in infinite loop. This makes difficult to estimate how much CPU usage is really used to send/receive packets by the DPDK application. For example, even if no incoming packets arriving, CPU usage lo

Re: [dpdk-dev] [PATCH] net/qede: change driver's default load parameter

2020-12-03 Thread Rasesh Mody
Hi Devendra, >From: Devendra Singh Rawat >Sent: Thursday, December 03, 2020 5:30 AM > >Loading ecore clients (PMD or linux kernel driver) having different ecore >versions concurrently, is problematic from device's init/config perspective. >This fix changes driver's default load parameter to have

Re: [dpdk-dev] [PATCH v2] net/i40e: fix incorrect checksum flag of L4 checksum

2020-12-03 Thread Guo, Jia
Acked-by: Jeff Guo > -Original Message- > From: Murphy Yang > Sent: Thursday, December 3, 2020 3:51 PM > To: dev@dpdk.org > Cc: Yang, Qiming ; Yang, SteveX > ; Guo, Jia ; Xing, Beilei > ; Yang, MurphyX > Subject: [PATCH v2] net/i40e: fix incorrect checksum flag of L4 checksum > > When

[dpdk-dev] [Bug 590] Transmit errors with i40e VF using SR-IOV on a vSphere VM

2020-12-03 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=590 Bug ID: 590 Summary: Transmit errors with i40e VF using SR-IOV on a vSphere VM Product: DPDK Version: 20.11 Hardware: x86 OS: Linux Status: UNCONFIRMED

[dpdk-dev] [PATCH v2 9/9] net/ionic: minor logging fixups

2020-12-03 Thread Andrew Boyer
Expose ionic_opcode_to_str() so it can be used for dev cmds, too. Store the device name in struct adapter. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic.h | 1 + drivers/net/ionic/ionic_dev.c | 5 +++ drivers/net/ionic/ionic_dev.h | 2 + drivers/net/ionic/ionic_ethde

[dpdk-dev] [PATCH v2 8/9] net/ionic: minor refactorings and helper variables

2020-12-03 Thread Andrew Boyer
This makes the code clearer and conserves resources. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_ethdev.c | 5 ++--- drivers/net/ionic/ionic_lif.c| 15 ++- drivers/net/ionic/ionic_main.c | 18 +++--- 3 files changed, 19 insertions(+), 19 deletions(-) d

[dpdk-dev] [PATCH v2 7/9] net/ionic: warn if loopback mode is requested

2020-12-03 Thread Andrew Boyer
The ionic FW does not support loopback mode at this time. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_ethdev.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ionic_ethdev.c index cd79f250e..ce6ca9671 100644 --- a/drivers/ne

[dpdk-dev] [PATCH v2 6/9] net/ionic: convert 'deferred' boolean to a flag bit

2020-12-03 Thread Andrew Boyer
This conserves resources. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_lif.c | 4 ++-- drivers/net/ionic/ionic_lif.h | 2 +- drivers/net/ionic/ionic_rxtx.c | 10 ++ 3 files changed, 9 insertions(+), 7 deletions(-) diff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/i

[dpdk-dev] [PATCH v2 5/9] net/ionic: remove some unused fields

2020-12-03 Thread Andrew Boyer
This conserves resources. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic.h| 1 - drivers/net/ionic/ionic_dev.c| 5 + drivers/net/ionic/ionic_dev.h| 7 +-- drivers/net/ionic/ionic_ethdev.c | 2 -- drivers/net/ionic/ionic_lif.c| 22 ++ d

[dpdk-dev] [PATCH v2 4/9] net/ionic: check for cmd completion more frequently

2020-12-03 Thread Andrew Boyer
This reduces the startup time a bit. Signed-off-by: Neel Patel Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_dev.h | 4 +++- drivers/net/ionic/ionic_main.c | 36 - drivers/net/ionic/ionic_osdep.h | 10 - 3 files changed, 21 insertions(+), 29

[dpdk-dev] [PATCH v2 3/9] net/ionic: update documentation and MAINTAINERS

2020-12-03 Thread Andrew Boyer
The UNMAINTAINED flag will be removed in a future patch. Signed-off-by: Andrew Boyer --- MAINTAINERS| 3 ++- doc/guides/nics/features/ionic.ini | 2 ++ doc/guides/nics/ionic.rst | 13 +++-- 3 files changed, 11 insertions(+), 7 deletions(-) diff --git a

[dpdk-dev] [PATCH v2 2/9] net/ionic: update interface file to the latest version

2020-12-03 Thread Andrew Boyer
The ionic_if.h file contains the firmware interface definitions. Signed-off-by: Andrew Boyer --- drivers/net/ionic/ionic_if.h | 1343 ++-- drivers/net/ionic/ionic_regs.h |3 - 2 files changed, 930 insertions(+), 416 deletions(-) diff --git a/drivers/net/ionic/i

[dpdk-dev] [PATCH v2 1/9] net/ionic: connect ionic to the build system

2020-12-03 Thread Andrew Boyer
Otherwise the ionic PMD is never built. Signed-off-by: Andrew Boyer --- drivers/net/meson.build | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/meson.build b/drivers/net/meson.build index 29f477750..6e4aa6bf3 100644 --- a/drivers/net/meson.build +++ b/drivers/net/meson.build @@ -

[dpdk-dev] [PATCH v2 0/9] net/ionic: minor updates and documentation

2020-12-03 Thread Andrew Boyer
These patches make some minor changes to the ionic PMD. Signed-off-by: Andrew Boyer --- v2: * Address Ferruh's comments (thank you!) * Switch to net/ionic: prefix * Leave UNMAINTAINED flag for now * Split patch 8 into patch 8 & 9; drop whitespace changes Andrew Boyer (9): net/ionic: connect i

Re: [dpdk-dev] [PATCH] app/testpmd: fix IP checksum calculation

2020-12-03 Thread Stephen Hemminger
On Thu, 3 Dec 2020 10:35:50 -0600 George Prekas wrote: > On 12/3/2020 10:08 AM, Stephen Hemminger wrote: > > On Thu, 3 Dec 2020 07:59:54 -0600 > > George Prekas wrote: > > > >> Insert a compiler barrier to make sure that the IP checksum calculation > >> happens after setting all the fields of

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

2020-12-03 Thread Lance Richardson
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 by disabling EOP padding. Fixes: f4253e97e7b0 ("net/bnxt: set pa

[dpdk-dev] [PATCH] app/testpmd: fix IP checksum calculation

2020-12-03 Thread George Prekas
Insert a compiler barrier to make sure that the IP checksum calculation happens after setting all the fields of the IP header. Signed-off-by: George Prekas --- app/test-pmd/flowgen.c | 1 + 1 file changed, 1 insertion(+) diff --git a/app/test-pmd/flowgen.c b/app/test-pmd/flowgen.c index acf3e24

Re: [dpdk-dev] [PATCH] app/testpmd: fix IP checksum calculation

2020-12-03 Thread George Prekas
On 12/3/2020 10:08 AM, Stephen Hemminger wrote: On Thu, 3 Dec 2020 07:59:54 -0600 George Prekas wrote: Insert a compiler barrier to make sure that the IP checksum calculation happens after setting all the fields of the IP header. Signed-off-by: George Prekas I don't think this is necessary.

Re: [dpdk-dev] [PATCH v3] ip_frag: recalculate data length of fragment

2020-12-03 Thread Aaron Conole
Yicai Lu writes: > In some situations, we would get several ip fragments, which total > data length is less than minimum frame(64) and padding with zeros. > Examples: Second Fragment "a0a1 a2a3 a4a5 a6a7 ..." > and Third Fragment "a8a9 aaab acad aeaf b0b1 b2b3 ...". > Finally, we would

Re: [dpdk-dev] [PATCH] app/testpmd: fix IP checksum calculation

2020-12-03 Thread Stephen Hemminger
On Thu, 3 Dec 2020 07:59:54 -0600 George Prekas wrote: > Insert a compiler barrier to make sure that the IP checksum calculation > happens after setting all the fields of the IP header. > > Signed-off-by: George Prekas I don't think this is necessary. All other OS's don't have to do this. The

[dpdk-dev] [Bug 586] Virtio: virtio_dev_rx_queue_done() does not support packed ring

2020-12-03 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=586 Bug ID: 586 Summary: Virtio: virtio_dev_rx_queue_done() does not support packed ring Product: DPDK Version: 19.11 Hardware: All OS: All Status: UNCONFIRM

[dpdk-dev] [PATCH] net/qede: change driver's default load parameter

2020-12-03 Thread Devendra Singh Rawat
Loading ecore clients (PMD or linux kernel driver) having different ecore versions concurrently, is problematic from device's init/config perspective. This fix changes driver's default load parameter to have ecore version validated by MFW. All ecore clients having matching ecore version should be a

[dpdk-dev] [PATCH v3] ip_frag: recalculate data length of fragment

2020-12-03 Thread Yicai Lu
In some situations, we would get several ip fragments, which total data length is less than minimum frame(64) and padding with zeros. Examples: Second Fragment "a0a1 a2a3 a4a5 a6a7 ..." and Third Fragment "a8a9 aaab acad aeaf b0b1 b2b3 ...". Finally, we would reassemble Second and Third F

[dpdk-dev] DPDK Release Status Meeting 3/12/2020

2020-12-03 Thread Ferruh Yigit
Meeting minutes of 3 December 2020 -- Agenda: * Release Dates * 20.11 retrospective * LTS * OvS * Opens Participants: * Arm * Debian/Microsoft * Intel * Nvidia * NXP * Red Hat Release Dates - * v20.11 is released on Friday, 27 November 2020 * http

Re: [dpdk-dev] [PATCH v4 3/6] app/testpmd: remove restriction on txpkts set

2020-12-03 Thread Ferruh Yigit
On 12/3/2020 9:45 AM, Slava Ovsiienko wrote: Hi, Ferruh -Original Message- From: Ferruh Yigit Sent: Wednesday, December 2, 2020 14:07 To: Slava Ovsiienko ; NBU-Contact-Thomas Monjalon ; Wei Hu (Xavier) Cc: dev@dpdk.org; xavier.hu...@huawei.com Subject: Re: [dpdk-dev] [PATCH v4 3/6] ap

Re: [dpdk-dev] [PATCH v4 3/6] app/testpmd: remove restriction on txpkts set

2020-12-03 Thread Slava Ovsiienko
Hi, Ferruh > -Original Message- > From: Ferruh Yigit > Sent: Wednesday, December 2, 2020 14:07 > To: Slava Ovsiienko ; NBU-Contact-Thomas > Monjalon ; Wei Hu (Xavier) > > Cc: dev@dpdk.org; xavier.hu...@huawei.com > Subject: Re: [dpdk-dev] [PATCH v4 3/6] app/testpmd: remove restriction on

Re: [dpdk-dev] [PATCH v2] license: add licenses for exception cases

2020-12-03 Thread Bruce Richardson
On Wed, Dec 02, 2020 at 07:01:40PM +, John McNamara wrote: > The license/exceptions.txt file lists a small number of files > that have licenses that are exceptions to the three main > licenses defined in the Intellectual Property Policy of the > DPDK Charter. > > The three exception licenses a

[dpdk-dev] 19.11.6 patches review and test

2020-12-03 Thread luca . boccassi
Hi all, Here is a list of patches targeted for stable release 19.11.6. The planned date for the final release is December 17. Please help with testing and validation of your use cases and report any issues/results with reply-all to this mail. For the final release the fixes and reported validati

Re: [dpdk-dev] [PATCH v5] usertools/devbind: fix binding for built-in kernel drivers

2020-12-03 Thread Liu, Yongxin
Hi Anatoly, Do you have any further comments on this v5? Or you can submit your own patch directly. I am really expecting this issue to be fixed. Thank you very much. Yongxin > -Original Message- > From: dev On Behalf Of Yongxin Liu > Sent: Monday, November 23, 2020 11:06 > To: dev@d