blob: f509e077e1cb08ebd549632499dc16c4a752c9e1 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
|
.. include:: ../disclaimer-zh_CN.rst
:Original: :ref:`Documentation/process/6.Followthrough.rst <development_followthrough>`
:Translator: Alex Shi <alex.shi@linux.alibaba.com>
.. _cn_development_followthrough:
跟进
====
在这一点上,您已经遵循了到目前为止给出的指导方针,并且,随着您自己的工程技能
的增加,已经发布了一系列完美的补丁。即使是经验丰富的内核开发人员也能犯的最大
错误之一是,认为他们的工作现在已经完成了。事实上,发布补丁意味着进入流程的下
一个阶段,可能还需要做很多工作。
一个补丁在第一次发布时就非常出色,没有改进的余地,这是很罕见的。内核开发流程
认识到这一事实,因此,它非常注重对已发布代码的改进。作为代码的作者,您应该与
内核社区合作,以确保您的代码符合内核的质量标准。如果不参与这个过程,很可能会
阻止将补丁包含到主线中。
与审阅者合作
------------
任何意义上的补丁都会导致其他开发人员在审查代码时发表大量评论。对于许多开发
人员来说,与审查人员合作可能是内核开发过程中最令人生畏的部分。但是,如果你
记住一些事情,生活会变得容易得多:
- 如果你已经很好地解释了你的补丁,评论人员会理解它的价值,以及为什么你会
费尽心思去写它。但是这个并不能阻止他们提出一个基本的问题:五年或十年后
用这个代码维护一个内核会是什么感觉?你可能被要求做出的许多改变——从编码风格
的调整到大量的重写——都来自于对Linux的理解,即从现在起十年后,Linux仍将在
开发中。
- 代码审查是一项艰苦的工作,这是一项相对吃力不讨好的工作;人们记得谁编写了
内核代码,但对于那些审查它的人来说,几乎没有什么持久的名声。因此,评论
人员可能会变得暴躁,尤其是当他们看到同样的错误被一遍又一遍地犯下时。如果
你得到了一个看起来愤怒、侮辱或完全冒犯你的评论,抵制以同样方式回应的冲动。
代码审查是关于代码的,而不是关于人的,代码审查人员不会亲自攻击您。
- 同样,代码审查人员也不想以牺牲你雇主的利益为代价来宣传他们雇主的议程。
内核开发人员通常希望今后几年能在内核上工作,但他们明白他们的雇主可能会改
变。他们真的,几乎毫无例外地,致力于创造他们所能做到的最好的内核;他们并
没有试图给雇主的竞争对手造成不适。
所有这些归根结底都是,当审阅者向您发送评论时,您需要注意他们正在进行的技术
观察。不要让他们的表达方式或你自己的骄傲阻止这种事情的发生。当你在一个补丁
上得到评论时,花点时间去理解评论人想说什么。如果可能的话,请修复审阅者要求
您修复的内容。然后回复审稿人:谢谢他们,并描述你将如何回答他们的问题。
请注意,您不必同意审阅者建议的每个更改。如果您认为审阅者误解了您的代码,请
解释到底发生了什么。如果您对建议的更改有技术上的异议,请描述它并证明您对该
问题的解决方案是正确的。如果你的解释有道理,审稿人会接受的。不过,如果你的
解释不能证明是有说服力的,尤其是当其他人开始同意审稿人的观点时,请花些时间
重新考虑一下。你很容易对自己解决问题的方法视而不见,以至于你没有意识到某个
问题根本是错误的,或者你甚至没有解决正确的问题。
Andrew Morton建议,每一条不会导致代码更改的评论都应该导致额外的代码注释;
这可以帮助未来的评论人员避免出现第一次出现的问题。
一个致命的错误是忽视评论,希望它们会消失。他们不会走的。如果您在没有对之前
收到的注释做出响应的情况下重新发布代码,那么很可能会发现补丁毫无用处。
说到重新发布代码:请记住,审阅者不会记住您上次发布的代码的所有细节。因此,
提醒审查人员以前提出的问题以及您如何处理这些问题总是一个好主意;补丁变更
日志是提供此类信息的好地方。审阅者不必搜索列表档案来熟悉上次所说的内容;
如果您帮助他们开始运行,当他们重新访问您的代码时,他们的心情会更好。
如果你已经试着做正确的事情,但事情仍然没有进展呢?大多数技术上的分歧都可以
通过讨论来解决,但有时人们只需要做出决定。如果你真的认为这个决定对你不利,
你可以试着向更高的权力上诉。在这篇文章中,更高的权力倾向于Andrew Morton。
Andrew在内核开发社区中受i很大的尊重;他经常为似乎被绝望地阻塞事情清障。
尽管如此,对Andrew的呼吁不应轻而易举,也不应在所有其他替代方案都被探索之前
使用。当然,记住,他也可能不同意你的意见。
接下来会发生什么
----------------
如果一个补丁被认为是添加到内核中的一件好事,并且一旦大多数审查问题得到解决,
下一步通常是进入子系统维护人员的树中。工作方式因子系统而异;每个维护人员都
有自己的工作方式。特别是,可能有不止一棵树——一棵树,也许,专门用于计划下一
个合并窗口的补丁,另一棵树用于长期工作。
对于应用于没有明显子系统树(例如内存管理修补程序)的区域的修补程序,默认树
通常以-mm结尾。影响多个子系统的补丁也可以最终通过-mm树。
包含在子系统树中可以提高补丁的可见性。现在,使用该树的其他开发人员将默认获
得补丁。子系统树通常也为Linux提供支持,使其内容对整个开发社区可见。在这一点
上,您很可能会从一组新的审阅者那里得到更多的评论;这些评论需要像上一轮那样
得到回答。
在这一点上也会发生什么,这取决于你的补丁的性质,是与其他人正在做的工作发生
冲突。在最坏的情况下,严重的补丁冲突可能会导致一些工作被搁置,以便剩余的补丁
可以成形并合并。另一些时候,冲突解决将涉及到与其他开发人员合作,可能还会
在树之间移动一些补丁,以确保所有的应用都是干净的。这项工作可能是一件痛苦的
事情,但要计算您的福祉:在Linux下一棵树出现之前,这些冲突通常只在合并窗口
中出现,必须迅速解决。现在可以在合并窗口打开之前,在空闲时解决这些问题。
有朝一日,如果一切顺利,您将登录并看到您的补丁已经合并到主线内核中。祝贺你!
然而,一旦庆祝活动完成(并且您已经将自己添加到维护人员文件中),就值得记住
一个重要的小事实:工作仍然没有完成。并入主线带来了自身的挑战。
首先,补丁的可见性再次提高。可能会有新一轮的开发者评论,他们以前不知道这
个补丁。忽略它们可能很有诱惑力,因为您的代码不再存在任何被合并的问题。但是,
要抵制这种诱惑,您仍然需要对有问题或建议的开发人员作出响应。
不过,更重要的是:将代码包含在主线中会将代码交给更大的一组测试人员。即使您
为尚未提供的硬件提供了驱动程序,您也会惊讶于有多少人会将您的代码构建到内核
中。当然,如果有测试人员,也会有错误报告。
最糟糕的错误报告是回归。如果你的补丁导致回归,你会发现很多不舒服的眼睛盯着
你;回归需要尽快修复。如果您不愿意或无法修复回归(其他人都不会为您修复),
那么在稳定期内,您的补丁几乎肯定会被移除。除了否定您为使补丁进入主线所做的
所有工作之外,如果由于未能修复回归而取消补丁,很可能会使将来的工作更难合并。
在处理完任何回归之后,可能还有其他普通的bug需要处理。稳定期是修复这些错误并
确保代码在主线内核版本中的首次发布尽可能可靠的最好机会。所以,请回答错误
报告,并尽可能解决问题。这就是稳定期的目的;一旦解决了旧补丁的任何问题,就
可以开始创建酷的新补丁。
别忘了,还有其他里程碑也可能会创建bug报告:下一个主线稳定版本,当著名的发行
商选择包含补丁的内核版本时,等等。继续响应这些报告是您工作的基本骄傲。但是,
如果这不是足够的动机,那么也值得考虑的是,开发社区会记住那些在合并后对代码
失去兴趣的开发人员。下一次你发布补丁时,他们会以你以后不会在身边维护它为假
设来评估它。
其他可能发生的事情
------------------
有一天,你可以打开你的邮件客户端,看到有人给你寄了一个代码补丁。毕竟,这是
让您的代码公开存在的好处之一。如果您同意这个补丁,您可以将它转发给子系统
维护人员(确保包含一个正确的From:行,这样属性是正确的,并添加一个您自己
的签准),或者回复一个Acked-by,让原始发送者向上发送它。
如果您不同意补丁,请发送一个礼貌的回复,解释原因。如果可能的话,告诉作者需要
做哪些更改才能让您接受补丁。对于代码的编写者和维护者所反对的合并补丁,存在着
一定的阻力,但仅此而已。如果你被认为不必要的阻碍了好的工作,那么这些补丁最
终会经过你身边并进入主线。在Linux内核中,没有人对任何代码拥有绝对的否决权。
除了Linus。
在非常罕见的情况下,您可能会看到完全不同的东西:另一个开发人员发布了针对您
的问题的不同解决方案。在这一点上,两个补丁中的一个可能不会合并,“我的在这里
是第一个”不被认为是一个令人信服的技术论据。如果有人的补丁取代了你的补丁而进
入了主线,那么只有一种方法可以回应你:高兴你的问题得到解决,继续你的工作。
以这种方式把一个人的工作推到一边可能会伤害和气馁,但是在他们忘记了谁的补丁
真正被合并很久之后,社区会记住你的反应。
|