mwifiex: maintain outstanding packet count for RA list instead of packet size
Maintain total outstanding packet count for RA list instead of total outstanding size as packet count metric seems more reasonable for checking threshold etc. Signed-off-by: Avinash Patil <patila@marvell.com> Signed-off-by: Bing Zhao <bzhao@marvell.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
f0cb84f8a1
commit
c7d9ed9ec9
|
@ -189,7 +189,7 @@ mwifiex_11n_aggregate_pkt(struct mwifiex_private *priv,
|
|||
|
||||
skb_src = skb_dequeue(&pra_list->skb_head);
|
||||
|
||||
pra_list->total_pkts_size -= skb_src->len;
|
||||
pra_list->total_pkt_count--;
|
||||
|
||||
atomic_dec(&priv->wmm.tx_pkts_queued);
|
||||
|
||||
|
@ -268,7 +268,7 @@ mwifiex_11n_aggregate_pkt(struct mwifiex_private *priv,
|
|||
|
||||
skb_queue_tail(&pra_list->skb_head, skb_aggr);
|
||||
|
||||
pra_list->total_pkts_size += skb_aggr->len;
|
||||
pra_list->total_pkt_count++;
|
||||
|
||||
atomic_inc(&priv->wmm.tx_pkts_queued);
|
||||
|
||||
|
|
|
@ -204,11 +204,11 @@ struct mwifiex_ra_list_tbl {
|
|||
struct list_head list;
|
||||
struct sk_buff_head skb_head;
|
||||
u8 ra[ETH_ALEN];
|
||||
u32 total_pkts_size;
|
||||
u32 is_11n_enabled;
|
||||
u16 max_amsdu;
|
||||
u16 ba_pkt_count;
|
||||
u8 ba_packet_thr;
|
||||
u16 total_pkt_count;
|
||||
};
|
||||
|
||||
struct mwifiex_tid_tbl {
|
||||
|
|
|
@ -120,7 +120,7 @@ mwifiex_wmm_allocate_ralist_node(struct mwifiex_adapter *adapter, u8 *ra)
|
|||
|
||||
memcpy(ra_list->ra, ra, ETH_ALEN);
|
||||
|
||||
ra_list->total_pkts_size = 0;
|
||||
ra_list->total_pkt_count = 0;
|
||||
|
||||
dev_dbg(adapter->dev, "info: allocated ra_list %p\n", ra_list);
|
||||
|
||||
|
@ -679,8 +679,8 @@ mwifiex_wmm_add_buf_txqueue(struct mwifiex_private *priv,
|
|||
|
||||
skb_queue_tail(&ra_list->skb_head, skb);
|
||||
|
||||
ra_list->total_pkts_size += skb->len;
|
||||
ra_list->ba_pkt_count++;
|
||||
ra_list->total_pkt_count++;
|
||||
|
||||
if (atomic_read(&priv->wmm.highest_queued_prio) <
|
||||
tos_to_tid_inv[tid_down])
|
||||
|
@ -1037,7 +1037,7 @@ mwifiex_send_single_packet(struct mwifiex_private *priv,
|
|||
tx_info = MWIFIEX_SKB_TXCB(skb);
|
||||
dev_dbg(adapter->dev, "data: dequeuing the packet %p %p\n", ptr, skb);
|
||||
|
||||
ptr->total_pkts_size -= skb->len;
|
||||
ptr->total_pkt_count--;
|
||||
|
||||
if (!skb_queue_empty(&ptr->skb_head))
|
||||
skb_next = skb_peek(&ptr->skb_head);
|
||||
|
@ -1062,7 +1062,7 @@ mwifiex_send_single_packet(struct mwifiex_private *priv,
|
|||
|
||||
skb_queue_tail(&ptr->skb_head, skb);
|
||||
|
||||
ptr->total_pkts_size += skb->len;
|
||||
ptr->total_pkt_count++;
|
||||
ptr->ba_pkt_count++;
|
||||
tx_info->flags |= MWIFIEX_BUF_FLAG_REQUEUED_PKT;
|
||||
spin_unlock_irqrestore(&priv->wmm.ra_list_spinlock,
|
||||
|
|
Loading…
Reference in New Issue