* [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