On Tue, 10 Nov 2020 12:23:05 -0500 Jeff Dike wrote:
> Commit 58956317c8de ("neighbor: Improve garbage collection")
> guarantees neighbour table entries a five-second lifetime.  Processes
> which make heavy use of multicast can fill the neighour table with
> multicast addresses in five seconds.  At that point, neighbour entries
> can't be GC-ed because they aren't five seconds old yet, the kernel
> log starts to fill up with "neighbor table overflow!" messages, and
> sends start to fail.
> 
> This patch allows multicast addresses to be thrown out before they've
> lived out their five seconds.  This makes room for non-multicast
> addresses and makes messages to all addresses more reliable in these
> circumstances.

We should add 

Fixes: 58956317c8de ("neighbor: Improve garbage collection")

right?

> Signed-off-by: Jeff Dike <jd...@akamai.com>

> diff --git a/net/ipv4/arp.c b/net/ipv4/arp.c
> index 687971d83b4e..097aa8bf07ee 100644
> --- a/net/ipv4/arp.c
> +++ b/net/ipv4/arp.c
> @@ -125,6 +125,7 @@ static int arp_constructor(struct neighbour *neigh);
>  static void arp_solicit(struct neighbour *neigh, struct sk_buff *skb);
>  static void arp_error_report(struct neighbour *neigh, struct sk_buff *skb);
>  static void parp_redo(struct sk_buff *skb);
> +static int arp_is_multicast(const void *pkey);
>  
>  static const struct neigh_ops arp_generic_ops = {
>       .family =               AF_INET,
> @@ -156,6 +157,7 @@ struct neigh_table arp_tbl = {
>       .key_eq         = arp_key_eq,
>       .constructor    = arp_constructor,
>       .proxy_redo     = parp_redo,
> +     .is_multicast   = arp_is_multicast,

extreme nit pick - please align the = sign using tabs like the
surrounding code does.

>       .id             = "arp_cache",
>       .parms          = {
>               .tbl                    = &arp_tbl,
> @@ -928,6 +930,10 @@ static void parp_redo(struct sk_buff *skb)
>       arp_process(dev_net(skb->dev), NULL, skb);
>  }
>  
> +static int arp_is_multicast(const void *pkey)
> +{
> +     return ipv4_is_multicast(*((__be32 *)pkey));
> +}
>  
>  /*
>   *   Receive an arp request from the device layer.
> diff --git a/net/ipv6/ndisc.c b/net/ipv6/ndisc.c
> index 27f29b957ee7..67457cfadcd2 100644
> --- a/net/ipv6/ndisc.c
> +++ b/net/ipv6/ndisc.c
> @@ -81,6 +81,7 @@ static void ndisc_error_report(struct neighbour *neigh, 
> struct sk_buff *skb);
>  static int pndisc_constructor(struct pneigh_entry *n);
>  static void pndisc_destructor(struct pneigh_entry *n);
>  static void pndisc_redo(struct sk_buff *skb);
> +static int ndisc_is_multicast(const void *pkey);
>  
>  static const struct neigh_ops ndisc_generic_ops = {
>       .family =               AF_INET6,
> @@ -115,6 +116,7 @@ struct neigh_table nd_tbl = {
>       .pconstructor = pndisc_constructor,
>       .pdestructor =  pndisc_destructor,
>       .proxy_redo =   pndisc_redo,
> +     .is_multicast = ndisc_is_multicast,

looks like the character after = is expected to be a tab, for better or
worse

>       .allow_add  =   ndisc_allow_add,
>       .id =           "ndisc_cache",
>       .parms = {
> @@ -1706,6 +1708,11 @@ static void pndisc_redo(struct sk_buff *skb)
>       kfree_skb(skb);
>  }
>  
> +static int ndisc_is_multicast(const void *pkey)
> +{
> +     return ipv6_addr_is_multicast((struct in6_addr *)pkey);
> +}
> +
>  static bool ndisc_suppress_frag_ndisc(struct sk_buff *skb)
>  {
>       struct inet6_dev *idev = __in6_dev_get(skb->dev);

Reply via email to