[d-kernel] [PATCH 0/1] Отказ от floppy драйвера

[d-kernel] [PATCH 0/1] Отказ от floppy драйвера

[dutyrok]

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

Стоит также учесть что, последний коммит в драйвер (drivers/block/floppy.c) был
2 года назад (8 мая 2022). А разработчики относятся к нему как к забагованному
легаси коду, который необходимо переписать
(https://lore.kernel.org/all/7df3e30a-aa31-495c-9d59-cb6080364f61@kernel.dk/).

В Syzbot также отказались от фаззинга драйвера
(https://github.com/google/syzkaller/commit/42e1d5241be0f7564d28dbd5f704d67c77d4dd74),
а найденные раннее срабатывания просто оставили неисправленными
(https://syzkaller.appspot.com/bug?id=09bfcd00c069466f6c3d07539d679e7c49e398e7).

Просьба прокомментировать насколько floppy драйвер востребован в текущий
момент, и можем ли мы отказаться от него?

[d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD

[dutyrok]

From: Alexandr Shashkin <dutyrok@altlinux.org>

Disable floppy disk driver, that is almost unmaintained and has enough legacy
code.

Signed-off-by: Alexandr Shashkin <dutyrok@altlinux.org>
---
 config | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config b/config
index 2dbf4ce44aa3..e95f17fcb75f 100644
--- a/config
+++ b/config
@@ -2536,7 +2536,7 @@ CONFIG_PNP_DEBUG_MESSAGES=y
 CONFIG_PNPACPI=y
 CONFIG_BLK_DEV=y
 # CONFIG_BLK_DEV_NULL_BLK is not set
-CONFIG_BLK_DEV_FD=m
+CONFIG_BLK_DEV_FD is not set
 CONFIG_CDROM=m
 # CONFIG_PARIDE is not set
 
-- 
2.33.8

Re: [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD

[Vitaly Chikunov]

On Tue, May 21, 2024 at 12:27:32PM +0300, dutyrok@altlinux.org wrote:
> From: Alexandr Shashkin <dutyrok@altlinux.org>
> 
> Disable floppy disk driver, that is almost unmaintained and has enough legacy
> code.
> 
> Signed-off-by: Alexandr Shashkin <dutyrok@altlinux.org>
> ---
>  config | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/config b/config
> index 2dbf4ce44aa3..e95f17fcb75f 100644
> --- a/config
> +++ b/config
> @@ -2536,7 +2536,7 @@ CONFIG_PNP_DEBUG_MESSAGES=y
>  CONFIG_PNPACPI=y
>  CONFIG_BLK_DEV=y
>  # CONFIG_BLK_DEV_NULL_BLK is not set
> -CONFIG_BLK_DEV_FD=m
> +CONFIG_BLK_DEV_FD is not set

Ошибка.

Applied, thanks

  a490baeecdd9 [kernelbot/un-def/p10] config: unset CONFIG_BLK_DEV_FD
  f62d247705c2 [kernelbot/std-def/p10] config: unset CONFIG_BLK_DEV_FD
  6e1bddfec669 [kernelbot/un-def/p11] config: unset CONFIG_BLK_DEV_FD
  5deb999ffcc2 [kernelbot/std-def/p11] config: unset CONFIG_BLK_DEV_FD

>  CONFIG_CDROM=m
>  # CONFIG_PARIDE is not set
>  
> -- 
> 2.33.8
> 
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel

Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера

[Alexey V. Vissarionov]

Good ${greeting_time}!

On 2024-05-21 12:27:31 +0300, dutyrok@altlinux.org wrote:

 > Этот драйвер был написан довольно давно, и в нём, по словам
 > мейнтейнера Ефремова Дениса, очень много легаси-вещей, которые
 > невозможно адекватным образом поправить. Его попытка добавить
 > тесты для драйвера тоже не имела успеха, так как "никому это
 > было неинтересно".
 > [...]
 > Просьба прокомментировать насколько floppy драйвер востребован
 > в текущий момент, и можем ли мы отказаться от него?

От поддержки x86_32 мы вроде как отказались, а на платах x86_64
этот контроллер отсутствует вот уже 20 лет (если у кого-то есть
практический интерес, можно и выяснить, начиная с какого именно
"южного моста").

Выкидывайте его нахрен нещадно.


-- 
Alexey V. Vissarionov
gremlin ПРИ altlinux ТЧК org; +vii-cmiii-ccxxix-lxxix-xlii
GPG: 0D92F19E1C0DC36E27F61A29CD17E2B43D879005 @ hkp://keys.gnupg.net

Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера

[Vitaly Chikunov]

On Tue, May 21, 2024 at 12:27:31PM +0300, dutyrok@altlinux.org wrote:
> Этот драйвер был написан довольно давно, и в нём, по словам мейнтейнера Ефремова
> Дениса, очень много легаси-вещей, которые невозможно адекватным образом
> поправить. Его попытка добавить тесты для драйвера тоже не имела успеха, так как
> "никому это было неинтересно".
> 
> Стоит также учесть что, последний коммит в драйвер (drivers/block/floppy.c) был
> 2 года назад (8 мая 2022). А разработчики относятся к нему как к забагованному
> легаси коду, который необходимо переписать
> (https://lore.kernel.org/all/7df3e30a-aa31-495c-9d59-cb6080364f61@kernel.dk/).
> 
> В Syzbot также отказались от фаззинга драйвера
> (https://github.com/google/syzkaller/commit/42e1d5241be0f7564d28dbd5f704d67c77d4dd74),
> а найденные раннее срабатывания просто оставили неисправленными
> (https://syzkaller.appspot.com/bug?id=09bfcd00c069466f6c3d07539d679e7c49e398e7).
> 
> Просьба прокомментировать насколько floppy драйвер востребован в текущий
> момент, и можем ли мы отказаться от него?

Это модуль и ему можно сделать blacklist.

  debian.master/config/config.common.ubuntu:1193:CONFIG_BLK_DEV_FD=m
  redhat/configs/fedora/generic/CONFIG_BLK_DEV_FD:1:CONFIG_BLK_DEV_FD=m
  redhat/configs/rhel/generic/CONFIG_BLK_DEV_FD:1:# CONFIG_BLK_DEV_FD is not set
  debian/config/kernelarch-x86/config:264:CONFIG_BLK_DEV_FD=m

Почему у нас должно быть иначе - там что есть LPE?


> 
> 
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel

Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера

[Alexey V. Vissarionov]

Good ${greeting_time}!

On 2024-06-10 16:28:35 +0300, Vitaly Chikunov wrote:

 >> Просьба прокомментировать насколько floppy драйвер востребован
 >> в текущий момент, и можем ли мы отказаться от него?
 > Это модуль и ему можно сделать blacklist.
 > debian.master/config/config.common.ubuntu:1193:CONFIG_BLK_DEV_FD=m
 > redhat/configs/fedora/generic/CONFIG_BLK_DEV_FD:1:CONFIG_BLK_DEV_FD=m
 > redhat/configs/rhel/generic/CONFIG_BLK_DEV_FD:1:#
 > CONFIG_BLK_DEV_FD is not set
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 > debian/config/kernelarch-x86/config:264:CONFIG_BLK_DEV_FD=m
 > Почему у нас должно быть иначе - там что есть LPE?

В данном случае есть смысл взять пример с RHEL, а не с Fedora.

Аналогичная ситуация (даже чуть похуже) с CONFIG_ATA_OVER_ETH -
подробности будут опубликованы чуть позже, но выкинуть его из
конфигурации следовало бы еще месяц назад.


-- 
Alexey V. Vissarionov
gremlin ПРИ altlinux ТЧК org; +vii-cmiii-ccxxix-lxxix-xlii
GPG: 0D92F19E1C0DC36E27F61A29CD17E2B43D879005 @ hkp://keys.gnupg.net

Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера

[Alexandr Shashkin]

Насколько мне известно, выявленных неисправленных LPE в драйвере Floppy 
нет.

10.06.2024 16:28, Vitaly Chikunov пишет:
> On Tue, May 21, 2024 at 12:27:31PM +0300, dutyrok@altlinux.org wrote:
>> Этот драйвер был написан довольно давно, и в нём, по словам мейнтейнера Ефремова
>> Дениса, очень много легаси-вещей, которые невозможно адекватным образом
>> поправить. Его попытка добавить тесты для драйвера тоже не имела успеха, так как
>> "никому это было неинтересно".
>>
>> Стоит также учесть что, последний коммит в драйвер (drivers/block/floppy.c) был
>> 2 года назад (8 мая 2022). А разработчики относятся к нему как к забагованному
>> легаси коду, который необходимо переписать
>> (https://lore.kernel.org/all/7df3e30a-aa31-495c-9d59-cb6080364f61@kernel.dk/).
>>
>> В Syzbot также отказались от фаззинга драйвера
>> (https://github.com/google/syzkaller/commit/42e1d5241be0f7564d28dbd5f704d67c77d4dd74),
>> а найденные раннее срабатывания просто оставили неисправленными
>> (https://syzkaller.appspot.com/bug?id=09bfcd00c069466f6c3d07539d679e7c49e398e7).
>>
>> Просьба прокомментировать насколько floppy драйвер востребован в текущий
>> момент, и можем ли мы отказаться от него?
> Это модуль и ему можно сделать blacklist.
>
>    debian.master/config/config.common.ubuntu:1193:CONFIG_BLK_DEV_FD=m
>    redhat/configs/fedora/generic/CONFIG_BLK_DEV_FD:1:CONFIG_BLK_DEV_FD=m
>    redhat/configs/rhel/generic/CONFIG_BLK_DEV_FD:1:# CONFIG_BLK_DEV_FD is not set
>    debian/config/kernelarch-x86/config:264:CONFIG_BLK_DEV_FD=m
>
> Почему у нас должно быть иначе - там что есть LPE?
>
>
>>
>> _______________________________________________
>> devel-kernel mailing list
>> devel-kernel@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/devel-kernel
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel

-- 
c уважением,
Александр Шашкин,
ALT Linux Team.