[net-next,0/8] net: ipa: abstract status parsing

Message

Alex Elder Jan. 25, 2023, 8:45 p.m. UTC

Under some circumstances, IPA generates a "packet status" structure
that describes information about a packet.  This is used, for
example, when offload hardware detects an error in a packet, or
otherwise discovers a packet needs special handling.  In this case,
the status is delivered (along with the packet it describes) to a
"default" endpoint so that it can be handled by the AP.

Until now, the structure of this status information hasn't changed.
However, to support more than 32 endpoints, this structure required
some changes, such that some fields are rearranged in ways that are
tricky to represent using C code.

This series updates code related to the IPA status structure.  The
first patch uses a local variable to avoid recomputing a packet
length more than once.  The second stops using sizeof() to determine
the size of an IPA packet status structure.  Patches 3-5 extend the
definitions for values held in packet status fields.  Patch 6 does a
little general cleanup to make patch 7 simpler.  Patch 7 stops using
a C structure to represent packet status; instead, a new function
fetches values "by name" from a buffer containing such a structure.
The last patch updates this function so it also supports IPA v5.0+.

					-Alex

Alex Elder (8):
  net: ipa: refactor status buffer parsing
  net: ipa: stop using sizeof(status)
  net: ipa: define all IPA status mask bits
  net: ipa: rename the NAT enumerated type
  net: ipa: define remaining IPA status field values
  net: ipa: IPA status preparatory cleanups
  net: ipa: introduce generalized status decoder
  net: ipa: add IPA v5.0 packet status support

 drivers/net/ipa/ipa_endpoint.c | 280 +++++++++++++++++++++++++--------
 drivers/net/ipa/ipa_reg.h      |  10 +-
 2 files changed, 217 insertions(+), 73 deletions(-)

Comments

[e-mail] Jan. 27, 2023, 11:20 a.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (master)
by David S. Miller <[e-mail]>:

On Wed, 25 Jan 2023 14:45:37 -0600 you wrote:
> Under some circumstances, IPA generates a "packet status" structure
> that describes information about a packet.  This is used, for
> example, when offload hardware detects an error in a packet, or
> otherwise discovers a packet needs special handling.  In this case,
> the status is delivered (along with the packet it describes) to a
> "default" endpoint so that it can be handled by the AP.
> 
> [...]

Here is the summary with links:
  - [net-next,1/8] net: ipa: refactor status buffer parsing
    https://git.kernel.org/netdev/net-next/c/63a560b5289a
  - [net-next,2/8] net: ipa: stop using sizeof(status)
    https://git.kernel.org/netdev/net-next/c/b8dc7d0eea5a
  - [net-next,3/8] net: ipa: define all IPA status mask bits
    https://git.kernel.org/netdev/net-next/c/8e71708bb25e
  - [net-next,4/8] net: ipa: rename the NAT enumerated type
    https://git.kernel.org/netdev/net-next/c/cbea4761173d
  - [net-next,5/8] net: ipa: define remaining IPA status field values
    https://git.kernel.org/netdev/net-next/c/ec4c24f6a511
  - [net-next,6/8] net: ipa: IPA status preparatory cleanups
    https://git.kernel.org/netdev/net-next/c/02c5077439fc
  - [net-next,7/8] net: ipa: introduce generalized status decoder
    https://git.kernel.org/netdev/net-next/c/ebd2a82ecea8
  - [net-next,8/8] net: ipa: add IPA v5.0 packet status support
    https://git.kernel.org/netdev/net-next/c/55c6eae70ff1

You are awesome, thank you!