linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* ath10k: controlling TX aggregation
@ 2017-05-12 14:43 Krishna Chaitanya
  2017-05-12 14:45 ` Ben Greear
  0 siblings, 1 reply; 6+ messages in thread
From: Krishna Chaitanya @ 2017-05-12 14:43 UTC (permalink / raw)
  To: linux-wireless, ath10k

Hi,

I am doing some comparative tests, wanted to configure common
settings btw all STA's. wanted to limit agg_size to 24 in ath10k with
the below command.

echo "3 24" > /sys/kernel/debug/ieee80211/phy0/ath10k/htt_max_amsdu_ampdu

The ADDBA request still has 64 and when i run iperf it is aggregating 64 frames.

Except for 1 no number takes effect. If i configure 1 there is no ADDBA exchange
initiated by ath10k.

Is this expected? Can we control the aggregation size through driver?

latest wireless-drivers-next and linux-firmware git is used.

-- 
Thanks,
Regards,
Chaitanya T K.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: ath10k: controlling TX aggregation
  2017-05-12 14:43 ath10k: controlling TX aggregation Krishna Chaitanya
@ 2017-05-12 14:45 ` Ben Greear
  2017-05-15  9:32   ` Krishna Chaitanya
  0 siblings, 1 reply; 6+ messages in thread
From: Ben Greear @ 2017-05-12 14:45 UTC (permalink / raw)
  To: Krishna Chaitanya, linux-wireless, ath10k

What chipset, and what firmware version?

Thanks,
Ben

On 05/12/2017 07:43 AM, Krishna Chaitanya wrote:
> Hi,
>
> I am doing some comparative tests, wanted to configure common
> settings btw all STA's. wanted to limit agg_size to 24 in ath10k with
> the below command.
>
> echo "3 24" > /sys/kernel/debug/ieee80211/phy0/ath10k/htt_max_amsdu_ampdu
>
> The ADDBA request still has 64 and when i run iperf it is aggregating 64 frames.
>
> Except for 1 no number takes effect. If i configure 1 there is no ADDBA exchange
> initiated by ath10k.
>
> Is this expected? Can we control the aggregation size through driver?
>
> latest wireless-drivers-next and linux-firmware git is used.
>

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: ath10k: controlling TX aggregation
  2017-05-12 14:45 ` Ben Greear
@ 2017-05-15  9:32   ` Krishna Chaitanya
  2017-05-20 16:21     ` Krishna Chaitanya
  0 siblings, 1 reply; 6+ messages in thread
From: Krishna Chaitanya @ 2017-05-15  9:32 UTC (permalink / raw)
  To: Ben Greear; +Cc: linux-wireless, ath10k

On Fri, May 12, 2017 at 8:15 PM, Ben Greear <greearb@candelatech.com> wrote:
> What chipset, and what firmware version?
>
> Thanks,
> Ben

Hi Ben,

I have missed your response. PFB details.

 chip: QCA986x/988x PCIe
 FW ver: 10.2.4.70.54 api 5
 kernel: 4.11.0-rc+7 (wireless-drivers-next)


> On 05/12/2017 07:43 AM, Krishna Chaitanya wrote:
>>
>> Hi,
>>
>> I am doing some comparative tests, wanted to configure common
>> settings btw all STA's. wanted to limit agg_size to 24 in ath10k with
>> the below command.
>>
>> echo "3 24" > /sys/kernel/debug/ieee80211/phy0/ath10k/htt_max_amsdu_ampdu
>>
>> The ADDBA request still has 64 and when i run iperf it is aggregating 64
>> frames.
>>
>> Except for 1 no number takes effect. If i configure 1 there is no ADDBA
>> exchange
>> initiated by ath10k.
>>
>> Is this expected? Can we control the aggregation size through driver?
>>
>> latest wireless-drivers-next and linux-firmware git is used.
>>
>
> --
> Ben Greear <greearb@candelatech.com>
> Candela Technologies Inc  http://www.candelatech.com



