Re: [PATCH] wifi: ath11k: fix warning when unbinding

[Rameshkumar Sundaram]

On 5/8/2026 4:01 PM, Jose Ignacio Tornos Martinez wrote:
> Hello Rameshkumar,
>
> > What is the exact failure? Do you see any driver error logs when it occurs?
> No error log, just the warning.
>
> > Got it. I was just thinking along with the proposed fix — whether we
> > might also need to handle the sequencing on QMI failure.
> > In other words, do you think the issue(double free) would still be
> > reproducible if we include a change like below ?
> Yes, I think so and in addition the code is more robust.

I agree that setting tx_status to NULL makes ath11k_dp_free() more
defensive, and it matches the ath12k fix.

However, i am still wondering how the second ath11k_dp_free() is reached 
if ATH11K_FLAG_QMI_FAIL is set.

In ath11k_pci_remove(), when ATH11K_FLAG_QMI_FAIL is set, we take the
qmi_fail path and skip ath11k_core_deinit(). So the normal remove path:

    ath11k_pci_remove()
      ath11k_core_deinit()
        ath11k_core_soc_destroy()
          ath11k_dp_free()

should not run.

So if the double free is still reproducible with QMI_FAIL set (with the 
change i proposed), either the flag is not actually set in this failure 
case, or there is another path calling ath11k_dp_free() ?


--
Ramesh