From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-bw0-f219.google.com ([209.85.218.219]:48347 "EHLO mail-bw0-f219.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750881AbZHIKS0 (ORCPT ); Sun, 9 Aug 2009 06:18:26 -0400 Received: by bwz19 with SMTP id 19so2130806bwz.37 for ; Sun, 09 Aug 2009 03:18:26 -0700 (PDT) To: Johannes Berg Cc: John Linville , linux-wireless Subject: Re: [PATCH] cfg80211: add missing device list locking References: <1249811489.4636.2.camel@johannes.local> From: Kalle Valo Date: Sun, 09 Aug 2009 13:18:23 +0300 In-Reply-To: <1249811489.4636.2.camel@johannes.local> (Johannes Berg's message of "Sun\, 09 Aug 2009 11\:51\:29 +0200") Message-ID: <87iqgxl28g.fsf@litku.valot.fi> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-wireless-owner@vger.kernel.org List-ID: Johannes Berg writes: > When calling into the wext code from the NETDEV_UP > notifier, we need to hold the devlist_mtx mutex as > the wext code ends up calling into channel checks. > > Reported-by: Kalle Valo > Signed-off-by: Johannes Berg Thanks for fixing this. I wasn't able to reproduce the warning after one suspend/resume cycle, but I'll test this patch a bit more before I send my Tested-by tag. -- Kalle Valo