Return-path: Received: from wolverine01.qualcomm.com ([199.106.114.254]:32749 "EHLO wolverine01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759693AbdAKKt1 (ORCPT ); Wed, 11 Jan 2017 05:49:27 -0500 From: "Valo, Kalle" To: Christian Lamparter CC: Mohammed Shafi Shajakhan , linux-wireless , "ath10k@lists.infradead.org" Subject: Re: [1/2] ath10k: add accounting for the extended peer statistics Date: Wed, 11 Jan 2017 10:49:20 +0000 Message-ID: <87y3yhoorc.fsf@kamboji.qca.qualcomm.com> (sfid-20170111_114931_627662_07830210) References: <992a4e2676037a06f482cdbe2d3d39e287530be5.1480974623.git.chunkeey@googlemail.com> <20170103052827.GA8204@atheros-ThinkPad-T61> <2769662.Qa6eUCGVkY@debian64> In-Reply-To: <2769662.Qa6eUCGVkY@debian64> (Christian Lamparter's message of "Wed, 04 Jan 2017 21:06:20 +0100") Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: Christian Lamparter writes: > Hello Shafi and Kalle, > > On Tuesday, January 3, 2017 10:58:27 AM CET Mohammed Shafi Shajakhan wrot= e: >> On Fri, Dec 30, 2016 at 03:35:10PM +0100, Christian Lamparter wrote: >> > On Thu, Dec 29, 2016 at 3:11 PM, Kalle Valo w= rote: >> > > Christian Lamparter wrote: >> > >> 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 updat= e") >> > >> Signed-off-by: Christian Lamparter >> > > >> > > My understanding is that I should skip this patch 1. Please let me k= now if >> > > I misunderstood. But I'm still plannning to apply patch 2. >> >=20 >> > 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. >> >=20 >> > 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). >>=20 >> [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=20 > "Re: [PATCH] ath10k: merge extended peer info data with existing peers in= fo" [0]. > out of my spam-bucket. Kalle, please look if your copy of the mail got=20 > flagged/deleted as well. Judging from the reply in this thread, I think y= ou > overlooked it as well?=20 I think I just read the discussion to hastily as it was rather long, sorry about that. After really long or confusin discussions, just to help the maintainers and also avoid miscommunication between participants, it's usually a good idea to summarise the conclusion. If us maintainers need to figure out the conclusion ourselves from a long discussion we are bound to make mistakes, just like I did here. So something like this would help me a lot: "Kalle, please drop these patches. I need to work on these a bit more." Or:=20 "Kalle, me and John came to agreement about foo. So these should be good to apply." > After reading it, I think the previous post and the request to put the pa= tch > on wait was unnecessary. As of now, it seems to me that the open question= s > between us have been settled amically (so to speak). Kalle, do you have a= ny > concerns or can you put this in the next round then? If you both are happy with the patch, I'm happy to take it :) I actived the patch again in my queue by moving the state from Deferred to New: https://patchwork.kernel.org/patch/9461631/ If all goes well I'm expecting to apply it later this week. --=20 Kalle Valo=