From patchwork Sat Nov  7 17:37:37 2009
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
X-Patchwork-Submitter: Felix Fietkau <nbd@openwrt.org>
X-Patchwork-Id: 58327
Received: from vger.kernel.org (vger.kernel.org [209.132.176.167])
	by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id nA7Hatls013685
	for <patchwork-linux-wireless@patchwork.kernel.org>;
	Sat, 7 Nov 2009 17:37:44 GMT
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1752590AbZKGRhi (ORCPT
	<rfc822;patchwork-linux-wireless@patchwork.kernel.org>);
	Sat, 7 Nov 2009 12:37:38 -0500
Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752581AbZKGRhi
	(ORCPT <rfc822;linux-wireless-outgoing>);
	Sat, 7 Nov 2009 12:37:38 -0500
Received: from nbd.name ([88.198.39.176]:36229 "EHLO ds10.mine.nu"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1752579AbZKGRhh (ORCPT <rfc822; linux-wireless@vger.kernel.org>);
	Sat, 7 Nov 2009 12:37:37 -0500
Received: from p54853c0f.dip.t-dialin.net ([84.133.60.15] helo=pi.local)
	by ds10.mine.nu with esmtpsa (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32)
	(Exim 4.63) (envelope-from <nbd@openwrt.org>)
	id 1N6pI6-0002Fz-Kg; Sat, 07 Nov 2009 18:41:34 +0100
Message-ID: <4AF5B061.60701@openwrt.org>
Date: Sat, 07 Nov 2009 18:37:37 +0100
From: Felix Fietkau <nbd@openwrt.org>
User-Agent: Thunderbird 2.0.0.23 (Macintosh/20090812)
MIME-Version: 1.0
To: linux-wireless <linux-wireless@vger.kernel.org>
CC: "John W. Linville" <linville@tuxdriver.com>,
	Johannes Berg <johannes@sipsolutions.net>, Michael Buesch <mb@bu3sch.de>
Subject: [PATCH] b43: work around a locking issue in ->set_tim()
X-Enigmail-Version: 0.96.0
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-wireless.vger.kernel.org>
X-Mailing-List: linux-wireless@vger.kernel.org


--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -4534,9 +4534,8 @@ static int b43_op_beacon_set_tim(struct
 {
 	struct b43_wl *wl = hw_to_b43_wl(hw);

-	mutex_lock(&wl->mutex);
+	/* FIXME: add locking */
 	b43_update_templates(wl);
-	mutex_unlock(&wl->mutex);

 	return 0;
 }