摘要:这一次,Torvalds 的怒火主要集中在 Linux 6.15 版本中的一段测试代码——hdrtest。他在邮件中直言不讳地表达了不满,甚至用上了 “disgusting(恶心)” 和 “turd(垃圾文件)” 这样的词汇——那么,hdrtest 究竟是什么
“这玩意儿必须被干掉!”
“别让所有人都看到这个‘恶心’的东西,让整个源码树变得一团糟!”
没错,Linux 之父 Linus Torvalds 又“发飙”了!
这一次,Torvalds 的怒火主要集中在 Linux 6.15 版本中的一段测试代码——hdrtest。他在邮件中直言不讳地表达了不满,甚至用上了 “disgusting(恶心)” 和 “turd(垃圾文件)” 这样的词汇——那么,hdrtest 究竟是什么?
最近,Linux 6.15 的大规模开源图形驱动(DRM)更新被合并到内核代码仓库,然而这次合并却引发了一场风波:Torvalds 在合并代码后,发现了一段新的 hdrtest 代码,而它的存在不仅拖慢了内核构建速度,还会在系统中留下“垃圾文件”。
这让 Torvalds 愤怒不已,于是在 Linux 内核开发邮件列表上发表了一封措辞极具其个人特色的邮件,其中写道:
“我完成了拉取,解决了一些(简单的)冲突,但我发现里面竟然包含了恶心的'hdrtest'代码:
(a)拖慢构建速度,因为这玩意儿被纳入了 allmodconfig 的常规构建,而不是作为一个按需运行的单独测试工具。
(b)在 include 目录里留下了随机的 hdrtest 垃圾文件。”
更让 Torvalds 恼火的是,他发现这些“垃圾文件”不仅影响了 Git 操作,还破坏了 Linux 内核开发中常用的文件名补全功能,让开发体验大打折扣。对此,Torvalds 愤怒质问道:“为什么要把这个测试代码作为常规构建的一部分?” “别让所有人都看到这个“恶心”的东西,让整个源码树变得一团糟!”
许多开发者在看到 Torvalds 的邮件后,第一反应是:“HDR 代码?是与 Linux 桌面高动态范围(HDR)显示技术相关的新测试?”——事实并非如此。
hdrtest 代码并不是 “高动态范围(High Dynamic Range)” 相关,而是与 C 语言“头文件(Header Files)” 相关的测试工具。它主要用于 Intel Xe 内核驱动,目的是检查 DRM 头文件的完整性,确保它们可以正确通过 kernel-doc 规范。
虽然这个测试代码本身有其技术价值,但按照 Torvalds 的说法,它也带来了不少问题:
(1)它被强制纳入内核的常规构建流程,导致所有 Linux 内核开发者在编译时都要额外运行它,浪费时间。
(2)它会在 include 目录下留下许多无用的测试文件,而这些文件没有被 .gitignore 忽略,导致 Git 提示开发者有未跟踪的文件,影响工作流。
(3)它还破坏了终端的文件名补全功能,让开发者在使用 Bash 进行文件操作时遇到不必要的干扰。
换句话说,这段 hdrtest 代码本意是为了优化 Linux 内核开发体验,但其实现方式太过粗暴,反而恶化了开发体验,最终触怒了 Torvalds——要知道,这位大神平时对烂代码的容忍度本就极低。
“如果你们非要搞这个 hdrtest 测试代码,就把它放在你们自己的检查流程里。别让所有人都看到这个“恶心”的东西,让整个源码树变得一团糟!
我建议不要把这部分放进 Kconfig 设置和常规构建,而是做成你们可以单独运行的测试(比如'make drm-hdrtest'这种形式,而不是常规构建的一部分)。”
同时,Torvalds 表示他已经直接将 hdrtest 标记为损坏(BROKEN),让其无法运行:“我现在就把它标记为 BROKEN 来禁用,你们自己想办法解决,但别强迫其他人也接受这堆垃圾。”
不仅如此,Torvalds 最后还补充道,即便是简单地将这些“垃圾文件”添加到 .gitignore 也不能真正解决问题,因为这只是掩盖了问题,并没有真正移除它。
参考链接:https://www.phoronix.com/news/Linux-6.15-hdrtest-Turd
来源:不秃头程序员一点号