Re: [d-kernel] [PATCH 6.15 0/4] config: Remove obsolete options

[Vitaly Chikunov <vt@altlinux.org>]

Vasiliy,

On Sun, Apr 20, 2025 at 10:15:06AM +0300, Vasiliy Kovalev wrote:
> 20.04.2025 06:18, Vitaly Chikunov wrote:
> > On Sun, Apr 20, 2025 at 05:35:38AM +0300, Vasiliy Kovalev wrote:
> > > 19.04.2025 22:37, Vitaly Chikunov wrote:
> > > > On Sat, Apr 19, 2025 at 06:29:58PM +0300, Vasiliy Kovalev wrote:
> > > > > файлах для удобства дальнейшей оценки конфигурации.
> > > > 
> > > > Пожалуйста, не используйте исходник конифига для оценки
> > > > конфигурации, используйте итоговый конфиг из собранных ядер.
> > > 
> > > Согласен и такой подход конечно правильный, но это также и противоречие,
> > > т.к. Вы сами анализируете исходники перед вынесением оценки конфигурации,
> > > например тут https://lore.altlinux.org/devel-kernel/20240610132835.qnphs5hv4rrym363@altlinux.org/
> > > вероятно по той же причине - удобство и скорость первичной оценки.
> > 
> > Исходный конфиг показывает намерение маинтайнеров.
> 
> Извините, но это противоречивая позиция - нельзя одновременно утверждать,
> что исходные
> конфиги не следует использовать для оценки, и в то же время обосновывать их
> использование "намерениями маинтайнеров". Либо исходные конфиги информативны
> и полезны для анализа (и тогда мои выводы об устаревших опциях обоснованы),
> либо нет.

Намерение - осознанное целенаправленное действие.

Используя исходный конфиг вы оцениваете намерение маинтайнеров.
Используя итоговый конфиг вы оцениваете конфигурацию ядра.
Между этими конфигами есть разница в том что огромная часть опций включаются
автоматически по формулам и наследуется из апстрима.

Например, в SUSE синхронизируют конфиг в каждом билде. И поэтому они
часто сами не понимают какие опции они включили зачем-то, а какие включились
автоматически. https://www.youtube.com/watch?v=ymw96lGq5hk


> 
> Отмечу, что предложенные патчи как раз основаны на анализе обоих типов
> конфигов
> -  как исходного, так и итогового из собранных ядер.
> 
> > 
> > > 
> > > --
> > > 
> > > Итого, чтобы придти к какому-то компромиссу, предлагаю для веток
> > > sisyphus (6.12+) и остальных (на базе 5.10 и 6.1) следующие действия:
> > > 
> > > 1) выполнить бэкпорт (cherry-pick) патчей для 6.13:
> > > f495aad09ab5 ("config: unset CONFIG_BLK_DEV_FD")
> > > 240723f0ad50 ("config: Enable CONFIG_NTSYNC=m")
> > > ee7084f38bcf ("config: Enable more Intel hardware")
> > > 
> > > 2) выполнить бэкпорт (cherry-pick) патчей для 6.12:
> > > 240723f0ad50 ("config: Enable CONFIG_NTSYNC=m")
> > > ee7084f38bcf ("config: Enable more Intel hardware")
> > 
> > В принципе, так и надо сделать.
> > 
> > > 
> > > 3) применить один патч для всех (sisyphus 6.12-6.15) веток из этой серии:
> > > [PATCH 6.15 1/4] config: Remove obsolete options (unused in v6.12-v6.15)
> > > 
> > > 4) применить остальные отправленные (*Remove obsolete options*) патчи
> > > для соотв.  веток на базе 5.10 и 6.1 ядер.
> > 
> > Эти патчи не приносят пользы, но вносят ещё больший рассинхрон в конфиги.
> > 
> > Желательно или придумать удобную систему конфигурирования 13 ядер
> > (умножить на кол-во архитектур), или не ломать то, что есть. То что
> > есть -- не так плохо как может показаться на первый взгляд.
> 
> $ git diff --stat kernelbot/un-def/p9..kernelbot/un-def/p10  -- config | tee
>  config | 1217
> ++++++++++++++++++++++++++++++++++++++++------------------------
>  1 file changed, 763 insertions(+), 454 deletions(-)
> 
> $ git diff --stat kernelbot/un-def/p10..kernelbot/6.12/sisyphus  -- config |
> tee
>  config | 367
> +++++++++++++++++++++++++++++++++++++++++++++++++++++------------
>  1 file changed, 300 insertions(+), 67 deletions(-)
> 
> $ git diff --stat kernelbot/un-def/p9..kernelbot/6.12/sisyphus  -- config |
> tee
>  config | 1538
> +++++++++++++++++++++++++++++++++++++++++++---------------------
>  1 file changed, 1040 insertions(+), 498 deletions(-)
> 
> где un-def/p9 - ветка с ядром 5.10
> un-def/p10 - ветка с ядром 6.1
> 
> При таких масштабных расхождениях аргумент о "небольшом рассинхроне"
> и удобстве бэкпортирования уже неактуален - конфиги давно радикально
> отличаются,
> и простой cherri-pick не всегда сработает без ручного вмешательства.
> 
> Удаление ~400 устаревших строк значительно не усугубит ситуацию, но улучшит
> читаемость и понимание конфигов.

Увеличение рассинхрона ухудшает бекпортиуремость, а
уменьшение рассинхрона улучшает бекпортиуремость.

Неполный их синхрон не означает, что идея синхрона
опровергнута. Так же есть цена/качество у внесения изменений. Например,
если изменение желательно протестировать, то дешевле пойти на временный
рассинхрон.


> 
> > У других
> > дистрибутивов тоже не идеальные системы, я разбирался в вопросе.
> 
> До возможного появления "системы конфигурирования"

Её никогда не будет так что можно это не обсуждать.

> чистка от устаревших
> опций - это
> скорее шаг к упрощению, а не усложнению. Меньше "шума" в конфигах сделает и
> текущую
> поддержку, и будущую систематизацию более прозрачной.

Увеличение степени рассинхрона увеличивает вероятность, что коммит для
конфига не наложится. Это увеличит работу по подготовке таких коммитов,
по их прохождению в ядра, увеличит вероятность ошибок.

Желательно, чтоб конфиги были синхронизированы, но полная синхронизация
это идеал. Если её нет, то не значит, что она не желательна. Но, полная
синхронизация не самоцель, могут быть ситуации, что где-то лучше оставить
рассинхрон, так как синзхрон себе дороже.

> 
> > > 
> > > > > 
> > > > > Эти опции выявлены анализом исходного кода ядра и
> > > > > конечного конфиг файла (/boot/config*), поставляемого
> > > > > kernel-image*.rpm пакетом соотв. ветки.
> > > > > 
> > > > > [PATCH 6.15 1/4] config: Remove obsolete options (unused in v6.12-v6.15)
> > > > > [PATCH 6.15 2/4] config: Remove obsolete options (unused in v6.13-v6.15)
> > > > > [PATCH 6.15 3/4] config: Remove obsolete options (unused in v6.14-v6.15)
> > > > > [PATCH 6.15 4/4] config: Remove obsolete options (unused in v6.15)
> > > > 
> > > > > _______________________________________________
> > > > > 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
> > > 
> > > -- 
> > > --
> > > Thanks,
> > > Vasiliy
> 
> -- 
> --
> Thanks,
> Vasiliy