Re: [PATCH] wifi: mwifiex: fix error recovery in PCIE buffer descriptor management

[Brian Norris <briannorris@chromium.org>]

On Mon, Jul 31, 2023 at 10:43:07AM +0300, Dmitry Antipov wrote:
> Add missing 'kfree_skb()' in 'mwifiex_init_rxq_ring()' and never do
> 'kfree(card->rxbd_ring_vbase)' because this area is DMAed and should
> be released with 'dma_free_coherent()'. The latter is performed in
> 'mwifiex_pcie_delete_rxbd_ring()', which is now called to recover
> from possible errors in 'mwifiex_pcie_create_rxbd_ring()'. Likewise
> for 'mwifiex_pcie_init_evt_ring()', 'kfree(card->evtbd_ring_vbase)'
> 'mwifiex_pcie_delete_evtbd_ring()' and 'mwifiex_pcie_create_rxbd_ring()'.
> 
> Fixes: 0732484b47b5 ("mwifiex: separate ring initialization and ring creation routines")

I'm not sure that's truly an appropriate "Fixes" target, as that commit
just shuffled the existing bad code around. I'd either remove that line,
or else also add:

Fixes: d930faee141b ("mwifiex: add support for Marvell pcie8766 chipset")

where the buggy stuff was first introduced.

I don't think you need to resubmit just for this; Kalle can usually make
small adjustments like this when applying. And even if not, the issue is
trivial.

> Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru>
> ---
>  drivers/net/wireless/marvell/mwifiex/pcie.c | 25 ++++++++++++++-------
>  1 file changed, 17 insertions(+), 8 deletions(-)

Patch looks good, thanks:

Acked-by: Brian Norris <briannorris@chromium.org>