[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