Ashok Raj Nagarajan <arnagara@xxxxxxxxxxxxxxxx> wrote: > In the wake tx queue path, we are not checking if the frame to be sent > takes management path or not. For eg. QOS null func frame coming here will > take the management path. Since we are not incrementing the descriptor > counter (num_pending_mgmt_tx) w.r.t tx management, on tx completion it is > possible to see negative values. > > When the above counter reaches a negative value, we will not be sending a > probe response out. > > if (is_presp && > ar->hw_params.max_probe_resp_desc_thres < htt->num_pending_mgmt_tx) > > For IPQ4019, max_probe_resp_desc_thres (u32) is 24 is compared against > num_pending_mgmt_tx (int) and the above condtions comes true if the counter > is negative and we drop the probe response. > > To avoid this, check on the wake tx queue path as well for the tx path of > the frame and increment the appropriate counters > > Fixes: cac085524cf1 "ath10k: move mgmt descriptor limit handle under mgmt_tx" > Signed-off-by: Ashok Raj Nagarajan <arnagara@xxxxxxxxxxxxxxxx> Thanks, 1 patch applied to ath-next branch of ath.git: e4fd726f21cd ath10k: fix sending frame in management path in push txq logic -- Sent by pwcli https://patchwork.kernel.org/patch/9287191/