ALT Linux kernel packages development
 help / color / mirror / Atom feed
* [d-kernel] [PATCH 0/1] Отказ от floppy драйвера
@ 2024-05-21  9:27 dutyrok
  2024-05-21  9:27 ` [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD dutyrok
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: dutyrok @ 2024-05-21  9:27 UTC (permalink / raw)
  To: devel-kernel; +Cc: kovalevvv, dutyrok, oficerovas

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

Стоит также учесть что, последний коммит в драйвер (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 драйвер востребован в текущий
момент, и можем ли мы отказаться от него?




^ permalink raw reply	[flat|nested] 7+ messages in thread

* [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD
  2024-05-21  9:27 [d-kernel] [PATCH 0/1] Отказ от floppy драйвера dutyrok
@ 2024-05-21  9:27 ` dutyrok
  2024-07-20 18:18   ` Vitaly Chikunov
  2024-05-21  9:58 ` [d-kernel] [PATCH 0/1] Отказ от floppy драйвера Alexey V. Vissarionov
  2024-06-10 13:28 ` Vitaly Chikunov
  2 siblings, 1 reply; 7+ messages in thread
From: dutyrok @ 2024-05-21  9:27 UTC (permalink / raw)
  To: devel-kernel; +Cc: kovalevvv, dutyrok, oficerovas

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



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера
  2024-05-21  9:27 [d-kernel] [PATCH 0/1] Отказ от floppy драйвера dutyrok
  2024-05-21  9:27 ` [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD dutyrok
@ 2024-05-21  9:58 ` Alexey V. Vissarionov
  2024-06-10 13:28 ` Vitaly Chikunov
  2 siblings, 0 replies; 7+ messages in thread
From: Alexey V. Vissarionov @ 2024-05-21  9:58 UTC (permalink / raw)
  To: ALT Linux kernel packages development

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера
  2024-05-21  9:27 [d-kernel] [PATCH 0/1] Отказ от floppy драйвера dutyrok
  2024-05-21  9:27 ` [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD dutyrok
  2024-05-21  9:58 ` [d-kernel] [PATCH 0/1] Отказ от floppy драйвера Alexey V. Vissarionov
@ 2024-06-10 13:28 ` Vitaly Chikunov
  2024-06-10 14:00   ` Alexey V. Vissarionov
  2024-07-17  9:37   ` Alexandr Shashkin
  2 siblings, 2 replies; 7+ messages in thread
From: Vitaly Chikunov @ 2024-06-10 13:28 UTC (permalink / raw)
  To: ALT Linux kernel packages development; +Cc: oficerovas, dutyrok, kovalevvv

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера
  2024-06-10 13:28 ` Vitaly Chikunov
@ 2024-06-10 14:00   ` Alexey V. Vissarionov
  2024-07-17  9:37   ` Alexandr Shashkin
  1 sibling, 0 replies; 7+ messages in thread
From: Alexey V. Vissarionov @ 2024-06-10 14:00 UTC (permalink / raw)
  To: ALT Linux kernel packages development

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [d-kernel] [PATCH 0/1] Отказ от floppy драйвера
  2024-06-10 13:28 ` Vitaly Chikunov
  2024-06-10 14:00   ` Alexey V. Vissarionov
@ 2024-07-17  9:37   ` Alexandr Shashkin
  1 sibling, 0 replies; 7+ messages in thread
From: Alexandr Shashkin @ 2024-07-17  9:37 UTC (permalink / raw)
  To: devel-kernel

Насколько мне известно, выявленных неисправленных 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.



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD
  2024-05-21  9:27 ` [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD dutyrok
@ 2024-07-20 18:18   ` Vitaly Chikunov
  0 siblings, 0 replies; 7+ messages in thread
From: Vitaly Chikunov @ 2024-07-20 18:18 UTC (permalink / raw)
  To: ALT Linux kernel packages development; +Cc: oficerovas, dutyrok, kovalevvv

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2024-07-20 18:18 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-21  9:27 [d-kernel] [PATCH 0/1] Отказ от floppy драйвера dutyrok
2024-05-21  9:27 ` [d-kernel] [PATCH 1/1] config: unset CONFIG_BLK_DEV_FD dutyrok
2024-07-20 18:18   ` Vitaly Chikunov
2024-05-21  9:58 ` [d-kernel] [PATCH 0/1] Отказ от floppy драйвера Alexey V. Vissarionov
2024-06-10 13:28 ` Vitaly Chikunov
2024-06-10 14:00   ` Alexey V. Vissarionov
2024-07-17  9:37   ` Alexandr Shashkin

ALT Linux kernel packages development

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-kernel/0 devel-kernel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel-kernel devel-kernel/ http://lore.altlinux.org/devel-kernel \
		devel-kernel@altlinux.org devel-kernel@altlinux.ru devel-kernel@altlinux.com
	public-inbox-index devel-kernel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-kernel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git