mbox series

[net-next,0/7] octeontx2-af: cleanup and extend parser config

Message ID 20200929092820.22487-1-skardach@marvell.com
Headers show
Series octeontx2-af: cleanup and extend parser config | expand

Message

Stanislaw Kardach Sept. 29, 2020, 9:28 a.m. UTC 
Current KPU configuration data is spread over multiple files which makes
it hard to read. Clean this up by gathering all configuration data in a
single structure and also in a single file (npc_profile.h). This should
increase the readability of KPU handling code (since it always
references same structure), simplify updates to the CAM key extraction
code and allow abstracting out the configuration source.
Additionally extend and fix the parser config to support additional DSA
types, NAT-T-ESP and IPv6 fields.

Patch 1 ensures that CUSTOMx LTYPEs are not aliased with meaningful
LTYPEs where possible.

Patch 2 gathers all KPU profile related data into a single struct and
creates an adapter structure which provides an interface to the KPU
profile for the octeontx2-af driver.

Patches 3-4 add support for Extended DSA, eDSA and Forward DSA.

Patches 5-6 adds IPv6 fields to CAM key extraction and optimize the
parser performance for fragmented IPv6 packets.

Patch 7 refactors ESP handling in the parser to support NAT-T-ESP.

Abhijit Ayarekar (1):
  octeontx2-af: optimize parsing of IPv6 fragments

Hariprasad Kelam (1):
  octeontx2-af: add parser support for Forward DSA

Kiran Kumar K (1):
  octeontx2-af: add parser support for NAT-T-ESP

Satha Rao (1):
  octeontx2-af: fix Extended DSA and eDSA parsing

Stanislaw Kardach (2):
  octeontx2-af: fix LD CUSTOM LTYPE aliasing
  octeontx2-af: cleanup KPU config data

Vidhya Vidhyaraman (1):
  octeontx2-af: Add IPv6 fields to default MKEX

 .../net/ethernet/marvell/octeontx2/af/npc.h   |  43 +-
 .../marvell/octeontx2/af/npc_profile.h        | 473 +++++++++++++++---
 .../net/ethernet/marvell/octeontx2/af/rvu.h   |  17 +
 .../ethernet/marvell/octeontx2/af/rvu_nix.c   |  36 +-
 .../ethernet/marvell/octeontx2/af/rvu_npc.c   | 202 +++-----
 5 files changed, 557 insertions(+), 214 deletions(-)

Comments

Jakub Kicinski Sept. 29, 2020, 5:56 p.m. UTC | #1 
On Tue, 29 Sep 2020 11:28:13 +0200 Stanislaw Kardach wrote:
> Current KPU configuration data is spread over multiple files which makes
> it hard to read. Clean this up by gathering all configuration data in a
> single structure and also in a single file (npc_profile.h). This should
> increase the readability of KPU handling code (since it always
> references same structure), simplify updates to the CAM key extraction
> code and allow abstracting out the configuration source.
> Additionally extend and fix the parser config to support additional DSA
> types, NAT-T-ESP and IPv6 fields.

Acked-by: Jakub Kicinski <kuba@kernel.org>
David Miller Sept. 29, 2020, 8:12 p.m. UTC | #2 
From: Stanislaw Kardach <skardach@marvell.com>

Date: Tue, 29 Sep 2020 11:28:13 +0200

> Current KPU configuration data is spread over multiple files which makes

> it hard to read. Clean this up by gathering all configuration data in a

> single structure and also in a single file (npc_profile.h). This should

> increase the readability of KPU handling code (since it always

> references same structure), simplify updates to the CAM key extraction

> code and allow abstracting out the configuration source.

> Additionally extend and fix the parser config to support additional DSA

> types, NAT-T-ESP and IPv6 fields.

> 

> Patch 1 ensures that CUSTOMx LTYPEs are not aliased with meaningful

> LTYPEs where possible.

> 

> Patch 2 gathers all KPU profile related data into a single struct and

> creates an adapter structure which provides an interface to the KPU

> profile for the octeontx2-af driver.

> 

> Patches 3-4 add support for Extended DSA, eDSA and Forward DSA.

> 

> Patches 5-6 adds IPv6 fields to CAM key extraction and optimize the

> parser performance for fragmented IPv6 packets.

> 

> Patch 7 refactors ESP handling in the parser to support NAT-T-ESP.


This looks fine, series applied, thanks.