Return-path: Received: from mail-qk0-f181.google.com ([209.85.220.181]:33162 "EHLO mail-qk0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752010AbdF0WHO (ORCPT ); Tue, 27 Jun 2017 18:07:14 -0400 Received: by mail-qk0-f181.google.com with SMTP id r62so37254303qkf.0 for ; Tue, 27 Jun 2017 15:07:14 -0700 (PDT) Subject: Re: [PATCH 3/4] brcmfmac: rework headroom check in .start_xmit() To: Kalle Valo Cc: linux-wireless@vger.kernel.org, James Hughes 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> From: Arend van Spriel Message-ID: <0d8f0be4-4487-5f0d-c903-c87656c96bac@broadcom.com> (sfid-20170628_000718_742408_DE28A8F3) Date: Wed, 28 Jun 2017 00:07:10 +0200 MIME-Version: 1.0 In-Reply-To: <874lv15xp9.fsf@purkki.adurom.net> Content-Type: text/plain; charset=utf-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: 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. Regards, Arend