On Thu, 2013-02-07 at 14:49 +0530, Krishna Chaitanya wrote: > On Thu, Feb 7, 2013 at 1:50 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > On Thu, 2013-02-07 at 00:01 -0800, chaitanyatk@xxxxxxxxxxx wrote: > > > > > Hmm. I'm not exactly happy with using (one of) the last control flag for > > a pure debug facility. > > > Hmm...initially i tried to add qos header in the debugfs itself but to avoid > code redundancy went with the TX_CTL. But in either case we need some kind > of flag to skip adding the qos header by mac80211 (or) to set the AMSDU > bit in the qos control header. I guess the question is whether we really need this code in the tree at all? It seems like a (relatively) obscure debug feature for which I'm not sure we should touch the TX fastpath? 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