URL: https://www.opennet.dev/cgi-bin/openforum/vsluhboard.cgi
Форум: vsluhforumID3
Нить номер: 139223
[ Назад ]

Исходное сообщение
"Линус Торвальдс отверг изменения к подсистеме MMC из-за низкого качества кода"

Отправлено opennews , 12-Фев-26 17:08 
Линус Торвальдс отказался принимать предназначенный для ядра 7.0  набор изменений от сопровождающего подсистему MMC (MultiMedia Card), назвав присланный патчи "полным мусорм", который даже не компилируется. По его предположению изменения не были предварительно обкатаны в ветке linux-next и не проходили сборочное  тестирование...

Подробнее: https://www.opennet.dev/opennews/art.shtml?num=64784


Содержание

Сообщения в этом обсуждении
"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 17:15 
> Компиляция завершалась ошибкой из-за
> повторного переопределения переменной.

Но как?!
Как PR с некомпиляющимся кодом вообще отправился на ревью?

А... ну да, у нас же тут диды, которые почтой PR отправляют!

Не то что у этих смузихлебов всякие CI, интерационные тесты, еще какая-то новомодная хрень.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 17:18 
Нейрослоп наверняка.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено вайбкодер , 12-Фев-26 17:43 
Не гони на прохресс!

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено НяшМяш , 12-Фев-26 17:54 
Это даже не нейрослоп, это просто кекспертный уровень. Вайбкодер хотя бы попросил нейронку это скомпилировать и починить ошибки.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:05 
> Нейрослоп наверняка.

Не... нейрослоп как минимум попытался бы скомпилить свой выс... код.
И сам бы попытался исправил. Ну или хотя бы сообщил.

А настоящий ядерный погромист даже не компилял :)


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Дмитрий , 12-Фев-26 18:26 
"Нейрослоп" хорошо решается залогом:
К патче прикреплаешь например 1 бикойн
Если код плохой Биткойн не возвращается. Хороший - возвращается.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:33 
И остаешься без коммитеров)
Потому что не у каждого есть биткойн "вынь да положь".

Проще забить и путь другие немамонты пишут ядро.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Витюшка , 12-Фев-26 17:19 
Насяльника, какой такой тистиравания? A KPI по количеству строк кода (возможно сгенерированного ИИ) кто выдерживать будет?

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 17:22 
Ладно кол-во и ИИ.
Просто прогоните тесты.. если они у вас есть))
В нормальных проектах CI проверил сборку ветки, пробежали юнит и прочие тесты.

Но ядро выше чем все эти мелочи, просто отправь то что написякал голубиной почтой, пусть кто-то там разбирается.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено нах. , 12-Фев-26 17:39 
я сказаль ыы прогони теста. ии сказаль все работаит софтастрои.

а оно -- вотъ...


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 17:51 
> The following changes since commit 63804fed149a6750ffd28610c5c1c98cce6bd377:
>   Linux 6.19-rc7 (2026-01-25 14:11:24 -0800)
> are available in the Git repository at:
>   git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc.git tags/mmc-v7.0

Нет никакой голубиной почты, всё через репозиторий. Коммитер закоммитил своё творчество, не удосужившись предварительно даже просто собрать ядро.

> В нормальных проектах CI проверил сборку ветки, пробежали юнит и прочие тесты.

Этот коммитер точно так же не стал бы прогонять тесты и результат был бы тот же, несмотря на все ваши новомодные тесты и сиаи. А если ваша апелляция к тому, что что-то сфейлилось бы на билд-сервере, так оно по сути и сфейлилось - Линус всё проверил и в релиз это не ушло.

Ну и про юнит тесты - было бы интересо послушать ваше мнение подробнее. Я лично, с одной стороны, вижу как ядро постоянно переделывается между релизами по всему коду, так что раз в полгода-год та же нвидия перестаёт собираться. С другой стороны, я вижу знакомого, работающего в крупной компании, с юнит-тестами по 30 строк каждый (подготовка, выполнение действий, и несколько проверок результата) и его производительностью 5 новых тестов в день. Ему проще перед релизом вручную протыкать за пару недель весь функционал, чем потратить полгода, чтобы переделать все эти тесты под новую логику. Как вы предполагаете объединять неповоротливость юнит-тестов и неконтролируемую скорость разработки ядра?


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:14 
> Нет никакой голубиной почты, всё через репозиторий. Коммитер закоммитил своё творчество, не удосужившись предварительно даже просто собрать ядро.

А как же почтовая рассылка?
Где в message 25/43 вам могли написать серьезное замечание, а в message 30/43 попросить заменить color на colour и побыстрее так как это оскорбляет чьи то чувства.

> Этот коммитер точно так же не стал бы прогонять тесты и результат был бы тот же, несмотря на все ваши новомодные тесты и сиаи.

Такой же, да не совсем.

> А если ваша апелляция к тому, что что-то сфейлилось бы на билд-сервере, так оно по сути и сфейлилось - Линус всё проверил и в релиз это не ушло.

Ну давайте Линус будет билдить каждый мердж-риквест.
У него наверное времени много.

> Ну и про юнит тесты - было бы интересо послушать ваше мнение подробнее.

Я могу просто поделиться своим опытом.
Проект на 10 лямов LOC.
Есть базовые юнит тесты, есть расширенный набор который бегает раз в несколько дней.
Плюс чеки на сборку под 3 платформы.

