Торвальдс обозвал %^!@$% код Intel Xe для DRM в Linux 6.8, сам исправил его и призвал разработчиков тестировать патчи

12 января 2024 года Линус Торвальдс обозвал (https://habr.com/ru/news/786196/) %^!@$% новый код Intel Xe для DRM (Direct Rendering Manager) в Linux 6.8. Он сам исправил его огрехи и призвал сторонних разработчиков и мейнтейнеров проекта тестировать, тестировать и ещё раз тестировать свои патчи.

В списке рассылки Торвальдс выразил (https://lore.kernel.org/dri-devel/CAHk-=wgPJttFz8yrdpPTN-ypMmDXHOKw9yi1nZSEq+7+tGftZA@mail.gmail.com/) разочарование по поводу тестирования, которого «серьёзно не хватает», когда дело касается фрагментов нового кода GPU Intel Xe. В итоге присланные коммиты от разработчиков даже не билдятся, так как там изменили тип переменной intel_wakeref_t с depot_stack_handle_t на unsigned long, а в итоге посыпались ошибки и сборка завершается с ошибкой. Также в файлах заголовков drm есть эта отвратительная вещь: файл header включает в себя файл C, что противно.

«Я пересобрал патч, исправив сломанный заголовочный файл совместимости с Xe, но это определенно НЕ то, как все должно было работать. Как это вообще до меня дошло без какого-либо тестирования сборки? И почему %^!@$% файл заголовка включает файл C? Это неправильно, независимо от этой ошибки», — заявил Торвальдс.

Торвальдс обозвал %^!@$% код Intel Xe для DRM в Linux 6.8, сам исправил его и призвал разработчиков тестировать патчи 12 января 2024 года Линус Торвальдс обозвал (https://habr.com/ru/news/786196/) %^!@$% новый код Intel Xe для DRM (Direct Rendering Manager) в Linux 6.8. Он сам исправил его огрехи и призвал сторонних разработчиков и мейнтейнеров проекта тестировать, тестировать и ещё раз тестировать свои патчи. В списке рассылки Торвальдс выразил (https://lore.kernel.org/dri-devel/CAHk-=wgPJttFz8yrdpPTN-ypMmDXHOKw9yi1nZSEq+7+tGftZA@mail.gmail.com/) разочарование по поводу тестирования, которого «серьёзно не хватает», когда дело касается фрагментов нового кода GPU Intel Xe. В итоге присланные коммиты от разработчиков даже не билдятся, так как там изменили тип переменной intel_wakeref_t с depot_stack_handle_t на unsigned long, а в итоге посыпались ошибки и сборка завершается с ошибкой. Также в файлах заголовков drm есть эта отвратительная вещь: файл header включает в себя файл C, что противно. «Я пересобрал патч, исправив сломанный заголовочный файл совместимости с Xe, но это определенно НЕ то, как все должно было работать. Как это вообще до меня дошло без какого-либо тестирования сборки? И почему %^!@$% файл заголовка включает файл C? Это неправильно, независимо от этой ошибки», — заявил Торвальдс.
Нравится
2
·644 Visualizações ·0 Anterior