From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.taht.net (mail.taht.net [176.58.107.8]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.bufferbloat.net (Postfix) with ESMTPS id D63DA3B29E for ; Fri, 1 Dec 2017 13:11:51 -0500 (EST) Received: from nemesis.taht.net (unknown [IPv6:2603:3024:1536:86f0:2e0:4cff:fec1:1206]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.taht.net (Postfix) with ESMTPSA id 99EEB21367; Fri, 1 Dec 2017 18:11:40 +0000 (UTC) From: Dave Taht To: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= Cc: make-wifi-fast@lists.bufferbloat.net References: <87o9ni36p4.fsf@toke.dk> Date: Fri, 01 Dec 2017 10:11:39 -0800 In-Reply-To: <87o9ni36p4.fsf@toke.dk> ("Toke \=\?utf-8\?Q\?H\=C3\=B8iland-J\?\= \=\?utf-8\?Q\?\=C3\=B8rgensen\=22's\?\= message of "Fri, 01 Dec 2017 14:42:47 +0100") Message-ID: <87efoegvxg.fsf@nemesis.taht.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Make-wifi-fast] Fwd: Re: [PATCH 2/2] ath10k: add per peer tx stats support for 10.2.4 X-BeenThere: make-wifi-fast@lists.bufferbloat.net X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 01 Dec 2017 18:11:57 -0000 Toke H=C3=B8iland-J=C3=B8rgensen writes: > Seems that ath10k has gained the ability to report airtime used by > clients, which hopefully will enable airtime fairness for those devices > as well (once we finish moving the implementation up to mac80211). Oh, cool. Maybe I won't regret buying these as much as I have so far. https://www.amazon.com/Ubiquiti-Networks-UAP-AC-M-US-Wide-Area-Dual-Band/dp= /B076B4ZVF2/ref=3Dsr_1_2?ie=3DUTF8&qid=3D1511125487&sr=3D8-2&keywords=3Dubn= t+uap&dpID=3D51-Rhp-ZYuL&preST=3D_SY300_QL70_&dpSrc=3Dsrch > > -Toke > > From: akolli@codeaurora.org > Subject: Re: [PATCH 2/2] ath10k: add per peer tx stats support for 10.2.4 > To: Kalle Valo > Cc: Toke H=C3=B8iland-J=C3=B8rgensen , > ath10k@lists.infradead.org, linux-wireless@vger.kernel.org, Anilkumar Ko= lli > , linux-wireless-owner@vger.kernel.org > Date: Fri, 01 Dec 2017 09:53:30 +0530 (13 hours, 46 minutes, 49 seconds a= go) > > On 2017-11-30 22:08, Kalle Valo wrote: >> Toke H=C3=B8iland-J=C3=B8rgensen writes: >>=20 >>>>> +struct ath10k_10_2_peer_tx_stats { >>>>> + u8 ratecode[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + u8 success_pkts[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + __le16 success_bytes[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + u8 retry_pkts[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + __le16 retry_bytes[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + u8 failed_pkts[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + __le16 failed_bytes[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + u8 flags[PEER_STATS_FOR_NO_OF_PPDUS]; >>>>> + __le32 tx_duration; >>>>> + u8 tx_ppdu_cnt; >>>>> + u8 peer_id; >>>>> +} __packed; >>>>=20 >>>> Toke, hopefully the tx_duration value here helps with ATF >>>> implementation >>>> using QCA988X. >>>=20 >>> Awesome! What's the semantics of this field? Just total >>> duration spent serving that station in the reporting interval? >>> Does it include retry attempts? >>=20 >> I have no clue :) I just noticed this while I was reviewing the patch >> internally and immediately recalled our discussions at Seoul. I can try >> to find out, but that will take a long time as I have way too much=20 >> stuff >> pending at the moment. Hopefully someone more knowledgeable=20 >> (Anilkumar?) >> can chime in and help. > > tx_duration is aggregate time duration of 4 PPDU sent to STA. > FW sends these values for retry packets also. > > Thanks, > Anil. > ---------- > > _______________________________________________ > Make-wifi-fast mailing list > Make-wifi-fast@lists.bufferbloat.net > https://lists.bufferbloat.net/listinfo/make-wifi-fast