Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5631becb authored by Arend van Spriel's avatar Arend van Spriel Committed by John W. Linville
Browse files

brcmfmac: add debugfs statistics for firmware-signalling 



Added statistics for flow-control and packets dropped by the
driver.

Reviewed-by: default avatarHante Meuleman <meuleman@broadcom.com>
Reviewed-by: default avatarPieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent fa587d4b

drivers/net/wireless/brcm80211/brcmfmac/dhd_dbg.c

+7 −1
Original line number Diff line number Diff line
@@ -156,8 +156,11 @@ ssize_t brcmf_debugfs_fws_stats_read(struct file *f, char __user *data, 
			"txs_suppr_core:    %u\n"

			"txs_suppr_ps:      %u\n"

			"txs_tossed:        %u\n"

			"txs_host_tossed:   %u\n"

			"bus_flow_block:    %u\n"

			"fws_flow_block:    %u\n"

			"send_pkts:         BK:%u BE:%u VO:%u VI:%u BCMC:%u\n"

			"fifo_credits_sent: BK:%u BE:%u VO:%u VI:%u BCMC:%u\n",

			"requested_sent:    BK:%u BE:%u VO:%u VI:%u BCMC:%u\n",

			fwstats->header_pulls,

			fwstats->header_only_pkt,

			fwstats->tlv_parse_failed,
@@ -176,6 +179,9 @@ ssize_t brcmf_debugfs_fws_stats_read(struct file *f, char __user *data, 
			fwstats->txs_supp_core,

			fwstats->txs_supp_ps,

			fwstats->txs_tossed,

			fwstats->txs_host_tossed,

			fwstats->bus_flow_block,

			fwstats->fws_flow_block,

			fwstats->send_pkts[0], fwstats->send_pkts[1],

			fwstats->send_pkts[2], fwstats->send_pkts[3],

			fwstats->send_pkts[4],

drivers/net/wireless/brcm80211/brcmfmac/dhd_dbg.h

+3 −0
Original line number Diff line number Diff line
@@ -157,6 +157,9 @@ struct brcmf_fws_stats { 
	u32 txs_supp_core;

	u32 txs_supp_ps;

	u32 txs_tossed;

	u32 txs_host_tossed;

	u32 bus_flow_block;

	u32 fws_flow_block;

};



struct brcmf_pub;

drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c

+9 −4
Original line number Diff line number Diff line
@@ -584,7 +584,7 @@ static int brcmf_fws_hanger_mark_suppressed(struct brcmf_fws_hanger *h, 
	if (slot_id >= BRCMF_FWS_HANGER_MAXITEMS)

		return -ENOENT;



	if (h->items[slot_id].state != BRCMF_FWS_HANGER_ITEM_STATE_INUSE) {

	if (h->items[slot_id].state == BRCMF_FWS_HANGER_ITEM_STATE_FREE) {

		brcmf_err("entry not in use\n");

		return -EINVAL;

	}
@@ -894,8 +894,10 @@ brcmf_fws_flow_control_check(struct brcmf_fws_info *fws, struct pktq *pq, 
		brcmf_txflowblock_if(ifp,

				     BRCMF_NETIF_STOP_REASON_FWS_FC, false);

	if (!(ifp->netif_stop & BRCMF_NETIF_STOP_REASON_FWS_FC) &&

	    pq->len >= BRCMF_FWS_FLOWCONTROL_HIWATER)

	    pq->len >= BRCMF_FWS_FLOWCONTROL_HIWATER) {

		fws->stats.fws_flow_block++;

		brcmf_txflowblock_if(ifp, BRCMF_NETIF_STOP_REASON_FWS_FC, true);

	}

	return;

}
@@ -1296,9 +1298,10 @@ brcmf_fws_txs_process(struct brcmf_fws_info *fws, u8 flags, u32 hslot, 
	} else if (flags == BRCMF_FWS_TXSTATUS_FW_PS_SUPPRESS) {

		fws->stats.txs_supp_ps++;

		remove_from_hanger = false;

	} else if ((flags == BRCMF_FWS_TXSTATUS_FW_TOSSED) ||

		   (flags == BRCMF_FWS_TXSTATUS_HOST_TOSSED))

	} else if (flags == BRCMF_FWS_TXSTATUS_FW_TOSSED)

		fws->stats.txs_tossed++;

	else if (flags == BRCMF_FWS_TXSTATUS_HOST_TOSSED)

		fws->stats.txs_host_tossed++;

	else

		brcmf_err("unexpected txstatus\n");
@@ -2030,4 +2033,6 @@ void brcmf_fws_bus_blocked(struct brcmf_pub *drvr, bool flow_blocked) 
	fws->bus_flow_blocked = flow_blocked;

	if (!flow_blocked)

		brcmf_fws_schedule_deq(fws);

	else

		fws->stats.bus_flow_block++;

}