Search Linux Wireless

Re: [RFC 00/14] ath10k: Refactor rx path little bit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11 March 2014 11:04, Kalle Valo <kvalo@xxxxxxxxxxxxxxxx> wrote:
> Janusz Dziedzic <janusz.dziedzic@xxxxxxxxx> writes:
>
>> Refactor rx path, mainly split/reduce calculation we
>> did before. Now we caculate variable/flags only once
>> per-ppdu, while before we did some calculations for
>> each mpdu. Kill some not needed code and fill directly
>> ieee80211_rx_status structure.
>> TP veryfication required.
>>
>> Janusz Dziedzic (14):
>>   ath10k: add ath10k_htt_rx_amsdu_allowed function
>>   ath10k: Fill per-ppdu info in rx_info only once
>>   ath10k: move rx related functions to htt_rx.c
>>   ath10k: rename process_rx_rates to ath10k_htt_rx_h_rates
>>   ath10k: introduce ieee80211_rx_status to htt_rx_info
>>   ath10k: kill signal field in htt_rx_info
>>   ath10k: setup rx channel per ppdu
>>   ath10k: kill rate substruct and tsf from htt_rx_info
>>   ath10k: kill fcs_err from htt_rx_info
>>   ath10k: kill mic_err/amsdu_more from htt_rx_info
>>   ath10k: kill status from htt_rx_info
>>   ath10k: kill encrypt_type from htt_rx_info
>>   ath10k: return error when ath10k_htt_rx_amsdu_pop() fail
>>   ath10k: improve way we play with attention flags
>
> I did now a quick review:
>
> I didn't fully understand your idea with rx status template.
>
Mainly I split calculations we have to do per-ppdu and per-mpdu.
Before, we did calculation for each mpdu even that wasn't required. In
case of heavy
traffic we will have eg. 15 mpdus in one ppdu. In case of attention flags
we did 4 times __cpu_to_le32() for each rx`ed packet - while one calculation
will be enough. For me this seems like resources (CPU) usage improvement.

After some refactoring tries I decided to introduce this
ieee80211_rx_status template. Seems this is easiest option here.
This will be also similar code as we have already for fragmented RX packets.

Some patches are just some code cleanup (moving code, grouping in
function etc...)

> I think the patchset is too long, it will be easier if you split these
> into two. First set with all the easy and obvious stuff and a second one
> for the rx status template.
>
Will send V2

> Some of the patches are missing commit logs. Even if the patch is
> obvious, please write a small commit log. Just oneliner would be enough.
>
> I would prefer the commit logs to answer more why the change is needed.
>
> --
> Kalle Valo
--
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




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux