Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1462680 > unrolled thread

[BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

Started byMasami Hiramatsu <mhiramat@kernel.org>
First post2016-08-15 11:50 +0200
Last post2016-08-24 15:20 +0200
Articles 7 — 4 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-15 11:50 +0200
    Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when  removing interface Rafał Miłecki <zajec5@gmail.com> - 2016-08-15 12:50 +0200
      Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface Kalle Valo <kvalo@codeaurora.org> - 2016-08-15 13:00 +0200
        Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when  removing interface Rafał Miłecki <zajec5@gmail.com> - 2016-08-15 14:00 +0200
          Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when  removing interface Arend Van Spriel <arend.vanspriel@broadcom.com> - 2016-08-15 23:50 +0200
            Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when  removing interface Masami Hiramatsu <mhiramat@kernel.org> - 2016-08-16 04:30 +0200
    Re: [BUGFIX,  1/2] brcmfmac: Check rtnl_lock is locked when removing interface Kalle Valo <kvalo@codeaurora.org> - 2016-08-24 15:20 +0200

#1462680 — [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

FromMasami Hiramatsu <mhiramat@kernel.org>
Date2016-08-15 11:50 +0200
Subject[BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface
Message-ID<s6mGR-3T5-9@gated-at.bofh.it>
Check rtnl_lock is locked in brcmf_p2p_ifp_removed() by passing
rtnl_locked flag. Actually the caller brcmf_del_if() checks whether
the rtnl_lock is locked, but doesn't pass it to brcmf_p2p_ifp_removed().

Without this fix, wpa_supplicant goes softlockup with rtnl_lock
holding (this means all other process using netlink are locked up too)

e.g.
[ 4495.876627] INFO: task wpa_supplicant:7307 blocked for more than 10 seconds.
[ 4495.876632]       Tainted: G        W       4.8.0-rc1+ #8
[ 4495.876635] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
[ 4495.876638] wpa_supplicant  D ffff974c647b39a0     0  7307      1 0x00000000
[ 4495.876644]  ffff974c647b39a0 0000000000000000 ffff974c00000000 ffff974c7dc59c58
[ 4495.876651]  ffff974c6b7417c0 ffff974c645017c0 ffff974c647b4000 ffffffff86f16c08
[ 4495.876657]  ffff974c645017c0 0000000000000246 00000000ffffffff ffff974c647b39b8
[ 4495.876664] Call Trace:
[ 4495.876671]  [<ffffffff868aeccc>] schedule+0x3c/0x90
[ 4495.876676]  [<ffffffff868af065>] schedule_preempt_disabled+0x15/0x20
[ 4495.876682]  [<ffffffff868b0996>] mutex_lock_nested+0x176/0x3b0
[ 4495.876686]  [<ffffffff867a2067>] ? rtnl_lock+0x17/0x20
[ 4495.876690]  [<ffffffff867a2067>] rtnl_lock+0x17/0x20
[ 4495.876720]  [<ffffffffc0ae9a5d>] brcmf_p2p_ifp_removed+0x4d/0x70 [brcmfmac]
[ 4495.876741]  [<ffffffffc0aebde6>] brcmf_remove_interface+0x196/0x1b0 [brcmfmac]
[ 4495.876760]  [<ffffffffc0ae9901>] brcmf_p2p_del_vif+0x111/0x220 [brcmfmac]
[ 4495.876777]  [<ffffffffc0adefab>] brcmf_cfg80211_del_iface+0x21b/0x270 [brcmfmac]
[ 4495.876820]  [<ffffffffc097b39e>] nl80211_del_interface+0xfe/0x3a0 [cfg80211]
[ 4495.876825]  [<ffffffff867ca335>] genl_family_rcv_msg+0x1b5/0x370
[ 4495.876832]  [<ffffffff860e5d8d>] ? trace_hardirqs_on+0xd/0x10
[ 4495.876836]  [<ffffffff867ca56d>] genl_rcv_msg+0x7d/0xb0
[ 4495.876839]  [<ffffffff867ca4f0>] ? genl_family_rcv_msg+0x370/0x370
[ 4495.876846]  [<ffffffff867c9a47>] netlink_rcv_skb+0x97/0xb0
[ 4495.876849]  [<ffffffff867ca168>] genl_rcv+0x28/0x40
[ 4495.876854]  [<ffffffff867c93c3>] netlink_unicast+0x1d3/0x2f0
[ 4495.876860]  [<ffffffff867c933b>] ? netlink_unicast+0x14b/0x2f0
[ 4495.876866]  [<ffffffff867c97cb>] netlink_sendmsg+0x2eb/0x3a0
[ 4495.876870]  [<ffffffff8676dad8>] sock_sendmsg+0x38/0x50
[ 4495.876874]  [<ffffffff8676e4df>] ___sys_sendmsg+0x27f/0x290
[ 4495.876882]  [<ffffffff8628b935>] ? mntput_no_expire+0x5/0x3f0
[ 4495.876888]  [<ffffffff8628b9be>] ? mntput_no_expire+0x8e/0x3f0
[ 4495.876894]  [<ffffffff8628b935>] ? mntput_no_expire+0x5/0x3f0
[ 4495.876899]  [<ffffffff8628bd44>] ? mntput+0x24/0x40
[ 4495.876904]  [<ffffffff86267830>] ? __fput+0x190/0x200
[ 4495.876909]  [<ffffffff8676f125>] __sys_sendmsg+0x45/0x80
[ 4495.876914]  [<ffffffff8676f172>] SyS_sendmsg+0x12/0x20
[ 4495.876918]  [<ffffffff868b5680>] entry_SYSCALL_64_fastpath+0x23/0xc1
[ 4495.876924]  [<ffffffff860e2b8f>] ? trace_hardirqs_off_caller+0x1f/0xc0

Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
---
 .../wireless/broadcom/brcm80211/brcmfmac/core.c    |    2 +-
 .../net/wireless/broadcom/brcm80211/brcmfmac/p2p.c |    8 +++++---
 .../net/wireless/broadcom/brcm80211/brcmfmac/p2p.h |    2 +-
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
index 8d16f02..65e8c87 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
@@ -743,7 +743,7 @@ static void brcmf_del_if(struct brcmf_pub *drvr, s32 bsscfgidx,
 		 * serious troublesome side effects. The p2p module will clean
 		 * up the ifp if needed.
 		 */
-		brcmf_p2p_ifp_removed(ifp);
+		brcmf_p2p_ifp_removed(ifp, rtnl_locked);
 		kfree(ifp);
 	}
 }
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
index 66f942f..de19c7c 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c
@@ -2297,7 +2297,7 @@ int brcmf_p2p_del_vif(struct wiphy *wiphy, struct wireless_dev *wdev)
 	return err;
 }
 
