- 19,791
- 44
- 8 Ноя 2022
Одна строчка с макросом стала символом провала первой партии патчей RISC-V.
Линус Торвальдс резко раскритиковал первую партию патчей для архитектуры RISC-V, предложенных для включения в Linux 6.17. Поводом стало то, что изменения пришли слишком поздно и содержали, по его словам, «мусор», не имеющий отношения к RISC-V и затрагивающий общие заголовочные файлы ядра.
Особое раздражение у него вызвал новый вспомогательный макрос <code>make_u32_from_two_u16()</code>, который, по мнению Торвальдса, делает код менее понятным и только ухудшает ситуацию. Он отметил, что простая запись вида <code>(a << 16) + b</code> сразу показывает, что происходит, тогда как использование «хелпера» скрывает порядок слов и вводит двусмысленность.
Торвальдс подчеркнул, что подобные изменения не должны появляться в общих заголовках, а тем более вноситься на завершающем этапе окна слияния. Он предупредил, что больше не примет поздних запросов на включение изменений и не допустит «мусора» за пределами дерева архитектуры RISC-V.
По его словам, авторы смогут повторно попытаться внести эти изменения только в версии 6.18 и только в самом начале окна слияния, причём уже без спорных и бесполезных правок.
Линус Торвальдс резко раскритиковал первую партию патчей для архитектуры RISC-V, предложенных для включения в Linux 6.17. Поводом стало то, что изменения пришли слишком поздно и содержали, по его словам, «мусор», не имеющий отношения к RISC-V и затрагивающий общие заголовочные файлы ядра.
Особое раздражение у него вызвал новый вспомогательный макрос <code>make_u32_from_two_u16()</code>, который, по мнению Торвальдса, делает код менее понятным и только ухудшает ситуацию. Он отметил, что простая запись вида <code>(a << 16) + b</code> сразу показывает, что происходит, тогда как использование «хелпера» скрывает порядок слов и вводит двусмысленность.
Торвальдс подчеркнул, что подобные изменения не должны появляться в общих заголовках, а тем более вноситься на завершающем этапе окна слияния. Он предупредил, что больше не примет поздних запросов на включение изменений и не допустит «мусора» за пределами дерева архитектуры RISC-V.
По его словам, авторы смогут повторно попытаться внести эти изменения только в версии 6.18 и только в самом начале окна слияния, причём уже без спорных и бесполезных правок.
- Источник новости
- www.securitylab.ru
