On Tue, Sep 16, 2008 at 9:37 AM, Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
> On Mon, 2008-09-15 at 20:17 -0700, Luis R. Rodriguez wrote:
>
>> +/**
>> + * is_part_ampdu - tells us whether this buffer is part of an AMPDU
>> + *
>> + * @skb: the buffer we want to check
>> + * @hw: the &struct ieee80211_hw to check the queue mapping on
>> + */
>> +static inline bool is_part_ampdu(struct sk_buff *skb, struct ieee80211_hw *hw)
>> +{
>> + return (skb_get_queue_mapping(skb) >=
>> + ieee80211_num_regular_queues(hw));
>> +}
>> +
>
> This is making the patch needlessly large and makes it change drivers
> etc, please just keep the flag and set it based on this in master xmit
> after we've cleared the info.
>
Yes, please keep the flag. Otherwise our rate scale algorithm won't work
Thanks
Tomas
--
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