-void brcmf_p2p_ifp_removed(struct brcmf_if *ifp)
+void brcmf_p2p_ifp_removed(struct brcmf_if *ifp, bool rtnl_locked)
 {
 	struct brcmf_cfg80211_info *cfg;
 	struct brcmf_cfg80211_vif *vif;
@@ -2306,9 +2306,11 @@ void brcmf_p2p_ifp_removed(struct brcmf_if *ifp)
 	vif = ifp->vif;
 	cfg = wdev_to_cfg(&vif->wdev);
 	cfg->p2p.bss_idx[P2PAPI_BSSCFG_DEVICE].vif = NULL;
-	rtnl_lock();
+	if (!rtnl_locked)
+		rtnl_lock();
 	cfg80211_unregister_wdev(&vif->wdev);
-	rtnl_unlock();
+	if (!rtnl_locked)
+		rtnl_unlock();
 	brcmf_free_vif(vif);
 }
 
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.h b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.h
index a3bd18c..8ce9447 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.h
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.h
@@ -155,7 +155,7 @@ struct wireless_dev *brcmf_p2p_add_vif(struct wiphy *wiphy, const char *name,
 int brcmf_p2p_del_vif(struct wiphy *wiphy, struct wireless_dev *wdev);
 int brcmf_p2p_ifchange(struct brcmf_cfg80211_info *cfg,
 		       enum brcmf_fil_p2p_if_types if_type);
-void brcmf_p2p_ifp_removed(struct brcmf_if *ifp);
+void brcmf_p2p_ifp_removed(struct brcmf_if *ifp, bool rtnl_locked);
 int brcmf_p2p_start_device(struct wiphy *wiphy, struct wireless_dev *wdev);
 void brcmf_p2p_stop_device(struct wiphy *wiphy, struct wireless_dev *wdev);
 int brcmf_p2p_scan_prep(struct wiphy *wiphy,

[toc] | [next] | [standalone]

#1462703 — Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

FromRafał Miłecki <zajec5@gmail.com>
Date2016-08-15 12:50 +0200
SubjectRe: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface
Message-ID<s6nCV-4sn-5@gated-at.bofh.it>
In reply to#1462680
On 08/15/2016 11:40 AM, Masami Hiramatsu wrote:
> Check rtnl_lock is locked in brcmf_p2p_ifp_removed() by passing
> rtnl_locked flag. Actually the caller brcmf_del_if() checks whether
> the rtnl_lock is locked, but doesn't pass it to brcmf_p2p_ifp_removed().
>
> Without this fix, wpa_supplicant goes softlockup with rtnl_lock
> holding (this means all other process using netlink are locked up too)
>
> e.g.
> [ 4495.876627] INFO: task wpa_supplicant:7307 blocked for more than 10 seconds.
> [ 4495.876632]       Tainted: G        W       4.8.0-rc1+ #8
> [ 4495.876635] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
> [ 4495.876638] wpa_supplicant  D ffff974c647b39a0     0  7307      1 0x00000000
> [ 4495.876644]  ffff974c647b39a0 0000000000000000 ffff974c00000000 ffff974c7dc59c58
> [ 4495.876651]  ffff974c6b7417c0 ffff974c645017c0 ffff974c647b4000 ffffffff86f16c08
> [ 4495.876657]  ffff974c645017c0 0000000000000246 00000000ffffffff ffff974c647b39b8
> [ 4495.876664] Call Trace:
> [ 4495.876671]  [<ffffffff868aeccc>] schedule+0x3c/0x90
> [ 4495.876676]  [<ffffffff868af065>] schedule_preempt_disabled+0x15/0x20
> [ 4495.876682]  [<ffffffff868b0996>] mutex_lock_nested+0x176/0x3b0
> [ 4495.876686]  [<ffffffff867a2067>] ? rtnl_lock+0x17/0x20
> [ 4495.876690]  [<ffffffff867a2067>] rtnl_lock+0x17/0x20
> [ 4495.876720]  [<ffffffffc0ae9a5d>] brcmf_p2p_ifp_removed+0x4d/0x70 [brcmfmac]
> [ 4495.876741]  [<ffffffffc0aebde6>] brcmf_remove_interface+0x196/0x1b0 [brcmfmac]
> [ 4495.876760]  [<ffffffffc0ae9901>] brcmf_p2p_del_vif+0x111/0x220 [brcmfmac]
> [ 4495.876777]  [<ffffffffc0adefab>] brcmf_cfg80211_del_iface+0x21b/0x270 [brcmfmac]
> [ 4495.876820]  [<ffffffffc097b39e>] nl80211_del_interface+0xfe/0x3a0 [cfg80211]
> [ 4495.876825]  [<ffffffff867ca335>] genl_family_rcv_msg+0x1b5/0x370
> [ 4495.876832]  [<ffffffff860e5d8d>] ? trace_hardirqs_on+0xd/0x10
> [ 4495.876836]  [<ffffffff867ca56d>] genl_rcv_msg+0x7d/0xb0
> [ 4495.876839]  [<ffffffff867ca4f0>] ? genl_family_rcv_msg+0x370/0x370
> [ 4495.876846]  [<ffffffff867c9a47>] netlink_rcv_skb+0x97/0xb0
> [ 4495.876849]  [<ffffffff867ca168>] genl_rcv+0x28/0x40
> [ 4495.876854]  [<ffffffff867c93c3>] netlink_unicast+0x1d3/0x2f0
> [ 4495.876860]  [<ffffffff867c933b>] ? netlink_unicast+0x14b/0x2f0
> [ 4495.876866]  [<ffffffff867c97cb>] netlink_sendmsg+0x2eb/0x3a0
> [ 4495.876870]  [<ffffffff8676dad8>] sock_sendmsg+0x38/0x50
> [ 4495.876874]  [<ffffffff8676e4df>] ___sys_sendmsg+0x27f/0x290
> [ 4495.876882]  [<ffffffff8628b935>] ? mntput_no_expire+0x5/0x3f0
> [ 4495.876888]  [<ffffffff8628b9be>] ? mntput_no_expire+0x8e/0x3f0
> [ 4495.876894]  [<ffffffff8628b935>] ? mntput_no_expire+0x5/0x3f0
> [ 4495.876899]  [<ffffffff8628bd44>] ? mntput+0x24/0x40
> [ 4495.876904]  [<ffffffff86267830>] ? __fput+0x190/0x200
> [ 4495.876909]  [<ffffffff8676f125>] __sys_sendmsg+0x45/0x80
> [ 4495.876914]  [<ffffffff8676f172>] SyS_sendmsg+0x12/0x20
> [ 4495.876918]  [<ffffffff868b5680>] entry_SYSCALL_64_fastpath+0x23/0xc1
> [ 4495.876924]  [<ffffffff860e2b8f>] ? trace_hardirqs_off_caller+0x1f/0xc0

This is probably caused by my commit:
a63b09872c1d ("brcmfmac: delete interface directly in code that sent fw request")
https://git.kernel.org/cgit/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/?id=a63b09872c1dc0ce0da3628647da67a112b484bf

I changed condition for calling brcmf_remove_interface and it seems it broke P2P. Unfortunately I couldn't fully test my change due to firmware not supporting P2P.

I did similar fix for error path for P2P with commit
b50ddfa8530e ("brcmfmac: fix lockup when removing P2P interface after event timeout")
https://git.kernel.org/cgit/linux/kernel/git/kvalo/wireless-drivers-next.git/commit/?id=b50ddfa8530e9b5f52e873fdd6ff04f327a88799
so your change looks like a proper follow-up.


> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>

Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that sent fw request")
Acked-by: Rafał Miłecki <rafal@milecki.pl>

Kalle: I'm acking this as bugfix for 4.8 release.

[toc] | [prev] | [next] | [standalone]

#1462708

FromKalle Valo <kvalo@codeaurora.org>
Date2016-08-15 13:00 +0200
Message-ID<s6nMB-4vM-5@gated-at.bofh.it>
In reply to#1462703
Rafał Miłecki <zajec5@gmail.com> writes:

>> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
>
> Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that sent fw request")
> Acked-by: Rafał Miłecki <rafal@milecki.pl>
>
> Kalle: I'm acking this as bugfix for 4.8 release.

Ok. I'll wait few days for more comments before I apply this.

(I assume you are talking only about patch 1)

-- 
Kalle Valo

[toc] | [prev] | [next] | [standalone]

#1462732 — Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

FromRafał Miłecki <zajec5@gmail.com>
Date2016-08-15 14:00 +0200
SubjectRe: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface
Message-ID<s6oIG-56q-17@gated-at.bofh.it>
In reply to#1462708
On 15 August 2016 at 12:57, Kalle Valo <kvalo@codeaurora.org> wrote:
> Rafał Miłecki <zajec5@gmail.com> writes:
>
>>> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
>>
>> Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that sent fw request")
>> Acked-by: Rafał Miłecki <rafal@milecki.pl>
>>
>> Kalle: I'm acking this as bugfix for 4.8 release.
>
> Ok. I'll wait few days for more comments before I apply this.

Sure.


> (I assume you are talking only about patch 1)

Yes, I'll leave mutex vs. spinlock to the experts :)

-- 
Rafał

[toc] | [prev] | [next] | [standalone]

#1463197 — Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

FromArend Van Spriel <arend.vanspriel@broadcom.com>
Date2016-08-15 23:50 +0200
SubjectRe: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface
Message-ID<s6xVD-2Er-11@gated-at.bofh.it>
In reply to#1462732

On 15-8-2016 13:52, Rafał Miłecki wrote:
> On 15 August 2016 at 12:57, Kalle Valo <kvalo@codeaurora.org> wrote:
>> Rafał Miłecki <zajec5@gmail.com> writes:
>>
>>>> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
>>>
>>> Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that sent fw request")
>>> Acked-by: Rafał Miłecki <rafal@milecki.pl>
>>>
>>> Kalle: I'm acking this as bugfix for 4.8 release.
>>
>> Ok. I'll wait few days for more comments before I apply this.
> 
> Sure.
> 
> 
>> (I assume you are talking only about patch 1)
> 
> Yes, I'll leave mutex vs. spinlock to the experts :)

Don't know who the experts are. Surely not me :-p

I made an uneducated design decision using a mutex for this. The
reasoning for using a regular spinlock make sense. So I will go and ack
that patch.

Regards,
Arend

[toc] | [prev] | [next] | [standalone]

#1463350 — Re: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

FromMasami Hiramatsu <mhiramat@kernel.org>
Date2016-08-16 04:30 +0200
SubjectRe: [BUGFIX PATCH 1/2] brcmfmac: Check rtnl_lock is locked when removing interface
Message-ID<s6CiB-5zO-5@gated-at.bofh.it>
In reply to#1463197
On Mon, 15 Aug 2016 23:44:05 +0200
Arend Van Spriel <arend.vanspriel@broadcom.com> wrote:

> 
> 
> On 15-8-2016 13:52, Rafał Miłecki wrote:
> > On 15 August 2016 at 12:57, Kalle Valo <kvalo@codeaurora.org> wrote:
> >> Rafał Miłecki <zajec5@gmail.com> writes:
> >>
> >>>> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
> >>>
> >>> Fixes: a63b09872c1d ("brcmfmac: delete interface directly in code that sent fw request")
> >>> Acked-by: Rafał Miłecki <rafal@milecki.pl>
> >>>
> >>> Kalle: I'm acking this as bugfix for 4.8 release.
> >>
> >> Ok. I'll wait few days for more comments before I apply this.

Thanks!

> > 
> > Sure.
> > 
> > 
> >> (I assume you are talking only about patch 1)
> > 
> > Yes, I'll leave mutex vs. spinlock to the experts :)
> 
> Don't know who the experts are. Surely not me :-p
> 
> I made an uneducated design decision using a mutex for this. The
> reasoning for using a regular spinlock make sense. So I will go and ack
> that patch.

As far as I can see, that change is very local and
at least my environment it works well :)

Regards,

> 
> Regards,
> Arend


-- 
Masami Hiramatsu <mhiramat@kernel.org>

[toc] | [prev] | [next] | [standalone]

#1469412 — Re: [BUGFIX, 1/2] brcmfmac: Check rtnl_lock is locked when removing interface

FromKalle Valo <kvalo@codeaurora.org>
Date2016-08-24 15:20 +0200
SubjectRe: [BUGFIX, 1/2] brcmfmac: Check rtnl_lock is locked when removing interface
Message-ID<s9Gg1-2I8-9@gated-at.bofh.it>
In reply to#1462680
mhiramat@kernel.org wrote:
> Check rtnl_lock is locked in brcmf_p2p_ifp_removed() by passing
> rtnl_locked flag. Actually the caller brcmf_del_if() checks whether
> the rtnl_lock is locked, but doesn't pass it to brcmf_p2p_ifp_removed().
> 
> Without this fix, wpa_supplicant goes softlockup with rtnl_lock
> holding (this means all other process using netlink are locked up too)
> 
> e.g.
> [ 4495.876627] INFO: task wpa_supplicant:7307 blocked for more than 10 seconds.
> [ 4495.876632]       Tainted: G        W       4.8.0-rc1+ #8
> [ 4495.876635] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
> [ 4495.876638] wpa_supplicant  D ffff974c647b39a0     0  7307      1 0x00000000
> [ 4495.876644]  ffff974c647b39a0 0000000000000000 ffff974c00000000 ffff974c7dc59c58
> [ 4495.876651]  ffff974c6b7417c0 ffff974c645017c0 ffff974c647b4000 ffffffff86f16c08
> [ 4495.876657]  ffff974c645017c0 0000000000000246 00000000ffffffff ffff974c647b39b8
> [ 4495.876664] Call Trace:
> [ 4495.876671]  [<ffffffff868aeccc>] schedule+0x3c/0x90
> [ 4495.876676]  [<ffffffff868af065>] schedule_preempt_disabled+0x15/0x20
> [ 4495.876682]  [<ffffffff868b0996>] mutex_lock_nested+0x176/0x3b0
> [ 4495.876686]  [<ffffffff867a2067>] ? rtnl_lock+0x17/0x20
> [ 4495.876690]  [<ffffffff867a2067>] rtnl_lock+0x17/0x20
> [ 4495.876720]  [<ffffffffc0ae9a5d>] brcmf_p2p_ifp_removed+0x4d/0x70 [brcmfmac]
> [ 4495.876741]  [<ffffffffc0aebde6>] brcmf_remove_interface+0x196/0x1b0 [brcmfmac]
> [ 4495.876760]  [<ffffffffc0ae9901>] brcmf_p2p_del_vif+0x111/0x220 [brcmfmac]
> [ 4495.876777]  [<ffffffffc0adefab>] brcmf_cfg80211_del_iface+0x21b/0x270 [brcmfmac]
> [ 4495.876820]  [<ffffffffc097b39e>] nl80211_del_interface+0xfe/0x3a0 [cfg80211]
> [ 4495.876825]  [<ffffffff867ca335>] genl_family_rcv_msg+0x1b5/0x370
> [ 4495.876832]  [<ffffffff860e5d8d>] ? trace_hardirqs_on+0xd/0x10
> [ 4495.876836]  [<ffffffff867ca56d>] genl_rcv_msg+0x7d/0xb0
> [ 4495.876839]  [<ffffffff867ca4f0>] ? genl_family_rcv_msg+0x370/0x370
> [ 4495.876846]  [<ffffffff867c9a47>] netlink_rcv_skb+0x97/0xb0
> [ 4495.876849]  [<ffffffff867ca168>] genl_rcv+0x28/0x40
> [ 4495.876854]  [<ffffffff867c93c3>] netlink_unicast+0x1d3/0x2f0
> [ 4495.876860]  [<ffffffff867c933b>] ? netlink_unicast+0x14b/0x2f0
> [ 4495.876866]  [<ffffffff867c97cb>] netlink_sendmsg+0x2eb/0x3a0
> [ 4495.876870]  [<ffffffff8676dad8>] sock_sendmsg+0x38/0x50
> [ 4495.876874]  [<ffffffff8676e4df>] ___sys_sendmsg+0x27f/0x290
> [ 4495.876882]  [<ffffffff8628b935>] ? mntput_no_expire+0x5/0x3f0
> [ 4495.876888]  [<ffffffff8628b9be>] ? mntput_no_expire+0x8e/0x3f0
> [ 4495.876894]  [<ffffffff8628b935>] ? mntput_no_expire+0x5/0x3f0
> [ 4495.876899]  [<ffffffff8628bd44>] ? mntput+0x24/0x40
> [ 4495.876904]  [<ffffffff86267830>] ? __fput+0x190/0x200
> [ 4495.876909]  [<ffffffff8676f125>] __sys_sendmsg+0x45/0x80
> [ 4495.876914]  [<ffffffff8676f172>] SyS_sendmsg+0x12/0x20
> [ 4495.876918]  [<ffffffff868b5680>] entry_SYSCALL_64_fastpath+0x23/0xc1
> [ 4495.876924]  [<ffffffff860e2b8f>] ? trace_hardirqs_off_caller+0x1f/0xc0
> 
> Signed-off-by: Masami Hiramatsu <mhiramat@kernel.org>
> Acked-by: Rafał Miłecki <rafal@milecki.pl>

Thanks, 2 patches applied to wireless-drivers.git:

15dacf880e49 brcmfmac: Check rtnl_lock is locked when removing interface
b64abcb7dae6 brcmfmac: Change vif_event_lock to spinlock

-- 
Sent by pwcli
https://patchwork.kernel.org/patch/9280681/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web