samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacmn: Drop undecoded frames err mask not configured

Browse Source 
1. Drop the received undecoded frame if error code
   abort reason not matched with configured error mask.
2. Reset phy error mask during undecoded capture mode reset.

Change-Id: I162ac1b60ceb5a4b5a87ff2d2186f2e67815f580
CRs-Fixed: 3134562
This commit is contained in:
Basamma Yakkanahalli
2022-02-20 08:52:55 +05:30
committed by Madan Koyyalamudi
parent 70f4513196
commit 831a3ce7d5
2 changed files with 30 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
dp/wifi3.0/monitor/1.0/dp_mon_filter_1.0.c
View File

@@ -253,6 +253,8 @@ void dp_mon_filter_reset_undecoded_metadata_capture_1_0(struct dp_pdev *pdev)
filter.tlv_filter.phy_err_mask = 0;
filter.tlv_filter.phy_err_mask_cont = 0;
mon_pdev->phyrx_error_mask = 0;
mon_pdev->phyrx_error_mask_cont = 0;
filter.tlv_filter.phy_err_filter_valid = 1;

34
dp/wifi3.0/monitor/dp_rx_mon.c
View File

@@ -1074,6 +1074,7 @@ dp_rx_handle_ppdu_stats(struct dp_soc *soc, struct dp_pdev *pdev,
#endif/* QCA_ENHANCED_STATS_SUPPORT */
#ifdef QCA_UNDECODED_METADATA_SUPPORT
#define RX_PHYERR_MASK_GET64(_val1, _val2) (((uint64_t)(_val2) << 32) | (_val1))
/**
* dp_rx_populate_cdp_indication_ppdu_undecoded_metadata() - Populate cdp
* rx indication structure
@@ -1216,6 +1217,24 @@ dp_rx_populate_cdp_indication_ppdu_undecoded_metadata(struct dp_pdev *pdev,
dp_rx_populate_cdp_indication_ppdu_user(pdev, ppdu_info, cdp_rx_ppdu);
}
/**
* dp_rx_is_valid_undecoded_frame() - Check unencoded frame received valid
* or not against configured error mask
* @err_mask: configured err mask
* @err_code: Received error reason code for phy abort
*
* Return: true / false
*/
static inline bool
dp_rx_is_valid_undecoded_frame(uint64_t err_mask, uint8_t err_code)
{
if (err_code < CDP_PHYRX_ERR_MAX &&
(err_mask & (1L << err_code)))
return true;
return false;
}
void
dp_rx_handle_ppdu_undecoded_metadata(struct dp_soc *soc, struct dp_pdev *pdev,
struct hal_rx_ppdu_info *ppdu_info)
@@ -1224,11 +1243,19 @@ dp_rx_handle_ppdu_undecoded_metadata(struct dp_soc *soc, struct dp_pdev *pdev,
struct cdp_rx_indication_ppdu *cdp_rx_ppdu;
uint8_t abort_reason = 0;
struct dp_mon_pdev *mon_pdev = pdev->monitor_pdev;
uint64_t mask64;
/* Return if RX_ABORT not set */
if (ppdu_info->rx_status.phyrx_abort == 0)
return;
mask64 = RX_PHYERR_MASK_GET64(mon_pdev->phyrx_error_mask,
mon_pdev->phyrx_error_mask_cont);
abort_reason = ppdu_info->rx_status.phyrx_abort_reason;
if (!dp_rx_is_valid_undecoded_frame(mask64, abort_reason))
return;
ppdu_nbuf = qdf_nbuf_alloc(soc->osdev,
sizeof(struct cdp_rx_indication_ppdu),
0, 0, FALSE);
@@ -1247,12 +1274,7 @@ dp_rx_handle_ppdu_undecoded_metadata(struct dp_soc *soc, struct dp_pdev *pdev,
}
mon_pdev->rx_mon_stats.rx_undecoded_count++;
abort_reason = cdp_rx_ppdu->phyrx_abort_reason;
if (abort_reason < CDP_PHYRX_ERR_MAX) {
mon_pdev->rx_mon_stats.rx_undecoded_error[abort_reason] += 1;
} else {
mon_pdev->rx_mon_stats.rx_undecoded_error[CDP_PHYRX_ERR_OTHER] += 1;
}
mon_pdev->rx_mon_stats.rx_undecoded_error[abort_reason] += 1;
dp_wdi_event_handler(WDI_EVENT_RX_PPDU_DESC_UNDECODED_METADATA,
soc, ppdu_nbuf, HTT_INVALID_PEER,