Hi,
I'm sorry, but quota character in "Fixes" tags is still wrong, causing the
checking script errors.
It should be " (0x22 ASCII), not ' (0x27 ASCII).
Beside this:
Acked-by: Viacheslav Ovsiienko
> -Original Message-
> From: Dey, Souvik
> Sent: Tuesday, February 2, 2021 19:49
> To: R
> -Original Message-
> From: Wang, Haiyue
> Sent: Wednesday, February 3, 2021 10:19 AM
> To: dev@dpdk.org
> Cc: pvale...@redhat.com; acon...@redhat.com;
> david.march...@redhat.com; Zhang, Qi Z ; Rong, Leyi
> ; Tu, Lijuan ;
> ruifeng.w...@arm.com; feifei.wa...@arm.com; Wang, Haiyue
> ;
The stored metadata in all registers C were lost in E-Switch egress
mirroring flows due to HW limitation. The register C0 keeps the
source vport index that also was used as one of the flow matcher.
While sample action and jump action (jump to table X) was in the
E-Switch egress flow, the flow in t
On Wed, Feb 3, 2021 at 9:07 AM Zhang, Qi Z wrote:
> > -Original Message-
> > From: Wang, Haiyue
> > Sent: Wednesday, February 3, 2021 10:19 AM
> > To: dev@dpdk.org
> > Cc: pvale...@redhat.com; acon...@redhat.com;
> > david.march...@redhat.com; Zhang, Qi Z ; Rong, Leyi
> > ; Tu, Lijuan ;
>
When doing a mempool dump or an audit, the application can panic because
the length of the cache is greater than the flush threshold, which is
seen as a fatal error. But this can temporarily happen when the mempool
is in use.
Fix the panic condition to abort only when the cache length is greater
t
Hi Jack,
>-Original Message-
>From: dev On Behalf Of Ori Kam
>Sent: Wednesday, February 3, 2021 9:51 AM
>To: Jack Min
>Cc: dev@dpdk.org
>Subject: Re: [dpdk-dev] [PATCH 1/2] doc: more info of rte flow shared action
>
>Hi Jack,
>
>> -Original Message-
>> From: Jack Min
>> Subject:
Hi Jack,
Regards,
Asaf Penso
>-Original Message-
>From: dev On Behalf Of Jack Min
>Sent: Tuesday, February 2, 2021 2:24 PM
>To: Matan Azrad ; Shahaf Shuler
>; Slava Ovsiienko
>Cc: dev@dpdk.org
>Subject: [dpdk-dev] [PATCH 2/2] doc: group mlx5 shared actions
>
>Put all supported shared ac
On 2/3/2021 7:46 AM, Lijun Ou wrote:
This patch series add some bugfixes and some small
functions. In order to fix some bugfixes, we had to
do some code rework, for example patch[13/17].
Chengchang Tang (6):
net/hns3: support module EEPROM dump
net/hns3: add more registers to dump
net/h
Hi Huawei,
On 2/1/21 8:43 AM, 谢华伟(此时此刻） wrote:
>
> On 2021/1/29 11:25, chris wrote:
>> Hi ferruh and maxime:
>>
>> v6 changes:
>>
>> send v6. Let us discuss if merge in this or early next release.
>
> Ping.
The -rc2 was released on the 29th, so I think it is too late for this
release.
As Ferru
On 1/29/21 4:18 AM, 谢华伟(此时此刻） wrote:
> From: "huawei.xhw"
>
> Currently virtio PMD asssumes legacy device uses PIO bar.
> There are three ways to get PIO(PortIO) address for virtio legacy device.
> under igb_uio, get pio address from uio/uio# sysfs attribute
> under uio_pci_generic:
>
On 1/29/21 4:18 AM, 谢华伟(此时此刻） wrote:
> |From: "huawei.xhw" With IO BAR, we get
> PIO(programmed IO) address. With MMIO BAR, we get mapped virtual
> address. We distinguish PIO(Programmed IO) and MMIO(memory mapped IO) by
> their address like how kernel does. ioread/write8/16/32 is provided to
>
On 1/29/2021 11:57 AM, Ferruh Yigit wrote:
On 1/29/2021 9:22 AM, Lijun Ou wrote:
Here series add some updates for hns3 rst and TM feature
note for hns3 as well as FEC feature notes.
v3:
1. merge patch[2/3] into patch[1] base on v2
2. fix some format errors
v2:
1. fix a meson build warning for
On 1/31/21 1:36 PM, Cheng Jiang wrote:
> It makes more sense to add error propagation for rte_ioat_completed_ops.
>
> Signed-off-by: Cheng Jiang
> ---
> examples/vhost/ioat.c | 14 ++
> 1 file changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/examples/vhost/ioat.c b/exam
On 2/2/21 7:17 AM, Cheng Jiang wrote:
> Remove async inflight packet counter since there is no need to keep
> tracking it. Increase MAX_ENQUEUED_SIZE to prevent packet segment number
> tracking ring from being exhausted.
Is that an optimization or a fix?
If the former, let's move it to v21.05.
On 2/2/21 8:24 AM, Cheng Jiang wrote:
> Change the type of buff_idx from uint64_t to uint32_t to fix coverity
> issue.
>
> Coverity issue: 366264
> Fixes: a68ba8e0a6b6 ("examples/vhost: refactor vhost data path")
>
> Signed-off-by: Cheng Jiang
> ---
> v2: fixed the format of git log
>
> exa
On 2/1/2021 7:56 PM, Ferruh Yigit wrote:
On 1/31/2021 10:36 AM, Heinrich Kuhn wrote:
Release-on-close has been implemented for the NFP PMD. Remove the
UNMAINTAINED flag.
Signed-off-by: Heinrich Kuhn
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAIN
Existing code to create shared RSS action doesn't fully check
driver capabilities.
Using older driver, if DevX capabilities are insufficient,
the IBV operations are used.
In this case the ind_table_modify operation is not supported, and
shared RSS action can't be modified after creation.
This patc
Implementation of mlx5_hrxq_get() tries to get pointer to a hrxq
object, by reusing an existing hrxq, or creating a new one.
There is no check of the obtained pointer, so using it might result
in error.
This patch adds check of the pointer before using it, and return 0
if the pointer is NULL to in
On Wed, Feb 3, 2021 at 8:58 AM Thomas Monjalon wrote:
>
> From: Asaf Penso
>
> Adding more information about the release milestones.
> This includes the scope of change, expectations, etc.
>
> Signed-off-by: Asaf Penso
> Signed-off-by: Thomas Monjalon
> Acked-by: John McNamara
> ---
> v2: fix
On 2/2/2021 7:09 PM, Pallavi Kadam wrote:
Add documentation to support i40e PMD on Windows.
Update the release notes and features list for the same.
v3 changes:
- Replaced the commit link with netuio README.rst
documentation link
v2 changes:
- updated doc/guides/nics/f
03/02/2021 11:14, David Marchand:
> On Wed, Feb 3, 2021 at 8:58 AM Thomas Monjalon wrote:
> > +Roadmap
> > +~~~
> > +
> > +* Content: new features in libraries, drivers, applications, and examples.
> > +* Timeframe: to be published before the first day of the release cycle.
>
> On the form, r
> -Original Message-
> From: Power, Ciara
> Sent: Tuesday, 2 February, 2021 17:58
> To: dev@dpdk.org
> Cc: akhil.go...@nxp.com; Doherty, Declan ;
> Dybkowski, AdamX ; Power, Ciara
>
> Subject: [PATCH v2 1/3] doc: update release notes for crypto perf script
>
> Add release note for the ne
> -Original Message-
> From: Power, Ciara
> Sent: Tuesday, 2 February, 2021 17:58
> To: dev@dpdk.org
> Cc: akhil.go...@nxp.com; Doherty, Declan ;
> Dybkowski, AdamX ; Power, Ciara
>
> Subject: [PATCH v2 2/3] doc: fix crypto perf script guide
>
> The guide for using the crypto perf graphi
> -Original Message-
> From: Power, Ciara
> Sent: Tuesday, 2 February, 2021 17:58
> To: dev@dpdk.org
> Cc: akhil.go...@nxp.com; Doherty, Declan ;
> Dybkowski, AdamX ; Power, Ciara
>
> Subject: [PATCH v2 3/3] test/cryptodev: fix handling for config parameters
>
> The crypto perf graphing
The script build-dict.sh was added in DPDK 20.08.
It generates a better dictionary for spell checking
done via checkpatch.
Signed-off-by: Thomas Monjalon
---
doc/guides/contributing/patches.rst | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/doc/guides/contributing
On 2/2/2021 3:23 PM, Ivan Malov wrote:
Innermost IP length and outer UDP datagram length must be
greater than or equal to the corresponding values derived
from the MSS; otherwise, the checksum offloads will break.
Fixes: c1ce2ba218f8 ("net/sfc: support tunnel TSO on EF10 native Tx datapath")
Fix
On 2/3/2021 10:22 AM, Ferruh Yigit wrote:
On 2/2/2021 7:09 PM, Pallavi Kadam wrote:
Add documentation to support i40e PMD on Windows.
Update the release notes and features list for the same.
v3 changes:
- Replaced the commit link with netuio README.rst
documentation link
v2 changes:
在 2021/2/3 17:24, Ferruh Yigit 写道:
On 2/3/2021 7:46 AM, Lijun Ou wrote:
This patch series add some bugfixes and some small
functions. In order to fix some bugfixes, we had to
do some code rework, for example patch[13/17].
Chengchang Tang (6):
net/hns3: support module EEPROM dump
net/hn
Hi,
> -Original Message-
> From: Alexander Kozyrev
> Sent: Tuesday, February 2, 2021 4:08 AM
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Raslan Darawsheh ; Slava
> Ovsiienko ; Matan Azrad
> Subject: [PATCH] net/mlx5: check FW miniCQE format capabilities
>
> miniCQE formats for Flow Tag an
Hi,
> -Original Message-
> From: Alexander Kozyrev
> Sent: Monday, February 1, 2021 7:17 PM
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Raslan Darawsheh ; Slava
> Ovsiienko ; Matan Azrad
> Subject: [PATCH] net/mlx5: fix miniCQE configuration for Verbs
>
> Verbs cannot be used to configure
On 2/3/2021 11:05 AM, oulijun wrote:
在 2021/2/3 17:24, Ferruh Yigit 写道:
On 2/3/2021 7:46 AM, Lijun Ou wrote:
This patch series add some bugfixes and some small
functions. In order to fix some bugfixes, we had to
do some code rework, for example patch[13/17].
Chengchang Tang (6):
net/hns3:
Hi,
> -Original Message-
> From: Maxime Coquelin
> Sent: Wednesday, February 3, 2021 5:49 PM
> To: Jiang, Cheng1 ; Xia, Chenbo
>
> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
> ; Wang, Yinan
> Subject: Re: [PATCH] examples/vhost: add error propagation in ioat ops
>
>
>
> On 1/31/21 1
On 2/3/2021 9:02 AM, Asaf Penso wrote:
Hi Jack,
-Original Message-
From: dev On Behalf Of Ori Kam
Sent: Wednesday, February 3, 2021 9:51 AM
To: Jack Min
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH 1/2] doc: more info of rte flow shared action
Hi Jack,
-Original Message-
On 2/2/2021 12:23 PM, Jack Min wrote:
This series update documents as following:
1. Add more information about shared action in rte_flow.rst.
2. Group all supported shared actions in mlx5.rst
Xiaoyu Min (2):
doc: more info of rte flow shared action
doc: group mlx5 shared actions
Series
Hi Slava,
Initially v2 of the patch has " instead of ' in the Fixes tags, but it
gave some warnings as wrong quota. So thought of changing it to '. I can change
it back again, do you suggest me to submit v4 with with corrected quota
character or its ok to have the v3 of the patch itself
Hi,
> -Original Message-
> From: Maxime Coquelin
> Sent: Wednesday, February 3, 2021 5:52 PM
> To: Jiang, Cheng1 ; Xia, Chenbo
>
> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
> ; Wang, Yinan
> Subject: Re: [PATCH v3] examples/vhost: remove async inflight packet
> counter
>
>
>
> On 2
On Wed, Feb 03, 2021 at 11:30:57AM +0100, Thomas Monjalon wrote:
> The script build-dict.sh was added in DPDK 20.08.
> It generates a better dictionary for spell checking
> done via checkpatch.
>
> Signed-off-by: Thomas Monjalon
> ---
> doc/guides/contributing/patches.rst | 10 +++---
> 1 fi
在 2021/2/3 19:46, Ferruh Yigit 写道:
On 2/3/2021 11:05 AM, oulijun wrote:
在 2021/2/3 17:24, Ferruh Yigit 写道:
On 2/3/2021 7:46 AM, Lijun Ou wrote:
This patch series add some bugfixes and some small
functions. In order to fix some bugfixes, we had to
do some code rework, for example patch[13/
From: Huisong Li
When link information is updated in the firmware, the link information
is updated first and then the link status is updated. In a 1s periodic
task, PF driver queries the link information and then obtains link status.
It may lead to a 1s time difference for obtaining valid link in
Based on some suggestions, we reorganize the fixes
and resend the only fixes.
Chengchang Tang (4):
net/hns3: fix stats flip overflow
net/hns3: replace all atomic type with C11 atomic builtins
net/hns3: fix cmdq cleared during firmware process
net/hns3: fix VF reset after MBX failed
Chengw
the driver should not use the fixed value as the validity check of
RSS indirection table size with HW supported. As a result, it will
cause misjudgment when the RSS reta size with HW supported have
changed.
Fixes: c37ca66f2b27 ("net/hns3: support RSS")
Cc: sta...@dpdk.org
Signed-off-by: Lijun Ou
From: Huisong Li
When the hardware link status changes, the firmware proactively
reports the link status change message, and then driver update
link status. This feature is lack of a switch to control in pf
driver. Otherwise, this feature does not take effect when the
kernel PF driver that suppor
From: Huisong Li
When jumbo frame is enabled, the MTU size needs to be modified
based on 'max_rx_pkt_len'. Driver needs to check the validity
of 'max_rx_pkt_len'. And it should be in the range of
HNS3_DEFAULT_FRAME_LEN and HNS3_MAX_FRAME_LEN. Otherwise, it may
cause that the MTU size is inconsist
From: Chengchang Tang
Currently, statistics may overflow in some scenarios.
For example, if HW statistics are reset by stats reset operation,
but there are still a lot of residual packets exist in the HW
queues and these packets are error packets, flip may occurred
because the ipacket is obtaine
From: Chengchang Tang
Currently, during the VF reset, the VF will send a MBX to inform
PF to reset it and the disable command bit will be set whether
the MBX is successful. Generally, multiple reset attempts are made
after a failure. However, because the command is disabled, all
subsequent reset
From: Chengwen Feng
User could config Port or TC's peak rate by TM ops, but hardware does
not support peak rate which lower than 1Mbps. So we constraint tm
peak rate must be at least 1Mbps.
Fixes: c09c7847d892 ("net/hns3: support traffic management")
Signed-off-by: Chengwen Feng
Signed-off-by:
From: Chengwen Feng
The Kunpeng920 and Kunpeng930 don't support parse MPLS packet, so
remove the type from supported flow items.
Fixes: fcba820d9b9e ("net/hns3: support flow director")
Cc: sta...@dpdk.org
Signed-off-by: Chengwen Feng
Signed-off-by: Lijun Ou
---
drivers/net/hns3/hns3_flow.c |
From: Chengchang Tang
There are two scenarios that command queue uninit performed
concurrently with the firmware command: asynchronous command
and timeout command.
For asynchronous command, if a large number of functions send
commands, these commands may need to be queued to wait for
firmware pr
From: Chengwen Feng
After FD rule config success, driver will malloc fdir_rule to hold the
rule info, if malloc fail the FD rule in hardware was not cleanup.
Fixes: fcba820d9b9e ("net/hns3: support flow director")
Cc: sta...@dpdk.org
Signed-off-by: Chengwen Feng
Signed-off-by: Lijun Ou
---
d
From: Chengchang Tang
Replace all the atomic type with C11 atomic builtins in hns3
PMD.
Signed-off-by: Chengchang Tang
Signed-off-by: Lijun Ou
---
drivers/net/hns3/hns3_cmd.c | 13 +++--
drivers/net/hns3/hns3_ethdev.c| 21 -
drivers/net/hns3/hns3_ethdev.h
From: Chengchang Tang
Because of the '9ca2f16' have merged, the current hns3
pmd driver can not be directly complied on the kunpeng930
FPGA board. Therefore, we need to fix the meson build.
Besides, add kunpeng 930 SoC meson cross compile target.
Fixes: 9ca2f16faa7f ("config/arm: isolate generic
From: Chengchang Tang
Because of the '9ca2f16' have merged, the current hns3
pmd driver can not be directly complied on the kunpeng920
server board. Therefore, we need to fix the meson build.
Besides, add kunpeng 920 SoC meson cross compile target.
Fixes: 9ca2f16faa7f ("config/arm: isolate gener
This series fix meson build for kunpeng920 and
kunpeng930 boards.
Chengchang Tang (2):
config/arm: fix Hisilicon kunpeng920 SoC build
config/arm: fix Hisilicon kunpeng930 Soc build
config/arm/arm64_kunpeng920_linux_gcc | 19 +++
config/arm/arm64_kunpeng930_linux_gcc
On 2/3/21 12:52 PM, Jiang, Cheng1 wrote:
> Hi,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, February 3, 2021 5:49 PM
>> To: Jiang, Cheng1 ; Xia, Chenbo
>>
>> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
>> ; Wang, Yinan
>> Subject: Re: [PATCH] examples/vhost: a
03/02/2021 13:18, Bruce Richardson:
> On Wed, Feb 03, 2021 at 11:30:57AM +0100, Thomas Monjalon wrote:
> > -
> > https://raw.githubusercontent.com/codespell-project/codespell/master/codespell_lib/data/dictionary.txt
> > + git clone https://github.com/codespell-project/codespell.git
> >
>
>
On 2/3/21 1:11 PM, Jiang, Cheng1 wrote:
> Hi,
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, February 3, 2021 5:52 PM
>> To: Jiang, Cheng1 ; Xia, Chenbo
>>
>> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
>> ; Wang, Yinan
>> Subject: Re: [PATCH v3] examples/vhost:
https://bugs.dpdk.org/show_bug.cgi?id=633
Bug ID: 633
Summary: Secondary process crash with virtio PMD
Product: DPDK
Version: unspecified
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
> -Original Message-
> From: Maxime Coquelin
> Sent: Wednesday, February 3, 2021 8:49 PM
> To: Jiang, Cheng1 ; Xia, Chenbo
>
> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
> ; Wang, Yinan
> Subject: Re: [PATCH] examples/vhost: add error propagation in ioat ops
>
>
>
> On 2/3/21 12:52
https://bugs.dpdk.org/show_bug.cgi?id=634
Bug ID: 634
Summary: i40e: rte_flow doesn't allow RSS action on IPV4 only
Product: DPDK
Version: 20.11
Hardware: x86
OS: Linux
Status: UNCONFIRMED
Severity: normal
> -Original Message-
> From: Maxime Coquelin
> Sent: Wednesday, February 3, 2021 8:55 PM
> To: Jiang, Cheng1 ; Xia, Chenbo
>
> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
> ; Wang, Yinan
> Subject: Re: [PATCH v3] examples/vhost: remove async inflight packet
> counter
>
>
>
> On 2/3/
On Wed, Feb 03, 2021 at 01:53:58PM +0100, Thomas Monjalon wrote:
> 03/02/2021 13:18, Bruce Richardson:
> > On Wed, Feb 03, 2021 at 11:30:57AM +0100, Thomas Monjalon wrote:
> > > -
> > > https://raw.githubusercontent.com/codespell-project/codespell/master/codespell_lib/data/dictionary.txt
> > > +
The current way of specifying Arm configuration options is insufficient
since we can't identify the SoC we're building for from the MIDR
information. For example, we can't distinguish between N1SDP, Graviton2
or Ampere Altra.
We also want to be able to enable/disable which drivers are built
withou
Add support for enabling or disabling drivers for Arm cross build. Do
not implement any enable/disable lists yet.
Enabling drivers is useful when building for an SoC where we only want
to build a few drivers. That way the list won't be too long.
Similarly, disabling drivers is useful when we want
Add Arm SoC configuration sets to Arm meson.build and add an arch
agnostic meson option, 'platform', to select from these SoC
configurations for meson native builds. This is preferable to
specifying a cross file when doing aarch64 -> aarch64 builds, since the
cross file specifies the toolchain as w
Fix the implementer and part number of DPAA and ARMADA SoCs.
The current values of 16 cores and 1 NUMA node don't cover all SoCs from
the Arm implementer, e.g. Taishan 2280 has 64 cores and 4 NUMA nodes.
Increase these to 64 and 4 to widen the coverage.
Add configuration to SoC options where smalle
> back again, do you suggest me to submit v4 with with corrected quota
> character or its ok to have the v3 of the patch itself as you have already
> acked ?
OK, let's ask Raslan to fix this minor issue while integrating the patch.
Raslan, could you, please, fix the quota character in the commit m
Hi,
> -Original Message-
> From: Viacheslav Ovsiienko
> Sent: Tuesday, February 2, 2021 4:28 PM
> To: dev@dpdk.org
> Cc: Raslan Darawsheh ; NBU-Contact-Thomas
> Monjalon ; Asaf Penso ;
> carsten.andr...@tu-ilmenau.de; sta...@dpdk.org
> Subject: [PATCH] doc: fix supported feature table in
Since commit 14e7a4b37b4f ("net/ice/base: support configuring device in
double VLAN mode"), the VLAN strip design is changed, double VLAN mode
is not enabled by updating the VSI setting, it depends on the DDP and FW
running support.
Also, in double VLAN mode, the VLAN strip is applied on outer, si
On 2/3/21 2:42 PM, Jiang, Cheng1 wrote:
>
>
>> -Original Message-
>> From: Maxime Coquelin
>> Sent: Wednesday, February 3, 2021 8:55 PM
>> To: Jiang, Cheng1 ; Xia, Chenbo
>>
>> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
>> ; Wang, Yinan
>> Subject: Re: [PATCH v3] examples/vhost: re
> -Original Message-
> From: Raslan Darawsheh
> Sent: Wednesday, February 3, 2021 16:19
> To: Slava Ovsiienko ; dev@dpdk.org
> Cc: NBU-Contact-Thomas Monjalon ; Asaf Penso
> ; carsten.andr...@tu-ilmenau.de; sta...@dpdk.org
> Subject: RE: [PATCH] doc: fix supported feature table in mlx5 gui
> -Original Message-
> From: Slava Ovsiienko
> Sent: Wednesday, February 3, 2021 4:25 PM
> To: Raslan Darawsheh ; dev@dpdk.org
> Cc: NBU-Contact-Thomas Monjalon ; Asaf Penso
> ; carsten.andr...@tu-ilmenau.de; sta...@dpdk.org
> Subject: RE: [PATCH] doc: fix supported feature table in mlx5 g
Hi,
> -Original Message-
> From: Jiawei Wang
> Sent: Tuesday, February 2, 2021 6:43 PM
> To: Matan Azrad ; Slava Ovsiienko
> ; Ori Kam ; NBU-Contact-
> Thomas Monjalon
> Cc: dev@dpdk.org; Raslan Darawsheh ;
> sta...@dpdk.org
> Subject: [PATCH] net/mlx5: fix validation for counter and age
> -Original Message-
> From: Maxime Coquelin
> Sent: Wednesday, February 3, 2021 10:21 PM
> To: Jiang, Cheng1 ; Xia, Chenbo
>
> Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX
> ; Wang, Yinan
> Subject: Re: [PATCH v3] examples/vhost: remove async inflight packet
> counter
>
>
>
> On 2/3
> -Original Message-
> From: Slava Ovsiienko
> Sent: Wednesday, February 3, 2021 4:15 PM
> To: Dey, Souvik ; Raslan Darawsheh
> ; Matan Azrad
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: RE: [PATCH v3] common/mlx5: fix storing the synched MAC to
> internal table
>
> > back again, do yo
It makes more sense to add error propagation for rte_ioat_completed_ops.
And change the type of dev_id from int to uint16_t.
Signed-off-by: Cheng Jiang
---
v2: mentioned dev_id type changes in git log
examples/vhost/ioat.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
d
Packet forwarding is not working when infinite Rx feature is used with
large .pcap files that has high number of packets.
The problem is number of allocated mbufs are less than the infinite Rx
ring size, and all mbufs consumed to fill the ring, so there is no mbuf
left for forwarding.
Current log
03/02/2021 14:59, Bruce Richardson:
> On Wed, Feb 03, 2021 at 01:53:58PM +0100, Thomas Monjalon wrote:
> > 03/02/2021 13:18, Bruce Richardson:
> > > On Wed, Feb 03, 2021 at 11:30:57AM +0100, Thomas Monjalon wrote:
> > > > -
> > > > https://raw.githubusercontent.com/codespell-project/codespell/ma
The Virtio rework series mistakenly moved the rte_pci_device
pointer to struct virtio_hw, which is shared between the two
processes. But this structure is per-process, so this change
made secondary process to try accessing primary process-only
memory, leading to a crash
This patch reverts to prope
>
> Hi Feifei,
>
> >
> > Hi, Honnappa, Konstantin and Stephen
> >
> > Thanks very much for your attention of this patch. Based on your
> > opinion, Ruifeng and I discuss about this and make a summary：
> >
> __
>
> > __
On 2/2/21 7:17 AM, Cheng Jiang wrote:
> Remove async inflight packet counter since there is no need to keep
> tracking it. Increase MAX_ENQUEUED_SIZE to prevent packet segment number
> tracking ring from being exhausted.
>
> Fixes: a68ba8e0a6b6 ("examples/vhost: refactor vhost data path")
>
>
On 2/3/21 3:29 PM, Cheng Jiang wrote:
> It makes more sense to add error propagation for rte_ioat_completed_ops.
> And change the type of dev_id from int to uint16_t.
I would reword it to:
"
This patch adds add error propagation for rte_ioat_completed_ops call,
and also changes dev_id type fro
Hi,
On 2/2/21 9:38 AM, Wan Junjie wrote:
> If vhost device's IOMMU feature is disabled, iotlb mempool allocation
> is unnecessary.
>
> Reported-by: Peng He
> Signed-off-by: Wan Junjie
Welcome to the community, looking forward for more contributions!
> Reviewed-by: Zhihong Wang
And welcome b
Fixes: f0073621940c ("event/dlb: add eventdev stop and close")
Coverity issue: 366200
Signed-off-by: Timothy McDaniel
---
drivers/event/dlb/dlb.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/event/dlb/dlb.c b/drivers/event/dlb/dlb.c
index fd03f87..8b26d1d 100644
Fixes: f0073621940c ("event/dlb: add eventdev stop and close")
Coverity issue: 366202
Signed-off-by: Timothy McDaniel
---
drivers/event/dlb/dlb.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/event/dlb/dlb.c b/drivers/event/dlb/dlb.c
index b580dc9..fd03f87 100644
Fixes: eb14a3421afd ("event/dlb: add eventdev start")
Coverity issue: 366205
Signed-off-by: Timothy McDaniel
---
drivers/event/dlb/dlb.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/event/dlb/dlb.c b/drivers/event/dlb/dlb.c
index 64e6df7..b580dc9 100644
--- a/dr
On Wed, Feb 3, 2021 at 10:26 PM Timothy McDaniel
wrote:
>
Please add a comment and also squash the similar patches.
> Fixes: eb14a3421afd ("event/dlb: add eventdev start")
> Coverity issue: 366205
>
> Signed-off-by: Timothy McDaniel
> ---
> drivers/event/dlb/dlb.c |2 +-
> 1 file changed,
On Wed, Feb 3, 2021 at 4:58 PM Maxime Coquelin
wrote:
>
> The Virtio rework series mistakenly moved the rte_pci_device
> pointer to struct virtio_hw, which is shared between the two
> processes. But this structure is per-process, so this change
> made secondary process to try accessing primary pro
On 2/3/21 4:58 PM, Maxime Coquelin wrote:
> The Virtio rework series mistakenly moved the rte_pci_device
> pointer to struct virtio_hw, which is shared between the two
> processes. But this structure is per-process, so this change
> made secondary process to try accessing primary process-only
>
On 2/3/21 3:29 PM, Cheng Jiang wrote:
> |It makes more sense to add error propagation for
> rte_ioat_completed_ops. And change the type of dev_id from int to
> uint16_t. Signed-off-by: Cheng Jiang --- v2:
> mentioned dev_id type changes in git log examples/vhost/ioat.c | 14
> ++ 1 f
On 2/2/21 8:24 AM, Cheng Jiang wrote:
> |Change the type of buff_idx from uint64_t to uint32_t to fix coverity
> issue. Coverity issue: 366264 Fixes: a68ba8e0a6b6 ("examples/vhost:
> refactor vhost data path") Signed-off-by: Cheng Jiang
> --- v2: fixed the format of git log
> examples/vhost/mai
On 2/3/21 5:27 PM, Maxime Coquelin wrote:
>
>
> On 2/2/21 7:17 AM, Cheng Jiang wrote:
>> Remove async inflight packet counter since there is no need to keep
>> tracking it. Increase MAX_ENQUEUED_SIZE to prevent packet segment number
>> tracking ring from being exhausted.
>>
>> Fixes: a68ba8e0a
On 2/1/21 9:48 AM, Peng He wrote:
> vhost_new_device might be called in different threads at the same time.
> thread 1(config thread)
> rte_vhost_driver_start
>->vhost_user_start_client
>->vhost_user_add_connection
> -> vhost_n
Drop Tx path in pcap is Tx that just drops the packets, which is used
for the case only Rx from a pcap file is requested/matters.
The byte stats was calculated using first mbuf segment, which gives
wrong values for multi segmented mbufs, updated to use packet length
instead.
Bugzilla ID: 597
Fixe
This patch updates the PMD to initialize response fields
prior to calling into the PF layer.
Fixes the following coverity issues:
Fixes: eb14a3421afd ("event/dlb: add eventdev start")
Coverity issue: 366205
Fixes: f0073621940c ("event/dlb: add eventdev stop and close")
Coverity issue: 366202
Fi
This patch updates the PMD to initialize response fields
prior to calling into the PF layer.
Fixes the following coverity issues:
Fixes: eb14a3421afd ("event/dlb: add eventdev start")
Coverity issue: 366205
Fixes: f0073621940c ("event/dlb: add eventdev stop and close")
Coverity issue: 366202
Fi
From: Qi Zhang
Clarify what is the scope and impact of the UDP port tunnel API.
There are still missing infos to be improved in future:
- no capability flag
- dependency between ports of the same device
- required privilege
Signed-off-by: Qi Zhang
Signed-off-by: Thomas
AMD CPU can present a high number of NUMA nodes.
The default should be 32 for better compatibility.
Signed-off-by: Thomas Monjalon
---
meson_options.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meson_options.txt b/meson_options.txt
index 5c382487da..6eff62e47d 100644
-
Return a PPC specific value for get_tsc_freq_arch() rather than
depending on the EAL framework to estimate the frequency.
Signed-off-by: David Christensen
---
lib/librte_eal/ppc/rte_cycles.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/librte_eal/ppc/rte_cycles.c b/l
get value from pcap_stats.ps_drop (see man pcap_stats)
the value is adjusted in this cases:
- port stop - pcap is closed and will lose count
- stats reset - pcap doesn't provide reset api
- rollover - pcap counter size is u_32 only
Signed-off-by: Ido Goshen
---
v3:
* code cleanup by dedicated
On 2/3/2021 11:07 PM, Ido Goshen wrote:
get value from pcap_stats.ps_drop (see man pcap_stats)
the value is adjusted in this cases:
- port stop - pcap is closed and will lose count
- stats reset - pcap doesn't provide reset api
- rollover - pcap counter size is u_32 only
Signed-off-by: Ido
1 - 100 of 121 matches
Mail list logo
