On Thu, 2014-01-23 at 14:42 +0100, Helmut Schaa wrote: > On Thu, Jan 23, 2014 at 7:08 AM, Grumbach, Emmanuel > <emmanuel.grumbach@xxxxxxxxx> wrote: > >> >> So, I guess you are taking what I sent ? > >> > > >> >Haven't really made up my mind yet ... I think it's more correct, so I > >> >should, but I also don't really want to break the ralink drivers over > >> >what seems to me to be a fairly small issue. > >> > >> I think I'm fine with this now. Let's just see if someone experiences any > >> issues ... > >> > >> Furthermore I think i even remember that you can force ralink HW to stop a > >> BA session. But all in all lets better comply with the spec ... > > > > Well - you need to see what "stop a BA session" mean. If you are able to tell the HW no AMPDUs anymore *now*, then ok - but you still have the out-of-order TX problem. Unless you use the same TX queue for AMPUs and non-AMPDUs and don't fear reordering issues > > We shouldn't have reordering issues in rt2x00 since AMPDUs and MPDUs > go through the same hw queue. > However, at the moment the hw will keep on sending AMPDUs that are in > the TX queue even after we received a DELBA. > But this is an issue with the ralink drivers ... Ok. Karl, can you rewrite the commit log, maybe referencing the original commit and this discussion (I'd suggest using http://mid.gmane.org/1390391564-18481-1-git-send-email-karl.beldan@xxxxxxxxx) please? Then I'll apply that. Thanks, 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