«Это мусор» — Линус Торвальдс о первой партии патчей RISC-V для Linux 6.17

Линус Торвальдс резко раскритиковал первую партию патчей для архитектуры RISC-V, предложенных для включения в Linux 6.17. Поводом стало то, что изменения пришли слишком поздно и содержали, по его словам, «мусор», не имеющий отношения к RISC-V и затрагивающий общие заголовочные файлы ядра.

Особое раздражение у него вызвал новый вспомогательный макрос make_u32_from_two_u16(), который, по мнению Торвальдса, делает код менее понятным и только ухудшает ситуацию. Он отметил, что простая запись вида (a << 16) + b сразу показывает, что происходит, тогда как использование «хелпера» скрывает порядок слов и вводит двусмысленность.

Торвальдс подчеркнул, что подобные изменения не должны появляться в общих заголовках, а тем более вноситься на завершающем этапе окна слияния. Он предупредил, что больше не примет поздних запросов на включение изменений и не допустит «мусора» за пределами дерева архитектуры RISC-V.

По его словам, авторы смогут повторно попытаться внести эти изменения только в версии 6.18 и только в самом начале окна слияния, причём уже без спорных и бесполезных правок.