Если чеки не прошли (не собралось) то даже добавить ревьюверов можно только специальной командой.
Если оно не собралось - кнопка мердж не активна.
Если тесты не прошли - кнопка мердж не активна.
Если минимум 2 ревьювера не поставили свои апрувы - да кнопка тоже не активна)

До Линуса оно даже не дошло бы.
CI попробовал бы собрать, не смог и отправил горе пограммисту письмо "твой овнокод даже не собрался!"

И это бы предотвратило такие ситуации opennet.me/opennews/art.shtml?num=62555
"патч ... был подан нестандартно - но был принят, хотя не получил ни одного подтверждения (ACK) от мэйнтайнеров"

> Как вы предполагаете объединять неповоротливость юнит-тестов и неконтролируемую скорость разработки ядра?

Тяжело)) Как и всё что связано с разработкой ядра.
Я бы выделил отдельные подсистемы и поставил задачу для их разработчиков покрыть их тестами. Далеко подсистемы не все меняются активно.
Это будет долгий процесс, но ядро не год делается.

Как минимум стоит посмотреть Linux Test Project и kernel Autotest и узнать какие у них трудности.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 17:38 
Claudes C Compiler использовать для компиляции пробовал? Может этот нейрослоп был для компиляции им заточен?

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено нах. , 12-Фев-26 17:41 
может кто-то в промпте забыл сказать "и оно должно компилироваться!"
бывает. Или модель проигнорировала указивку где-то в середине текста (они у штирлица учились и прекрасно знают что помнить надо первое и последнее предложения, а как размножаются йожики можно сразу забыть)


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 17:54 
Ну, и х@м этот Линус.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Doctor , 12-Фев-26 18:08 
Троллинг засчитан.
По чесноку, если говорить прямо, то всё правильно сделал.
Ладно тестирование, ладно юниттесты, ладно CI, но можно просто попытаться скомпилить это на самом мин. для этого железе и т.п.?
Как и сказали комментаторы выше, попахивает очередным нейрослопом, или хотя бы автоматической отправкой PR.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:15 
> Компиляция завершалась ошибкой из-за повторного переопределения переменной.
> но можно просто попытаться скомпилить

Я не в теме, поэтому вопрос. А че, статических анализаторов кода через какой-нибудь LSP для Си нет, которые еще на этапе редактирования такие простейшие моменты найдут?


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:18 
Они конечно есть.
Но на них Настоящие Ядреные Пограммисты кладут болт.

Чтобы заставить ленивых м--ов что-то сделать, нужно вводить обязаловку.
Не прошли чеки на сборку? Иди в пень, даже ревьюверов не сможешь добавить, чтобы их лишний раз не отвлекать.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:27 
Товальдс, молодец).

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:11 
Я за первый случай коммита несобирающегося кода депремировал, за второй штрафовал + депремирование, на третий увольнял. А ругаться бессмысленно.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:20 
Самого за бытность джуном заставляли на каждом дейли коммитить собирающийся и рабочий код в репозиторий?

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:36 
Кого ты собираешься штрафовать? линусу бесплатно в ядро комьюнят

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено ятупойтролль , 12-Фев-26 18:54 
уже лет 15 не бесплатно, вокруг ядра собрались крупные корпорации, которым плевать на линукс, им нужна только прибыль. отправить нейрослоп для них плевое дело потому, что это дешевле найма настоящего разраба на сях.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 19:11 
Разве корпорация г** отправила , а не частный любитель во всяких gpt сидеть ?

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено нах. , 12-Фев-26 19:32 
там адресок @linaro - т.е. не то чтоб совсем частный любитель, скорее - проедатель донатов. Не прокатилло, денежки наивных л=в переходят в другие карманцы.

в корпорациях помимо пряников еще и кнут предусмотрен, если ты не удосужишься даже проверить что оно собирается.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 19:12 
Это как-то опровергает утверждение, что Линусу коммитят бесплатно, и как следствие он не может штрафовать на зарплату?

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено нах. , 12-Фев-26 19:34 
> Это как-то опровергает утверждение, что Линусу коммитят бесплатно, и как следствие он
> не может штрафовать на зарплату?

ну он в целом именно это и сделал - запрет на прием изменений от неудачника = хрен он получит очередные денежки линары. Потому что те платят все же за успешные успехи а не за потраченное время.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 19:33 
Расскажите как уволить человека? я на полном серьёзе, мне интересна юридическая сторона. Если вы конечно не в США.

"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено нах. , 12-Фев-26 19:39 
> Расскажите как уволить человека? я на полном серьёзе, мне интересна юридическая сторона.
> Если вы конечно не в США.

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

Можно поиграть с мышкой - сперва предупреждение о неполном, с занесением в трудовую. Минус премии и прочие не явно гарантированные трудовым договором радости. Чаще всего в современной реальности это треть зарплаты. Через пол-года вторая аттестация. Второе предупреждение.
На третьем - увольнение.

Оспорить такое в суде - желаю всяческих узбеков. Найти с такими записями работу - ну тоже интересно.

Если бы твое руководство на самом деле интересовало качество нанимаемого персонала - были бы еще аттестации по итогам испытательного срока, где можно уволить сразу и без объяснений.


"Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."
Отправлено Аноним , 12-Фев-26 18:22 
Ещё кто-то желает комьютить в ядро?