mbox series

[v2,net-next,00/11] ENA driver XDP changes

Message ID 20240101190855.18739-1-darinzon@amazon.com (mailing list archive)
Headers show
Series ENA driver XDP changes | expand

Message

Arinzon, David Jan. 1, 2024, 7:08 p.m. UTC
From: David Arinzon <darinzon@amazon.com>

This patchset contains multiple XDP-related changes
in the ENA driver, including moving the XDP code to
dedicated files.

Changes in v2:
- Moved changes to right commits in order to avoid compilation errors

David Arinzon (11):
  net: ena: Move XDP code to its new files
  net: ena: Pass ena_adapter instead of net_device to ena_xmit_common()
  net: ena: Put orthogonal fields in ena_tx_buffer in a union
  net: ena: Introduce total_tx_size field in ena_tx_buffer struct
  net: ena: Use tx_ring instead of xdp_ring for XDP channel TX
  net: ena: Don't check if XDP program is loaded in ena_xdp_execute()
  net: ena: Refactor napi functions
  net: ena: Add more debug prints to XDP related function
  net: ena: Always register RX queue info
  net: ena: Make queue stats code cleaner by removing the if block
  net: ena: Take xdp packets stats into account in ena_get_stats64()

 .../device_drivers/ethernet/amazon/ena.rst    |   1 +
 drivers/net/ethernet/amazon/ena/Makefile      |   2 +-
 drivers/net/ethernet/amazon/ena/ena_ethtool.c |  18 +-
 drivers/net/ethernet/amazon/ena/ena_netdev.c  | 689 ++----------------
 drivers/net/ethernet/amazon/ena/ena_netdev.h  |  99 ++-
 drivers/net/ethernet/amazon/ena/ena_xdp.c     | 468 ++++++++++++
 drivers/net/ethernet/amazon/ena/ena_xdp.h     | 151 ++++
 7 files changed, 736 insertions(+), 692 deletions(-)
 create mode 100644 drivers/net/ethernet/amazon/ena/ena_xdp.c
 create mode 100644 drivers/net/ethernet/amazon/ena/ena_xdp.h

Comments

Jakub Kicinski Jan. 2, 2024, 6:08 p.m. UTC | #1
On Mon, 1 Jan 2024 19:08:44 +0000 darinzon@amazon.com wrote:
> Changes in v2:
> - Moved changes to right commits in order to avoid compilation errors

Please read:

https://www.kernel.org/doc/html/next/process/maintainer-netdev.html#tl-dr
Arinzon, David Jan. 3, 2024, 7:08 a.m. UTC | #2
> On Mon, 1 Jan 2024 19:08:44 +0000 darinzon@amazon.com wrote:
> > Changes in v2:
> > - Moved changes to right commits in order to avoid compilation errors
> 
> Please read:
> 
> https://www.kernel.org/doc/html/next/process/maintainer-
> netdev.html#tl-dr
> --
> pv-bot: 24h

Apologies, I've noticed the failures in patchwork and decided to act quickly.
Thank you for the guidelines link.

Shall I resend v3 after the waiting time?
Jakub Kicinski Jan. 3, 2024, 10:18 p.m. UTC | #3
On Wed, 3 Jan 2024 07:08:02 +0000 Arinzon, David wrote:
> Apologies, I've noticed the failures in patchwork and decided to act quickly.
> Thank you for the guidelines link.
> 
> Shall I resend v3 after the waiting time?

No need, just a note for future submissions.
I'll get to reviewing the code itself later today, fingers crossed,
and then we'll see if v3 is in order :)
patchwork-bot+netdevbpf@kernel.org Jan. 4, 2024, 2:10 a.m. UTC | #4
Hello:

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

On Mon, 1 Jan 2024 19:08:44 +0000 you wrote:
> From: David Arinzon <darinzon@amazon.com>
> 
> This patchset contains multiple XDP-related changes
> in the ENA driver, including moving the XDP code to
> dedicated files.
> 
> Changes in v2:
> - Moved changes to right commits in order to avoid compilation errors
> 
> [...]

Here is the summary with links:
  - [v2,net-next,01/11] net: ena: Move XDP code to its new files
    https://git.kernel.org/netdev/net-next/c/d000574d0287
  - [v2,net-next,02/11] net: ena: Pass ena_adapter instead of net_device to ena_xmit_common()
    https://git.kernel.org/netdev/net-next/c/39a044f4dcfe
  - [v2,net-next,03/11] net: ena: Put orthogonal fields in ena_tx_buffer in a union
    https://git.kernel.org/netdev/net-next/c/009b387659d3
  - [v2,net-next,04/11] net: ena: Introduce total_tx_size field in ena_tx_buffer struct
    https://git.kernel.org/netdev/net-next/c/23ec97498026
  - [v2,net-next,05/11] net: ena: Use tx_ring instead of xdp_ring for XDP channel TX
    https://git.kernel.org/netdev/net-next/c/911a8c960110
  - [v2,net-next,06/11] net: ena: Don't check if XDP program is loaded in ena_xdp_execute()
    https://git.kernel.org/netdev/net-next/c/436c79358595
  - [v2,net-next,07/11] net: ena: Refactor napi functions
    https://git.kernel.org/netdev/net-next/c/b626fd9627d4
  - [v2,net-next,08/11] net: ena: Add more debug prints to XDP related function
    https://git.kernel.org/netdev/net-next/c/2b02e332c151
  - [v2,net-next,09/11] net: ena: Always register RX queue info
    https://git.kernel.org/netdev/net-next/c/ea5c460023aa
  - [v2,net-next,10/11] net: ena: Make queue stats code cleaner by removing the if block
    https://git.kernel.org/netdev/net-next/c/4f28e789be76
  - [v2,net-next,11/11] net: ena: Take xdp packets stats into account in ena_get_stats64()
    https://git.kernel.org/netdev/net-next/c/782345d24874

You are awesome, thank you!