Re: [PATCH] bridge: Fix a deadlock when enabling multicast snooping

2020-12-03 Thread Nikolay Aleksandrov
l.org; linux-kernel@vger.kernel.org; Linus Lüssing >> >> Subject: RE: [PATCH] bridge: Fix a deadlock when enabling multicast snooping >> >>> From: Nikolay Aleksandrov >>> Sent: Thursday, December 3, 2020 3:47 PM >>> To: Jakub Kicinski ; Huang, Jos

Re: [PATCH] bridge: Fix a deadlock when enabling multicast snooping

2020-12-03 Thread Huang, Joseph
tion.org; > > net...@vger.kernel.org; linux-kernel@vger.kernel.org; Linus Lüssing > > > > Subject: Re: [PATCH] bridge: Fix a deadlock when enabling multicast > > snooping > > > > On 03/12/2020 20:28, Jakub Kicinski wrote: > > > On Tue,

RE: [PATCH] bridge: Fix a deadlock when enabling multicast snooping

2020-12-03 Thread Huang, Joseph
ct: Re: [PATCH] bridge: Fix a deadlock when enabling multicast > snooping > > On 03/12/2020 20:28, Jakub Kicinski wrote: > > On Tue, 1 Dec 2020 16:40:47 -0500 Joseph Huang wrote: > >> When enabling multicast snooping, bridge module deadlocks on > >> multicast_lock if

Re: [PATCH] bridge: Fix a deadlock when enabling multicast snooping

2020-12-03 Thread Nikolay Aleksandrov
On 03/12/2020 20:28, Jakub Kicinski wrote: > On Tue, 1 Dec 2020 16:40:47 -0500 Joseph Huang wrote: >> When enabling multicast snooping, bridge module deadlocks on multicast_lock >> if 1) IPv6 is enabled, and 2) there is an existing querier on the same L2 >> network. >> >> The deadlock was caused

Re: [PATCH] bridge: Fix a deadlock when enabling multicast snooping

2020-12-03 Thread Jakub Kicinski
On Tue, 1 Dec 2020 16:40:47 -0500 Joseph Huang wrote: > When enabling multicast snooping, bridge module deadlocks on multicast_lock > if 1) IPv6 is enabled, and 2) there is an existing querier on the same L2 > network. > > The deadlock was caused by the following sequence: While holding the lock,

[PATCH] bridge: Fix a deadlock when enabling multicast snooping

2020-12-01 Thread Joseph Huang
When enabling multicast snooping, bridge module deadlocks on multicast_lock if 1) IPv6 is enabled, and 2) there is an existing querier on the same L2 network. The deadlock was caused by the following sequence: While holding the lock, br_multicast_open calls br_multicast_join_snoopers, which