On Mon, Feb 14, 2011 at 3:58 PM, Mohammed Shafi <shafi.wireless@xxxxxxxxx> wrote: > On Mon, Feb 14, 2011 at 3:42 PM, Johannes Berg > <johannes@xxxxxxxxxxxxxxxx> wrote: >> On Tue, 2011-02-08 at 16:41 +0530, Mohammed Shafi wrote: >>> On Tue, Feb 1, 2011 at 6:19 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: >>> > From: Johannes Berg <johannes.berg@xxxxxxxxx> >>> > >>> > At least the WMM spec says we should reply to >>> > unicast probe request frames that go to us. >>> > >>> > Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx> >>> > --- >>> > net/mac80211/ibss.c | 2 +- >>> > 1 file changed, 1 insertion(+), 1 deletion(-) >>> > >>> > --- wireless-testing.orig/net/mac80211/ibss.c 2011-02-01 12:58:16.000000000 +0100 >>> > +++ wireless-testing/net/mac80211/ibss.c 2011-02-01 13:47:44.000000000 +0100 >>> > @@ -689,7 +689,7 @@ static void ieee80211_rx_mgmt_probe_req( >>> > mgmt->bssid, tx_last_beacon); >>> > #endif /* CONFIG_MAC80211_IBSS_DEBUG */ >>> > >>> > - if (!tx_last_beacon) >>> > + if (!tx_last_beacon && !(status->rx_flags & IEEE80211_RX_RA_MATCH)) >>> > return; >>> > >>> > if (memcmp(mgmt->bssid, ifibss->bssid, ETH_ALEN) != 0 && >>> >>> Hi Johannes, >>> tested with the same thing (in addition of passing the >>> rx_status structure) and it is working fine. >> >> Yeah, I clearly missed that, guess I wasn't paying attention. >> >> The real question though is -- do we want/need this? I can only find >> reference to this behaviour in the WMM spec. > > Yes, Test Engineers say this is necessary for IBSS test case-5.3.1 to pass. Sorry for the incomplete information, few details of the test case is 5.3.1 IBSS Active Scanning Test Purpose and Description Active scan tests verify that the STAUT can create an IBSS, respond to active scans, and pass data. References Test case 6.1 in “Wi-Fi 802.11 with WPA2, WPA, and WEP System Interoperability Test Plan for IEEE 802.11a, b & g Devices” For the Active Probe STAUT responds with Basic rate enumerated in beacon message is per default channel is correct record BSSID for alter comparison to ensure that it is a random value > thanks, > shafi > >> >> johannes >> >> > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
