Return-path: Received: from packetmixer.de ([79.140.42.25]:56012 "EHLO mail.mail.packetmixer.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754493Ab3KURUk (ORCPT ); Thu, 21 Nov 2013 12:20:40 -0500 From: Simon Wunderlich To: johannes@sipsolutions.net Cc: linux-wireless@vger.kernel.org, Simon Wunderlich Subject: [PATCHv2 0/5] Fix CSA locking issues Date: Thu, 21 Nov 2013 18:19:48 +0100 Message-Id: <1385054393-17238-1-git-send-email-sw@simonwunderlich.de> (sfid-20131121_182043_536429_ACFB11D3) Sender: linux-wireless-owner@vger.kernel.org List-ID: Unfortunately my channel switch announcement patches contained locking issues which I didn't notice during my test due to bad configuration. Johannes pointed them out, so here is my attempt to fix them. This series attempts to fix two issues: * There are "suspicious rcu_dereference_protected() usage" problems for ieee80211_assign_beacon and ieee80211_ibss_csa_beacon because the respective locking was missing * There is a circular dependency between the wdev/sdata-lock and the csa_finalize worker lock (see description in patch 3). This is another update. Changes from the PATCH series: * split first patch into mac80211 and cfg80211 part * simplify memory leak fix Cheers, Simon Simon Wunderlich (5): cfg80211: protect beacon changing functions with wdev-lock mac80211: modify beacon using sdata/wdev-lock, not rtnl lock cfg80211/mac80211/ath6kl: acquire wdev lock outside ch_switch_notify mac80211: don't cancel csa finalize work within stop_ap mac80211: don't leak next beacon when csa is aborted drivers/net/wireless/ath/ath6kl/cfg80211.c | 2 ++ include/net/cfg80211.h | 3 +- net/mac80211/cfg.c | 45 ++++++++++++++++++---------- net/mac80211/ieee80211_i.h | 3 ++ net/wireless/nl80211.c | 31 +++++++++++++------ 5 files changed, 59 insertions(+), 25 deletions(-) -- 1.7.10.4