diff options
| author | Ferry Zhou <tianguiz@codeaurora.org> | 2017-03-08 17:35:30 +0800 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2017-07-19 05:09:22 -0700 |
| commit | 1a97ed1da4b37d4dc9e73cda183649aa7cf53451 (patch) | |
| tree | 9e73344e5dabba1652f41f1332a735a0c8f64ad9 | |
| parent | be12aeb853ae7112e8a8b3e33f0ec2704d619bf5 (diff) | |
qcacld-2.0: Collect only DSRC tx packet statistics
The predicate adf_nbuf_headroom(msdu) >=
sizeof(struct htt_tx_msdu_desc_ext_t) is not always right
to judge if the packet is of DSRC or other protocols.
When IPv6 packet length is 1514, we think it should be false
but it's true. So, the bug happens.
Make choice by tx_ctrl_header_found, which is always right,
since only DSRC packet has ocb_tx_ctrl_hdr.
Change-Id: I704fde2d993c8a2a172f95d8edf9c2463b03ad00
CRs-Fixed: 1113905
| -rw-r--r-- | CORE/CLD_TXRX/TXRX/ol_tx.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/CORE/CLD_TXRX/TXRX/ol_tx.c b/CORE/CLD_TXRX/TXRX/ol_tx.c index a6bd7272ef68..eaf5f048d20d 100644 --- a/CORE/CLD_TXRX/TXRX/ol_tx.c +++ b/CORE/CLD_TXRX/TXRX/ol_tx.c @@ -1059,12 +1059,10 @@ ol_tx_hl_base( * Per OCB channel, it includes a default MAC address * for broadcast DSRC frame in this channel. */ - if (tx_ctrl_header_found) + if (tx_ctrl_header_found) { dsrc_update_broadcast_frame_sa(vdev, &tx_ctrl, msdu); - /* TODO: only collect dsrc tx packet stats.*/ - if (adf_nbuf_headroom(msdu) >= - sizeof(struct htt_tx_msdu_desc_ext_t)) { + /* only collect dsrc tx packet stats.*/ ol_collect_per_pkt_tx_stats(vdev, &tx_ctrl, tx_desc->id); } } |