From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=unavailable autolearn_force=no version=3.4.1 Message-ID: <9b7ffa46-fdbe-89e0-bfb9-a311e86a4b85@basealt.ru> Date: Sun, 20 Apr 2025 05:35:38 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Content-Language: en-US To: Vitaly Chikunov References: <20250419153002.925933-1-kovalev@altlinux.org> <20250419193755.q7njma6cqjsetmdn@altlinux.org> From: Vasiliy Kovalev In-Reply-To: <20250419193755.q7njma6cqjsetmdn@altlinux.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Cc: ALT Linux kernel packages development Subject: Re: [d-kernel] [PATCH 6.15 0/4] config: Remove obsolete options X-BeenThere: devel-kernel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux kernel packages development List-Id: ALT Linux kernel packages development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 20 Apr 2025 02:35:41 -0000 Archived-At: List-Archive: List-Post: 19.04.2025 22:37, Vitaly Chikunov wrote: > Vasiliy, > > On Sat, Apr 19, 2025 at 06:29:58PM +0300, Vasiliy Kovalev wrote: >> Эта серия патчей и последующие в течение ~5 мин., >> имеющие однотипный шаблон *Remove obsolete options*, >> удаляют устаревшие опции в соответствующих config* > > Нет цели все чистить, так что отсылка на чистку не обоснование > изменения. Vitaly, можете тогда прояснить обратную цель - какое обоснование оставлять устаревшие опции в ядрах, которые не оказывают никакого влияния на сборку ядра и зачем их наследовать в следующие версии веток? Например, с таким подходом в конфиге ветки 6.12 набралось почти 400 лишних строк: (патч "config: Remove obsolete options (unused in v6.12-v6.15)" поверх 6.12) git diff --stat HEAD~1 config | 382 -------------------------------------------------------------- 1 file changed, 382 deletions(-) Вероятность возвращения связанного функционала близка к нулю, особенно для стабильных веток. Даже если такое и произойдет, вернуть конкретную опцию не является существенной проблемой, к тому же часть популярных опций задается по-умолчанию в исходном коде. > Тем более, это делает будущие патчи на конфиги не применимы > ко всем конифгам - придется делать для каждого бранча/флейвора отдельный > патчсет. Что осложнит и их разработку, и применение. Но ведь это уже нарушено, например, ваши коммиты в ветках 6.14+ ee7084f38bcf ("config: Enable more Intel hardware") 240723f0ad50 ("config: Enable CONFIG_NTSYNC=m") не бэкпортированы в ближайшие 6.12/6/13 , причем, помимо консистентности конфигов, патчи на первый взгляд также актуальны. Ещё момент, на который стоит обратить внимание, изменения коммита f495aad09ab5 ("config: unset CONFIG_BLK_DEV_FD") есть в ветках 6.12/6.14/6.15 но нет в ветке 6.13 В противном случае, я бы для всех веток sisyphus ограничился бы отправкой одного патча из этой серии: [PATCH 6.15 1/4] config: Remove obsolete options (unused in v6.12-v6.15) Отправленные патчи "чистки конфигов" для остальных веток (на базе 6.1 и 5.10 ядер) этой стратегии придерживаются. >> файлах для удобства дальнейшей оценки конфигурации. > > Пожалуйста, не используйте исходник конифига для оценки > конфигурации, используйте итоговый конфиг из собранных ядер. Согласен и такой подход конечно правильный, но это также и противоречие, т.к. Вы сами анализируете исходники перед вынесением оценки конфигурации, например тут https://lore.altlinux.org/devel-kernel/20240610132835.qnphs5hv4rrym363@altlinux.org/ вероятно по той же причине - удобство и скорость первичной оценки. -- Итого, чтобы придти к какому-то компромиссу, предлагаю для веток 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 ядер. >> >> Эти опции выявлены анализом исходного кода ядра и >> конечного конфиг файла (/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