On Mon, 2013-01-21 at 12:18 +0100, Marco Porsch wrote: [long changelog] I think the changelog is pretty much unreadable. Can you boil it down, or maybe start with a "tl;dr" version? All the little technical details are also less important (like debugfs files, who cares). > @@ -656,6 +684,13 @@ void ieee80211_stop_mesh(struct ieee80211_sub_if_data *sdata) > sta_info_flush(sdata); > mesh_path_flush_by_iface(sdata); > > + atomic_set(&ifmsh->ps.num_sta_ps, 0); /* not reset by sta_info_flush */ > + atomic_set(&ifmsh->num_mpsp, 0); This comment tells me that you have bugs elsewhere, in particular in mesh housekeeping. In fact, looking there, I see nothing that causes a sleeping station to be accounted for correctly when it expires. > +static void mps_set_sta_peer_pm(struct sta_info *sta, > + struct ieee80211_hdr *hdr) > +void ieee80211_mpsp_trigger_process(struct ieee80211_hdr *hdr, > + struct sta_info *sta, bool tx, bool acked) I wonder if these functions should get the QoS CTL field instead of the header, as otherwise nothing really verifies that they're passed QoS frames to start with. Yes, that's the case today, but it's not obvious that this is really a requirement. 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
