Re: [PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-02-01 Thread Jakub Kicinski
On Wed, 27 Jan 2021 22:24:54 +0530 Chinmay Agarwal wrote: > Following race condition was detected: > - neigh_flush_dev() is under execution and calls > neigh_mark_dead(n) marking the neighbour entry 'n' as dead. > > - Executing: __netif_receive_skb() -> > __netif_receive_skb_core() -> arp_rcv()

Re: [PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-30 Thread David Ahern
On 1/27/21 9:54 AM, Chinmay Agarwal wrote: > Following race condition was detected: > - neigh_flush_dev() is under execution and calls > neigh_mark_dead(n) marking the neighbour entry 'n' as dead. > > - Executing: __netif_receive_skb() -> > __netif_receive_skb_core() -> arp_rcv() -> arp_process(

Re: [PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-27 Thread Cong Wang
On Wed, Jan 27, 2021 at 8:55 AM Chinmay Agarwal wrote: > > Following race condition was detected: > - neigh_flush_dev() is under execution and calls > neigh_mark_dead(n) marking the neighbour entry 'n' as dead. > > - Executing: __netif_receive_skb() -> > __netif_receive_skb_core() -> arp_rcv() -

[PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-27 Thread Chinmay Agarwal
Following race condition was detected: - neigh_flush_dev() is under execution and calls neigh_mark_dead(n) marking the neighbour entry 'n' as dead. - Executing: __netif_receive_skb() -> __netif_receive_skb_core() -> arp_rcv() -> arp_process().arp_process() calls __neigh_lookup() which takes a re

Re: [PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-25 Thread Cong Wang
On Mon, Jan 25, 2021 at 11:59 AM Chinmay Agarwal wrote: > > Following race condition was detected: > - neigh_flush_dev() is under execution and calls > neigh_mark_dead(n), > marking the neighbour entry 'n' as dead. > > - Executing: __netif_receive_skb() -> __netif_receive_skb_core() > -> arp_rc

Re: [PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-25 Thread Jakub Kicinski
On Tue, 26 Jan 2021 01:29:37 +0530 Chinmay Agarwal wrote: > Following race condition was detected: > - neigh_flush_dev() is under execution and calls > neigh_mark_dead(n), > marking the neighbour entry 'n' as dead. > > - Executing: __netif_receive_skb() -> __netif_receive_skb_core() > -> arp_rc

[PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-25 Thread Chinmay Agarwal
Following race condition was detected: - neigh_flush_dev() is under execution and calls neigh_mark_dead(n), marking the neighbour entry 'n' as dead. - Executing: __netif_receive_skb() -> __netif_receive_skb_core() -> arp_rcv() -> arp_process().arp_process() calls __neigh_lookup() which takes a r

[PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-25 Thread Chinmay Agarwal
Following race condition was detected: - neigh_flush_dev() is under execution and calls neigh_mark_dead(n), marking the neighbour entry 'n' as dead. - Executing: __netif_receive_skb() -> __netif_receive_skb_core() -> arp_rcv() -> arp_process().arp_process() calls __neigh_lookup() which takes a r