netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: ast@kernel.org
Cc: netdev@vger.kernel.org, bpf@vger.kernel.org, hawk@kernel.org,
	amritha.nambiar@intel.com, aleksander.lobakin@intel.com,
	Jakub Kicinski <kuba@kernel.org>
Subject: [PATCH bpf-next v2 0/3] net: struct netdev_rx_queue and xdp.h reshuffling
Date: Wed,  2 Aug 2023 18:02:27 -0700	[thread overview]
Message-ID: <20230803010230.1755386-1-kuba@kernel.org> (raw)

While poking at struct netdev_rx_queue I got annoyed by
the huge rebuild times. I split it out from netdevice.h
and then realized that it was the main reason we included
xdp.h in there. So I removed that dependency as well.

This gives us very pleasant build times for both xdp.h
and struct netdev_rx_queue changes.

I'm sending this for bpf-next because I think it'd be easiest
if it goes in there, and then bpf-next gets flushed soon after?
I can also make a branch on merge-base for net-next and bpf-next..

v2:
 - build fix
 - reorder some includes
v1: https://lore.kernel.org/all/20230802003246.2153774-1-kuba@kernel.org/

Jakub Kicinski (3):
  eth: add missing xdp.h includes in drivers
  net: move struct netdev_rx_queue out of netdevice.h
  net: invert the netdevice.h vs xdp.h dependency

 drivers/net/bonding/bond_main.c               |  1 +
 drivers/net/ethernet/amazon/ena/ena_netdev.h  |  1 +
 drivers/net/ethernet/engleder/tsnep.h         |  1 +
 .../net/ethernet/freescale/dpaa2/dpaa2-eth.h  |  1 +
 drivers/net/ethernet/freescale/enetc/enetc.h  |  1 +
 drivers/net/ethernet/freescale/fec.h          |  1 +
 .../ethernet/fungible/funeth/funeth_txrx.h    |  1 +
 drivers/net/ethernet/google/gve/gve.h         |  1 +
 drivers/net/ethernet/intel/igc/igc.h          |  1 +
 .../ethernet/microchip/lan966x/lan966x_main.h |  1 +
 drivers/net/ethernet/microsoft/mana/mana_en.c |  1 +
 drivers/net/ethernet/stmicro/stmmac/stmmac.h  |  1 +
 drivers/net/ethernet/ti/cpsw_priv.h           |  1 +
 drivers/net/hyperv/hyperv_net.h               |  1 +
 drivers/net/tap.c                             |  1 +
 drivers/net/virtio_net.c                      |  1 +
 include/linux/filter.h                        | 17 ------
 include/linux/netdevice.h                     | 55 ++-----------------
 include/net/busy_poll.h                       |  1 +
 include/net/mana/mana.h                       |  2 +
 include/net/netdev_rx_queue.h                 | 53 ++++++++++++++++++
 include/net/xdp.h                             | 29 ++++++++--
 include/trace/events/xdp.h                    |  1 +
 kernel/bpf/btf.c                              |  1 +
 kernel/bpf/offload.c                          |  1 +
 kernel/bpf/verifier.c                         |  1 +
 net/bpf/test_run.c                            |  1 +
 net/core/dev.c                                |  1 +
 net/core/net-sysfs.c                          |  1 +
 net/netfilter/nf_conntrack_bpf.c              |  1 +
 net/xdp/xsk.c                                 |  1 +
 31 files changed, 110 insertions(+), 72 deletions(-)
 create mode 100644 include/net/netdev_rx_queue.h

-- 
2.41.0


             reply	other threads:[~2023-08-03  1:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-03  1:02 Jakub Kicinski [this message]
2023-08-03  1:02 ` [PATCH bpf-next v2 1/3] eth: add missing xdp.h includes in drivers Jakub Kicinski
2023-08-03 14:09   ` Jesper Dangaard Brouer
2023-08-03 19:34   ` Horatiu Vultur
2023-08-03  1:02 ` [PATCH bpf-next v2 2/3] net: move struct netdev_rx_queue out of netdevice.h Jakub Kicinski
2023-08-03 14:24   ` Jesper Dangaard Brouer
2023-08-03 15:52   ` Alexander Lobakin
2023-08-03  1:02 ` [PATCH bpf-next v2 3/3] net: invert the netdevice.h vs xdp.h dependency Jakub Kicinski
2023-08-03 13:59   ` Jesper Dangaard Brouer
2023-08-03 15:50 ` [PATCH bpf-next v2 0/3] net: struct netdev_rx_queue and xdp.h reshuffling patchwork-bot+netdevbpf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230803010230.1755386-1-kuba@kernel.org \
    --to=kuba@kernel.org \
    --cc=aleksander.lobakin@intel.com \
    --cc=amritha.nambiar@intel.com \
    --cc=ast@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=hawk@kernel.org \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).