Re: [PATCH bpf-next v2] net: don't include filter.h from net/sock.h

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 29/12/2021 02:49, Jakub Kicinski wrote:
> sock.h is pretty heavily used (5k objects rebuilt on x86 after
> it's touched). We can drop the include of filter.h from it and
> add a forward declaration of struct sk_filter instead.
> This decreases the number of rebuilt objects when bpf.h
> is touched from ~5k to ~1k.
> 
> There's a lot of missing includes this was masking. Primarily
> in networking tho, this time.
> 
> Acked-by: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>
> Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx>
> ---
> v2: https://lore.kernel.org/all/20211228192519.386913-1-kuba@xxxxxxxxxx/
>  - fix build in bond on ia64
>  - fix build in ip6_fib with randconfig
> 
> CC: marcel@xxxxxxxxxxxx
> CC: johan.hedberg@xxxxxxxxx
> CC: luiz.dentz@xxxxxxxxx
> CC: dledford@xxxxxxxxxx
> CC: jgg@xxxxxxxx
> CC: mustafa.ismail@xxxxxxxxx
> CC: shiraz.saleem@xxxxxxxxx
> CC: leon@xxxxxxxxxx
> CC: ap420073@xxxxxxxxx
> CC: wg@xxxxxxxxxxxxxx
> CC: woojung.huh@xxxxxxxxxxxxx
> CC: andrew@xxxxxxx
> CC: vivien.didelot@xxxxxxxxx
> CC: f.fainelli@xxxxxxxxx
> CC: olteanv@xxxxxxxxx
> CC: george.mccollister@xxxxxxxxx
> CC: michael.chan@xxxxxxxxxxxx
> CC: jesse.brandeburg@xxxxxxxxx
> CC: anthony.l.nguyen@xxxxxxxxx
> CC: ast@xxxxxxxxxx
> CC: daniel@xxxxxxxxxxxxx
> CC: hawk@xxxxxxxxxx
> CC: john.fastabend@xxxxxxxxx
> CC: tariqt@xxxxxxxxxx
> CC: saeedm@xxxxxxxxxx
> CC: ecree.xilinx@xxxxxxxxx
> CC: habetsm.xilinx@xxxxxxxxx
> CC: jreuter@xxxxxxxx
> CC: dsahern@xxxxxxxxxx
> CC: kvalo@xxxxxxxxxxxxxx
> CC: pkshih@xxxxxxxxxxx
> CC: trond.myklebust@xxxxxxxxxxxxxxx
> CC: anna.schumaker@xxxxxxxxxx
> CC: viro@xxxxxxxxxxxxxxxxxx
> CC: andrii@xxxxxxxxxx
> CC: mcgrof@xxxxxxxxxx
> CC: keescook@xxxxxxxxxxxx
> CC: yzaikin@xxxxxxxxxx
> CC: nikolay@xxxxxxxxxx
> CC: jiri@xxxxxxxxxx
> CC: wintera@xxxxxxxxxxxxx
> CC: wenjia@xxxxxxxxxxxxx
> CC: pablo@xxxxxxxxxxxxx
> CC: kadlec@xxxxxxxxxxxxx
> CC: fw@xxxxxxxxx
> CC: ralf@xxxxxxxxxxxxxx
> CC: jhs@xxxxxxxxxxxx
> CC: xiyou.wangcong@xxxxxxxxx
> CC: kgraul@xxxxxxxxxxxxx
> CC: sgarzare@xxxxxxxxxx
> CC: steffen.klassert@xxxxxxxxxxx
> CC: herbert@xxxxxxxxxxxxxxxxxxx
> CC: arnd@xxxxxxxx
> CC: linux-bluetooth@xxxxxxxxxxxxxxx
> CC: linux-rdma@xxxxxxxxxxxxxxx
> CC: linux-can@xxxxxxxxxxxxxxx
> CC: intel-wired-lan@xxxxxxxxxxxxxxxx
> CC: bpf@xxxxxxxxxxxxxxx
> CC: linux-hams@xxxxxxxxxxxxxxx
> CC: ath11k@xxxxxxxxxxxxxxxxxxx
> CC: linux-wireless@xxxxxxxxxxxxxxx
> CC: linux-nfs@xxxxxxxxxxxxxxx
> CC: linux-fsdevel@xxxxxxxxxxxxxxx
> CC: bridge@xxxxxxxxxxxxxxxxxxxxxxxxxx
> CC: linux-decnet-user@xxxxxxxxxxxxxxxxxxxxx
> CC: linux-s390@xxxxxxxxxxxxxxx
> CC: netfilter-devel@xxxxxxxxxxxxxxx
> CC: coreteam@xxxxxxxxxxxxx
> CC: virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
> ---
[snip]
>  net/bridge/br_ioctl.c                             | 1 +
[snip
>  70 files changed, 80 insertions(+), 1 deletion(-)
> 

For the bridge:
Acked-by: Nikolay Aleksandrov <nikolay@xxxxxxxxxx>





[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux