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

4. Сообщение от Аноним (4), 12-Фев-26, 17:18   +13 +/
Нейрослоп наверняка.
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #10, #14, #18, #28

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

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

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #5 Ответы: #8, #12, #46

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

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

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

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

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

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

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

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #6 Ответы: #23

13. Сообщение от Аноним (13), 12-Фев-26, 17:54   –5 +/
Ну, и х@м этот Линус.
Ответить | Правка | Наверх | Cообщить модератору
Ответы: #20, #30, #44

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

18. Сообщение от Аноним (18), 12-Фев-26, 18:05   +3 +/
> Нейрослоп наверняка.

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

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

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

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

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

23. Сообщение от Аноним (23), 12-Фев-26, 18:14   +1 +/
> Нет никакой голубиной почты, всё через репозиторий. Коммитер закоммитил своё творчество, не удосужившись предварительно даже просто собрать ядро.

А как же почтовая рассылка?
Где в 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ообщить модератору
Родитель: #12 Ответы: #43

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

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #20 Ответы: #25

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

41. Сообщение от нах. (?), 12-Фев-26, 19:34   +/
> Это как-то опровергает утверждение, что Линусу коммитят бесплатно, и как следствие он
> не может штрафовать на зарплату?

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

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

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

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

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

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

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

Ответить | Правка | Наверх | Cообщить модератору
Родитель: #40 Ответы: #48

43. Сообщение от нах. (?), 12-Фев-26, 19:52   +/
> А как же почтовая рассылка?

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

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

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

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

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

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

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

тоже нет.

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

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

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

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

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

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

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

44. Сообщение от Аноним (-), 12-Фев-26, 20:07    Скрыто ботом-модератором+/
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #13

45. Сообщение от Аноним (45), 12-Фев-26, 20:09   +/
Попахивает истеричками на самом деле, ну, вполне в духе Линуса. Факап тут в отсталом пайплайне, я согласен с этим.
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #20

46. Сообщение от Аноним (46), 12-Фев-26, 20:10   +/
> нормальных проектах CI проверил сборку ветки, пробежали юнит и прочие тесты.

https://patchwork.kernel.org/project/netdevbpf/list/

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

47. Сообщение от мелстрой (?), 12-Фев-26, 20:17   +/
Я думаю нужна практика линчевания, плохой код? Отпилят палец. Не компилится - руку, а за уязвимости, повесят за причинное местт
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #28

48. Сообщение от мелстрой (?), 12-Фев-26, 20:21   +/
А спрашивают потом, Зачем манагерам из галер процессы саботировать?
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #42

49. Сообщение от Аноним (49), 12-Фев-26, 20:24   +/
По методу Paul Le Roux. Ну или Саддамки. Или Виссарионыча. Этот метод практиковали многие...
Ответить | Правка | Наверх | Cообщить модератору
Родитель: #40


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

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




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

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