Can you capitalise BAR in the patch subjects please? I think of an
actual bar when reading that :)
> --- a/include/net/mac80211.h
> +++ b/include/net/mac80211.h
> @@ -235,6 +235,8 @@ struct ieee80211_bss_conf {
> * @IEEE80211_TX_STAT_ACK: Frame was acknowledged
> * @IEEE80211_TX_STAT_AMPDU: The frame was aggregated, so status
> * is for the whole aggregation.
> + * @IEEE80211_TX_STAT_AMPDU_NO_BACK: no block ack was returned,
> + * so consider using block ack request (BAR).
This flag I don't understand. When does the driver set this? And what
does mac80211 do to "consider", it seems to always just try to set up a
block-ack session.
I'd still loke to see a "HT operation" document that describes how all
this works together so other people can write HT capable drivers :)
> + bar_control |= (u16)IEEE80211_BAR_CTRL_ACK_POLICY_NORMAL;
??
> + bar_control |= (u16)IEEE80211_BAR_CTRL_CBMTID_COMPRESSED_BA;
> + bar_control |= (u16)(tid << 12);
> + bar->control = cpu_to_le16(bar_control);
> + bar->start_seq_num = cpu_to_le16(ssn);
> +
> + /* must be sent through voice AC */
> + skb->priority = 3;
Does that actually work? The current wme.c code will still call
classify_1d() and that will use the IP TOS unless you set the priority
to 256..263.
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
