Skip to contentSkip to conversation

Implement ath9k mesh point operation

RFC In review Pat Erley 3 participants 7 messages Mar 16, 2009 – Mar 17, 2009 Networking drivers (wireless)

lore
Share & tools
lore

Pat Erley

author
#0 lore raw

RFC 1/1 Implement ath9k mesh point operation

This patch enables mesh point operation for ath9k. Tested with b43 and ath5k as peers.

---
Currently, you need to trigger a scan on the mesh point interface to enable beaconing. This is the same behavior the ath5k, rt2500usb and rt61pci drivers exhibit. Is this a mac80211 design flaw, or something in all of these drivers?

drivers/net/wireless/ath9k/beacon.c +3 -1
index 039c781..68e05ed 100644
@@ -70,7 +70,8 @@ static void ath_beacon_setup(struct ath_softc *sc, struct ath_vif *avp,
 	ds = bf->bf_desc;
 	flags = ATH9K_TXDESC_NOACK;
 
-	if (sc->sc_ah->opmode == NL80211_IFTYPE_ADHOC &&
+	if (((sc->sc_ah->opmode == NL80211_IFTYPE_ADHOC) ||
+	     (sc->sc_ah->opmode == NL80211_IFTYPE_MESH_POINT)) &&
 	    (ah->caps.hw_caps & ATH9K_HW_CAP_VEOL)) {
 		ds->ds_link = bf->bf_daddr; /* self-linked */
 		flags |= ATH9K_TXDESC_VEOL;
@@ -728,6 +729,7 @@ void ath_beacon_config(struct ath_softc *sc, struct ieee80211_vif *vif)
 			ath_beacon_config_ap(sc, &conf, avp);
 			break;
 		case NL80211_IFTYPE_ADHOC:
+		case NL80211_IFTYPE_MESH_POINT:
 			ath_beacon_config_adhoc(sc, &conf, avp, vif);
 			break;
 		case NL80211_IFTYPE_STATION:
drivers/net/wireless/ath9k/hw.c +2 -0
index ea550cc..a69328e 100644
@@ -1424,6 +1424,7 @@ static void ath9k_hw_set_operating_mode(struct ath_hw *ah, int opmode)
 		REG_CLR_BIT(ah, AR_CFG, AR_CFG_AP_ADHOC_INDICATION);
 		break;
 	case NL80211_IFTYPE_ADHOC:
+	case NL80211_IFTYPE_MESH_POINT:
 		REG_WRITE(ah, AR_STA_ID1, val | AR_STA_ID1_ADHOC
 			  | AR_STA_ID1_KSRCH_MODE);
 		REG_SET_BIT(ah, AR_CFG, AR_CFG_AP_ADHOC_INDICATION);
@@ -3129,6 +3130,7 @@ void ath9k_hw_beaconinit(struct ath_hw *ah, u32 next_beacon, u32 beacon_period)
 		flags |= AR_TBTT_TIMER_EN;
 		break;
 	case NL80211_IFTYPE_ADHOC:
+	case NL80211_IFTYPE_MESH_POINT:
 		REG_SET_BIT(ah, AR_TXCFG,
 			    AR_TXCFG_ADHOC_BEACON_ATIM_TX_POLICY);
 		REG_WRITE(ah, AR_NEXT_NDP_TIMER,
drivers/net/wireless/ath9k/main.c +12 -12
index a2f5af6..0f9f6ca 100644
@@ -1591,7 +1591,8 @@ void ath_set_hw_capab(struct ath_softc *sc, struct ieee80211_hw *hw)
 	hw->wiphy->interface_modes =
 		BIT(NL80211_IFTYPE_AP) |
 		BIT(NL80211_IFTYPE_STATION) |
-		BIT(NL80211_IFTYPE_ADHOC);
+		BIT(NL80211_IFTYPE_ADHOC) |
+		BIT(NL80211_IFTYPE_MESH_POINT);
 
 	hw->wiphy->reg_notifier = ath9k_reg_notifier;
 	hw->wiphy->strict_regulatory = true;
@@ -2199,18 +2200,13 @@ static int ath9k_add_interface(struct ieee80211_hw *hw,
 		ic_opmode = NL80211_IFTYPE_STATION;
 		break;
 	case NL80211_IFTYPE_ADHOC:
-		if (sc->nbcnvifs >= ATH_BCBUF) {
-			ret = -ENOBUFS;
-			goto out;
-		}
-		ic_opmode = NL80211_IFTYPE_ADHOC;
-		break;
 	case NL80211_IFTYPE_AP:
+	case NL80211_IFTYPE_MESH_POINT:
 		if (sc->nbcnvifs >= ATH_BCBUF) {
 			ret = -ENOBUFS;
 			goto out;
 		}
-		ic_opmode = NL80211_IFTYPE_AP;
+		ic_opmode = conf->type;
 		break;
 	default:
 		DPRINTF(sc, ATH_DBG_FATAL,
@@ -2246,7 +2242,8 @@ static int ath9k_add_interface(struct ieee80211_hw *hw,
 	 * Note we only do this (at the moment) for station mode.
 	 */
 	if ((conf->type == NL80211_IFTYPE_STATION) ||
-	    (conf->type == NL80211_IFTYPE_ADHOC)) {
+	    (conf->type == NL80211_IFTYPE_ADHOC) ||
+	    (conf->type == NL80211_IFTYPE_MESH_POINT)) {
 		if (ath9k_hw_phycounters(sc->sc_ah))
 			sc->imask |= ATH9K_INT_MIB;
 		sc->imask |= ATH9K_INT_TSFOOR;
@@ -2293,8 +2290,9 @@ static void ath9k_remove_interface(struct ieee80211_hw *hw,
 	del_timer_sync(&sc->ani.timer);
 
 	/* Reclaim beacon resources */
-	if (sc->sc_ah->opmode == NL80211_IFTYPE_AP ||
-	    sc->sc_ah->opmode == NL80211_IFTYPE_ADHOC) {
+	if ((sc->sc_ah->opmode == NL80211_IFTYPE_AP) ||
+	    (sc->sc_ah->opmode == NL80211_IFTYPE_ADHOC) ||
+	    (sc->sc_ah->opmode == NL80211_IFTYPE_MESH_POINT)) {
 		ath9k_hw_stoptxdma(sc->sc_ah, sc->beacon.beaconq);
 		ath_beacon_return(sc, avp);
 	}
@@ -2427,6 +2425,7 @@ static int ath9k_config_interface(struct ieee80211_hw *hw,
 		switch (vif->type) {
 		case NL80211_IFTYPE_STATION:
 		case NL80211_IFTYPE_ADHOC:
+		case NL80211_IFTYPE_MESH_POINT:
 			/* Set BSSID */
 			memcpy(sc->curbssid, conf->bssid, ETH_ALEN);
 			memcpy(avp->bssid, conf->bssid, ETH_ALEN);
@@ -2450,7 +2449,8 @@ static int ath9k_config_interface(struct ieee80211_hw *hw,
 	}
 
 	if ((vif->type == NL80211_IFTYPE_ADHOC) ||
-	    (vif->type == NL80211_IFTYPE_AP)) {
+	    (vif->type == NL80211_IFTYPE_AP) ||
+	    (vif->type == NL80211_IFTYPE_MESH_POINT)) {
 		if ((conf->changed & IEEE80211_IFCC_BEACON) ||
 		    (conf->changed & IEEE80211_IFCC_BEACON_ENABLED &&
 		     conf->enable_beacon)) {
drivers/net/wireless/ath9k/rc.c +1 -0
index 8327356..31a6a2a 100644
@@ -1615,6 +1615,7 @@ static void ath_rate_init(void *priv, struct ieee80211_supported_band *sband,
 	/* Choose rate table first */
 
 	if ((sc->sc_ah->opmode == NL80211_IFTYPE_STATION) ||
+	    (sc->sc_ah->opmode == NL80211_IFTYPE_MESH_POINT) ||
 	    (sc->sc_ah->opmode == NL80211_IFTYPE_ADHOC)) {
 		rate_table = ath_choose_rate_table(sc, sband->band,
 						   sta->ht_cap.ht_supported,
#1 lore raw

Hi Pat,

Quoted 2 lines · Pat Erley

Currently, you need to trigger a scan on the mesh point interface to enable beaconing. =A0This is the same behavior the ath5k, rt2500usb a=

nd

Quoted 1 line

rt61pci drivers exhibit. =A0Is this a mac80211 design flaw, or someth=

ing

Quoted 1 line

in all of these drivers?

Are you really seeing this behavior on ath5k? I have never seen that happen, if it's new then it's a regression. rt2x00 does have this problem at this time.

-Andrey
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

author
#2 lore raw
Quoted 8 lines · Andrey Yurovsky

Hi Pat,

Quoted 4 lines · Pat Erley

Currently, you need to trigger a scan on the mesh point interface to enable beaconing. This is the same behavior the ath5k, rt2500usb and rt61pci drivers exhibit. Is this a mac80211 design flaw, or something in all of these drivers?

Quoted 5 lines

Are you really seeing this behavior on ath5k? I have never seen that happen, if it's new then it's a regression. rt2x00 does have this problem at this time.

-Andrey

Yep, here's my lspci output and relevant dmesg messages when I load ath5k:

Atheros Communications Inc. AR5212/AR5213 Multiprotocol MAC/baseband processor (rev 01)

ath5k 0000:02:0a.0: PCI INT A -> Link[APC3] -> GSI 18 (level, low) -> IRQ 18 ath5k 0000:02:0a.0: registered as 'phy5'
phy5: Selected rate control algorithm 'minstrel'
ath5k phy5: Atheros AR5213A chip found (MAC: 0x59, PHY: 0x43) ath5k phy5: RF5112B multiband radio found (0x36)
udev: renamed network interface wlan1 to wlan3

(yes, phy5, I have a ton of wireless stuff plugged into this dev system)

and here's the procedure I use:

iw dev wlan3 del
iw phy phy5 interface add mp0 type mp mesh_id FooBarMesh ifconfig mp0 10.0.0.5 netmask 255.255.255.0 up (tshark on rtl8187 shows no beaconing)
iw dev mp0 scan trigger
(tshark on rtl8187 now shows beaconing)

Out of all the devices I have, only b43 seems to beacon before a scan. Is there a chance I need to wait like 45 seconds for beaconing to start on ath5k like how ibss mode tries to find a peer already broadcasting before it enables beacons?

Pat Erley

author
#3 lore raw
In reply to Pat Erley

Also, I was able to do the development of this thanks to hardware donated by Johannes Berg (b43 card that was used as a baseline known working device) and an ath9k card donated by Oxford TEC[1]. What's the best way to add this 'credit' to the commit, or is it not appropriate?

Pat Erley

[1] http://www.oxfordtec.com/

maintainer
#4 lore raw
Quoted 5 lines · Pat Erley

Also, I was able to do the development of this thanks to hardware donated by Johannes Berg (b43 card that was used as a baseline known working device) and an ath9k card donated by Oxford TEC[1]. What's the best way to add this 'credit' to the commit, or is it not appropriate?

For all I'm personally concerned, it doesn't matter at all.

johannes

maintainer
#5 lore raw
In reply to Pat Erley
Quoted 4 lines · Pat Erley

Currently, you need to trigger a scan on the mesh point interface to enable beaconing. This is the same behavior the ath5k, rt2500usb and rt61pci drivers exhibit. Is this a mac80211 design flaw, or something in all of these drivers?

It's mac80211. We never call ieee80211_if_config() when we enable a mesh interface, and thus conf.enable_beacon with IEEE80211_IFCC_BEACON_ENABLED isn't ever set correctly.

johannes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
Url : http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20090317/8064f19a/attachment.pgp

#6 lore raw

ieee80211_start_mesh() does this:

ieee80211_if_config(sdata, IEEE80211_IFCC_BEACON |

                                   IEEE80211_IFCC_BEACON_ENABLED);

and it's called by ieee80211_open()

Quoted 4 lines · Johannes Berg
Quoted 2 lines · Pat Erley

Currently, you need to trigger a scan on the mesh point interface to enable beaconing. =A0This is the same behavior the ath5k, rt2500usb =

and

Quoted 1 line
Quoted 1 line

rt61pci drivers exhibit. =A0Is this a mac80211 design flaw, or somet=

hing

Quoted 3 lines
Quoted 1 line

in all of these drivers?

It's mac80211. We never call ieee80211_if_config() when we enable a m=

esh

Quoted 5 lines

interface, and thus conf.enable_beacon with IEEE80211_IFCC_BEACON_ENABLED isn't ever set correctly.

johannes

--=20
Andrey Yurovsky
cozybit Inc.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles= s" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html