11
原来 Linux 内核贡献第二是这么来的
source link: https://www.v2ex.com/t/784789
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
237 条回复 • 2021-06-22 02:26:22 +08:00
acr0ss 7 小时 47 分钟前 
2
Fact 不可置喙，Opinion 才有讨论空间。
再此，我想提出几个困惑：
1. 某司是否真有内核 PR KPI ？
2. 大材偏偏小用，是个体自由，还是需要杜绝的“浪费”？
3. 追求细节是否有必要？
4. 是单次提交，还是多次提交？如果是多次，为什么不合并提交？
5. 部分人偷懒（偷懒前有贡献），能否否定整个组织？
这些问题的是否，显然会极大影响我的判断。
再此，我想提出几个困惑：
1. 某司是否真有内核 PR KPI ？
2. 大材偏偏小用，是个体自由，还是需要杜绝的“浪费”？
3. 追求细节是否有必要？
4. 是单次提交，还是多次提交？如果是多次，为什么不合并提交？
5. 部分人偷懒（偷懒前有贡献），能否否定整个组织？
这些问题的是否，显然会极大影响我的判断。
nicevar 7 小时 24 分钟前 1
@jhdxr 请问什么，就这个帖子下面的人一堆都是不正常的，嘲讽华为可以，动不动上升到民族和国家是什么臭习惯，你以为所有人都迁就你，看你那些冷嘲热讽的评论？
NSAgold 7 小时 20 分钟前 4
@acr0ss #201
"I'm not saying cleanup is not important, in fact we have routinely cleanups of typos/grammar for btrfs. (And I guess mostly caused by myself?)
Please at least merge all those small fixes into a larger patchset, and with a good cover letter to explain the reason (and auto-tool to do the change if possible) for all the involved maintainers, so that all of us are on the same page."
这是今天的回复的原文（来源： https://lore.kernel.org/linux-btrfs/[email protected]/）
大致机翻如下：“我不是说清理工作不重要，事实上我们对 btrfs 的错别字 /语法进行了例行清理。(我猜大部分是由我自己造成的？)
请至少将所有这些小的修正合并到一个更大的补丁集中，并为所有参与的维护者写一封很好的封面信来解释原因（如果可能的话，还可以用自动工具来进行修改），这样我们所有人都会站在同一战线上。”
因此，应该是多次提交只改动了一个文件几行的 commit 。
从其历史提交可以看出 19 年他在写驱动，21 年之后基本都是 fix typo，而以这种形式来提交，很难让人不觉得他是在刷 KPI 。
类比一下就是新闻网站小编把明明一片长文章能写完的报道，每个自然段单独发一篇新闻，发了洋洋洒洒几十篇，每篇你还得联系上下文。追求细节有必要，但这种大可不必。
"I'm not saying cleanup is not important, in fact we have routinely cleanups of typos/grammar for btrfs. (And I guess mostly caused by myself?)
Please at least merge all those small fixes into a larger patchset, and with a good cover letter to explain the reason (and auto-tool to do the change if possible) for all the involved maintainers, so that all of us are on the same page."
这是今天的回复的原文（来源： https://lore.kernel.org/linux-btrfs/[email protected]/）
大致机翻如下：“我不是说清理工作不重要，事实上我们对 btrfs 的错别字 /语法进行了例行清理。(我猜大部分是由我自己造成的？)
请至少将所有这些小的修正合并到一个更大的补丁集中，并为所有参与的维护者写一封很好的封面信来解释原因（如果可能的话，还可以用自动工具来进行修改），这样我们所有人都会站在同一战线上。”
因此，应该是多次提交只改动了一个文件几行的 commit 。
从其历史提交可以看出 19 年他在写驱动，21 年之后基本都是 fix typo，而以这种形式来提交，很难让人不觉得他是在刷 KPI 。
类比一下就是新闻网站小编把明明一片长文章能写完的报道，每个自然段单独发一篇新闻，发了洋洋洒洒几十篇，每篇你还得联系上下文。追求细节有必要，但这种大可不必。
ELloop 7 小时 19 分钟前 via Android
@atwoodSoInterest 劣币驱逐良币，奖励对应的不是找到宝藏，而是献宝，那么效益最大化的点是输送，也就是抢注抄袭即可，而且如果没有吃的了那就可能会反噬吧。
wazggcd 4 小时 48 分钟前 via Android 12
@Devin 自己的祖国为啥不能黑？我能黑美国，能黑日本，能黑毛子，能黑津巴布韦，当然就能黑中国。我大中国不过是几百个国家中的普通的一个，除了人多没啥特殊的。别把自己的祖国搞得像幼儿园中的问题儿童一样，我的祖国就是一个正常的国家
wazggcd 4 小时 28 分钟前 via Android 13
@kikyous 没有东西能做到让人人都爱，钱都不行，何况一个国家呢。我在这里生活，在这里长大，该交税交税，该上班上班，不害人不违法。我在这里只是因为我出生在这里，习惯了这里，并不代表我就爱这里，也不代表我不爱这里。我也知道太平洋没有盖，但我不游过去，因为我习惯在这里。就是这么简单。
jhdxr 3 小时 25 分钟前
@nicevar 我又看了一下这个帖子的回复。
1. 我对你的回复是我在这个帖子的第一条回复。并没有任何涉及民族 /国家的部分。
2. 在你那条回复之前，除了『辱华』两个字勉强沾边（华也可以解释为华为），其他没有涉及民族 /国家的。就算考虑到可能有被我 block 的人发了相关评论，但考虑到 block 的评论不影响翻页，就算有也是极少数。
所以说，为华为洗地洗的脑子都没了？看不惯冷嘲热讽那一开始就别做那些恶心人的事，再不行就自戳双目去吧。反正华为养的狗，死了也算为环保做贡献了。
1. 我对你的回复是我在这个帖子的第一条回复。并没有任何涉及民族 /国家的部分。
2. 在你那条回复之前，除了『辱华』两个字勉强沾边（华也可以解释为华为），其他没有涉及民族 /国家的。就算考虑到可能有被我 block 的人发了相关评论，但考虑到 block 的评论不影响翻页，就算有也是极少数。
所以说，为华为洗地洗的脑子都没了？看不惯冷嘲热讽那一开始就别做那些恶心人的事，再不行就自戳双目去吧。反正华为养的狗，死了也算为环保做贡献了。
shiji 3 小时 19 分钟前
@chnyuwen
你说的不严谨。
好的个人行为要上升公司，上升国家。 感谢国家感谢组织感谢领导。
坏的个人行为要留给个人，切割关联。 个人觉悟不深，意识不强，愧对公司和国家的培养。
“当你穿着校服的时候，在校外的一举一动都代表着你的学校。” （这句话耳熟吧）
你说的不严谨。
好的个人行为要上升公司，上升国家。 感谢国家感谢组织感谢领导。
坏的个人行为要留给个人，切割关联。 个人觉悟不深，意识不强，愧对公司和国家的培养。
“当你穿着校服的时候，在校外的一举一动都代表着你的学校。” （这句话耳熟吧）
jinliming2 3 小时 5 分钟前 1
@twor2 #161 > 吹就吹，你不信就行了。哪个公司不是拼命吹自己的产品？所以点不在于吹，而是你恶心他打的爱国牌。我们再分析，为什么你恶心爱国牌，细想一下。
细想一下，恶心的并不是爱国牌。
中国民族品牌多了去了，百度上就有列表。在其他品牌那里，没怎么听说“恨国”的，唯独就这个手机行业，风气极差，动不动就“恨国”。
比如通信行业，中国移动、中国电信，国内也有骂的，墙中墙、运营商劫持、资费套餐连环套，但怎么就没有上升到“恨国”这方面来呢？也有人为了使用美国的一些服务，专门申请 T 、V 、GV 之类的美国运营商号码来使用，就是不选电信、移动，这也没上升到“崇洋媚外”的方面去啊。
偏偏就是这个手机行业，乌烟瘴气。为啥呢？
并且在手机品牌里面，华为、小米、OV 、一加……为啥又各不相同呢？
说到底，实际上恶心的不是国产，而是某国产品牌 A 打的爱国牌，让某品牌 A 的“脑残粉”沸腾，然后这些“脑残粉”开始疯狂输出：“爱国就得用某品牌 A”、“用某品牌 B 的都不爱国”、“用某品牌 C 的都是卖器官炫耀的”、“用的每一个品牌 D 都是打在你身上的子弹”，这种捧一踩一的行为才最终产生的对某品牌 A 的绝对反感。
细想一下，恶心的并不是爱国牌。
中国民族品牌多了去了，百度上就有列表。在其他品牌那里，没怎么听说“恨国”的，唯独就这个手机行业，风气极差，动不动就“恨国”。
比如通信行业，中国移动、中国电信，国内也有骂的，墙中墙、运营商劫持、资费套餐连环套，但怎么就没有上升到“恨国”这方面来呢？也有人为了使用美国的一些服务，专门申请 T 、V 、GV 之类的美国运营商号码来使用，就是不选电信、移动，这也没上升到“崇洋媚外”的方面去啊。
偏偏就是这个手机行业，乌烟瘴气。为啥呢？
并且在手机品牌里面，华为、小米、OV 、一加……为啥又各不相同呢？
说到底，实际上恶心的不是国产，而是某国产品牌 A 打的爱国牌，让某品牌 A 的“脑残粉”沸腾，然后这些“脑残粉”开始疯狂输出：“爱国就得用某品牌 A”、“用某品牌 B 的都不爱国”、“用某品牌 C 的都是卖器官炫耀的”、“用的每一个品牌 D 都是打在你身上的子弹”，这种捧一踩一的行为才最终产生的对某品牌 A 的绝对反感。
coolcoffee 2 小时 24 分钟前
某为喜欢享受捧可以，虽然有可能成功，但是无形间影响了各种资源的倾斜，不这么做的公司就会受到损失。
像前段时间鸿蒙所谓的开源发布，很多公司都说即将适配鸿蒙，反正蹭热点又不花钱也不会费力。然后小米、VIVO 等没有任何回应微博都被刷屏刷爆了。
像前段时间鸿蒙所谓的开源发布，很多公司都说即将适配鸿蒙，反正蹭热点又不花钱也不会费力。然后小米、VIVO 等没有任何回应微博都被刷屏刷爆了。
kennylam777 2 小时 1 分钟前
dayeye2006199 1 小时 12 分钟前 1
这类 fix 是有效的，只是提交者的方法有问题。
首先这类清理 debug 信息+typo 可以由机器人来做到，人来提交这类 fix 是比较浪费时间。
第二，这类 fix 最好合并之后作为一个大的 fix 一并提交，并且给出 fix 是如何产生的（比如写明 grep 的规则等），比较节约 reviewer 的时间。
huawei 的工程师提交的 fix 都是有效的，但是提交的方法有问题，对 reviewer 不友好，浪费了别人的时间。但是做的事情没问题。
SUSE 的这位工程师，根据提交者的邮箱后缀对别人下价值判断，这是一个 over-generalize 的做法，是有问题的。这样的社区运营态度是不可取的。但是他提出的建议是合理的（合并之后 review ）。
我觉得不应该过于政治化这样的争论，reviewer 觉得 committer 提交方法有问题，应该提出建议和改进的方法，不应该根据公司就提出价值判断（ KPI 导向，浪费时间等主观词）。
committer 应该注重与人协作的方法，尊重他人时间。
首先这类清理 debug 信息+typo 可以由机器人来做到，人来提交这类 fix 是比较浪费时间。
第二，这类 fix 最好合并之后作为一个大的 fix 一并提交，并且给出 fix 是如何产生的（比如写明 grep 的规则等），比较节约 reviewer 的时间。
huawei 的工程师提交的 fix 都是有效的，但是提交的方法有问题，对 reviewer 不友好，浪费了别人的时间。但是做的事情没问题。
SUSE 的这位工程师，根据提交者的邮箱后缀对别人下价值判断，这是一个 over-generalize 的做法，是有问题的。这样的社区运营态度是不可取的。但是他提出的建议是合理的（合并之后 review ）。
我觉得不应该过于政治化这样的争论，reviewer 觉得 committer 提交方法有问题，应该提出建议和改进的方法，不应该根据公司就提出价值判断（ KPI 导向，浪费时间等主观词）。
committer 应该注重与人协作的方法，尊重他人时间。
ryd994 59 分钟前 via Android
@akring #210 Qu 的意思是这人根本没有用过这段代码。提交的 patch 也没有实际意义。
这是一段测试代码。本身没有生产用途。这个 patch 删了一行分配内存失败后的错误消息。理由是既然你都内存不足了这行消息根本打不出来嘛。
Qu 的意思是你丫根本没做过内核的测试吧？压力测试时主动注入内存分配错误，验证程序在极端资源紧缺情况下的正确性，这是很正常的做法。这时看起来会有各种内存分配失败，但是并没有实际内存不足。
我同意 Qu 。虽然没有测试过 Linux 内核。但是 Windows 内核也是有这个功能的，还是自带的。driver verifier 了解一下。
这是一段测试代码。本身没有生产用途。这个 patch 删了一行分配内存失败后的错误消息。理由是既然你都内存不足了这行消息根本打不出来嘛。
Qu 的意思是你丫根本没做过内核的测试吧？压力测试时主动注入内存分配错误，验证程序在极端资源紧缺情况下的正确性，这是很正常的做法。这时看起来会有各种内存分配失败，但是并没有实际内存不足。
我同意 Qu 。虽然没有测试过 Linux 内核。但是 Windows 内核也是有这个功能的，还是自带的。driver verifier 了解一下。
ryd994 50 分钟前 via Android 1
@dayeye2006199 自称满级大佬还打新手村，抢新人的怪？新手村的怪也是怪，但是打再多也成不了满级大佬。只能获得一万个欢迎来到本游戏的称号。
我们嘲笑的是不是打新手村。而是自称满级大佬这一点。大佬只会默默扫掉一万个小怪。而不会宣传本大佬今天又清理了一万个怪物，为大家做出了巨大贡献。
如果你觉得这些 patch 是有价值的，非常建议你去看一下 Greg 的 I don't want your code 。Linux 社区早就不缺开发者了。最值钱的是管理员的时间。
我们嘲笑的是不是打新手村。而是自称满级大佬这一点。大佬只会默默扫掉一万个小怪。而不会宣传本大佬今天又清理了一万个怪物，为大家做出了巨大贡献。
如果你觉得这些 patch 是有价值的，非常建议你去看一下 Greg 的 I don't want your code 。Linux 社区早就不缺开发者了。最值钱的是管理员的时间。
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK