RE: [PATCH net-next v1 3/5] net: lan743x: quiesce TX before freeing the rings

[Woojung.Huh]

Hi Chris,

..

> @@ -3336,6 +3337,9 @@ static int lan743x_netdev_close(struct net_device
> *netdev)
> 
>         lan743x_phylink_disconnect(adapter);
> 
> +       synchronize_net();
> +       netif_tx_disable(netdev);

synchronize_net() is called inside of lan743x_tx_close() and lan743x_rx_close()
via netif_napi_del(). Do you see current order of calling is an issue?

> +
>         for (index = 0; index < adapter->used_tx_channels; index++)
>                 lan743x_tx_close(&adapter->tx[index]);
> 
> --
> 2.34.1

Thanks
Woojung