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]>