Re: [PATCH net 1/3] net: fealnx: fix teardown order in remove

From: Жамбакиев Радий Рикардинович

Date: Mon Sep 28 2026 - 09:27:27 EST


On Fri, 2026-09-25 at 10:46 +0000, netdev-bot+sashiko@xxxxxxxxxx wrote:
> - [High] The commit says unregister_netdev() -> dev_close() ->
>   netdev_close() "deletes the timers ... before the rings are freed".
> - [High] The fix assumes that once unregister_netdev() returns,
>   netdev_close() has stopped the Rx/Tx DMA engines, so…
>
> Pre-existing issues:
> - [Low] At this commit, fealnx_init_one() calls pci_enable_device()
> and
>   pci_set_master() (lines 497/499).
>
> --

I think [High] issues is pre-existing. Sashiko points to the code, that
already been there and was not touched by this patch. It might be the
wording of the patch, that made Sashiko mark issues as freshly
introduced.