Intel Xe驅(qū)動(dòng)代碼嚴(yán)重缺乏測(cè)試

Linux 6.8 的Direct Rendering Manager (DRM)內(nèi)核圖形 / 顯示驅(qū)動(dòng)程序更新包括新的英特爾 "Xe"DRM 和 PowerVR Imagination 驅(qū)動(dòng)程序、實(shí)驗(yàn)形式的 AMD 色彩管理屬性、Raspberry Pi 5 圖形支持等。 但 Linus Torvalds 最近在合并相關(guān)代碼時(shí)卻發(fā)現(xiàn)，一些新提交的 Intel Xe 驅(qū)動(dòng)程序代碼 "嚴(yán)重缺乏" 測(cè)試。對(duì)此，他在內(nèi)核郵件列表中表達(dá)了自己的不滿：

你的測(cè)試嚴(yán)重不足。甚至無法構(gòu)建，原因似乎在于該 commit b49e894c3fd8 ("drm/i915: Replace custom intel runtime_pm tracker with ref_tracker library")將 "intel_wakeref_t" 類型從 "deep_stack_handle_t" 改為了 "unsigned long"...... 真令人不悅。我已經(jīng)修復(fù)了那個(gè)損壞的 Xe compat 頭文件并完成了構(gòu)建，但這絕對(duì)不是事情的本來應(yīng)該有的樣子。我怎么會(huì)遇到這種情況？竟然會(huì)沒有進(jìn)行任何構(gòu)建測(cè)試。 為什么 %^!@$% 頭文件會(huì)包含 C 文件？無論如何，這個(gè)錯(cuò)誤都不應(yīng)該發(fā)生。

與以前的一些 “火爆” 回復(fù)相比，Linus 這次的措辭可以說是算的上溫和，并且也提出了一些合理的問題。事實(shí)上，在 2023 年底的 Linux 基金會(huì)的日本開源峰會(huì)上，Linus 就表示自己已經(jīng)收斂了脾氣，在吸取了一些教訓(xùn)之后不會(huì)再 “對(duì)一些公司豎中指”。 但也正如 Phoronix 所言，無論如何 Linus 已經(jīng)將新代碼合并到 Linux 6.8 中。希望這只是 Intel Xe 驅(qū)動(dòng)程序的一個(gè) one-off issue，而沒有更大的代碼質(zhì)量問題。

審核編輯：黃飛