Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:33092 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751596AbdF1QEJ (ORCPT ); Wed, 28 Jun 2017 12:04:09 -0400 From: Kalle Valo To: Arend van Spriel Cc: linux-wireless@vger.kernel.org, James Hughes Subject: Re: [PATCH 3/4] brcmfmac: rework headroom check in .start_xmit() References: <1498125664-25980-1-git-send-email-arend.vanspriel@broadcom.com> <1498125664-25980-4-git-send-email-arend.vanspriel@broadcom.com> <874lv15xp9.fsf@purkki.adurom.net> <0d8f0be4-4487-5f0d-c903-c87656c96bac@broadcom.com> Date: Wed, 28 Jun 2017 19:04:04 +0300 In-Reply-To: <0d8f0be4-4487-5f0d-c903-c87656c96bac@broadcom.com> (Arend van Spriel's message of "Wed, 28 Jun 2017 00:07:10 +0200") Message-ID: <87zicsqetn.fsf@kamboji.qca.qualcomm.com> (sfid-20170628_180419_070986_642ED54E) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-wireless-owner@vger.kernel.org List-ID: Arend van Spriel writes: > On 27-06-17 16:09, Kalle Valo wrote: >> Arend van Spriel writes: >> >>> Since commit 9cc4b7cb86cb ("brcmfmac: Make skb header writable >>> before use") the headroom usage has been fixed. However, the >>> driver was keeping statistics that got lost. So reworking the >>> code so we get those driver statistics back for debugging. >>> >>> Cc: James Hughes >>> Reviewed-by: Hante Meuleman >>> Reviewed-by: Pieter-Paul Giesberts >>> Reviewed-by: Franky Lin >>> Signed-off-by: Arend van Spriel >>> --- >>> .../net/wireless/broadcom/brcm80211/brcmfmac/bus.h | 15 ++++++++++++-- >>> .../wireless/broadcom/brcm80211/brcmfmac/core.c | 23 +++++++++++++++------- >>> .../wireless/broadcom/brcm80211/brcmfmac/sdio.c | 13 +++++++----- >>> 3 files changed, 37 insertions(+), 14 deletions(-) >>> >>> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bus.h >>> b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bus.h >>> index e1a4d9e..163ddc4 100644 >>> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bus.h >>> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bus.h >>> @@ -113,6 +113,17 @@ struct brcmf_bus_msgbuf { >>> >>> >>> /** >>> + * struct brcmf_bus_stats - bus statistic counters. >>> + * >>> + * @pktcowed: packets cowed for extra headroom/unorphan. >>> + * @pktcow_failed: packets dropped due to failed cow-ing. >>> + */ >>> +struct brcmf_bus_stats { >>> + atomic_t pktcowed; >>> + atomic_t pktcow_failed; >>> +}; >> >> Same question as in the previous patch. I only see updates for these >> variables, but nobody reading them? > > Hi Kalle, > > You are right. My intention was to expose these to debugfs, but clearly > did not include it in this patch series. So how do you want to handle > this. Feel free to drop patch 2 and 3. I can resubmit them with > subsequent patches dealing with exposing it to debugfs if that makes > more sense. No need to resend if you are going to add the debugfs interface soon. But the general rule is that we don't add any dead code to kernel and there was no explanation in the commit log, that's why I asked. -- Kalle Valo