Re: [1/2] ath10k: add accounting for the extended peer statistics
From: Christian Lamparter <chunkeey@googlemail.com>
Date: 2017-01-04 20:06:24
Hello Shafi and Kalle, On Tuesday, January 3, 2017 10:58:27 AM CET Mohammed Shafi Shajakhan wrote:
On Fri, Dec 30, 2016 at 03:35:10PM +0100, Christian Lamparter wrote:quoted
On Thu, Dec 29, 2016 at 3:11 PM, Kalle Valo [off-list ref] wrote:quoted
Christian Lamparter [off-list ref] wrote:quoted
The 10.4 firmware adds extended peer information to the firmware's statistics payload. This additional info is stored as a separate data field and the elements are stored in their own "peers_extd" list. These elements can pile up in the same way as the peer information elements. This is because the ath10k_wmi_10_4_op_pull_fw_stats() function tries to pull the same amount (num_peer_stats) for every statistic data unit. Fixes: 4a49ae94a448faa ("ath10k: fix 10.4 extended peer stats update") Signed-off-by: Christian Lamparter <chunkeey@googlemail.com>My understanding is that I should skip this patch 1. Please let me know if I misunderstood. But I'm still plannning to apply patch 2.I added Mohammed (I hope, he can reply to the open question when he returns), Since I'm not sure what he wants or not. As far as I'm aware, the "extended" boolean serves no purpose because it was only used in once place in debugfs_sta which was removed in the patch. ( "ath10k_sta_update_stats_rx_duration" and "ath10k_sta_update_extd_stats_rx_duration" have been unified).[shafi] We will wait for Kalle to review from the de-ferred stage and get his opinion as well(regarding the design change). I have no concerns as long this does not changes the existing behaviour. thank you !
Thank you Shafi for sticking around. I just fished your response to "Re: [PATCH] ath10k: merge extended peer info data with existing peers info" [0]. out of my spam-bucket. Kalle, please look if your copy of the mail got flagged/deleted as well. Judging from the reply in this thread, I think you overlooked it as well? After reading it, I think the previous post and the request to put the patch on wait was unnecessary. As of now, it seems to me that the open questions between us have been settled amically (so to speak). Kalle, do you have any concerns or can you put this in the next round then? Regards, Christian [0] <https://www.mail-archive.com/ath10k@lists.infradead.org/msg06066.html>