Skip to content

I40E refactors

Anatoly Burakov dpdk-dev 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 3 repliesIn review
  2. v12 patches 0 repliesSuperseded

Changes in v2 from the cover letter

- Fix patch 1 returning non-NULL pointers to queue/key when lengths of either are 0

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2net/i40e: do not use flow RSS conf structdrivers/net/intel/i40e/i40e_ethdev.h drivers/net/intel/i40e/i40e_flow.c drivers/net/intel/i40e/i40e_hash.c +1 more+33 -311 reply
2/2net/i40e: move mutable fields out of RSS configdrivers/net/intel/i40e/i40e_ethdev.h drivers/net/intel/i40e/i40e_hash.c+50 -431 reply

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by.

PatchBruce Richardson
1/2A-b
2/2A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/2] I40E refactors

A small amount of refactor in hash engine to prepare for future code. Specifically, disentangling the mutable filter list metadata from immutable RSS configuration supplied from rte_flow API, and getting rid of embedded rte_flow RSS conf struct and replacing it with direct values embedded in config struct.

v1 -> v2:
- Fix patch 1 returning non-NULL pointers to queue/key when lengths of either are 0

Anatoly Burakov (2):
net/i40e: do not use flow RSS conf struct
net/i40e: move mutable fields out of RSS config

 drivers/net/intel/i40e/i40e_ethdev.h |  18 +++-
 drivers/net/intel/i40e/i40e_flow.c   |  13 ++-
 drivers/net/intel/i40e/i40e_hash.c   | 125 +++++++++++++--------------
 drivers/net/intel/i40e/i40e_hash.h   |   1 -
 4 files changed, 83 insertions(+), 74 deletions(-)
Signature · 1 line
2.52.0