mbox series

[net-next,v6,00/10] xdp: a fistful of generic changes pt. I

Message ID 20241203173733.3181246-1-aleksander.lobakin@intel.com (mailing list archive)
Headers show
Series xdp: a fistful of generic changes pt. I | expand

Message

Alexander Lobakin Dec. 3, 2024, 5:37 p.m. UTC
XDP for idpf is currently 6 chapters:
* convert Rx to libeth;
* convert Tx and stats to libeth;
* generic XDP and XSk code changes (you are here);
* generic XDP and XSk code additions;
* actual XDP for idpf via new libeth_xdp;
* XSk for idpf (via ^).

Part III does the following:
* improve &xdp_buff_xsk cacheline placement;
* does some cleanups with marking read-only bpf_prog and xdp_buff
  arguments const for some generic functions;
* allows attaching already registered XDP memory model to RxQ info;
* makes system percpu page_pools valid XDP memory models;
* starts using netmems in the XDP core code (1 function);
* allows mixing pages from several page_pools within one XDP frame;
* optimizes &xdp_frame layout and removes no-more-used field.

Bullets 4-6 are the most important ones. All of them are prereqs to
libeth_xdp.

Alexander Lobakin (9):
  xsk: align &xdp_buff_xsk harder
  bpf, xdp: constify some bpf_prog * function arguments
  xdp, xsk: constify read-only arguments of some static inline helpers
  xdp: allow attaching already registered memory model to xdp_rxq_info
  xsk: allow attaching XSk pool via xdp_rxq_info_reg_mem_model()
  netmem: add a couple of page helper wrappers
  page_pool: make page_pool_put_page_bulk() handle array of netmems
  page_pool: allow mixing PPs within one bulk
  xdp: get rid of xdp_frame::mem.id

Toke Høiland-Jørgensen (1):
  xdp: register system page pool as an XDP memory model

 include/net/page_pool/types.h                 |   6 +-
 include/linux/bpf.h                           |  12 +-
 include/linux/filter.h                        |   9 +-
 include/linux/netdevice.h                     |   7 +-
 include/linux/skbuff.h                        |   2 +-
 include/net/netmem.h                          |  78 +++++++++++-
 include/net/xdp.h                             |  93 ++++++++++----
 include/net/xdp_sock_drv.h                    |  11 +-
 include/net/xsk_buff_pool.h                   |   4 +-
 .../net/ethernet/freescale/dpaa/dpaa_eth.c    |   2 +-
 drivers/net/veth.c                            |   4 +-
 kernel/bpf/cpumap.c                           |   2 +-
 kernel/bpf/devmap.c                           |   8 +-
 net/bpf/test_run.c                            |   4 +-
 net/core/dev.c                                |  20 ++-
 net/core/filter.c                             |  41 +++---
 net/core/page_pool.c                          |  79 ++++++++----
 net/core/skbuff.c                             |   2 +-
 net/core/xdp.c                                | 118 +++++++++++-------
 19 files changed, 348 insertions(+), 154 deletions(-)

---
From v5[0]:
* split the overgrowth series into 2 parts: changes and additions
  (Jakub);
* 008: future-proof: make the touched function MP-agnostic to avoid
  double work in future;
* send to better fitting now bpf instead of netdev.

From v4[1]:
* 12: pick RB from Toke;
* 19: drop redundant ';'s (Jakub);
* 19: fix a couple context imbalance warnings by moving __acquires() /
  __releases() to the proper place (smatch);
* no functional changes.

From v3[2]:
* rebase on top of the latest net-next to solve conflict (Jakub);
* 09: use iterative approach instead of recursive to not blow the stack
  (Toke);
* 12: rephrase the commitmsg since the functionality changed, so that
  it's not actual anymore (Toke);
* align &xdp_buff_xsk a bit harder since its alignment degraded
  recently;
* pick RBs from Toke.

From v2[3]:
* cover: rename the series;
* collect RBs and Acks from Maciej;
* 007: reword the commitmsg;
* 011: fix typos in the commitmsg (M);
* 012: 'ts' -> 'tsize' (M; not 'truesize' to fit into 80 cols =\);
* 016: fix the intro sentence (M);
* no functional changes.

From v1[4]:
* rebase on top of the latest net-next;
* no other changes.

[0] https://lore.kernel.org/netdev/20241113152442.4000468-1-aleksander.lobakin@intel.com
[1] https://lore.kernel.org/netdev/20241107161026.2903044-1-aleksander.lobakin@intel.com
[2] https://lore.kernel.org/netdev/20241030165201.442301-1-aleksander.lobakin@intel.com
[3] https://lore.kernel.org/netdev/20241015145350.4077765-1-aleksander.lobakin@intel.com
[4] https://lore.kernel.org/netdev/20241009152756.3113697-1-aleksander.lobakin@intel.com

Comments

patchwork-bot+netdevbpf@kernel.org Dec. 6, 2024, 7:10 a.m. UTC | #1
Hello:

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

On Tue,  3 Dec 2024 18:37:23 +0100 you wrote:
> XDP for idpf is currently 6 chapters:
> * convert Rx to libeth;
> * convert Tx and stats to libeth;
> * generic XDP and XSk code changes (you are here);
> * generic XDP and XSk code additions;
> * actual XDP for idpf via new libeth_xdp;
> * XSk for idpf (via ^).
> 
> [...]

Here is the summary with links:
  - [net-next,v6,01/10] xsk: align &xdp_buff_xsk harder
    https://git.kernel.org/netdev/net-next/c/ca5c94949fac
  - [net-next,v6,02/10] bpf, xdp: constify some bpf_prog * function arguments
    https://git.kernel.org/netdev/net-next/c/7cd1107f48e2
  - [net-next,v6,03/10] xdp, xsk: constify read-only arguments of some static inline helpers
    https://git.kernel.org/netdev/net-next/c/dcf3827cde86
  - [net-next,v6,04/10] xdp: allow attaching already registered memory model to xdp_rxq_info
    https://git.kernel.org/netdev/net-next/c/f65966fe0178
  - [net-next,v6,05/10] xsk: allow attaching XSk pool via xdp_rxq_info_reg_mem_model()
    https://git.kernel.org/netdev/net-next/c/9e25dd9d65d2
  - [net-next,v6,06/10] xdp: register system page pool as an XDP memory model
    https://git.kernel.org/netdev/net-next/c/e77d9aee9513
  - [net-next,v6,07/10] netmem: add a couple of page helper wrappers
    https://git.kernel.org/netdev/net-next/c/9bd9f72a7434
  - [net-next,v6,08/10] page_pool: make page_pool_put_page_bulk() handle array of netmems
    https://git.kernel.org/netdev/net-next/c/024bfd2e9d80
  - [net-next,v6,09/10] page_pool: allow mixing PPs within one bulk
    (no matching commit)
  - [net-next,v6,10/10] xdp: get rid of xdp_frame::mem.id
    (no matching commit)

You are awesome, thank you!