The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Линус Торвальдс отверг изменения к подсистеме MMC из-за низкого качества кода"
Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Изначальное сообщение [ Отслеживать ]

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

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

Ответить | Правка | Cообщить модератору

Оглавление

Сообщения [Сортировка по времени | RSS]


3. Скрыто модератором  +5 +/
Сообщение от Аноним (3), 12-Фев-26, 17:15 
Ответить | Правка | Наверх | Cообщить модератору

4. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +12 +/
Сообщение от Аноним (4), 12-Фев-26, 17:18 
Нейрослоп наверняка.
Ответить | Правка | Наверх | Cообщить модератору

10. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +1 +/
Сообщение от вайбкодер (?), 12-Фев-26, 17:43 
Не гони на прохресс!
Ответить | Правка | Наверх | Cообщить модератору

14. Скрыто модератором  +/
Сообщение от НяшМяш (ok), 12-Фев-26, 17:54 
Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

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

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

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

Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

28. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +/
Сообщение от Дмитрий (??), 12-Фев-26, 18:26 
"Нейрослоп" хорошо решается залогом:
К патче прикреплаешь например 1 бикойн
Если код плохой Биткойн не возвращается. Хороший - возвращается.
Ответить | Правка | К родителю #4 | Наверх | Cообщить модератору

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

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

Ответить | Правка | Наверх | Cообщить модератору

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

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

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

Ответить | Правка | Наверх | Cообщить модератору

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

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

Ответить | Правка | Наверх | Cообщить модератору

12. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +/
Сообщение от Аноним (12), 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 новых тестов в день. Ему проще перед релизом вручную протыкать за пару недель весь функционал, чем потратить полгода, чтобы переделать все эти тесты под новую логику. Как вы предполагаете объединять неповоротливость юнит-тестов и неконтролируемую скорость разработки ядра?

Ответить | Правка | К родителю #6 | Наверх | Cообщить модератору

23. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +1 +/
Сообщение от Аноним (23), 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 и узнать какие у них трудности.

Ответить | Правка | Наверх | Cообщить модератору

43. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +/
Сообщение от нах. (?), 12-Фев-26, 19:52 
> А как же почтовая рассылка?

там _уважаемый_, ему можно (было) в пулреквесты

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

ну раз в его чудо-dvcs нельзя сделать это автоматически...

> Я могу просто поделиться своим опытом.

у твоего проекта либо не git, либо что-то странное используется. Т.е. вместо полноценной dvcs вы ухитрились гит свести к централизованной. А, ну судя по каким-то "merge" вместо pull request - так и есть.

У линуса на это просто нет ресурсов. Эти вот васянские репо с которыми предлагается пулл - ему не подконтрольны.

> И это бы предотвратило такие ситуации opennet.me/opennews/art.shtml?num=62555

тоже нет.

> "патч ... был подан нестандартно - но был принят, хотя не получил

т.е. у этого чувака были бы и права пооверрайдить "неактивную кнопку".

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

А они бы тебя и послали бы... оооот туда. Потому что зарплату им если и платят, то это точно не ты.

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

никаких кроме той что васян и не думал подключать к чему-то подобное СВОЙ васянский репо. Так работает гит.

Ответить | Правка | Наверх | Cообщить модератору

7. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  –3 +/
Сообщение от Аноним (7), 12-Фев-26, 17:38 
Claudes C Compiler использовать для компиляции пробовал? Может этот нейрослоп был для компиляции им заточен?
Ответить | Правка | Наверх | Cообщить модератору

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

Ответить | Правка | Наверх | Cообщить модератору

13. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  –3 +/
Сообщение от Аноним (13), 12-Фев-26, 17:54 
Ну, и х@м этот Линус.
Ответить | Правка | Наверх | Cообщить модератору

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

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

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

Ответить | Правка | Наверх | Cообщить модератору

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

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

Ответить | Правка | Наверх | Cообщить модератору

30. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +/
Сообщение от Аноним (30), 12-Фев-26, 18:27 
Товальдс, молодец).
Ответить | Правка | К родителю #13 | Наверх | Cообщить модератору

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

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

34. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +/
Сообщение от Аноним (34), 12-Фев-26, 18:36 
Кого ты собираешься штрафовать? линусу бесплатно в ядро комьюнят
Ответить | Правка | К родителю #22 | Наверх | Cообщить модератору

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

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

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

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

Ответить | Правка | Наверх | Cообщить модератору

37. Скрыто модератором  +/
Сообщение от Аноним (37), 12-Фев-26, 19:12 
Ответить | Правка | К родителю #35 | Наверх | Cообщить модератору

41. Скрыто модератором  +/
Сообщение от нах. (?), 12-Фев-26, 19:34 
Ответить | Правка | Наверх | Cообщить модератору

40. "Линус Торвальдс отверг изменения к подсистеме MMC из-за низк..."  +/
Сообщение от Аноним (40), 12-Фев-26, 19:33 
Расскажите как уволить человека? я на полном серьёзе, мне интересна юридическая сторона. Если вы конечно не в США.
Ответить | Правка | К родителю #22 | Наверх | Cообщить модератору

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

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

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

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

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

Ответить | Правка | Наверх | Cообщить модератору

27. Скрыто модератором  +/
Сообщение от Аноним (-), 12-Фев-26, 18:22 
Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2026 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру