[PATCH] ath10k: improve rx path when play with attention flags
Kalle Valo
kvalo at qca.qualcomm.com
Wed Feb 26 12:29:57 EST 2014
Michal Kazior <michal.kazior at tieto.com> writes:
> On 25 February 2014 08:13, Janusz Dziedzic <janusz.dziedzic at tieto.com> wrote:
>> Currently when we check attention flags we do __le32_to_cpu()
>> four times for each packet. This could have performance
>> impact for BIG endian platforms. This patch improve this
>> little bit.
>>
>> Signed-off-by: Janusz Dziedzic <janusz.dziedzic at tieto.com>
[...]
>> @@ -929,6 +873,9 @@ static void ath10k_htt_rx_handler(struct ath10k_htt *htt,
>> struct sk_buff *msdu_head, *msdu_tail;
>> enum htt_rx_mpdu_status status;
>> int msdu_chaining;
>> + struct htt_rx_desc *rxd;
>> + u32 att_flags;
>> +
>
> No need for an empty line I suppose? I would also prefer `attention`
> instead of `att_flags`, but no big deal.
Yeah, attention is better.
And also the variable declarations should be in the beginning of
function. Apparently I have been sloppy missed that here.
--
Kalle Valo
More information about the ath10k
mailing list