Am Donnerstag, 24. März 2011 schrieb Johannes Berg: > On Thu, 2011-03-24 at 14:09 +0100, Helmut Schaa wrote: > > > Ok, found out some more. The problem now seems to trigger only if the Intel STA > > leaves powersave and mac80211 dropped a frame while the STA was sleeping (either > > due to the size of the STA PS buffer or due to a race between rx/tx processing). > > Why do we even have aggregation sessions active while the station goes > to sleep? Is it scanning? I guess so, yes (as I wrote before this is an Intel 5100 client running Windows Vista and the latest Intel driver). It sends a nullfunc going to sleep and returns a few hundret ms later. And in the meantime I can see my rt2800 AP sending an AMPDU to the sleeping STA which of course times out and therefore ends up as filtered frame. > Maybe the PS buffer should be larger then? 128 frames per STA is already large, no? And also it's nothing unusual that some frames get dropped if the STA stays in powersave for a long time. > I don't see how we can lose a > frame due to rx/tx processing races either, how does that happen? I > thought we had the ability to avoid all those races now. Good point. Maybe this only happens with rt2x00. The frame exchange looks basically like (if you want to see the pcap just ask ;) ): STA -> AP nullfunc PM=1 AP -> STA AMPDU (seqnr 3106 - 3112) AP -> STA AMPDU (seqnr 3106 - 3112), retry AP -> STA AMPDU (seqnr 3106 - 3112), retry AP -> STA AMPDU (seqnr 3106 - 3112), retry AP -> STA AMPDU (seqnr 3106 - 3112), retry AP -> STA AMPDU (seqnr 3106 - 3112), retry ... STA -> AP nullfunc PM=0 ... AP -> STA AMPDU (seqnr 3108) STA -> AP BlockAck AP -> STA AMPDU (seqnr 3109 - 3114) STA -> AP BlockAck ... As you can see 3106 and 3107 somehow got lost and thus leave a hole in the STAs reorder buffer leading to the strange behavior I described before. > > This can also cause a sequence number hole in the STAs rx reordering buffer but > > we won't send a BAR in that case ending in the same situation that the Intel STA > > BlockAcks AMPDUs to it but doesn't pass them to the user space. > > > > Johannes, does it make sense to always send a BAR when a STA wakes up from > > sleep and we've got an active aggregation session? > > I don't know, you tell me, does it? :) Ha Ha ;) Sending a BAR at this point would at least ensure that the Intel STAs RX reorder buffer gets flushed (in case we dropped any frames while the STA was sleeping, which happened in this case). Helmut -- 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
