mbox series

[net-next,v2,0/2] Add L2 hw acceleration for airoha_eth driver

Message ID 20250409-airoha-flowtable-l2b-v2-0-4a1e3935ea92@kernel.org (mailing list archive)
Headers show
Series Add L2 hw acceleration for airoha_eth driver | expand

Message

Lorenzo Bianconi April 9, 2025, 9:47 a.m. UTC
Introduce the capability to offload L2 traffic defining flower rules in
the PSE/PPE engine available on EN7581 SoC.
Since the hw always reports L2/L3/L4 flower rules, link all L2 rules
sharing the same L2 info (with different L3/L4 info) in the L2 subflows
list of a given L2 PPE entry.

---
Changes in v2:
- squash patch 1/3 and 2/3
- explicitly initialize airoha_flow_table_entry type for
  FLOW_TYPE_L4 entry
- get rid of airoha_ppe_foe_flow_remove_entry_locked() and just rely on
  airoha_ppe_foe_flow_remove_entry()
- Link to v1: https://lore.kernel.org/r/20250407-airoha-flowtable-l2b-v1-0-18777778e568@kernel.org

---
Lorenzo Bianconi (2):
      net: airoha: Add l2_flows rhashtable
      net: airoha: Add L2 hw acceleration support

 drivers/net/ethernet/airoha/airoha_eth.c |   2 +-
 drivers/net/ethernet/airoha/airoha_eth.h |  22 ++-
 drivers/net/ethernet/airoha/airoha_ppe.c | 224 ++++++++++++++++++++++++++-----
 3 files changed, 212 insertions(+), 36 deletions(-)
---
base-commit: 61f96e684edd28ca40555ec49ea1555df31ba619
change-id: 20250313-airoha-flowtable-l2b-e0b50d4a3215

Best regards,

Comments

Michal Kubiak April 9, 2025, 11:58 a.m. UTC | #1
On Wed, Apr 09, 2025 at 11:47:13AM +0200, Lorenzo Bianconi wrote:
> Introduce the capability to offload L2 traffic defining flower rules in
> the PSE/PPE engine available on EN7581 SoC.
> Since the hw always reports L2/L3/L4 flower rules, link all L2 rules
> sharing the same L2 info (with different L3/L4 info) in the L2 subflows
> list of a given L2 PPE entry.
> 
> ---
> Changes in v2:
> - squash patch 1/3 and 2/3
> - explicitly initialize airoha_flow_table_entry type for
>   FLOW_TYPE_L4 entry
> - get rid of airoha_ppe_foe_flow_remove_entry_locked() and just rely on
>   airoha_ppe_foe_flow_remove_entry()
> - Link to v1: https://lore.kernel.org/r/20250407-airoha-flowtable-l2b-v1-0-18777778e568@kernel.org
> 
> ---
> Lorenzo Bianconi (2):
>       net: airoha: Add l2_flows rhashtable
>       net: airoha: Add L2 hw acceleration support
> 
>  drivers/net/ethernet/airoha/airoha_eth.c |   2 +-
>  drivers/net/ethernet/airoha/airoha_eth.h |  22 ++-
>  drivers/net/ethernet/airoha/airoha_ppe.c | 224 ++++++++++++++++++++++++++-----
>  3 files changed, 212 insertions(+), 36 deletions(-)
> ---
> base-commit: 61f96e684edd28ca40555ec49ea1555df31ba619
> change-id: 20250313-airoha-flowtable-l2b-e0b50d4a3215
> 
> Best regards,
> -- 
> Lorenzo Bianconi <lorenzo@kernel.org>
> 

The v2 addresses all the comments I had for the v1. Thank you!
It looks fine to me.

For the series:
Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>

Thanks,
Michal
patchwork-bot+netdevbpf@kernel.org April 12, 2025, 3:20 a.m. UTC | #2
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 09 Apr 2025 11:47:13 +0200 you wrote:
> Introduce the capability to offload L2 traffic defining flower rules in
> the PSE/PPE engine available on EN7581 SoC.
> Since the hw always reports L2/L3/L4 flower rules, link all L2 rules
> sharing the same L2 info (with different L3/L4 info) in the L2 subflows
> list of a given L2 PPE entry.
> 
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/2] net: airoha: Add l2_flows rhashtable
    https://git.kernel.org/netdev/net-next/c/b4916f67902e
  - [net-next,v2,2/2] net: airoha: Add L2 hw acceleration support
    https://git.kernel.org/netdev/net-next/c/cd53f622611f

You are awesome, thank you!