1.. SPDX-License-Identifier: GPL-2.0
2
3.. _tw_submittingpatches:
4
5.. include:: ../disclaimer-zh_TW.rst
6
7:Original: :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`
8
9譯者::
10
11        中文版維護者: 鍾宇 TripleX Chung <xxx.phy@gmail.com>
12        中文版翻譯者: 鍾宇 TripleX Chung <xxx.phy@gmail.com>
13                       時奎亮 Alex Shi <alex.shi@linux.alibaba.com>
14        中文版校譯者: 李陽 Li Yang <leoyang.li@nxp.com>
15                       王聰 Wang Cong <xiyou.wangcong@gmail.com>
16                       胡皓文 Hu Haowen <src.res.211@gmail.com>
17
18
19如何讓你的改動進入內核
20======================
21
22對於想要將改動提交到 Linux 內核的個人或者公司來說,如果不熟悉「規矩」,
23提交的流程會讓人畏懼。本文檔收集了一系列建議,這些建議可以大大的提高你
24的改動被接受的機會.
25
26以下文檔含有大量簡潔的建議, 具體請見:
27:ref:`Documentation/process <development_process_main>`
28同樣,:ref:`Documentation/translations/zh_TW/process/submit-checklist.rst <tw_submitchecklist>`
29給出在提交代碼前需要檢查的項目的列表。
30
31其中許多步驟描述了Git版本控制系統的默認行爲;如果您使用Git來準備補丁,
32您將發現它爲您完成的大部分機械工作,儘管您仍然需要準備和記錄一組合理的
33補丁。一般來說,使用git將使您作爲內核開發人員的生活更輕鬆。
34
35
360) 獲取當前源碼樹
37-----------------
38
39如果您沒有一個可以使用當前內核原始碼的存儲庫,請使用git獲取一個。您將要
40從主線存儲庫開始,它可以通過以下方式獲取::
41
42        git clone git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
43
44但是,請注意,您可能不希望直接針對主線樹進行開發。大多數子系統維護人員運
45行自己的樹,並希望看到針對這些樹準備的補丁。請參見MAINTAINERS文件中子系
46統的 **T:** 項以查找該樹,或者簡單地詢問維護者該樹是否未在其中列出。
47
48仍然可以通過tarballs下載內核版本(如下一節所述),但這是進行內核開發的
49一種困難的方式。
50
511) "diff -up"
52-------------
53
54使用 "diff -up" 或者 "diff -uprN" 來創建補丁。
55
56所有內核的改動,都是以補丁的形式呈現的,補丁由 diff(1) 生成。創建補丁的
57時候,要確認它是以 "unified diff" 格式創建的,這種格式由 diff(1) 的 '-u'
58參數生成。而且,請使用 '-p' 參數,那樣會顯示每個改動所在的C函數,使得
59產生的補丁容易讀得多。補丁應該基於內核原始碼樹的根目錄,而不是裡邊的任
60何子目錄。
61
62爲一個單獨的文件創建補丁,一般來說這樣做就夠了::
63
64        SRCTREE=linux
65        MYFILE=drivers/net/mydriver.c
66
67        cd $SRCTREE
68        cp $MYFILE $MYFILE.orig
69        vi $MYFILE      # make your change
70        cd ..
71        diff -up $SRCTREE/$MYFILE{.orig,} > /tmp/patch
72
73爲多個文件創建補丁,你可以解開一個沒有修改過的內核原始碼樹,然後和你自
74己的代碼樹之間做 diff 。例如::
75
76        MYSRC=/devel/linux
77
78        tar xvfz linux-3.19.tar.gz
79        mv linux-3.19 linux-3.19-vanilla
80        diff -uprN -X linux-3.19-vanilla/Documentation/dontdiff \
81                linux-3.19-vanilla $MYSRC > /tmp/patch
82
83"dontdiff" 是內核在編譯的時候產生的文件的列表,列表中的文件在 diff(1)
84產生的補丁里會被跳過。
85
86確定你的補丁里沒有包含任何不屬於這次補丁提交的額外文件。記得在用diff(1)
87生成補丁之後,審閱一次補丁,以確保準確。
88
89如果你的改動很散亂,你應該研究一下如何將補丁分割成獨立的部分,將改動分
90割成一系列合乎邏輯的步驟。這樣更容易讓其他內核開發者審核,如果你想你的
91補丁被接受,這是很重要的。請參閱:
92:ref:`tw_split_changes`
93
94如果你用 ``git`` , ``git rebase -i`` 可以幫助你這一點。如果你不用 ``git``,
95``quilt`` <https://savannah.nongnu.org/projects/quilt> 另外一個流行的選擇。
96
97.. _tw_describe_changes:
98
992) 描述你的改動
100---------------
101
102描述你的問題。無論您的補丁是一行錯誤修復還是5000行新功能,都必須有一個潛在
103的問題激勵您完成這項工作。讓審稿人相信有一個問題值得解決,讓他們讀完第一段
104是有意義的。
105
106描述用戶可見的影響。直接崩潰和鎖定是相當有說服力的,但並不是所有的錯誤都那麼
107明目張胆。即使在代碼審查期間發現了這個問題,也要描述一下您認爲它可能對用戶產
108生的影響。請記住,大多數Linux安裝運行的內核來自二級穩定樹或特定於供應商/產品
109的樹,只從上游精選特定的補丁,因此請包含任何可以幫助您將更改定位到下游的內容:
110觸發的場景、DMESG的摘錄、崩潰描述、性能回歸、延遲尖峯、鎖定等。
111
112量化優化和權衡。如果您聲稱在性能、內存消耗、堆棧占用空間或二進位大小方面有所
113改進,請包括支持它們的數字。但也要描述不明顯的成本。優化通常不是免費的,而是
114在CPU、內存和可讀性之間進行權衡;或者,探索性的工作,在不同的工作負載之間進
115行權衡。請描述優化的預期缺點,以便審閱者可以權衡成本和收益。
116
117一旦問題建立起來,就要詳細地描述一下您實際在做什麼。對於審閱者來說,用簡單的
118英語描述代碼的變化是很重要的,以驗證代碼的行爲是否符合您的意願。
119
120如果您將補丁描述寫在一個表單中,這個表單可以很容易地作爲「提交日誌」放入Linux
121的原始碼管理系統git中,那麼維護人員將非常感謝您。見 :ref:`tw_explicit_in_reply_to`.
122
123每個補丁只解決一個問題。如果你的描述開始變長,這就表明你可能需要拆分你的補丁。
124請見 :ref:`tw_split_changes`
125
126提交或重新提交修補程序或修補程序系列時,請包括完整的修補程序說明和理由。不要
127只說這是補丁(系列)的第幾版。不要期望子系統維護人員引用更早的補丁版本或引用
128URL來查找補丁描述並將其放入補丁中。也就是說,補丁(系列)及其描述應該是獨立的。
129這對維護人員和審查人員都有好處。一些評審者可能甚至沒有收到補丁的早期版本。
130
131描述你在命令語氣中的變化,例如「make xyzzy do frotz」而不是「[這個補丁]make
132xyzzy do frotz」或「[我]changed xyzzy to do frotz」,就好像你在命令代碼庫改變
133它的行爲一樣。
134
135如果修補程序修復了一個記錄的bug條目,請按編號和URL引用該bug條目。如果補丁來
136自郵件列表討論,請給出郵件列表存檔的URL;使用帶有 ``Message-ID`` 的
137https://lore.kernel.org/ 重定向,以確保連結不會過時。
138
139但是,在沒有外部資源的情況下,儘量讓你的解釋可理解。除了提供郵件列表存檔或
140bug的URL之外,還要總結需要提交補丁的相關討論要點。
141
142如果您想要引用一個特定的提交,不要只引用提交的 SHA-1 ID。還請包括提交的一行
143摘要,以便於審閱者了解它是關於什麼的。例如::
144
145        Commit e21d2170f36602ae2708 ("video: remove unnecessary
146        platform_set_drvdata()") removed the unnecessary
147        platform_set_drvdata(), but left the variable "dev" unused,
148        delete it.
149
150您還應該確保至少使用前12位 SHA-1 ID. 內核存儲庫包含*許多*對象,使與較短的ID
151發生衝突的可能性很大。記住,即使現在不會與您的六個字符ID發生衝突,這種情況
152可能五年後改變。
153
154如果修補程序修復了特定提交中的錯誤,例如,使用 ``git bisct`` ,請使用帶有前
15512個字符SHA-1 ID 的"Fixes:"標記和單行摘要。爲了簡化不要將標記拆分爲多個,
156行、標記不受分析腳本「75列換行」規則的限制。例如::
157
158        Fixes: 54a4f0239f2e ("KVM: MMU: make kvm_mmu_zap_page() return the number of pages it actually freed")
159
160下列 ``git config`` 設置可以添加讓 ``git log``, ``git show`` 漂亮的顯示格式::
161
162	[core]
163		abbrev = 12
164	[pretty]
165		fixes = Fixes: %h (\"%s\")
166
167.. _tw_split_changes:
168
1693) 拆分你的改動
170---------------
171
172將每個邏輯更改分隔成一個單獨的補丁。
173
174例如,如果你的改動里同時有bug修正和性能優化,那麼把這些改動拆分到兩個或
175者更多的補丁文件中。如果你的改動包含對API的修改,並且修改了驅動程序來適
176應這些新的API,那麼把這些修改分成兩個補丁。
177
178另一方面,如果你將一個單獨的改動做成多個補丁文件,那麼將它們合併成一個
179單獨的補丁文件。這樣一個邏輯上單獨的改動只被包含在一個補丁文件里。
180
181如果有一個補丁依賴另外一個補丁來完成它的改動,那沒問題。簡單的在你的補
182丁描述里指出「這個補丁依賴某補丁」就好了。
183
184在將您的更改劃分爲一系列補丁時,要特別注意確保內核在系列中的每個補丁之後
185都能正常構建和運行。使用 ``git bisect`` 來追蹤問題的開發者可能會在任何時
186候分割你的補丁系列;如果你在中間引入錯誤,他們不會感謝你。
187
188如果你不能將補丁濃縮成更少的文件,那麼每次大約發送出15個,然後等待審查
189和集成。
190
1914) 檢查你的更改風格
192-------------------
193
194檢查您的補丁是否存在基本樣式衝突,詳細信息可在
195:ref:`Documentation/translations/zh_TW/process/coding-style.rst <tw_codingstyle>`
196中找到。如果不這樣做,只會浪費審稿人的時間,並且會導致你的補丁被拒絕,甚至
197可能沒有被閱讀。
198
199一個重要的例外是在將代碼從一個文件移動到另一個文件時——在這種情況下,您不應
200該在移動代碼的同一個補丁中修改移動的代碼。這清楚地描述了移動代碼和您的更改
201的行爲。這大大有助於審查實際差異,並允許工具更好地跟蹤代碼本身的歷史。
202
203在提交之前,使用補丁樣式檢查程序檢查補丁(scripts/check patch.pl)。不過,
204請注意,樣式檢查程序應該被視爲一個指南,而不是作爲人類判斷的替代品。如果您
205的代碼看起來更好,但有違規行爲,那麼最好不要使用它。
206
207檢查者報告三個級別:
208
209 - ERROR:很可能出錯的事情
210 - WARNING:需要仔細審查的事項
211 - CHECK:需要思考的事情
212
213您應該能夠判斷您的補丁中存在的所有違規行爲。
214
2155) 選擇補丁收件人
216-----------------
217
218您應該總是在任何補丁上複製相應的子系統維護人員,以獲得他們維護的代碼;查看
219維護人員文件和原始碼修訂歷史記錄,以了解這些維護人員是誰。腳本
220scripts/get_Maintainer.pl在這個步驟中非常有用。如果您找不到正在工作的子系統
221的維護人員,那麼Andrew Morton(akpm@linux-foundation.org)將充當最後的維護
222人員。
223
224您通常還應該選擇至少一個郵件列表來接收補丁集的。linux-kernel@vger.kernel.org
225作爲最後一個解決辦法的列表,但是這個列表上的體積已經引起了許多開發人員的拒絕。
226在MAINTAINERS文件中查找子系統特定的列表;您的補丁可能會在那裡得到更多的關注。
227不過,請不要發送垃圾郵件到無關的列表。
228
229許多與內核相關的列表託管在vger.kernel.org上;您可以在
230http://vger.kernel.org/vger-lists.html 上找到它們的列表。不過,也有與內核相關
231的列表託管在其他地方。
232
233不要一次發送超過15個補丁到vger郵件列表!!!!
234
235Linus Torvalds 是決定改動能否進入 Linux 內核的最終裁決者。他的 e-mail
236地址是 <torvalds@linux-foundation.org> 。他收到的 e-mail 很多,所以一般
237的說,最好別給他發 e-mail。
238
239如果您有修復可利用安全漏洞的補丁,請將該補丁發送到 security@kernel.org。對於
240嚴重的bug,可以考慮短期暫停以允許分銷商向用戶發布補丁;在這種情況下,顯然不應
241將補丁發送到任何公共列表。
242
243修復已發布內核中嚴重錯誤的補丁程序應該指向穩定版維護人員,方法是放這樣的一行::
244
245        Cc: stable@vger.kernel.org
246
247進入補丁的簽准區(注意,不是電子郵件收件人)。除了這個文件之外,您還應該閱讀
248:ref:`Documentation/process/stable-kernel-rules.rst <stable_kernel_rules>`
249
250但是,請注意,一些子系統維護人員希望得出他們自己的結論,即哪些補丁應該被放到
251穩定的樹上。尤其是網絡維護人員,不希望看到單個開發人員在補丁中添加像上面這樣
252的行。
253
254如果更改影響到用戶和內核接口,請向手冊頁維護人員(如維護人員文件中所列)發送
255手冊頁補丁,或至少發送更改通知,以便一些信息進入手冊頁。還應將用戶空間API
256更改複製到 linux-api@vger.kernel.org。
257
2586) 沒有 MIME 編碼,沒有連結,沒有壓縮,沒有附件,只有純文本
259-----------------------------------------------------------
260
261Linus 和其他的內核開發者需要閱讀和評論你提交的改動。對於內核開發者來說
262,可以「引用」你的改動很重要,使用一般的 e-mail 工具,他們就可以在你的
263代碼的任何位置添加評論。
264
265因爲這個原因,所有的提交的補丁都是 e-mail 中「內嵌」的。
266
267.. warning::
268   如果你使用剪切-粘貼你的補丁,小心你的編輯器的自動換行功能破壞你的補丁
269
270不要將補丁作爲 MIME 編碼的附件,不管是否壓縮。很多流行的 e-mail 軟體不
271是任何時候都將 MIME 編碼的附件當作純文本發送的,這會使得別人無法在你的
272代碼中加評論。另外,MIME 編碼的附件會讓 Linus 多花一點時間來處理,這就
273降低了你的改動被接受的可能性。
274
275例外:如果你的郵遞員弄壞了補丁,那麼有人可能會要求你使用mime重新發送補丁
276
277請參閱 :ref:`Documentation/translations/zh_TW/process/email-clients.rst <tw_email_clients>`
278以獲取有關配置電子郵件客戶端以使其不受影響地發送修補程序的提示。
279
2807) e-mail 的大小
281----------------
282
283大的改動對郵件列表不合適,對某些維護者也不合適。如果你的補丁,在不壓縮
284的情況下,超過了300kB,那麼你最好將補丁放在一個能通過 internet 訪問的服
285務器上,然後用指向你的補丁的 URL 替代。但是請注意,如果您的補丁超過了
286300kb,那麼它幾乎肯定需要被破壞。
287
2888)回複評審意見
289---------------
290
291你的補丁幾乎肯定會得到評審者對補丁改進方法的評論。您必須對這些評論作出
292回應;讓補丁被忽略的一個好辦法就是忽略審閱者的意見。不會導致代碼更改的
293意見或問題幾乎肯定會帶來注釋或變更日誌的改變,以便下一個評審者更好地了解
294正在發生的事情。
295
296一定要告訴審稿人你在做什麼改變,並感謝他們的時間。代碼審查是一個累人且
297耗時的過程,審查人員有時會變得暴躁。即使在這種情況下,也要禮貌地回應並
298解決他們指出的問題。
299
3009)不要洩氣或不耐煩
301-------------------
302
303提交更改後,請耐心等待。審閱者是忙碌的人,可能無法立即訪問您的修補程序。
304
305曾幾何時,補丁曾在沒有評論的情況下消失在空白中,但開發過程比現在更加順利。
306您應該在一周左右的時間內收到評論;如果沒有收到評論,請確保您已將補丁發送
307到正確的位置。在重新提交或聯繫審閱者之前至少等待一周-在諸如合併窗口之類的
308繁忙時間可能更長。
309
31010)主題中包含 PATCH
311--------------------
312
313由於到linus和linux內核的電子郵件流量很高,通常會在主題行前面加上[PATCH]
314前綴. 這使Linus和其他內核開發人員更容易將補丁與其他電子郵件討論區分開。
315
31611)簽署你的作品-開發者原始認證
317-------------------------------
318
319爲了加強對誰做了何事的追蹤,尤其是對那些透過好幾層的維護者的補丁,我們
320建議在發送出去的補丁上加一個 「sign-off」 的過程。
321
322"sign-off" 是在補丁的注釋的最後的簡單的一行文字,認證你編寫了它或者其他
323人有權力將它作爲開放原始碼的補丁傳遞。規則很簡單:如果你能認證如下信息:
324
325開發者來源證書 1.1
326^^^^^^^^^^^^^^^^^^
327
328對於本項目的貢獻,我認證如下信息:
329
330      (a)這些貢獻是完全或者部分的由我創建,我有權利以文件中指出
331           的開放原始碼許可證提交它;或者
332      (b)這些貢獻基於以前的工作,據我所知,這些以前的工作受恰當的開放
333           原始碼許可證保護,而且,根據許可證,我有權提交修改後的貢獻,
334           無論是完全還是部分由我創造,這些貢獻都使用同一個開放原始碼許可證
335           (除非我被允許用其它的許可證),正如文件中指出的;或者
336      (c)這些貢獻由認證(a),(b)或者(c)的人直接提供給我,而
337           且我沒有修改它。
338      (d)我理解並同意這個項目和貢獻是公開的,貢獻的記錄(包括我
339           一起提交的個人記錄,包括 sign-off )被永久維護並且可以和這個項目
340           或者開放原始碼的許可證同步地再發行。
341
342那麼加入這樣一行::
343
344       Signed-off-by: Random J Developer <random@developer.example.org>
345
346使用你的真名(抱歉,不能使用假名或者匿名。)
347
348有人在最後加上標籤。現在這些東西會被忽略,但是你可以這樣做,來標記公司
349內部的過程,或者只是指出關於 sign-off 的一些特殊細節。
350
351如果您是子系統或分支維護人員,有時需要稍微修改收到的補丁,以便合併它們,
352因爲樹和提交者中的代碼不完全相同。如果你嚴格遵守規則(c),你應該要求提交者
353重新發布,但這完全是在浪費時間和精力。規則(b)允許您調整代碼,但是更改一個
354提交者的代碼並讓他認可您的錯誤是非常不禮貌的。要解決此問題,建議在最後一個
355由簽名行和您的行之間添加一行,指示更改的性質。雖然這並不是強制性的,但似乎
356在描述前加上您的郵件和/或姓名(全部用方括號括起來),這足以讓人注意到您對最
357後一分鐘的更改負有責任。例如::
358
359	Signed-off-by: Random J Developer <random@developer.example.org>
360	[lucky@maintainer.example.org: struct foo moved from foo.c to foo.h]
361	Signed-off-by: Lucky K Maintainer <lucky@maintainer.example.org>
362
363如果您維護一個穩定的分支機構,同時希望對作者進行致謝、跟蹤更改、合併修復並
364保護提交者不受投訴,那麼這種做法尤其有用。請注意,在任何情況下都不能更改作者
365的ID(From 頭),因爲它是出現在更改日誌中的標識。
366
367對回合(back-porters)的特別說明:在提交消息的頂部(主題行之後)插入一個補丁
368的起源指示似乎是一種常見且有用的實踐,以便於跟蹤。例如,下面是我們在3.x穩定
369版本中看到的內容::
370
371  Date:   Tue Oct 7 07:26:38 2014 -0400
372
373    libata: Un-break ATA blacklist
374
375    commit 1c40279960bcd7d52dbdf1d466b20d24b99176c8 upstream.
376
377還有, 這裡是一個舊版內核中的一個回合補丁::
378
379    Date:   Tue May 13 22:12:27 2008 +0200
380
381        wireless, airo: waitbusy() won't delay
382
383        [backport of 2.6 commit b7acbdfbd1f277c1eb23f344f899cfa4cd0bf36a]
384
38512)何時使用Acked-by:,CC:,和Co-Developed by:
386----------------------------------------------
387
388Singed-off-by: 標記表示簽名者參與了補丁的開發,或者他/她在補丁的傳遞路徑中。
389
390如果一個人沒有直接參與補丁的準備或處理,但希望表示並記錄他們對補丁的批准,
391那麼他們可以要求在補丁的變更日誌中添加一個 Acked-by:
392
393Acked-by:通常由受影響代碼的維護者使用,當該維護者既沒有貢獻也沒有轉發補丁時。
394
395Acked-by: 不像簽字人那樣正式。這是一個記錄,確認人至少審查了補丁,並表示接受。
396因此,補丁合併有時會手動將Acker的「Yep,looks good to me」轉換爲 Acked-By:(但
397請注意,通常最好要求一個明確的Ack)。
398
399Acked-by:不一定表示對整個補丁的確認。例如,如果一個補丁影響多個子系統,並且
400有一個:來自一個子系統維護者,那麼這通常表示只確認影響維護者代碼的部分。這裡
401應該仔細判斷。如有疑問,應參考郵件列表檔案中的原始討論。
402
403如果某人有機會對補丁進行評論,但沒有提供此類評論,您可以選擇在補丁中添加 ``Cc:``
404這是唯一一個標籤,它可以在沒有被它命名的人顯式操作的情況下添加,但它應該表明
405這個人是在補丁上抄送的。討論中包含了潛在利益相關方。
406
407Co-developed-by: 聲明補丁是由多個開發人員共同創建的;當幾個人在一個補丁上工
408作時,它用於將屬性賦予共同作者(除了 From: 所賦予的作者之外)。因爲
409Co-developed-by: 表示作者身份,所以每個共同開發人:必須緊跟在相關合作作者的
410簽名之後。標準的簽核程序要求:標記的簽核順序應儘可能反映補丁的時間歷史,而不
411管作者是通過 From :還是由 Co-developed-by: 共同開發的。值得注意的是,最後一
412個簽字人:必須始終是提交補丁的開發人員。
413
414注意,當作者也是電子郵件標題「發件人:」行中列出的人時,「From: 」 標記是可選的。
415
416作者提交的補丁程序示例::
417
418	<changelog>
419
420	Co-developed-by: First Co-Author <first@coauthor.example.org>
421	Signed-off-by: First Co-Author <first@coauthor.example.org>
422	Co-developed-by: Second Co-Author <second@coauthor.example.org>
423	Signed-off-by: Second Co-Author <second@coauthor.example.org>
424	Signed-off-by: From Author <from@author.example.org>
425
426合作開發者提交的補丁示例::
427
428	From: From Author <from@author.example.org>
429
430	<changelog>
431
432	Co-developed-by: Random Co-Author <random@coauthor.example.org>
433	Signed-off-by: Random Co-Author <random@coauthor.example.org>
434	Signed-off-by: From Author <from@author.example.org>
435	Co-developed-by: Submitting Co-Author <sub@coauthor.example.org>
436	Signed-off-by: Submitting Co-Author <sub@coauthor.example.org>
437
438
43913)使用報告人:、測試人:、審核人:、建議人:、修復人:
440--------------------------------------------------------
441
442Reported-by: 給那些發現錯誤並報告錯誤的人致謝,它希望激勵他們在將來再次幫助
443我們。請注意,如果bug是以私有方式報告的,那麼在使用Reported-by標記之前,請
444先請求權限。
445
446Tested-by: 標記表示補丁已由指定的人(在某些環境中)成功測試。這個標籤通知
447維護人員已經執行了一些測試,爲將來的補丁提供了一種定位測試人員的方法,並確
448保測試人員的信譽。
449
450Reviewed-by:相反,根據審查人的聲明,表明該補丁已被審查並被認爲是可接受的:
451
452
453審查人的監督聲明
454^^^^^^^^^^^^^^^^
455
456通過提供我的 Reviewed-by,我聲明:
457
458        (a) 我已經對這個補丁進行了一次技術審查,以評估它是否適合被包含到
459            主線內核中。
460
461        (b) 與補丁相關的任何問題、顧慮或問題都已反饋給提交者。我對提交者對
462            我的評論的回應感到滿意。
463
464        (c) 雖然這一提交可能會改進一些東西,但我相信,此時,(1)對內核
465            進行了有價值的修改,(2)沒有包含爭論中涉及的已知問題。
466
467        (d) 雖然我已經審查了補丁並認爲它是健全的,但我不會(除非另有明確
468            說明)作出任何保證或保證它將在任何給定情況下實現其規定的目的
469            或正常運行。
470
471Reviewed-by 是一種觀點聲明,即補丁是對內核的適當修改,沒有任何遺留的嚴重技術
472問題。任何感興趣的審閱者(完成工作的人)都可以爲一個補丁提供一個 Review-by
473標籤。此標籤用於向審閱者提供致謝,並通知維護者已在修補程序上完成的審閱程度。
474Reviewed-by: 當由已知了解主題區域並執行徹底檢查的審閱者提供時,通常會增加
475補丁進入內核的可能性。
476
477Suggested-by: 表示補丁的想法是由指定的人提出的,並確保將此想法歸功於指定的
478人。請注意,未經許可,不得添加此標籤,特別是如果該想法未在公共論壇上發布。
479這就是說,如果我們勤快地致謝我們的創意者,他們很有希望在未來得到鼓舞,再次
480幫助我們。
481
482Fixes: 指示補丁在以前的提交中修復了一個問題。它可以很容易地確定錯誤的來源,
483這有助於檢查錯誤修復。這個標記還幫助穩定內核團隊確定應該接收修復的穩定內核
484版本。這是指示補丁修復的錯誤的首選方法。請參閱 :ref:`tw_describe_changes`
485描述您的更改以了解更多詳細信息。
486
487.. _tw_the_canonical_patch_format:
488
48912)標準補丁格式
490----------------
491
492本節描述如何格式化補丁本身。請注意,如果您的補丁存儲在 ``Git`` 存儲庫中,則
493可以使用 ``git format-patch`` 進行正確的補丁格式設置。但是,這些工具無法創建
494必要的文本,因此請務必閱讀下面的說明。
495
496標準的補丁,標題行是::
497
498    Subject: [PATCH 001/123] 子系統:一句話概述
499
500標準補丁的信體存在如下部分:
501
502  - 一個 "from" 行指出補丁作者。後跟空行(僅當發送修補程序的人不是作者時才需要)。
503
504  - 解釋的正文,行以75列包裝,這將被複製到永久變更日誌來描述這個補丁。
505
506  - 一個空行
507
508  - 上面描述的「Signed-off-by」 行,也將出現在更改日誌中。
509
510  - 只包含 ``---`` 的標記線。
511
512  - 任何其他不適合放在變更日誌的注釋。
513
514  - 實際補丁( ``diff`` 輸出)。
515
516標題行的格式,使得對標題行按字母序排序非常的容易 - 很多 e-mail 客戶端都
517可以支持 - 因爲序列號是用零填充的,所以按數字排序和按字母排序是一樣的。
518
519e-mail 標題中的「子系統」標識哪個內核子系統將被打補丁。
520
521e-mail 標題中的「一句話概述」扼要的描述 e-mail 中的補丁。「一句話概述」
522不應該是一個文件名。對於一個補丁系列(「補丁系列」指一系列的多個相關補
523丁),不要對每個補丁都使用同樣的「一句話概述」。
524
525記住 e-mail 的「一句話概述」會成爲該補丁的全局唯一標識。它會蔓延到 git
526的改動記錄里。然後「一句話概述」會被用在開發者的討論里,用來指代這個補
527丁。用戶將希望通過 google 來搜索"一句話概述"來找到那些討論這個補丁的文
528章。當人們在兩三個月後使用諸如 ``gitk`` 或 ``git log --oneline`` 之類
529的工具查看數千個補丁時,也會很快看到它。
530
531出於這些原因,概述必須不超過70-75個字符,並且必須描述補丁的更改以及爲
532什麼需要補丁。既要簡潔又要描述性很有挑戰性,但寫得好的概述應該這樣做。
533
534概述的前綴可以用方括號括起來:「Subject: [PATCH <tag>...] <概述>」。標記
535不被視爲概述的一部分,而是描述應該如何處理補丁。如果補丁的多個版本已發
536送出來以響應評審(即「v1,v2,v3」)或「rfc」,以指示評審請求,那麼通用標記
537可能包括版本描述符。如果一個補丁系列中有四個補丁,那麼各個補丁可以這樣
538編號:1/4、2/4、3/4、4/4。這可以確保開發人員了解補丁應用的順序,並且他們
539已經查看或應用了補丁系列中的所有補丁。
540
541一些標題的例子::
542
543    Subject: [patch 2/5] ext2: improve scalability of bitmap searching
544    Subject: [PATCHv2 001/207] x86: fix eflags tracking
545
546"From" 行是信體裡的最上面一行,具有如下格式:
547        From: Patch Author <author@example.com>
548
549"From" 行指明在永久改動日誌里,誰會被確認爲作者。如果沒有 "From" 行,那
550麼郵件頭裡的 "From: " 行會被用來決定改動日誌中的作者。
551
552說明的主題將會被提交到永久的原始碼改動日誌里,因此對那些早已經不記得和
553這個補丁相關的討論細節的有能力的讀者來說,是有意義的。包括補丁程序定位
554錯誤的(內核日誌消息、OOPS消息等)症狀,對於搜索提交日誌以尋找適用補丁的人
555尤其有用。如果一個補丁修復了一個編譯失敗,那麼可能不需要包含所有編譯失敗;
556只要足夠讓搜索補丁的人能夠找到它就行了。與概述一樣,既要簡潔又要描述性。
557
558"---" 標記行對於補丁處理工具要找到哪裡是改動日誌信息的結束,是不可缺少
559的。
560
561對於 "---" 標記之後的額外註解,一個好的用途就是用來寫 diffstat,用來顯
562示修改了什麼文件和每個文件都增加和刪除了多少行。diffstat 對於比較大的補
563丁特別有用。其餘那些只是和時刻或者開發者相關的註解,不合適放到永久的改
564動日誌里的,也應該放這裡。
565使用 diffstat的選項 "-p 1 -w 70" 這樣文件名就會從內核原始碼樹的目錄開始
566,不會占用太寬的空間(很容易適合80列的寬度,也許會有一些縮進。)
567
568在後面的參考資料中能看到適當的補丁格式的更多細節。
569
570.. _tw_explicit_in_reply_to:
571
57215) 明確回覆郵件頭(In-Reply-To)
573-------------------------------
574
575手動添加回復補丁的的標題頭(In-Reply_To:) 是有幫助的(例如,使用 ``git send-email`` )
576將補丁與以前的相關討論關聯起來,例如,將bug修復程序連結到電子郵件和bug報告。
577但是,對於多補丁系列,最好避免在回復時使用連結到該系列的舊版本。這樣,
578補丁的多個版本就不會成爲電子郵件客戶端中無法管理的引用序列。如果連結有用,
579可以使用 https://lore.kernel.org/ 重定向器(例如,在封面電子郵件文本中)
580連結到補丁系列的早期版本。
581
58216) 發送git pull請求
583--------------------
584
585如果您有一系列補丁,那麼讓維護人員通過git pull操作將它們直接拉入子系統存儲
586庫可能是最方便的。但是,請注意,從開發人員那裡獲取補丁比從郵件列表中獲取補
587丁需要更高的信任度。因此,許多子系統維護人員不願意接受請求,特別是來自新的
588未知開發人員的請求。如果有疑問,您可以在封面郵件中使用pull 請求作爲補丁系列
589正常發布的一個選項,讓維護人員可以選擇使用其中之一。
590
591pull 請求的主題行中應該有[Git Pull]。請求本身應該在一行中包含存儲庫名稱和
592感興趣的分支;它應該看起來像::
593
594  Please pull from
595
596      git://jdelvare.pck.nerim.net/jdelvare-2.6 i2c-for-linus
597
598  to get these changes:
599
600
601pull 請求還應該包含一條整體消息,說明請求中將包含什麼,一個補丁本身的 ``Git shortlog``
602以及一個顯示補丁系列整體效果的 ``diffstat`` 。當然,將所有這些信息收集在一起
603的最簡單方法是讓 ``git`` 使用 ``git request-pull`` 命令爲您完成這些工作。
604
605一些維護人員(包括Linus)希望看到來自已簽名提交的請求;這增加了他們對你的
606請求信心。特別是,在沒有簽名標籤的情況下,Linus 不會從像 Github 這樣的公共
607託管站點拉請求。
608
609創建此類簽名的第一步是生成一個 GNRPG 密鑰,並由一個或多個核心內核開發人員對
610其進行簽名。這一步對新開發人員來說可能很困難,但沒有辦法繞過它。參加會議是
611找到可以簽署您的密鑰的開發人員的好方法。
612
613一旦您在Git 中準備了一個您希望有人拉的補丁系列,就用 ``git tag -s`` 創建一
614個簽名標記。這將創建一個新標記,標識該系列中的最後一次提交,並包含用您的私
615鑰創建的簽名。您還可以將changelog樣式的消息添加到標記中;這是一個描述拉請求
616整體效果的理想位置。
617
618如果維護人員將要從中提取的樹不是您正在使用的存儲庫,請不要忘記將已簽名的標記
619顯式推送到公共樹。
620
621生成拉請求時,請使用已簽名的標記作爲目標。這樣的命令可以實現::
622
623  git request-pull master git://my.public.tree/linux.git my-signed-tag
624
625參考文獻
626--------
627
628Andrew Morton, "The perfect patch" (tpp).
629  <https://www.ozlabs.org/~akpm/stuff/tpp.txt>
630
631Jeff Garzik, "Linux kernel patch submission format".
632  <https://web.archive.org/web/20180829112450/http://linux.yyz.us/patch-format.html>
633
634Greg Kroah-Hartman, "How to piss off a kernel subsystem maintainer".
635  <http://www.kroah.com/log/linux/maintainer.html>
636
637  <http://www.kroah.com/log/linux/maintainer-02.html>
638
639  <http://www.kroah.com/log/linux/maintainer-03.html>
640
641  <http://www.kroah.com/log/linux/maintainer-04.html>
642
643  <http://www.kroah.com/log/linux/maintainer-05.html>
644
645  <http://www.kroah.com/log/linux/maintainer-06.html>
646
647NO!!!! No more huge patch bombs to linux-kernel@vger.kernel.org people!
648  <https://lore.kernel.org/r/20050711.125305.08322243.davem@davemloft.net>
649
650Kernel Documentation/process/coding-style.rst:
651  :ref:`Documentation/translations/zh_TW/process/coding-style.rst <tw_codingstyle>`
652
653Linus Torvalds's mail on the canonical patch format:
654  <https://lore.kernel.org/r/Pine.LNX.4.58.0504071023190.28951@ppc970.osdl.org>
655
656Andi Kleen, "On submitting kernel patches"
657  Some strategies to get difficult or controversial changes in.
658
659  http://halobates.de/on-submitting-patches.pdf
660
661