Commit 8a61af65 authored by Johannes Berg's avatar Johannes Berg
Browse files

mac80211: fix channel context iteration 



During suspend/resume channel contexts might be
iterated even if they haven't been re-added to
the driver, keep track of this and skip them in
iteration. Also use the new status for sanity
checks.

Also clarify the fact that during HW restart all
contexts are iterated over (thanks Eliad.)

Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 529ba6e9

include/net/mac80211.h

+5 −0
Original line number Diff line number Diff line
@@ -3754,6 +3754,11 @@ void ieee80211_iter_keys(struct ieee80211_hw *hw, 
 * The iterator will not find a context that's being added (during

 * the driver callback to add it) but will find it while it's being

 * removed.

 *

 * Note that during hardware restart, all contexts that existed

 * before the restart are considered already present so will be

 * found while iterating, whether they've been re-added already

 * or not.

 */

void ieee80211_iter_chan_contexts_atomic(

	struct ieee80211_hw *hw,

net/mac80211/chan.c

+2 −1
Original line number Diff line number Diff line
@@ -381,6 +381,7 @@ void ieee80211_iter_chan_contexts_atomic( 


	rcu_read_lock();

	list_for_each_entry_rcu(ctx, &local->chanctx_list, list)

		if (ctx->driver_present)

			iter(hw, &ctx->conf, iter_data);

	rcu_read_unlock();

}

net/mac80211/driver-ops.h

+12 −3
Original line number Diff line number Diff line
@@ -913,6 +913,8 @@ static inline int drv_add_chanctx(struct ieee80211_local *local, 
	if (local->ops->add_chanctx)

		ret = local->ops->add_chanctx(&local->hw, &ctx->conf);

	trace_drv_return_int(local, ret);

	if (!ret)

		ctx->driver_present = true;



	return ret;

}
@@ -924,6 +926,7 @@ static inline void drv_remove_chanctx(struct ieee80211_local *local, 
	if (local->ops->remove_chanctx)

		local->ops->remove_chanctx(&local->hw, &ctx->conf);

	trace_drv_return_void(local);

	ctx->driver_present = false;

}



static inline void drv_change_chanctx(struct ieee80211_local *local,
@@ -931,8 +934,10 @@ static inline void drv_change_chanctx(struct ieee80211_local *local, 
				      u32 changed)

{

	trace_drv_change_chanctx(local, ctx, changed);

	if (local->ops->change_chanctx)

	if (local->ops->change_chanctx) {

		WARN_ON_ONCE(!ctx->driver_present);

		local->ops->change_chanctx(&local->hw, &ctx->conf, changed);

	}

	trace_drv_return_void(local);

}
@@ -945,10 +950,12 @@ static inline int drv_assign_vif_chanctx(struct ieee80211_local *local, 
	check_sdata_in_driver(sdata);



	trace_drv_assign_vif_chanctx(local, sdata, ctx);

	if (local->ops->assign_vif_chanctx)

	if (local->ops->assign_vif_chanctx) {

		WARN_ON_ONCE(!ctx->driver_present);

		ret = local->ops->assign_vif_chanctx(&local->hw,

						     &sdata->vif,

						     &ctx->conf);

	}

	trace_drv_return_int(local, ret);



	return ret;
@@ -961,10 +968,12 @@ static inline void drv_unassign_vif_chanctx(struct ieee80211_local *local, 
	check_sdata_in_driver(sdata);



	trace_drv_unassign_vif_chanctx(local, sdata, ctx);

	if (local->ops->unassign_vif_chanctx)

	if (local->ops->unassign_vif_chanctx) {

		WARN_ON_ONCE(!ctx->driver_present);

		local->ops->unassign_vif_chanctx(&local->hw,

						 &sdata->vif,

						 &ctx->conf);

	}

	trace_drv_return_void(local);

}

net/mac80211/ieee80211_i.h

+1 −0
Original line number Diff line number Diff line
@@ -685,6 +685,7 @@ struct ieee80211_chanctx { 


	enum ieee80211_chanctx_mode mode;

	int refcount;

	bool driver_present;



	struct ieee80211_chanctx_conf conf;

};

CRA Git | Maintained and supported by SUSTech CRA and CCSE