[lvc-project] [PATCH net v2 3/3] net: fealnx: allocate the card index from an IDA

[Fedor Pchelkin]

On Tue, 06. Oct 14:31, netdev-bot+sashiko at kernel.org wrote:
> - [Medium] The commit message says the IDA "reuses ids on re-add,
>   preserving the options[] and full_duplex[] mapping by probe order".

Не знаю, подход с IDA выглядит более уместным, как по мне.  Опасения
мэйнтейнеров понятны - это древний драйвер, любые изменения для которого
желательно тестировать в связке с физическим устройством.

===
Если организация включает данный драйвер в состав сертифицируемого ядра,
предполагается, что у неё такие возможности должны быть.  В идеальной
ситуации, понятное дело.  Иначе к сожалению нам всем приходится иметь дело
вот с такими реакциями мэйнтейнеров на происходящее.

Я к тому, что подавляющая часть работы над древними драйверами может быть
легко убрана выключением их из конфигураций ядер, направленных на
сертификацию.  Если же вдруг эти драйверы правда кому-то требуются,
возвращаемся к пункту, что должна иметься возможность тестировать.
===


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

> 
> Pre-existing issues:
> - [High] fealnx_init_one() (drivers/net/ethernet/fealnx.c:482-518) builds
>   the PCI resource name in the stack array `char boardname[18]` and…

Раз уж мы залезли в это болото, проблема выше тоже выглядит валидной.  И
должно быть, даже более "достижимой" и значимой на практике.  Подготовите
на неё четвертый патч?

Отправить будет сейчас достаточно в lvc-patches at linuxtesting.org.  В
сообществе, насколько понимаю, переводят серию на net-next и смогут
что-то добавить они туда через несколько недель только.