[lvc-project] [PATCH net 1/3] net: fealnx: fix teardown order in remove

Жамбакиев Радий Рикардинович r.zhambakiev at prosoftsystems.ru
Fri Oct 2 09:05:16 MSK 2026


On Fri, 2026-09-25 at 16:39 +0300, Fedor Pchelkin wrote:
> Касательно ответов мэйнтейнеров - их реакции показались токсичными,
> особенно Andrew Lunn.  Ну сделаем мы этот KISS как он написал, так их
> же
> sashiko-bot и сообщит, что это плохое исправление, захардкоживающее
> кол-во
> попыток пробинга драйвера, и потом они автоматом cr (changes-
> requested)
> поставят у себя в системе для патча.  Если сопровождающие netdev
> такое
> примут, а Andrew Lunn среди них не главный, будет странно.

Стоит ли как-то попробовать аппелировать его предложение или просто
направить вторую версию патча? Явно ответ такой скептический из-за
старого драйвера, будут ли восприняты аргументы в сторону IDA.

> Комментарии Francois Romieu тоже очень придирчивые.  С рефакторингом
> раз
> на раз не приходится - хотя в этих фикс-патчах проведенный
> одновременно
> рефакторинг, на мой взгляд, выглядит органично, вполне является
> логической
> частью изменений и не есть рефакторинг ради рефакторинга - но видимо
> в
> древние неподдерживаемые драйверы предлагают обходиться очень
> краткими
> фикс-патчами.
> 
> Стоит ли делать рефакторинг вообще, выделив его в отдельные патчи,
> можно
> уточнить у Francois.

Уточнил, судя по ответу, как минимум с тегом net рефакторинг, особенно
в старом драйвере, не приветствуется. В новой версии уберу его.


More information about the lvc-project mailing list