“这玩意儿必须被干掉!”
“别让所有人都看到这个
‘恶心’
的东西,让整个源码树变得一团糟!”
没错,
Linux 之父
Linus Torvalds 又“
发飙
”了!
这一次,
Torv
ald
s
的怒火主要集中在 Linux 6.15 版本中的一段测试代码——hdrtest。他在邮件中直言不讳地表达了不满,甚至用上了 “disgusting(恶心)” 和
“tu
r
d
(垃圾文件)” 这样的词汇
——
那么,hdrtest 究竟是什么?
具体
发生了什么?
最近,Linux 6.15 的大规模开源图形驱动(DRM)更新被合并到内核代码仓库,然而这次合并却引发了一场风波:Torvalds 在合并代码后,发现了一段新的 hdrtest 代码,而它的存在不仅拖慢了内核构建速度,还会在系统中留下“垃圾文件”。
这让 Torvalds 愤怒不已,
于是
在 Linux 内核开发邮件列表上发表了一封措辞极具其个人特色的
邮件,其中写道:
“我完成了拉取,解决了一些(简单的)冲突,但我发现里面竟然包含了恶心的'hdrtest'代码:
(a)拖慢构建速度,因为这玩意儿被纳入了 allmodconfig 的常规构建,而不是作为一个按需运行的单独测试工具。
(b)在 include 目录里留下了随机的 hdrtest 垃圾文件。”
更让 Torvalds 恼火的是,他发现这些“垃圾文件”不仅影响了 Git 操作,还破坏了 Linux 内核开发中常用的文件名补全功能,让开发体验大打折扣。对此
,
Torvalds
愤怒质问道:“为什么
要把这个测试代码作为常规构建的一部分?”
“别让所有人都看到这个“恶心”的东西,让整个源码树变得一团糟!”
hdrtest 代码是什么,为何激怒 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
——
要知道,这位大神平时对烂代码的容忍度本就极低。
Torvalds 的解决方案:自己玩去,别影响别人!
面对这个问题,Torvalds 并没有给 hdrtest 留下任何回旋余地,他明确表示:
“如果你们非要搞这个 hdrtest 测试代码,就把它放在你们自己的检查流程里。别让所有人都看到这个“恶心”的东西,让整个源码树变得一团糟!
我建议不要把这部分放进 Kconfig 设置和常规构建,而是做成你们可以单独运行的测试(比如'make drm-hdrtest'这种形式,而不是常规构建的一部分)。”
同时
,
Torvalds 表示他已经直接
将 hdrtest 标记为损坏(BROKEN),让其无法运行:“
我现在就把它标记为 BROKEN 来禁用,你们自己想办法解决,但别强迫其他人也接受这堆垃圾
。”
不仅如此
,
Torvalds 最后
还补充道,
即便是简单地将这些“垃圾文件”添加到 .gitignore 也不能真正解决问题,因为这只是掩盖了问题,并没有真正移除它。
参考链接
:
https://www.phoronix.com/news/Linux-6.15-hdrtest-Turd
好啦,今天的内容分享就到这,感觉不错的同学记得分享点赞哦!
PS:
程序员好物馆
持续分享程序员学习、面试相关干货,不见不散!