On Tue, May 20, 2008 at 9:41 AM, Tomas Winkler <tomasw@xxxxxxxxx> wrote: > On Tue, May 20, 2008 at 4:14 PM, Stefanik Gábor <netrolller.3d@xxxxxxxxx> wrote: >> (Resend of previous mail, his time also sending to the list.) >> On Tue, May 20, 2008 at 8:38 AM, Johannes Berg >> <johannes@xxxxxxxxxxxxxxxx> wrote: >>> >>>> <snip> >>> >>> That code is completely bogus anyway IMO, it should just be removed. >>> >>>> <snip> >>> >>> Same here, why drop a frame if there's no station for it, that just >>> means you can't do hw encryption which, well, cannot happen anyway since >>> mac80211 would tell you about the station for hw crypto. >>> >>>> <snip> >>> >>> That's the code I was complaining about. Just kill it if it works then. >>> >>>> <snip> >>> >>> Same code, other driver. What are you doing poking with the 802.11 >>> frames you're supposed to transmit anyway? >>> >>> I wish you guys at Intel would trust mac80211 and not try to work around >>> it all the time. >>> >>> johannes >>> >> >> Patch with nits addressed, a syslog issue fixed and also with a >> bracketing fix in 3945: >> >> <snip> > This seams Okay. >> <snip> > > Not sure about this > >> <snip> > The thing is that mac80211 and uCode are not exactly in sync on > association time. > I'm not sure but this would be also spec violation. > >> <snip> > send packet with invalid station will certainly crash uCode. > >> <snip> > Thanks for the feedback, here is a fixed patch: diff -rp -U 8 compat-wireless-2008-05-20.orig/drivers/net/wireless/iwlwifi/iwl3945-base.c compat-wireless-2008-05-20/drivers/net/wireless/iwlwifi/iwl3945-base.c --- compat-wireless-2008-05-20.orig/drivers/net/wireless/iwlwifi/iwl3945-base.c 2008-05-20 05:05:29.000000000 -0400 +++ compat-wireless-2008-05-20/drivers/net/wireless/iwlwifi/iwl3945-base.c 2008-05-20 10:11:58.809424284 -0400 @@ -2542,16 +2542,19 @@ static int iwl3945_get_sta_id(struct iwl return sta_id; IWL_DEBUG_DROP("Station %s not in station map. " "Defaulting to broadcast...\n", print_mac(mac, hdr->addr1)); iwl3945_print_hex_dump(IWL_DL_DROP, (u8 *) hdr, sizeof(*hdr)); return priv->hw_setting.bcast_sta_id; } + /* If we are in monitor mode, use BCAST */ + case IEEE80211_IF_TYPE_MNTR: + return priv->hw_setting.bcast_sta_id; default: IWL_WARNING("Unknown mode of operation: %d", priv->iw_mode); return priv->hw_setting.bcast_sta_id; } } /* * start REPLY_TX command process @@ -2579,18 +2582,18 @@ static int iwl3945_tx_skb(struct iwl3945 int rc; spin_lock_irqsave(&priv->lock, flags); if (iwl3945_is_rfkill(priv)) { IWL_DEBUG_DROP("Dropping - RF KILL\n"); goto drop_unlock; } - if (!priv->vif) { - IWL_DEBUG_DROP("Dropping - !priv->vif\n"); + if (!priv->vif && priv->iw_mode != IEEE80211_IF_TYPE_MNTR) { + IWL_DEBUG_DROP("Dropping - !priv->vif outside monitor mode\n"); goto drop_unlock; } if ((ctl->tx_rate->hw_value & 0xFF) == IWL_INVALID_RATE) { IWL_ERROR("ERROR: No TX rate available.\n"); goto drop_unlock; } @@ -2603,36 +2606,35 @@ static int iwl3945_tx_skb(struct iwl3945 if (ieee80211_is_auth(fc)) IWL_DEBUG_TX("Sending AUTH frame\n"); else if (ieee80211_is_assoc_request(fc)) IWL_DEBUG_TX("Sending ASSOC frame\n"); else if (ieee80211_is_reassoc_request(fc)) IWL_DEBUG_TX("Sending REASSOC frame\n"); #endif - /* drop all data frame if we are not associated */ - if ((!iwl3945_is_associated(priv) || + /* drop all data frame if we are not associated and not monitoring */ + if (((!iwl3945_is_associated(priv) && priv->iw_mode != IEEE80211_IF_TYPE_MNTR) || ((priv->iw_mode == IEEE80211_IF_TYPE_STA) && !priv->assoc_id)) && ((fc & IEEE80211_FCTL_FTYPE) == IEEE80211_FTYPE_DATA)) { - IWL_DEBUG_DROP("Dropping - !iwl3945_is_associated\n"); + IWL_DEBUG_DROP("Dropping - !iwl3945_is_associated outside monitor mode\n"); goto drop_unlock; } spin_unlock_irqrestore(&priv->lock, flags); hdr_len = ieee80211_get_hdrlen(fc); /* Find (or create) index into station table for destination station */ sta_id = iwl3945_get_sta_id(priv, hdr); + + /* If the station is invalid, assume broadcast */ if (sta_id == IWL_INVALID_STATION) { DECLARE_MAC_BUF(mac); - - IWL_DEBUG_DROP("Dropping - INVALID STATION: %s\n", - print_mac(mac, hdr->addr1)); - goto drop; + sta_id = priv->hw_setting.bcast_sta_id; } IWL_DEBUG_RATE("station Id %d\n", sta_id); qc = ieee80211_get_qos_ctrl(hdr); if (qc) { u8 tid = (u8)(le16_to_cpu(*qc) & 0xf); seq_number = priv->stations[sta_id].tid[tid].seq_number & @@ -6692,21 +6694,16 @@ static void iwl3945_mac_stop(struct ieee static int iwl3945_mac_tx(struct ieee80211_hw *hw, struct sk_buff *skb, struct ieee80211_tx_control *ctl) { struct iwl3945_priv *priv = hw->priv; IWL_DEBUG_MAC80211("enter\n"); - if (priv->iw_mode == IEEE80211_IF_TYPE_MNTR) { - IWL_DEBUG_MAC80211("leave - monitor\n"); - return -1; - } - IWL_DEBUG_TX("dev->xmit(%d bytes) at rate 0x%02x\n", skb->len, ctl->tx_rate->bitrate); if (iwl3945_tx_skb(priv, skb, ctl)) dev_kfree_skb_any(skb); IWL_DEBUG_MAC80211("leave\n"); return 0; diff -rp -U 8 compat-wireless-2008-05-20.orig/drivers/net/wireless/iwlwifi/iwl4965-base.c compat-wireless-2008-05-20/drivers/net/wireless/iwlwifi/iwl4965-base.c --- compat-wireless-2008-05-20.orig/drivers/net/wireless/iwlwifi/iwl4965-base.c 2008-05-20 05:05:29.000000000 -0400 +++ compat-wireless-2008-05-20/drivers/net/wireless/iwlwifi/iwl4965-base.c 2008-05-20 10:07:02.644103448 -0400 @@ -1746,16 +1746,19 @@ static int iwl4965_get_sta_id(struct iwl return sta_id; IWL_DEBUG_DROP("Station %s not in station map. " "Defaulting to broadcast...\n", print_mac(mac, hdr->addr1)); iwl_print_hex_dump(priv, IWL_DL_DROP, (u8 *) hdr, sizeof(*hdr)); return priv->hw_params.bcast_sta_id; + /* If we are in monitor mode, use BCAST */ + case IEEE80211_IF_TYPE_MNTR: + return priv->hw_params.bcast_sta_id; default: IWL_WARNING("Unknown mode of operation: %d", priv->iw_mode); return priv->hw_params.bcast_sta_id; } } /* * start REPLY_TX command process @@ -1784,18 +1787,18 @@ static int iwl4965_tx_skb(struct iwl_pri int rc; spin_lock_irqsave(&priv->lock, flags); if (iwl_is_rfkill(priv)) { IWL_DEBUG_DROP("Dropping - RF KILL\n"); goto drop_unlock; } - if (!priv->vif) { - IWL_DEBUG_DROP("Dropping - !priv->vif\n"); + if (!priv->vif && priv->iw_mode != IEEE80211_IF_TYPE_MNTR) { + IWL_DEBUG_DROP("Dropping - !priv->vif outside monitor mode\n"); goto drop_unlock; } if ((ctl->tx_rate->hw_value & 0xFF) == IWL_INVALID_RATE) { IWL_ERROR("ERROR: No TX rate available.\n"); goto drop_unlock; } @@ -1808,36 +1811,36 @@ static int iwl4965_tx_skb(struct iwl_pri if (ieee80211_is_auth(fc)) IWL_DEBUG_TX("Sending AUTH frame\n"); else if (ieee80211_is_assoc_request(fc)) IWL_DEBUG_TX("Sending ASSOC frame\n"); else if (ieee80211_is_reassoc_request(fc)) IWL_DEBUG_TX("Sending REASSOC frame\n"); #endif - /* drop all data frame if we are not associated */ + /* drop all data frame if we are not associated and not monitoring */ if (((fc & IEEE80211_FCTL_FTYPE) == IEEE80211_FTYPE_DATA) && - (!iwl_is_associated(priv) || + ((!iwl_is_associated(priv) && priv->iw_mode != IEEE80211_IF_TYPE_MNTR) || ((priv->iw_mode == IEEE80211_IF_TYPE_STA) && !priv->assoc_id) || !priv->assoc_station_added)) { - IWL_DEBUG_DROP("Dropping - !iwl_is_associated\n"); + IWL_DEBUG_DROP("Dropping - !iwl_is_associated outside monitor mode\n"); goto drop_unlock; } spin_unlock_irqrestore(&priv->lock, flags); hdr_len = ieee80211_get_hdrlen(fc); /* Find (or create) index into station table for destination station */ sta_id = iwl4965_get_sta_id(priv, hdr); + + /* If the station is invalid, assume broadcast */ if (sta_id == IWL_INVALID_STATION) { DECLARE_MAC_BUF(mac); - - IWL_DEBUG_DROP("Dropping - INVALID STATION: %s\n", - print_mac(mac, hdr->addr1)); + sta_id = priv->hw_params.bcast_sta_id; goto drop; } IWL_DEBUG_TX("station Id %d\n", sta_id); qc = ieee80211_get_qos_ctrl(hdr); if (qc) { u8 tid = (u8)(le16_to_cpu(*qc) & 0xf); @@ -5110,21 +5113,16 @@ static void iwl4965_mac_stop(struct ieee static int iwl4965_mac_tx(struct ieee80211_hw *hw, struct sk_buff *skb, struct ieee80211_tx_control *ctl) { struct iwl_priv *priv = hw->priv; IWL_DEBUG_MAC80211("enter\n"); - if (priv->iw_mode == IEEE80211_IF_TYPE_MNTR) { - IWL_DEBUG_MAC80211("leave - monitor\n"); - return -1; - } - IWL_DEBUG_TX("dev->xmit(%d bytes) at rate 0x%02x\n", skb->len, ctl->tx_rate->bitrate); if (iwl4965_tx_skb(priv, skb, ctl)) dev_kfree_skb_any(skb); IWL_DEBUG_MAC80211("leave\n"); return 0; -- 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