Linus 口吐芬芳:怒斥英特尔工程师提交的代码是“令人作呕的一坨”

Linux 6.15 的开源图形驱动程序更新集已合并,但 Linux 创始人 Linus Torvalds 对这次合并请求并不十分满意。

特别是,他对作为完整内核构建一部分构建的某些新的"hdrtest"测试代码以及它留下的“垃圾”感到不满,并认为这段代码“应该死去”,至少对于非 DRM 驱动程序开发者来说是这样。

根据他们在邮件列表的讨论,这些代码是由英特尔工程师 Jani Nikula 提交的 DRM 驱动相关代码。DRM 是 Linux 内核管理 GPU 渲染的核心子系统,负责硬件加速、视频播放等图形处理任务。

在周五夜晚,Linus Torvalds 在邮件列表上发表评论

“哼。我已经完成了合并,解决了(琐碎的)冲突,但注意到这最终包含了令人厌恶的“hdrtest”垃圾,

(a) 它减慢了构建速度,因为它是为常规 allmodconfig 构建,而不是作为一些简单的东西供你们按需运行

(b) 还会在包含目录中留下随机的 'hdrtest' 废物

人们已经分别对此进行了抱怨。它绝对不应该以这种破损的形式出现在我面前。

为什么这个测试会被当作构建的常规部分来执行?

该死的,我们不应该为依赖项添加随机的废物文件,这会让源代码树变得一团糟。

我注意到它仍然存在的原因是 "git status" 会提醒那些愚蠢的垃圾代码没有被忽略。

但更重要的是,这些垃圾代码还会破坏文件名补全功能!所以,将它们添加到 gitignore 中实际上并不能解决问题,它只会让我更快地注意到。

这东西需要 消失

如果你想做那个 hdrtest 的事情,就把它作为你 自己的 检查的一部分来做。不要让其他人看到那个令人厌恶的一坨 (disgusting turds),并在他们的树中留下这些废物。

我现在就通过将其标记为损坏来禁用它。你们可以自己决定怎么做,但强迫别人看到这些内容并不是解决问题的办法。

我建议你们 *不要* 将这部分内容纳入 Kconfig 设置和常规构建中,而应该是一个 你们 可以在测试中运行的部分(即像“make drm-hdrtest”那样,而不是作为常规构建的一部分)。

Linus

这里的“hdr”是指 C 头文件。新的“hdrtest”代码是为Intel Xe内核驱动编写的,目的是尝试确保 DRM 头文件是自包含的,并且能够通过内核文档测试。对包含的 DRM 头文件进行基本的维护检查,以确保它们都处于良好状态。

后续英特尔工程师 Jani Nikula 在邮件列表中进行了回应,承诺将测试文件移至.hdrtest 子目录,并通过 kconfig 选项隔离额外检查项。

猜你喜欢

转载自www.oschina.net/news/342394/linux-6-15-hdrtest-turd
今日推荐