Re: [PATCH v3 net-next 08/15] ipv4: Replace &arp_tbl with arp_table(net).

Nikolay Aleksandrov <[email protected]>
Newsgroups org.kernel.vger.netdev
Message-ID <[email protected]>
On 11/08/2026 05:23, Kuniyuki Iwashima wrote:
> We will allocate per-netns neigh_table in net->neigh_tables[].
> 
> Let's replace &arp_tbl with arp_table(net) and remove extern
> definition for arp_tbl.
> 
> Three notes:
> 
>    1. mlx5e_rep_netevent_event() and nfp_tun_neigh_event_handler()
>       have code assuming neigh_table other than &arp_tbl and
>       &nb_tbl, and the part is removed as it will be false once
>       per-netns table is allocated.
> 
>    2. prestera and rocker uses init_net because they set
>       dev->netns_immutable to true.
> 
>    3. mlx5e_tc_update_neigh_used_value() dereferences nhe->neigh_dev
>       in trace_mlx5e_tc_update_neigh_used_value().
> 
> Signed-off-by: Kuniyuki Iwashima <[email protected]>
> ---
> v2: Add note about mlx5e_tc_update_neigh_used_value()
> 
> Cc: Jason Gunthorpe <[email protected]>
> Cc: Leon Romanovsky <[email protected]>
> Cc: Elad Nachman <[email protected]>
> Cc: Saeed Mahameed <[email protected]>
> Cc: Tariq Toukan <[email protected]>
> Cc: Mark Bloch <[email protected]>
> Cc: Boris Pismenny <[email protected]>
> Cc: Petr Machata <[email protected]>
> Cc: Jiri Pirko <[email protected]>
> Cc: Edward Cree <[email protected]>
> Cc: Nikolay Aleksandrov <[email protected]>
> ---
>   drivers/infiniband/ulp/ipoib/ipoib_main.c     | 27 ++++--
>   .../marvell/prestera/prestera_router.c        | 10 ++-
>   .../mellanox/mlx5/core/en/rep/neigh.c         | 22 ++---
>   .../mellanox/mlx5/core/en/tc_tun_encap.c      | 23 ++---
>   .../mellanox/mlx5/core/en_accel/ipsec.c       |  6 +-
>   .../ethernet/mellanox/mlxsw/spectrum_router.c | 18 ++--
>   .../ethernet/mellanox/mlxsw/spectrum_span.c   |  5 +-
>   .../netronome/nfp/flower/tunnel_conf.c        | 10 +--
>   drivers/net/ethernet/rocker/rocker_main.c     |  2 +-
>   drivers/net/ethernet/rocker/rocker_ofdpa.c    |  2 +-
>   drivers/net/ethernet/sfc/tc_counters.c        |  6 +-
>   drivers/net/ethernet/sfc/tc_encap_actions.c   |  2 +-
>   drivers/net/vxlan/vxlan_core.c                | 10 ++-
>   include/net/arp.h                             |  6 +-
>   include/net/route.h                           |  7 +-
>   net/bridge/br_arp_nd_proxy.c                  |  2 +-
>   net/ipv4/arp.c                                | 88 +++++++++++--------
>   net/ipv4/devinet.c                            | 18 ++--
>   net/ipv4/fib_semantics.c                      |  5 +-
>   net/ipv4/route.c                              |  2 +-
>   20 files changed, 159 insertions(+), 112 deletions(-)
> 

Reviewed-by: Nikolay Aleksandrov <[email protected]>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.