-- 
Thanks,
Regards,
Chaitanya T K.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: ath10k: controlling TX aggregation
  2017-05-15  9:32   ` Krishna Chaitanya
@ 2017-05-20 16:21     ` Krishna Chaitanya
  2017-05-21 15:06       ` Ben Greear
  0 siblings, 1 reply; 6+ messages in thread
From: Krishna Chaitanya @ 2017-05-20 16:21 UTC (permalink / raw)
  To: Ben Greear; +Cc: linux-wireless, ath10k

On Mon, May 15, 2017 at 3:02 PM, Krishna Chaitanya
<chaitanya.mgit@gmail.com> wrote:
> On Fri, May 12, 2017 at 8:15 PM, Ben Greear <greearb@candelatech.com> wrote:
>> What chipset, and what firmware version?
>>
>> Thanks,
>> Ben
>
> Hi Ben,
>
> I have missed your response. PFB details.
>
>  chip: QCA986x/988x PCIe
>  FW ver: 10.2.4.70.54 api 5
>  kernel: 4.11.0-rc+7 (wireless-drivers-next)
>
Ben,

any clues?
>
>> On 05/12/2017 07:43 AM, Krishna Chaitanya wrote:
>>>
>>> Hi,
>>>
>>> I am doing some comparative tests, wanted to configure common
>>> settings btw all STA's. wanted to limit agg_size to 24 in ath10k with
>>> the below command.
>>>
>>> echo "3 24" > /sys/kernel/debug/ieee80211/phy0/ath10k/htt_max_amsdu_ampdu
>>>
>>> The ADDBA request still has 64 and when i run iperf it is aggregating 64
>>> frames.
>>>
>>> Except for 1 no number takes effect. If i configure 1 there is no ADDBA
>>> exchange
>>> initiated by ath10k.
>>>
>>> Is this expected? Can we control the aggregation size through driver?
>>>
>>> latest wireless-drivers-next and linux-firmware git is used.
>>>
>>
>> --
>> Ben Greear <greearb@candelatech.com>
>> Candela Technologies Inc  http://www.candelatech.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: ath10k: controlling TX aggregation
  2017-05-20 16:21     ` Krishna Chaitanya
@ 2017-05-21 15:06       ` Ben Greear
  2017-05-21 15:43         ` Krishna Chaitanya
  0 siblings, 1 reply; 6+ messages in thread
From: Ben Greear @ 2017-05-21 15:06 UTC (permalink / raw)
  To: Krishna Chaitanya; +Cc: linux-wireless, ath10k



On 05/20/2017 09:21 AM, Krishna Chaitanya wrote:
> On Mon, May 15, 2017 at 3:02 PM, Krishna Chaitanya
> <chaitanya.mgit@gmail.com> wrote:
>> On Fri, May 12, 2017 at 8:15 PM, Ben Greear <greearb@candelatech.com> wrote:
>>> What chipset, and what firmware version?
>>>
>>> Thanks,
>>> Ben
>>
>> Hi Ben,
>>
>> I have missed your response. PFB details.
>>
>>  chip: QCA986x/988x PCIe
>>  FW ver: 10.2.4.70.54 api 5
>>  kernel: 4.11.0-rc+7 (wireless-drivers-next)
>>
> Ben,
>
> any clues?

I haven't tried that test yet, and use a different
firmware version.  I did recently work on allowing larger
AMSDU aggregates and have that working in my CT firmware and
driver.

Thanks,
Ben

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: ath10k: controlling TX aggregation
  2017-05-21 15:06       ` Ben Greear
@ 2017-05-21 15:43         ` Krishna Chaitanya
  0 siblings, 0 replies; 6+ messages in thread
From: Krishna Chaitanya @ 2017-05-21 15:43 UTC (permalink / raw)
  To: Ben Greear; +Cc: linux-wireless, ath10k

On Sun, May 21, 2017 at 8:36 PM, Ben Greear <greearb@candelatech.com> wrote:
>
>
> On 05/20/2017 09:21 AM, Krishna Chaitanya wrote:
>>
>> On Mon, May 15, 2017 at 3:02 PM, Krishna Chaitanya
>> <chaitanya.mgit@gmail.com> wrote:
>>>
>>> On Fri, May 12, 2017 at 8:15 PM, Ben Greear <greearb@candelatech.com>
>>> wrote:
>>>>
>>>> What chipset, and what firmware version?
>>>>
>>>> Thanks,
>>>> Ben
>>>
>>>
>>> Hi Ben,
>>>
>>> I have missed your response. PFB details.
>>>
>>>  chip: QCA986x/988x PCIe
>>>  FW ver: 10.2.4.70.54 api 5
>>>  kernel: 4.11.0-rc+7 (wireless-drivers-next)
>>>
>> Ben,
>>
>> any clues?
>
>
> I haven't tried that test yet, and use a different
> firmware version.  I did recently work on allowing larger
> AMSDU aggregates and have that working in my CT firmware and
> driver.
ok, so at least its suppose to work.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-05-21 15:43 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-12 14:43 ath10k: controlling TX aggregation Krishna Chaitanya
2017-05-12 14:45 ` Ben Greear
2017-05-15  9:32   ` Krishna Chaitanya
2017-05-20 16:21     ` Krishna Chaitanya
2017-05-21 15:06       ` Ben Greear
2017-05-21 15:43         ` Krishna Chaitanya

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).