Existing sentence on cross-compilation that mentions ppc64 does not make much sense in today's perspective. Expand it for the benefits of testing against architectures of different word sizes and endianness. Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx> Cc: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> Cc: Tsugikazu Shibata <shibata@xxxxxxxxxxxxxxxxxxx> --- Documentation/process/submit-checklist.rst | 9 ++++++--- .../translations/ja_JP/process/submit-checklist.rst | 10 ++++++---- 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/Documentation/process/submit-checklist.rst b/Documentation/process/submit-checklist.rst index e531dd504b6c..88b6358258d7 100644 --- a/Documentation/process/submit-checklist.rst +++ b/Documentation/process/submit-checklist.rst @@ -91,9 +91,12 @@ Build your code fix any issues. 2) Builds on multiple CPU architectures by using local cross-compile tools - or some other build farm. Note that ppc64 is a good architecture for - cross-compilation checking because it tends to use ``unsigned long`` for - 64-bit quantities. + or some other build farm. + Note that testing against architectures of different word sizes + (32- and 64-bit) and different endianness (big- and little-) is effective + in catching various portability issues due to false assumptions on + representable quantity range, data alignment, or endianness, among + others. 3) Newly-added code has been compiled with ``gcc -W`` (use ``make KCFLAGS=-W``). This will generate lots of noise, but is good diff --git a/Documentation/translations/ja_JP/process/submit-checklist.rst b/Documentation/translations/ja_JP/process/submit-checklist.rst index b9da826d30ae..41af64d5a73d 100644 --- a/Documentation/translations/ja_JP/process/submit-checklist.rst +++ b/Documentation/translations/ja_JP/process/submit-checklist.rst @@ -112,10 +112,12 @@ Kconfig 変更のレビュー が無いこと。 ``make htmldocs`` または ``make pdfdocs`` でビルドし、問題があれば修正。 -2) ローカルのクロス・コンパイル・ツール、その他のビルド環境を使って、 - 複数の CPU アーキテクチャ向けのビルドができるか。 - ``unsigned long`` を 64 ビットの数量に対して使ってしまうことが多いという - 点で ppc64 は、クロス・コンパイルの検査対象として好適である。 +2) ローカルのクロス・コンパイル・ツール、その他のビルド環境を使って、複数の + CPU アーキテクチャ向けのビルドをしたか。 + 特に、ワードサイズ (32 ビットと 64 ビット) やエンディアン (ビッグとリトル) + の異なるアーキテクチャを対象とするテストは、表現可能数値範囲・データ整列・ + エンディアンなどについての誤った仮定に起因する様々な移植上の問題を捕える + のに効果的。 3) 新規に追加されたコードについて (``make KCFLAGS=-W`` を使って) ``gcc -W`` でコンパイルしたか。 -- 2.34.1