On Tue, 10 Nov 2020 16:37:55 +0100 Lorenzo Bianconi <lorenzo@xxxxxxxxxx> wrote: > XDP bulk APIs introduce a defer/flush mechanism to return > pages belonging to the same xdp_mem_allocator object > (identified via the mem.id field) in bulk to optimize > I-cache and D-cache since xdp_return_frame is usually run > inside the driver NAPI tx completion loop. > Convert mvneta, mvpp2 and mlx5 drivers to xdp_return_frame_bulk APIs. Series Acked-by: Jesper Dangaard Brouer <brouer@xxxxxxxxxx> > Changes since v4: > - fix comments > - introduce xdp_frame_bulk_init utility routine > - compiler annotations for I-cache code layout > - move rcu_read_lock outside fast-path > - mlx5 xdp bulking code optimization I've done a lot of these changes, and benchmarked them on mlx5, details in[1]. [1] https://github.com/xdp-project/xdp-project/blob/master/areas/mem/xdp_bulk_return01.org > Changes since v3: > - align DEV_MAP_BULK_SIZE to XDP_BULK_QUEUE_SIZE > - refactor page_pool_put_page_bulk to avoid code duplication > > Changes since v2: > - move mvneta changes in a dedicated patch > > Changes since v1: > - improve comments > - rework xdp_return_frame_bulk routine logic > - move count and xa fields at the beginning of xdp_frame_bulk struct > - invert logic in page_pool_put_page_bulk for loop > > Lorenzo Bianconi (5): > net: xdp: introduce bulking for xdp tx return path > net: page_pool: add bulk support for ptr_ring > net: mvneta: add xdp tx return bulking support > net: mvpp2: add xdp tx return bulking support > net: mlx5: add xdp tx return bulking support > > drivers/net/ethernet/marvell/mvneta.c | 10 ++- > .../net/ethernet/marvell/mvpp2/mvpp2_main.c | 10 ++- > .../net/ethernet/mellanox/mlx5/core/en/xdp.c | 22 ++++-- > include/net/page_pool.h | 26 +++++++ > include/net/xdp.h | 17 ++++- > net/core/page_pool.c | 69 ++++++++++++++++--- > net/core/xdp.c | 54 +++++++++++++++ > 7 files changed, 191 insertions(+), 17 deletions(-) > -- Best regards, Jesper Dangaard Brouer MSc.CS, Principal Kernel Engineer at Red Hat LinkedIn: http://www.linkedin.com/in/brouer
