Linux カーネルパッチ投稿チェックリスト¶
Note
【訳註】 この文書は、 Linux Kernel patch submission checklist の翻訳です。 免責条項については、 免責条項の抄訳 および、 Disclaimer (英語版) を参照してください。
以下は、カーネルパッチの投稿時に、そのスムーズな受け入れのために心がける べき基本的な事項です。
これは、 Submitting patches: the essential guide to getting your code into the kernel およびその他の Linux カーネルパッチ投稿に関する文書を踏まえ、それを補足するものです。
Note
【訳註】 可能な項目については、パッチもしくはパッチ内の更新を暗黙の主語として、 その望ましい状態を表す文体とします。その他、原義を損なわない範囲で 係り結びを調整するなど、簡潔で把握しやすい箇条書きを目指します。
コードのレビュー¶
利用する機能について、その機能を定義・宣言しているファイルを
#includeしている。 他のヘッダーファイル経由での取り込みに依存しない。Linux kernel coding style に詳述されている一般的なスタイル についてチェック済み。
メモリバリアー (例,
barrier(),rmb(),wmb()) について、 そのすべてに、作用と目的、及び必要理由についての説明がソースコード内の コメントとして記述されている。
Kconfig 変更のレビュー¶
新規の、もしくは変更された
CONFIGオプションについて、それが関係する コンフィグメニューへの悪影響がない。また、 Kconfig Language の “Menu attibutes: default value” に記載の例外条件を満たす場合を除き、 そのデフォルトが無効になっている。新規の
Kconfigオプションにヘルプテキストがある。妥当な
Kconfigの組み合わせについて注意深くレビューされている。 これをテストでやり切るのは困難で、知力が決め手となる。
ドキュメンテーションの作成¶
グローバルなカーネル API が kernel-doc の形式で ドキュメント化されている (静的関数には求められないが、付けてもよい)。
新規
/procエントリーが、すべてDocumentation/以下に記載されて いる。新規カーネル・ブート・パラメータが、すべて
Documentation/admin-guide/kernel-parameters.rstに記載されている。新規モジュール・パラメータが、すべて
MODULE_PARM_DESC()によって記述 されている。新規ユーザースペース・インターフェースが、すべて
Documentaion/ABI/以下に記載されている。詳しくは、 Linux ABI description (もしくはDocumentation/ABI/README) を参照。 ユーザースペース・インターフェースを変更するパッチは、 linux-api@vger.kernel.org にも CC すべし。なんらかの ioctl を追加するパッチは、
Documentation/userspace-api/ioctl/ioctl-number.rstの更新を伴う。
ツールによるコードのチェック¶
スタイル・チェッカー (
scripts/checkpatch.pl) によって、犯しがちな パッチ・スタイルの違反がないことを確認済み。 指摘される違反を残す場合は、それを正当化できること。sparse により入念にチェック済み。
make checkstackで指摘される問題があれば、それが修正済み。checkstackは問題点を明示的には指摘しないが、 スタック消費が 512 バイトを越える関数は見直しの候補。
コードのビルド¶
以下の条件でクリーンにビルドできる。
適用可能な、および
=y,=m,=nを変更したCONFIGオプションでのビルド。gccおよびリンカーからの警告・エラーがないこと。allnoconfigとallmodconfigがパスO=builddirを指定してのビルドDocumentation/ 以下の変更に関して、ドキュメントのビルドで新たな警告や エラーが出ない。
make htmldocsまたはmake pdfdocsでビルドし、問題があれば修正。
ローカルのクロス・コンパイル・ツール、その他のビルド環境 (訳註: build farm) を使って、複数の CPU アーキテクチャ向けにビルドできる。 特に、ワードサイズ (32 ビットと 64 ビット) やエンディアン (ビッグとリトル) の異なるアーキテクチャを対象とするテストは、表現可能数値範囲・データ整列・ エンディアンなどについての誤った仮定に起因する様々な移植上の問題を捕える のに効果的。
新規に追加されたコードについて (
make KCFLAGS=-Wを使って)gcc -Wでコンパイル。 これは多くのノイズを伴うが、warning: comparison between signed and unsignedの類いのバグをあぶり出すのに効果的。変更されるソースコードが、下記の
Kconfigシンボルに関連するカーネル API や機能に依存 (もしくは利用) する場合、それらのKconfigシンボルが、 無効、および (可能なら)=mの場合を組み合わせた複数のビルドを (全部まとめてではなく、いろいろなランダムの組み合わせで) テスト済み。CONFIG_SMP,CONFIG_SYSFS,CONFIG_PROC_FS,CONFIG_INPUT,CONFIG_PCI,CONFIG_BLOCK,CONFIG_PM,CONFIG_MAGIC_SYSRQ,CONFIG_NET,CONFIG_INET=n(ただし、後者はCONFIG_NET=yとの組み合わせ)。
コードのテスト¶
CONFIG_PREEMPT,CONFIG_DEBUG_PREEMPT,CONFIG_SLUB_DEBUG,CONFIG_DEBUG_PAGEALLOC,CONFIG_DEBUG_MUTEXES,CONFIG_DEBUG_SPINLOCK,CONFIG_DEBUG_ATOMIC_SLEEP,CONFIG_PROVE_RCUおよびCONFIG_DEBUG_OBJECTS_RCU_HEADをすべて 同時に有効にしてのテスト済み。CONFIG_SMPとCONFIG_PREEMPTが有効と無効の場合について、ビルドと ランタイムのテスト済み。lockdep の機能をすべて有効にしての実行で、すべてのコード経路が確認済み。
最低限、slab と ページ・アロケーションの失敗に関する誤り注入 (訳註: fault injection) によるチェック済み。 詳しくは、 Fault-injection を参照。 新規のコードが多い場合は、サブシステム対象の誤り注入を追加するのが望ましい 可能性あり。
linux-next の最新タグに対するテストにより、他でキューイングされている パッチや、VM、VFS、その他のサブシステム内のすべての変更と組み合わせての 動作を確認済み。