From 91d92caece5ce014a944d5bfd95d8b26f08704ff Mon Sep 17 00:00:00 2001 From: Adrian Chadd Date: Fri, 9 Mar 2012 22:41:09 +0000 Subject: [PATCH] Should the mcast queue be locked here? In case more multicast traffic comes along? This commit was brought to you via an Atheros AR5210, associated to an 3x3 HT40 11na access point. Yes, this driver still works with it. --- sys/dev/ath/if_ath.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sys/dev/ath/if_ath.c b/sys/dev/ath/if_ath.c index 21adfa89a00..bff28ee48fd 100644 --- a/sys/dev/ath/if_ath.c +++ b/sys/dev/ath/if_ath.c @@ -3059,7 +3059,9 @@ ath_beacon_generate(struct ath_softc *sc, struct ieee80211vap *vap) */ bf = avp->av_bcbuf; m = bf->bf_m; + /* XXX lock mcastq? */ nmcastq = avp->av_mcastq.axq_depth; + if (ieee80211_beacon_update(bf->bf_node, &avp->av_boff, m, nmcastq)) { /* XXX too conservative? */ bus_dmamap_unload(sc->sc_dmat, bf->bf_dmamap);