> -Original Message-
> From: Morten Brørup
> Sent: Monday, January 9, 2023 3:48 PM
> To: He, ShiyangX ; dev@dpdk.org
> Cc: Zhou, YidingX ; sta...@dpdk.org; Yang, Qiming
> ; Wu, Wenjun1 ; Remy
> Horton
> Subject: RE: [PATCH v2] net/ixgbe: fix ixgbe firmware version inconsistency
>
> >
Hi Jasvinder,
Thanks for the insights on the complexity of adding a layer.
As for the workaround that you suggested using multiple subports, if I
understand it correctly (*pls correct if I misunderstood*) it would not
meet our needs:
- We require multiple heterogeneous ports (i.e. ports with di
On Sun, Jan 15, 2023 at 05:53:15PM +0100, Thomas Monjalon wrote:
> The first parameter of the Meson function "find_library()"
> should be the library name without the "lib" prefix.
>
> Otherwise Meson prints this warning:
> WARNING: find_library('libexecinfo') starting in "lib"
> only
On 1/13/2023 6:44 PM, Stephen Hemminger wrote:
The function rte_ring_free() accepts NULL as vaild input
like free() and other functions.
Found with null_free_check.cocci.
Fixes: 16d6ebb65d59 ("crypto/ipsec_mb: fix null checks")
Cc: kai...@intel.com
Signed-off-by: Stephen Hemminger
---
driver
On Sun, Jan 15, 2023 at 5:53 PM Thomas Monjalon wrote:
>
> The first parameter of the Meson function "find_library()"
> should be the library name without the "lib" prefix.
>
> Otherwise Meson prints this warning:
> WARNING: find_library('libexecinfo') starting in "lib"
> only work
From: Satha Rao
When FC enable requested for CPT, mbox returns allocated BPID.
While configuring CQ consider this value to enable late back pressure.
Signed-off-by: Satha Rao
---
drivers/common/cnxk/roc_nix_fc.c| 2 ++
drivers/common/cnxk/roc_nix_priv.h | 1 +
drivers/common/cnxk/roc_ni
Take mbox lock before NDC sync to be thread safe.
Also release the lock only after access to response
is complete.
Fixes: 7a978bc4be6b ("common/cnxk: support mailbox locking")
Cc: rkuduruma...@marvell.com
Signed-off-by: Nithin Dabilpuram
---
drivers/common/cnxk/roc_nix_tm.c | 10 +++---
1 f
From: Satha Rao
New parameter added inside SQ structure to control the fc_hyst_bits.
Instead of count on all updates each SQ can tune his own hysteresis
level.
Signed-off-by: Satha Rao
---
drivers/common/cnxk/roc_nix.h | 1 +
drivers/common/cnxk/roc_nix_queue.c | 5 ++---
2 files changed
From: Rakesh Kudurumalla
reset pfc and flow control if pfc mode and flow
control are set respectively during unintilization
of pf or vf
Signed-off-by: Rakesh Kudurumalla
---
drivers/net/cnxk/cnxk_ethdev.c | 38 --
1 file changed, 23 insertions(+), 15 deletions(-
Dump inline device RQ context along with ethdev's RQ context.
Signed-off-by: Nithin Dabilpuram
---
drivers/common/cnxk/roc_nix_debug.c | 20
1 file changed, 20 insertions(+)
diff --git a/drivers/common/cnxk/roc_nix_debug.c
b/drivers/common/cnxk/roc_nix_debug.c
index 775325
Now that kernel is clearing parent info that is needed for flush,
free the resources in order from leaf to root so that when SMQ flush
is called there is always hierarchy present from SMQ till TL1.
Signed-off-by: Nithin Dabilpuram
---
drivers/common/cnxk/roc_nix_tm.c | 32 +++
From: Srujana Challa
Updates CPT inbound inline configuration mailbox message
format to set CPT credit threshold and bpid, which are
introduced for CN10KB.
This patch also fixes inline inbound config read API.
Fixes: 37da58509579 ("common/cnxk: update inbound inline IPsec config mailbox")
Cc: sc
From: Rakesh Kudurumalla
no action is taken when application calls
rte_eth_dev_flow_ctrl_get(), for sdp port
which is inline with rte_eth_dev_flow_ctrl_set()
for sdp port
Signed-off-by: Rakesh Kudurumalla
---
drivers/net/cnxk/cnxk_ethdev_ops.c | 3 +++
1 file changed, 3 insertions(+)
diff --g
Skip L4 checks on inline IPsec traffic as even first fragment
is set as valid ESP packet in order to send it via CPT.
Signed-off-by: Nithin Dabilpuram
---
drivers/common/cnxk/roc_nix_inl_dev.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/common/cnxk/roc_nix_inl_
16/01/2023 10:23, David Marchand:
> On Sun, Jan 15, 2023 at 5:53 PM Thomas Monjalon wrote:
> >
> > The first parameter of the Meson function "find_library()"
> > should be the library name without the "lib" prefix.
> >
> > Otherwise Meson prints this warning:
> > WARNING: find_library('lib
Ping
> -Original Message-
> From: Hanumanth Pothula
> Sent: Wednesday, December 21, 2022 7:37 AM
> To: Thomas Monjalon ; Ferruh Yigit
> ; Andrew Rybchenko
>
> Cc: dev@dpdk.org; viachesl...@nvidia.com; Jerin Jacob Kollanukkaran
> ; Nithin Kumar Dabilpuram
> ; Hanumanth Reddy Pothula
>
>
This patch removes unnecessary link status check.
Fixes: a4ba77367923 ("net/i40e: enable maximum frame size at port level")
Fixes: 2184f7cdeeaa ("net/i40e: fix max frame size config at port level")
Fixes: 719469f13b11 ("net/i40e: fix jumbo frame Rx with X722")
Cc: sta...@dpdk.org
Signed-off-by: S
Hi David,
> -Original Message-
> From: David Marchand
> Sent: Friday, January 13, 2023 9:53 PM
> To: Zhang, Helin
> Cc: Zhang, Yuying ; Xing, Beilei
> ; Mcnamara, John ;
> dev@dpdk.org; sta...@dpdk.org; Zhang, Qi Z ; Dapeng
> Yu ; Wenxuan Wu
> Subject: Re: [PATCH v2] net/i40e: don't che
On Mon, Jan 16, 2023 at 11:54 AM Simei Su wrote:
>
> This patch removes unnecessary link status check.
>
> Fixes: a4ba77367923 ("net/i40e: enable maximum frame size at port level")
> Fixes: 2184f7cdeeaa ("net/i40e: fix max frame size config at port level")
> Fixes: 719469f13b11 ("net/i40e: fix jum
Hi Venky,
Please see inline;
Thanks,
Jasvinder
From: Venky Venkatesh
Sent: Monday, January 16, 2023 8:06 AM
To: Singh, Jasvinder
Cc: dev@dpdk.org
Subject: Re: [2nd Try]:Re: Traffic Management API Questions
Hi Jasvinder,
Thanks for the insights on the complexity of adding a layer.
As for the
Hi Ferruh and Andrew,
This patch series optimizes some codes and bug.
Can you take a look at this patch series?
If there are no other questions, can it be merged?
Best,
Huisong
在 2022/12/19 15:06, Huisong Li 写道:
Some lib telemetry interfaces add the 'u32' and 'u64' data by the
rte_tel_data_add
Hi David,
> -Original Message-
> From: David Marchand
> Sent: Monday, January 16, 2023 7:19 PM
> To: Su, Simei
> Cc: Xing, Beilei ; Zhang, Yuying
> ; dev@dpdk.org; Zhang, Qi Z
> ; Yang, Qiming ;
> sta...@dpdk.org; Zhang, Helin
> Subject: Re: [PATCH] net/i40e: rework maximum frame size c
On 12/15/2022 10:56 AM, lihuisong (C) wrote:
>
> 在 2022/12/14 17:55, humin (Q) 写道:
>>
>> 在 2022/12/14 14:13, Chengwen Feng 写道:
>>> This patchset adds device private dump for bonding PMD, and use
>>> rte_eth_dev_priv_dump API to implement testpmd show bonding command.
>>>
>>> Chengwen Feng (3):
>
Copying with the same src and dst address has no effect; removed to
avoid compiler warning with decorated rte_memcpy.
Fixes: e7c9971a857a ("event/dlb2: add probe-time hardware init")
Cc: timothy.mcdan...@intel.com
Signed-off-by: Morten Brørup
Acked-by: Stephen Hemminger
---
v6:
* Add Fixes to p
Bugfix: The vlan in the bulletin does not contain a VLAN header, only the
VLAN ID, so only copy 2 byte, not 4. The target structure has padding
after the field, so copying 2 byte too many is effectively harmless.
There is no need to backport this patch.
Use RTE_PTR_ADD where copying arrays to the
Use RTE_PTR_ADD where copying to the offset of a field in a structure
holding multiple fields, to avoid compiler warnings with decorated
rte_memcpy.
Fixes: 16a7dbc4f69006cc1c96ca2a2c6d3e3c51a2ff50 ("net/mlx5: make flow modify
action list thread safe")
Cc: xuemi...@nvidia.com
Cc: ma...@nvidia.com
Add nonnull function attribute to help the compiler detect a NULL
pointer being passed to a function not accepting NULL pointers as an
argument at build time.
Add access function attributes to tell the compiler how a function
accesses memory pointed to by its pointer arguments.
Add these attribut
> From: Morten Brørup
> Sent: Monday, 9 January 2023 13.28
>
> > From: David Marchand [mailto:david.march...@redhat.com]
> > Sent: Monday, 9 January 2023 12.22
> > attributes
> >
> > On Wed, Dec 28, 2022 at 4:10 PM Morten Brørup
> > wrote:
> > >
> > > Add "nonnull" function attribute to help the
Bugfix: The vlan in the bulletin does not contain a VLAN header, only the
VLAN ID, so only copy 2 byte, not 4. The target structure has padding
after the field, so copying 2 byte too many is effectively harmless.
There is no need to backport this patch.
Use RTE_PTR_ADD where copying arrays to the
Copying with the same src and dst address has no effect; removed to
avoid compiler warning with decorated rte_memcpy.
Fixes: e7c9971a857a ("event/dlb2: add probe-time hardware init")
Cc: timothy.mcdan...@intel.com
Signed-off-by: Morten Brørup
Acked-by: Stephen Hemminger
---
v7:
* No changes.
v6
Use RTE_PTR_ADD where copying to the offset of a field in a structure
holding multiple fields, to avoid compiler warnings with decorated
rte_memcpy.
Fixes: 16a7dbc4f69006cc1c96ca2a2c6d3e3c51a2ff50 ("net/mlx5: make flow modify
action list thread safe")
Cc: xuemi...@nvidia.com
Cc: ma...@nvidia.com
Add nonnull function attribute to help the compiler detect a NULL
pointer being passed to a function not accepting NULL pointers as an
argument at build time.
Add access function attributes to tell the compiler how a function
accesses memory pointed to by its pointer arguments.
Add these attribut
On 12/12/2022 12:48 AM, Frank Du wrote:
> Get from /sys/class/net/{if}/device/numa_node.
>
> Signed-off-by: Frank Du
> ---
> drivers/net/af_xdp/rte_eth_af_xdp.c | 17 ++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/af_xdp/rte_eth_af_xdp.c
> b/d
> -Original Message-
> From: Stephen Hemminger
> Sent: Thursday, January 12, 2023 11:32 PM
> To: Zhang, Ke1X
> Cc: Zhang, Qi Z ; Zhang, Yuying
> ; dev@dpdk.org
> Subject: Re: [PATCH v3] net/ice: add devargs for disabling default mac
>
> On Thu, 12 Jan 2023 15:55:38 +0800
> Ke Zhang w
Thanks Jasvinder. I guess we are on the same page. With the design that you
mention we run short by 1 level of hierarchy -- which is why I was
originally asking for the difficulty of adding a layer. I think I
understand your assessment in that regard i.e. it is easier to add a shaped
dequeue at the
Minutes of Technical Board Meeting, 2022-Nov-16
Members Attending
-
∘ Maxime Coquelin
∘ Thomas Monjalon
∘ Kevin Traynor
∘ Hemant Agrawal
∘ Bruce Richardson
∘ Konstantin Ananyev
∘ Jerin Jacob
∘ Honnappa Nagarahalli
∘ Aaron Conole
NOTE: The technical board meetings every second Wed
This patchset fixes a couple of problems with stopping and restarting an
ioat DMA device. Following the two fixes, a series of improvements are
made to the dmadev unit tests to properly validate that dmadevs work
correctly as they are started and stopped, and ensure that no other or
future drivers
The HW DMA devices supported by IOAT driver do not transition to
the "active" state until the first operation is started by the HW.
Therefore, if the user calls "rte_dma_stop()" on a device without
triggering any operations, the sequence of commands to be sent to
the HW is different, as is the fina
As part of the process of restarting a dma instance, the IOAT driver
will reset the HW addresses and state values. The read and write
indexes for SW use need to be similarly reset to keep HW and SW in
sync.
Fixes: 583f046dd404 ("dma/ioat: add start and stop")
Cc: conor.wa...@intel.com
Cc: sta...@d
The DMA device stop API can return an error value so check that return
value when running dmadev unit tests.
Signed-off-by: Bruce Richardson
---
app/test/test_dmadev.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c
index f
The copy tests for dmadev had separate blocks in the test function for
single copy and burst copies. Separate out the single-copy block to its
own function so that it can be re-used if necessary.
Signed-off-by: Bruce Richardson
---
app/test/test_dmadev.c | 120 ++-
Validate device operation when a device is stopped or restarted.
The only complication - and gap in the dmadev ABI specification - is
what happens to the job ids on restart. Some drivers reset them to 0,
while others continue where things left off. Take account of both
posibilities in the test cas
Members Attending: 6
* Aaron Conole (chair)
* Hement Agarwal
* Maxime Coquelin
* Nathan Southern
* Stephen Hemminger
* Thomas Monjalon
NOTE: The Technical Board meetings take place every second Wednesday on
https://meet.jit.si/DPDK at 3 pm UTC.
Meetings are public, and DPDK community members are
Hi Bruce,
This patchset breaks the dmadev autotest for IOAT on IceLake.
Trace below:
### Test dmadev instance 0 [:00:01.0]
IOAT.status: ACTIVE [0x100242880]
DMA Dev 0: Running copy Tests
Ops submitted: 85120Ops completed: 85120Errors: 0
DMA Dev 0: Running stop-start Tests
IOAT.status
Hi Rongwei,
> -Original Message-
> From: Rongwei Liu
> Sent: Wednesday, 21 December 2022 10:43
>
> Add IPv6 routing extension header definition and no
> TLV support for now.
> At rte_flow layer, there are new items defined for matching
> type/nexthdr/segment_left field.
>
> Signed-off-
Hi Rongwei,
> -Original Message-
> From: Rongwei Liu
> Sent: Wednesday, 21 December 2022 10:43
>
> Add IPv6 protocol modify field definition.
>
> Signed-off-by: Rongwei Liu
> ---
> lib/ethdev/rte_flow.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/ethdev/rte_flow.h b/
On Mon, Jan 16, 2023 at 04:09:19PM +, Walsh, Conor wrote:
> Hi Bruce,
>
> This patchset breaks the dmadev autotest for IOAT on IceLake.
>
> Trace below:
>
> ### Test dmadev instance 0 [:00:01.0] IOAT.status: ACTIVE
> [0x100242880] DMA Dev 0: Running copy Tests Ops submitted: 85120Ops
On 1/16/2023 1:07 PM, Morten Brørup wrote:
> Add nonnull function attribute to help the compiler detect a NULL
> pointer being passed to a function not accepting NULL pointers as an
> argument at build time.
>
> Add access function attributes to tell the compiler how a function
> accesses memory p
These are the results of a script replacing 'if(' with 'if ('
in various places.
Stephen Hemminger (7):
test: fix whitespace
testpmd: fix whitespace
net/e1000: fix whitespace
i40e: fix whitespace
examples: fix whitespace
cmdline: fix whitespace
ip_frag: fix whitespace
app/test-pmd/
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
app/test/test_debug.c| 4 ++--
app/test/test_hash.c | 6 +++---
app/test/test_malloc.c | 10 +-
app/test/test_mbuf.c | 12 ++--
app/test/test_spinlock.c |
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
app/test-pmd/cmdline.c| 30 +++---
app/test-pmd/parameters.c | 8
app/test-pmd/testpmd.c| 2 +-
3 files changed, 20 insertions(+), 20 deletions(
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
drivers/net/i40e/i40e_pf.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/i40e/i40e_pf.c b/drivers/net/i40e/i40e_pf.c
index 15d9ff868f3a..7050e005
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
drivers/net/e1000/em_ethdev.c | 2 +-
drivers/net/e1000/igb_ethdev.c | 8
drivers/net/e1000/igb_pf.c | 2 +-
drivers/net/e1000/igb_rxtx.c | 6 +++---
4 files changed, 9 in
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
examples/ip_reassembly/main.c | 2 +-
examples/l3fwd-power/main.c| 4 ++--
examples/l3fwd/main.c | 4 ++--
examples/multi_process/symmetr
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
lib/cmdline/cmdline_rdline.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/cmdline/cmdline_rdline.c b/lib/cmdline/cmdline_rdline.c
index 5cf723a0126a..28fc
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
lib/ip_frag/rte_ipv4_reassembly.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ip_frag/rte_ipv4_reassembly.c
b/lib/ip_frag/rte_ipv4_reassembly.c
index 4a89a5f5
This patchset fixes a couple of problems with stopping and restarting an
ioat DMA device. Following the three fixes, a series of improvements are
made to the dmadev unit tests to properly validate that dmadevs work
correctly as they are started and stopped, and ensure that no other or
future driver
The HW DMA devices supported by IOAT driver do not transition to
the "active" state until the first operation is started by the HW.
Therefore, if the user calls "rte_dma_stop()" on a device without
triggering any operations, the sequence of commands to be sent to
the HW is different, as is the fina
As part of the process of restarting a dma instance, the IOAT driver
will reset the HW addresses and state values. The read and write
indexes for SW use need to be similarly reset to keep HW and SW in
sync.
Fixes: 583f046dd404 ("dma/ioat: add start and stop")
Cc: conor.wa...@intel.com
Cc: sta...@d
When the DMA device was stopped and restarted by the driver, the control
register specifying the behaviour on error was not getting correctly
reset. This caused unit tests to fail as explicitly introduced errors
were got getting reported back.
Fix by moving the setting of the register to the start
The DMA device stop API can return an error value so check that return
value when running dmadev unit tests.
Signed-off-by: Bruce Richardson
---
app/test/test_dmadev.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c
index f
The copy tests for dmadev had separate blocks in the test function for
single copy and burst copies. Separate out the single-copy block to its
own function so that it can be re-used if necessary.
Signed-off-by: Bruce Richardson
---
app/test/test_dmadev.c | 120 ++-
Validate device operation when a device is stopped or restarted.
The only complication - and gap in the dmadev ABI specification - is
what happens to the job ids on restart. Some drivers reset them to 0,
while others continue where things left off. Take account of both
possibilities in the test ca
These are the results of a script replacing 'if(' with 'if ('
in various places.
v2 - fix related style changes on same lines reported
by checkpatch
Stephen Hemminger (7):
test: fix whitespace
testpmd: fix whitespace
net/e1000: fix whitespace
i40e: fix whitespace
examples: fix whitespac
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
app/test/test_cmdline_cirbuf.c | 4 ++--
app/test/test_debug.c | 6 +++---
app/test/test_hash.c | 10 +-
app/test/test_lpm6_perf.c | 2 +-
app/test/test
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
app/test-pmd/cmdline.c| 33 +
app/test-pmd/parameters.c | 11 ++-
app/test-pmd/testpmd.c| 2 +-
3 files changed, 24 insertions(+), 22 dele
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
drivers/net/e1000/em_ethdev.c | 2 +-
drivers/net/e1000/igb_ethdev.c | 16
drivers/net/e1000/igb_pf.c | 2 +-
drivers/net/e1000/igb_rxtx.c | 6 +++---
4 files c
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
drivers/net/i40e/i40e_pf.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/i40e/i40e_pf.c b/drivers/net/i40e/i40e_pf.c
index 15d9ff868f3a..7050e005
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
examples/ip_reassembly/main.c | 2 +-
examples/l3fwd-power/main.c| 7 ---
examples/l3fwd/main.c | 7 ---
examples/multi_process/s
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
lib/cmdline/cmdline_rdline.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/lib/cmdline/cmdline_rdline.c b/lib/cmdline/cmdline_rdline.c
index 5cf723a0126
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
lib/ip_frag/rte_ipv4_reassembly.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ip_frag/rte_ipv4_reassembly.c
b/lib/ip_frag/rte_ipv4_reassembly.c
index 4a89a5f5
On Fri, 13 Jan 2023 13:12:47 +
Vipin P R wrote:
> add test case coverage to cover the ms_idx jump
>
> Cc: sta...@dpdk.org
>
> Signed-off-by: Vipin P R
> Acked-by: Kumara Parameshwaran
> ---
> Depends-on: 0001-Memory-Allocation-Fixes-ms_idx-jump-lookahead-during.patch
> Depends-on: 0002-Me
On Fri, 13 Jan 2023 13:19:23 +
Vipin P R wrote:
> In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR are
> unsupported.
> i40evf_execute_vf_cmd is invoked with these operations as part of
> i40evf_set_mc_addr_list()
>
> The cases are not handled in i40evf_execute_vf_cm
On Wed, Jan 11, 2023 at 5:11 AM John Levon wrote:
> On Sun, Jan 08, 2023 at 06:52:39PM -0700, Ashish Sadanandan wrote:
>
> > The code added for allowing --huge-dir to specify hugetlbfs
> > sub-directories has a bug where it incorrectly matches mounts that
> > contain a prefix of the specified --h
Acked-by: Kai Ji
> -Original Message-
> From: Stephen Hemminger
> Sent: Friday, January 13, 2023 6:44 PM
> To: dev@dpdk.org
> Cc: Stephen Hemminger ; Ji, Kai
> ; De Lara Guarch, Pablo ;
> Power, Ciara
> Subject: [PATCH] ipsec: remove unneccessary null check
>
> The function rte_ring_fr
These are the results of a script replacing 'if(' with 'if ('
in various places.
v3 - more checkpatch fallout
also fix places where 'for(' is used.
Stephen Hemminger (7):
test: fix whitespace
testpmd: fix whitespace
net/e1000: fix whitespace
i40e: fix whitespace
examples: fix white
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
app/test/test_cmdline_cirbuf.c | 4 ++--
app/test/test_debug.c | 6 +++---
app/test/test_hash.c | 10 +-
app/test/test_lpm6_perf.c | 2 +-
app/test/test
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
app/test-pmd/cmdline.c| 33 +
app/test-pmd/parameters.c | 12 +++-
app/test-pmd/testpmd.c| 2 +-
3 files changed, 25 insertions(+), 22 del
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
drivers/net/e1000/em_ethdev.c | 2 +-
drivers/net/e1000/igb_ethdev.c | 16
drivers/net/e1000/igb_pf.c | 2 +-
drivers/net/e1000/igb_rxtx.c | 6 +++---
4 files c
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
drivers/net/i40e/i40e_pf.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/i40e/i40e_pf.c b/drivers/net/i40e/i40e_pf.c
index 15d9ff868f3a..7050e005
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
examples/ip_reassembly/main.c | 2 +-
examples/l3fwd-power/main.c| 9 +
examples/l3fwd/main.c | 9 +
examples/multi_proce
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
lib/cmdline/cmdline_rdline.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/lib/cmdline/cmdline_rdline.c b/lib/cmdline/cmdline_rdline.c
index 5cf723a0126
The style standard is to use blank after keywords.
I.e "if (" not "if("
Signed-off-by: Stephen Hemminger
---
lib/ip_frag/rte_ipv4_reassembly.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ip_frag/rte_ipv4_reassembly.c
b/lib/ip_frag/rte_ipv4_reassembly.c
index 4a89a5f5
> -Original Message-
> From: Stephen Hemminger
> Sent: Tuesday, January 17, 2023 8:15 AM
> To: dev@dpdk.org
> Cc: Stephen Hemminger ; Zhang, Yuying
> ; Xing, Beilei
> Subject: [PATCH v3 4/7] i40e: fix whitespace
>
> The style standard is to use blank after keywords.
> I.e "if (" not "
> -Original Message-
> From: Xing, Beilei
> Sent: Friday, January 6, 2023 5:16 PM
> To: Wu, Jingjing
> Cc: dev@dpdk.org; Zhang, Qi Z ; Xing, Beilei
>
> Subject: [PATCH v2 07/15] common/idpf: add irq map/unmap
>
> From: Beilei Xing
>
> Add irq map/unmap functions in common module.
> -Original Message-
> From: Mike Pattrick
> Sent: Thursday, December 29, 2022 7:00 AM
> To: dev@dpdk.org
> Cc: Zhang, Qi Z ; tho...@monjalon.net;
> david.march...@redhat.com; Zhou, YidingX ;
> ktray...@redhat.com; Mike Pattrick ; sta...@dpdk.org
> Subject: [PATCH v4] net/iavf: add lock
> -Original Message-
> From: Stephen Hemminger
> Sent: Tuesday, January 17, 2023 8:15 AM
> To: dev@dpdk.org
> Cc: Stephen Hemminger ; Su, Simei
> ; Wu, Wenjun1 ; Burakov,
> Anatoly
> Subject: [PATCH v3 3/7] net/e1000: fix whitespace
>
> The style standard is to use blank after keyword
> -Original Message-
> From: Stephen Hemminger
> Sent: Tuesday, January 17, 2023 8:15 AM
> To: dev@dpdk.org
> Cc: Stephen Hemminger ; Su, Simei
> ; Wu, Wenjun1 ; Burakov,
> Anatoly
> Subject: [PATCH v3 3/7] net/e1000: fix whitespace
>
> The style standard is to use blank after keywords
> -Original Message-
> From: Yang, SteveX
> Sent: Monday, December 26, 2022 8:37 AM
> To: dev@dpdk.org
> Cc: Zhang, Yuying ; Xing, Beilei
> ; Yang, Qiming ; Zhang, Qi
> Z ; Yang, SteveX
> Subject: [PATCH v1 1/2] net/i40e: fix unsupported flow rule transfer attribute
>
> i40e doesn't s
> -Original Message-
> From: Zhang, Qi Z
> Sent: Tuesday, January 17, 2023 9:27 AM
> To: 'Yang, SteveX' ; dev@dpdk.org
> Cc: Zhang, Yuying ; Xing, Beilei
> ; Yang, Qiming
> Subject: RE: [PATCH v1 1/2] net/i40e: fix unsupported flow rule transfer
> attribute
>
>
>
> > -Original Me
> -Original Message-
> From: Yang, SteveX
> Sent: Monday, December 26, 2022 8:37 AM
> To: dev@dpdk.org
> Cc: Zhang, Yuying ; Xing, Beilei
> ; Yang, Qiming ; Zhang, Qi
> Z ; Yang, SteveX
> Subject: [PATCH v1 2/2] net/ice: fix unsupported flow rule transfer attribute
>
> ice doesn't sup
Hi ferruh,
Our application use rte_eth_dev_socket_id to query the socket that a NIC port
connected, then allocate lcore/memory according to this affinity.
The remote memory access is really slow compared to local.
Thanks,
Frank
-Original Message-
From: Ferruh Yigit,
Sent: Monday, Jan
> -Original Message-
> From: Vipin P R
> Sent: Friday, January 13, 2023 9:19 PM
> To: Wu, Jingjing ; Xing, Beilei
> Cc: dev@dpdk.org; Vipin P R ; sta...@dpdk.org
> Subject: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
>
> In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and
> -Original Message-
> From: David Marchand
> Sent: Thursday, January 5, 2023 9:58 PM
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Wu, Jingjing ; Xing, Beilei
> ; Zhang, Qi Z ; Yang, Qiming
>
> Subject: [PATCH] net/iavf: protect insertion in flow list
>
> Add missing lock acquire.
>
> F
> -Original Message-
> From: Zhichao Zeng
> Sent: Thursday, January 12, 2023 5:32 PM
> To: dev@dpdk.org
> Cc: sta...@dpdk.org; Zhou, YidingX ; Zeng,
> ZhichaoX ; Wu, Jingjing ;
> Xing, Beilei ; Doherty, Declan
> ; Sinha, Abhijit ;
> Nicolau, Radu
> Subject: [PATCH] net/iavf: fix buildi
> -Original Message-
> From: Su, Simei
> Sent: Tuesday, December 20, 2022 11:41 AM
> To: Zhang, Qi Z ; Guo, Junfeng
>
> Cc: dev@dpdk.org; Wu, Wenjun1 ; Su, Simei
>
> Subject: [PATCH 1/3] net/igc: code refactoring
>
> Move related structures for Rx/Tx queue from igc_txrx.c to igc_txrx
There are many high-performance DMA devices supported in DPDK now, and
these DMA devices can also be integrated into other modules of DPDK as
accelerators, such as Vhost. Before integrating DMA into applications,
developers need to know the performance of these DMA devices in various
scenarios and
HI Ori:
Those three matching items are redundant. Has been removed already.
Do I need to send RFC v2?
BR
Rongwei
> -Original Message-
> From: Ori Kam
> Sent: Tuesday, January 17, 2023 00:15
> To: Rongwei Liu ; Matan Azrad ;
> Slava Ovsiienko ; NBU-Contact-Thomas Monjalon
Hi Qi,
> -Original Message-
> From: Zhang, Qi Z
> Sent: Tuesday, January 17, 2023 10:25 AM
> To: Su, Simei ; Guo, Junfeng
> Cc: dev@dpdk.org; Wu, Wenjun1
> Subject: RE: [PATCH 1/3] net/igc: code refactoring
>
>
>
> > -Original Message-
> > From: Su, Simei
> > Sent: Tuesday,
On Sat, Jan 14, 2023 at 12:22 AM Tyler Retzlaff
wrote:
>
> Remove the rte_thread_getname API. The API is __rte_experimental and
> requires no deprecation notice.
>
> Fold the platform specific variants into the one place it is used as a
> special case to retain the functionality for linux only.
>
1 - 100 of 120 matches
Mail list logo
