The data endian of NFDk Tx descriptor should be little, and the related
logic also should modify.
Fixes: d7f6d9b21ffa ("net/nfp: adjust coding style for NFDk")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfdk/nfp_n
The data endian of NFD3 Tx descriptor should be little, and the related
logic also should modify.
Fixes: 3745dd9dd86f ("net/nfp: adjust coding style for NFD3")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/
This patch series standard the data endian of Rx/Tx descriptors into
little endian to sync with the kernel driver.
Chaoyong He (3):
net/nfp: standard the data endian of Rx descriptor
net/nfp: standard the data endian of NFD3 Tx descritor
net/nfp: standard the data endian of NFDk Tx
The data endian of Rx descriptor should be little endian, and the
related logic also should modify.
Fixes: 3745dd9dd86f ("net/nfp: adjust coding style for NFD3")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/
> On Fri, 11 Apr 2025 16:10:03 +0800
> Chaoyong He wrote:
>
> > From: Long Wu
> >
> > The command of setting VLAN filter IDs can only set one VLAN filter ID
> > at once. For example:
> > testpmd> rx_vlan add 100 0
> >
> > This is very in
f-by: Long Wu
Reviewed-by: Chaoyong He
---
drivers/net/nfp/nfp_net_cmsg.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/nfp/nfp_net_cmsg.c b/drivers/net/nfp/nfp_net_cmsg.c
index 8f77c5588a..1ebd66bb0f 100644
--- a/drivers/net/nfp/nfp_net_cmsg.c
+++ b/drive
k.org
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
drivers/net/nfp/nfp_net_flow.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/nfp/nfp_net_flow.c b/drivers/net/nfp/nfp_net_flow.c
index 7f3fff2186..0e5003ebe3 100644
--- a/drivers/net/nfp/nfp_net_flow.c
+++
This patch series fix one coredump problem and one data over flow
problem.
Long Wu (2):
net/nfp: fix flow rule free problem
net/nfp: fix control message over flow problem
drivers/net/nfp/nfp_net_cmsg.c | 3 ++-
drivers/net/nfp/nfp_net_flow.c | 2 +-
2 files changed, 3 insertions(+), 2 deleti
From: Long Wu
Added an API `rte_eth_dev_get_vlan_filter_conf()` to retrieve
the VLAN filter configuration of an Ethernet device.
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
doc/guides/rel_notes/release_25_07.rst | 5 +
lib/ethdev/ethdev_trace.h | 8
lib
From: Long Wu
Introduce a new command to show the VLAN identifiers
filtered by a port.
Usage example:
testpmd> rx_vlan show port 0
VLAN filter IDs:
1,4-5,9,12,14,18,3864-3865,3869,3874,3878,4076-4091,4093,4095,
testpmd>
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
app/te
rts the user set any number valid VLAN filter
IDs just through one command and it is also backwards compatible.
For example:
testpmd> rx_vlan add 100,102-103,200 0
The VLAN filter IDs 100, 102, 103, 200 will be added to port 0.
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
app/test-pmd/cmd
This patch series aims to enhance the vlan filter feature:
- Support add/remove multiple vlan IDs just in one comment.
- Support show the already configured vlan IDs.
Long Wu (3):
app/testpmd: add/remove multiple VLAN filter IDs at once
ethdev: retrieve VLAN filter configuration
app/testpmd:
Implement the logic of get RSS algorithm capability, also add the logic
of print RSS algorithm capability.
Signed-off-by: Chaoyong He
---
drivers/common/nfp/nfp_common.h | 1 +
drivers/common/nfp/nfp_common_ctrl.h | 3 +++
.../net/nfp/flower/nfp_flower_representor.c
Implement the logic of get RSS algorithm configuration, also do little
refactor to related functions.
Signed-off-by: Chaoyong He
---
drivers/net/nfp/nfp_net_common.c | 84 ++--
1 file changed, 57 insertions(+), 27 deletions(-)
diff --git a/drivers/net/nfp
Correct the RSS algorithm configuration logic, also do some refactor to
the related logic.
Signed-off-by: Chaoyong He
---
drivers/net/nfp/nfp_net_common.c | 121 +--
1 file changed, 81 insertions(+), 40 deletions(-)
diff --git a/drivers/net/nfp/nfp_net_common.c b
This patch series try to correct the RSS algorithm related logic.
Chaoyong He (3):
net/nfp: support get RSS algorithm capability
net/nfp: support get RSS algorithm configuration
net/nfp: correct RSS algorithm configuration logic
drivers/common/nfp/nfp_common.h | 1
API, read from this address will get wrong hash key length
and write to this address will overwrite the MAC address unexpected,
which will cause very strange problem.
Fixes: 934e4c60fbff ("nfp: add RSS")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zhang
34e4c60fbff ("nfp: add RSS")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_net_common.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/net/nfp/nfp_net_common.c b/drivers/net/nfp
unctions:
ipv4 ipv6
RSS key:
6D5A56DA255B0EC24167253D43A38FB0D0CA2BCBAE7B30B477CB2DA38030F20C6A42B7
3BBEAC01FA
testpmd>
--
Fixes: 8205e241b2b0 ("app/testpmd: add missing type to RSS hash commands")
Cc: nelio.laranje...@6wind.com
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
the PF ID to the BSP for firmware load command.
Fixes: 74fd1a71cec1 ("net/nfp: support loading firmware for card without DDR")
Cc: chaoyong...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
dri
Fix two bugs of error process logic.
Fixes: 19bd7cce5705 ("net/nfp: support different configuration BAR size")
Fixes: 636e133ec891 ("net/nfp: update Tx and Rx for multiple PF")
Cc: peng.zh...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zha
From: Long Wu
The 'ipackets'/'opackets' are used to record the number
of packets on represetor port received/sent. But the
code does not consider concurrent calculation of
'ipackets'/'opackets'. If multiple queues are calculated
'ipackets'/'opackets' simultaneously, it will result in
incorrect re
Thanks.
Acked-by: Chaoyong He
> -Original Message-
> From: Stephen Hemminger
> Sent: Sunday, February 9, 2025 4:22 AM
> To: dev@dpdk.org
> Cc: Stephen Hemminger ; Chaoyong He
>
> Subject: [PATCH 22/32] net/nfp: replace memcpy with assignment
>
> Prefe
Seems ok for me.
Acked-by: Chaoyong He
> -Original Message-
> From: David Marchand
> Sent: Friday, February 7, 2025 4:33 PM
> To: dev@dpdk.org
> Cc: tho...@monjalon.net; bruce.richard...@intel.com;
> andre...@linux.microsoft.com; Chaoyong He
> Subject: [RFC 07/10]
The original logic don't support load firmware for cards which has no
DDR, add the related logic to make it can load firmware successfully.
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfpcore/nfp_nsp.c | 131
This patch series aims to support load firmware for card without DDR
chip.
Chaoyong He (2):
net/nfp: fix logic of load firmware from flash
net/nfp: support load firmware for card without DDR
drivers/net/nfp/nfp_ethdev.c | 30 +++
drivers/net/nfp/nfpcore/nfp_nsp.c | 131
ic.
Fixes: 08ea495d624b ("net/nfp: support loading firmware from flash")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_ethdev.c | 30 +++---
1 file changed, 15 insertions(+), 15 deletions(-)
diff
ta...@dpdk.org
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
---
drivers/net/nfp/nfp_ethdev.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/net/nfp/nfp_ethdev.c b/drivers/net/nfp/nfp_ethdev.c
index f54483822f..b98eee6b90 100644
--- a/drivers/net/nfp/nfp_ethdev.c
+++ b/dri
ll
cause an error because 'true' will be regard as no problem.
Fixes: b4ae16eae01c ("net/nfp: support to add and delete flows to firmware")
Cc: chaoyong...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by:
From: Zerun Fu
Fix multiple PFs check from NSP problem in the logic of PF initialization.
Fixes: 8ad2cc8fec37 ("net/nfp: add flag for multiple PFs support")
Cc: peng.zh...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
R
initialize dedicated hardware Rx/Tx queue. If obtaining the
minimum queue size failed, use the default queue size.
Fixes: 112891cd27e5 ("net/bonding: add dedicated HW queues for LACP control")
Cc: tomaszx.kula...@intel.com
Cc: sta...@dpdk.org
Signed-off-by: Long Wu
Reviewed-by: Chaoyong H
> > On Wed, 4 Dec 2024 06:21:00 +0000
> > Chaoyong He wrote:
> >
> > > > The definition of what a "dedicated queue" is a bit confusing.
> > > > If it is only for LACP packets, it should never need to be very big.
> > > > O
Implement the burst mode get operation functions for both Rx and Tx.
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zhang
---
v2:
* Replace 'snprintf()' with 'strlcpy()'.
---
drivers/net/nfp/nfp_ethdev.c| 2 ++
drivers/net/nfp/nfp_ethdev_vf.c | 2 ++
drivers/net/nfp/
> On Wed, 18 Dec 2024 14:30:36 +0800
> Chaoyong He wrote:
>
> > +int
> > +nfp_net_rx_burst_mode_get(struct rte_eth_dev *eth_dev,
> > + uint16_t queue_id __rte_unused,
> > + struct rte_eth_burst_mode *mode)
> > +{
> > + eth_
are.
Fix these problems by modify the related logic, also add needed helper
function at the same time.
Fixes: eae7dadbe987 ("net/nfp: update link status reporting")
Cc: sta...@dpdk.org
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
drivers/net/nfp/flower/nf
Implement the burst mode get operation functions for both Rx and Tx.
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_ethdev.c| 2 ++
drivers/net/nfp/nfp_ethdev_vf.c | 2 ++
drivers/net/nfp/nfp_rxtx.c | 46 +
drivers/net/nfp
Make sure the Rx queue threshold value query by API
'rte_eth_rx_queue_info_get()' and 'rte_eth_dev_info_get()' unify.
Same for the Tx queue threshold value.
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_rxtx.c | 2 ++
1 file changed, 2 inserti
> On Wed, 4 Dec 2024 06:21:00 +
> Chaoyong He wrote:
>
> > > The definition of what a "dedicated queue" is a bit confusing.
> > > If it is only for LACP packets, it should never need to be very big.
> > > Only under a mis-configuration and DoS
lt;< (n)) - 1)
>
> -#define BITS_PER_LONG (__SIZEOF_LONG__ * 8)
> -#define BITS_PER_LONG_LONG (__SIZEOF_LONG_LONG__ * 8)
> +#define BITS_PER_LONG (sizeof(long) * 8)
> +#define BITS_PER_LONG_LONG (sizeof(long long) * 8)
This looks good to me, thanks.
Acked-by: Chaoyong He
>
> #define GENMASK(h, l) \
> ((~0UL << (l)) & (~0UL >> (BITS_PER_LONG - (h) - 1)))
> --
> 2.47.0.vfs.0.3
> On Fri, 11 Oct 2024 11:24:12 +0800
> Chaoyong He wrote:
>
> > From: Long Wu
> >
> > The testpmd application can not modify the value of dedicated hardware
> > Rx/Tx queue size, and hardcoded them as (128/512). This will cause the
> > bonding port sta
ule EEPROM for a specific port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
doc/guides/nics/features/nfp.ini | 2 +
.../net/nfp/flower/nfp_flower_representor.c | 76 ++
drivers/net/nfp/nfp_ethdev.c | 5 +
drivers/net/nfp/nfp_ne
Implement the necessary functions to allow user to visually identify a
physical port associated with a netdev by blinking an LED on that port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
doc/guides/nics/features/nfp.ini | 1 +
.../net/nfp/flower
Extract a helper function to check the physical representor port.
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c| 18 --
.../net/nfp/flower/nfp_flower_representor.h| 1 +
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/drivers
("net/nfp: extract function to allocate VF")
Cc: peng.zh...@corigine.com
Signed-off-by: Chaoyong He
---
drivers/net/nfp/flower/nfp_flower_representor.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c
b/drivers/net/nfp/flower/n
This patch series fix one problem imported by mistake and add support of
two new APIs.
---
v4:
* Also update the 'nfp.ini' file.
v3:
* Adapt to the multiple PF firmware.
v2:
* Add one missing commit.
---
Chaoyong He (4):
net/nfp: fix port index problem
net/nfp: extract functio
Implement the necessary functions to allow user to visually identify a
physical port associated with a netdev by blinking an LED on that port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c | 30
drivers/net/nfp
ule EEPROM for a specific port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c | 76 ++
drivers/net/nfp/nfp_ethdev.c | 5 +
drivers/net/nfp/nfp_net_common.c | 256 ++
drive
Extract a helper function to check the physical representor port.
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c| 18 --
.../net/nfp/flower/nfp_flower_representor.h| 1 +
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/drivers
("net/nfp: extract function to allocate VF")
Cc: peng.zh...@corigine.com
Signed-off-by: Chaoyong He
---
drivers/net/nfp/flower/nfp_flower_representor.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c
b/drivers/net/nfp/flower/n
This patch series fix one problem imported by mistake and add support of
two new APIs.
---
v3:
* Adapt to the multiple PF firmware.
v2:
* add one missing commit.
---
Chaoyong He (4):
net/nfp: fix port index problem
net/nfp: extract function to check physical reprsentor
net/nfp: add support
ule EEPROM for a specific port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c | 76 ++
drivers/net/nfp/nfp_ethdev.c | 5 +
drivers/net/nfp/nfp_net_common.c | 256 ++
drive
Implement the necessary functions to allow user to visually identify a
physical port associated with a netdev by blinking an LED on that port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c | 30
drivers/net/nfp
("net/nfp: extract function to allocate VF")
Cc: peng.zh...@corigine.com
Signed-off-by: Chaoyong He
---
drivers/net/nfp/flower/nfp_flower_representor.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c
b/drivers/net/nfp/flower/n
Extract a helper function to check the physical representor port.
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c| 18 --
.../net/nfp/flower/nfp_flower_representor.h| 1 +
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/drivers
This patch series fix one problem imported by mistake and add support of
two new APIs.
---
v2:
* add one missing commit.
---
Chaoyong He (4):
net/nfp: fix port index problem
net/nfp: extract function to check physical reprsentor
net/nfp: add support for EEPROM functions
net/nfp: add
Implement the necessary functions to allow user to visually identify a
physical port associated with a netdev by blinking an LED on that port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c | 30
drivers/net/nfp
ule EEPROM for a specific port.
Signed-off-by: James Hershaw
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c | 76 ++
drivers/net/nfp/nfp_ethdev.c | 5 +
drivers/net/nfp/nfp_net_common.c | 256 ++
drive
Extract a helper function to check the physical representor port.
Signed-off-by: Chaoyong He
---
.../net/nfp/flower/nfp_flower_representor.c| 18 --
.../net/nfp/flower/nfp_flower_representor.h| 1 +
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/drivers
This patch series fix one problem imported by mistake and add support of
two new APIs.
Chaoyong He (3):
net/nfp: extract function to check physical reprsentor
net/nfp: add support for EEPROM functions
net/nfp: add support for port identify
.../net/nfp/flower/nfp_flower_representor.c
From: Peng Zhang
Update the Tx function and Rx function for multiple PF and simplify the
Rx function to enhance the performance.
In the RX function, the packets are processed according the firmware type.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
drivers/net
From: Peng Zhang
When using the multiple PF firmware, add the new
operations and update the port start and stop
function.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
drivers/net/nfp/flower/nfp_flower.c | 21
drivers/net/nfp/flower
From: Peng Zhang
When using the multiple PF firmware, there is no PF representor port,
so we also need not care about the corresponding control messages anymore.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
drivers/net/nfp/flower/nfp_flower_ctrl.c | 10
From: Peng Zhang
When using multiple PF firmware, add the process of initialize
the representor port and revise the corresponding logic in the
close function.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
.../net/nfp/flower/nfp_flower_representor.c | 80
From: Peng Zhang
When using the multiple PF firmware, there is no need to
create the PF representor port anymore.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
drivers/net/nfp/flower/nfp_flower_representor.c | 12 +---
1 file changed, 9 insertions(+), 3
From: Peng Zhang
Extract the helper function to allocate the VF representor ports.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
.../net/nfp/flower/nfp_flower_representor.c | 85 +++
1 file changed, 49 insertions(+), 36 deletions(-)
diff
From: Peng Zhang
Extract the helper function to initialize the basic information
of the PF representor port.
The logic of check VF is updated because of changing the order
of some logics in the initialization function.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
From: Peng Zhang
Extract the helper function to allocate the PF representor port.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
.../net/nfp/flower/nfp_flower_representor.c | 58 ---
1 file changed, 38 insertions(+), 20 deletions(-)
diff --git
From: Peng Zhang
Extract the helper function to allocate the PHY representor port.
Signed-off-by: Peng Zhang
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
---
.../net/nfp/flower/nfp_flower_representor.c | 86 ---
1 file changed, 53 insertions(+), 33 deletions(-)
diff
This patch series add support of the multiple PF flower firmware.
Peng Zhang (9):
net/nfp: extract the function to allocate the PHY
net/nfp: extract the function to initialize the PF
net/nfp: extract the function to allocate PF
net/nfp: extract the function to allocate the VF
net/nfp: re
> From: Vignesh PS
>
> Add support to bonding PMD to allow user callback function registration for
> TX transmit policy.
>
> Signed-off-by: Vignesh PS
> ---
> .mailmap| 1 +
> drivers/net/bonding/eth_bond_private.h | 6 ++
> drivers/net/bonding/rte_eth_bo
> On Fri, 18 Oct 2024 05:50:20 +
> Chaoyong He wrote:
>
> > > On Fri, 18 Oct 2024 03:21:28 +0000
> > > Chaoyong He wrote:
> > >
> > > > > RTE_PKTMBUF_HEADROOM); }
> > > > > > +
> > > > >
> > > >
> On Fri, 18 Oct 2024 03:21:28 +
> Chaoyong He wrote:
>
> > > RTE_PKTMBUF_HEADROOM); }
> > > > +
> > >
> > > Not sure why this is needed? What is the problem with the original code?
> > > Are you trying to force packets to be segment
> On Fri, 18 Oct 2024 10:42:53 +0800
> Chaoyong He wrote:
>
> > From: Long Wu
> >
> > The previous code used a macro as the data size for mbuf to create the
> > mempool and users cannot modify the size.
> >
> > Now modify the code to support sett
lt size is still
'RTE_MBUF_DEFAULT_BUF_SIZE'.
Examples:
dpdk-l3fwd -l 0-3 -- -p 0x03 --mbuf-size=4096
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
Acked-by: Morten Brørup
---
v2:
* Modify some logic following the advices of reviewer.
* Add the 'Acked-by' tag.
Fix one return value check problem found by the CI.
Coverity issue: 445519
Fixes: 08ea495d624b ("net/nfp: support loading firmware from flash")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_ethdev.c | 8 +++-
1 file changed, 7
From: Zerun Fu
Correction of misspellings in logs.
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/flower/nfp_flower.c | 4 ++--
drivers/net/nfp/flower/nfp_flower_representor.c | 6 +++---
2 files changed, 5
From: Zerun Fu
Unified do not apply abbreviations in log.
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/flower/nfp_flower.c | 2 +-
drivers/net/nfp/flower/nfp_flower_flow.c | 8
drivers/net/nfp/nfp_ethdev.c
From: Zerun Fu
Capitalize the first letter of each sentence in the log.
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/common/nfp/nfp_common.c | 4 +-
drivers/common/nfp/nfp_common_pci.c | 2 +-
drivers
From: Zerun Fu
Here the invoke of `nfp_eth_read_ports()` may fail, but it's acceptable
because it will recover soon once the link status changed.
So modify the level of the log to avoid confusing users.
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by:
This patches unify the format of log, also adjust some log level.
Zerun Fu (6):
net/nfp: modify the level of some logs
net/nfp: modify the level of speed update log
net/nfp: reformat the header of logs
net/nfp: reformat the abbreviations of logs
net/nfp: reformat the period of logs
net
ned-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_net_common.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/nfp/nfp_net_common.c b/drivers/net/nfp/nfp_net_common.c
index f76d5a6895..de7140d
lt size is still
'RTE_MBUF_DEFAULT_BUF_SIZE'.
Examples:
dpdk-l3fwd -l 0-3 -- -p 0x03 --mbuf-size=4096
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
examples/l3fwd/main.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/examples/l3fwd/mai
nd add the logic of disable
NFP_NET_CFG_CTRL_VXLAN into the 'udp_tunnel_port_del()', thus the whole
solution more complete and easier to understand.
Fixes: 5126a904fae0 ("net/nfp: use offload flag to control VXLAN configuration")
Cc: sta...@dpdk.org
Signed-off-by: Lo
first initialization, only when
the Rx queue buffer size is greater than the current hardware set size
will it be modified.
Signed-off-by: Long Wu
Reviewed-by: Chaoyong He
---
drivers/net/nfp/nfp_net_common.h | 1 +
drivers/net/nfp/nfp_rxtx.c | 16 +++-
2 files changed, 16
Zhang
Reviewed-by: Chaoyong He
---
drivers/net/nfp/nfp_ethdev.c | 40 ++--
1 file changed, 38 insertions(+), 2 deletions(-)
diff --git a/drivers/net/nfp/nfp_ethdev.c b/drivers/net/nfp/nfp_ethdev.c
index 8cbbadb4de..b16fbe7db7 100644
--- a/drivers/net/nfp/nfp_eth
From: Huaxing Zhu
Add more logs to debug probe process, and modify some log level.
Also add an new entry to the mailmap file.
Signed-off-by: Huaxing Zhu
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
Reviewed-by: Chaoyong He
---
.mailmap | 1 +
drivers/net/nfp
y: Long Wu
Reviewed-by: Peng Zhang
Reviewed-by: Chaoyong He
---
drivers/net/nfp/nfp_ipsec.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers/net/nfp/nfp_ipsec.c b/drivers/net/nfp/nfp_ipsec.c
index 647bc2bb6d..89116af1b2 100644
--- a/drivers/net/nfp/nfp_ips
This patch series includes two bug fix commits and two debug enhancement
commits.
Huaxing Zhu (2):
net/nfp: add more logs to debug probe process
net/nfp: clear errors status of aer after soft reset
Shihong Wang (1):
net/nfp: do not set the IPv6 flag in transport mode
Zerun Fu (1):
net/nf
PF speed.
Fixes: e1124c4f8a45 ("net/nfp: add flower representor framework")
Cc: chaoyong...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Zerun Fu
Reviewed-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/flower/nfp_flower_representor.c | 3 +++
d
> On Sat, 12 Oct 2024 10:40:59 +0800
> Chaoyong He wrote:
>
> > The original logic keeps using the same name parameter when malloc
> > memory in secondary process, which may cause error when using multiple
> > PF cards.
> >
> > Fixes: 3b00109d2b65 (&qu
Fix one memory leak problem in the logic of VF initialization.
Fixes: d81e2b514dc9 ("net/nfp: move device info into process private data")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
Acked-by: Stephen Hemminger
---
drive
The comment of some control messages are not right, which conflict
with the data structure and may confuse the other developers.
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Acked-by: Stephen Hemminger
---
drivers/net/nfp/flower/nfp_flower_cmsg.h | 14 +++---
1 file changed, 7
The return value of 'nfp_eth_set_fec()' is three ways, the original
logic considered it as two ways wrongly.
Fixes: 37bd1b843a20 ("net/nfp: support setting FEC mode")
Cc: zerun...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by:
The return value of 'nfp_eth_config_commit_end()' is three ways, the
original logic considered it as two ways wrongly.
Fixes: 68aa35373a94 ("net/nfp: support setting pause frame switch mode")
Cc: zerun...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-b
The return value of 'nfp_eth_set_configured()' is three ways, the
original logic considered it as two ways wrongly.
Fixes: 61d4008fe6bb ("net/nfp: support setting link up/down")
Cc: sta...@dpdk.org
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhan
The 'nfp_eth_set_configured()' function is not always success, so
need to check the return value of it.
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
Acked-by: Stephen Hemminger
---
.../net/nfp/flower/nfp_flower_representor.c| 18 +-
d
Try our best to make the logic in secondary proess the same with the
primary process and improve the readability, by add helper functions
and rename function.
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
---
drivers/net/nfp/nfp_ethdev.c | 86
Refactor to the logic of initialize process to simplify some function
parameters, both for primary and secodary process.
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
Acked-by: Stephen Hemminger
---
drivers/net/nfp/nfp_ethdev.c | 153
The original logic keeps using the same name parameter when malloc
memory in secondary process, which may cause error when using
multiple PF cards.
Fixes: 3b00109d2b65 ("net/nfp: add PF ID used to format symbols")
Cc: peng.zh...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: C
Replace 'snprintf()' with 'strlcpy()' where applicable.
Using 'strlcpy()' is safe practice when copying strings, as it
will include a null terminator.
Signed-off-by: Chaoyong He
Reviewed-by: Long Wu
Reviewed-by: Peng Zhang
Acked-by: Stephen Hemminge
1 - 100 of 1245 matches
Mail list logo
