On Fri, 2016-10-07 at 22:29 +0200, Michael Braun wrote:
> 
> I guess the same is needed for packets with IEEE80211_TX_CTRL_AMSDU
> set.

I don't think that's true, since we don't get here after A-MSDU
building in mac80211.

>       /* preserve EOSP bit */
>       ack_policy = *p & IEEE80211_QOS_CTL_EOSP;
>  
> +     /* preserve A-MSDU bit for MONITOR interfaces to allow
> injecting
> +      * A-MSDU frames
> +      */
> +     if (info->flags & IEEE80211_TX_CTL_INJECTED ||
> +         info->control.flags & IEEE80211_TX_CTRL_AMSDU)
> +             ack_policy |= *p & IEEE80211_QOS_CTL_A_MSDU_PRESENT;

However, why not just unconditionally preserve it?

johannes

Reply via email to