lkml.org 
[lkml]   [2021]   [Apr]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v2] Documentation/translations/zh_CN/dev-tools/
Hi Bernard,

On Thu, Apr 01, 2021 at 06:27:16AM -0700, Bernard Zhao wrote:

Why the charset in your email header is 'y' ?
"Content-Type: text/plain; charset=y"
<https://lore.kernel.org/linux-doc/20210401132732.70612-1-bernard@vivo.com/raw>

> Add translations to dev-tools gcov
>
> Signed-off-by: Bernard Zhao <bernard@vivo.com>
> Reviewed-by: Wu X.C <bobwxc@email.cn>
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
This reviewed-by tag is invalid.

Please do not pick review-by tag before one give it.

> ---
> Changes since V1:
> * add index.rst in dev-tools and link to to zh_CN/index.rst
> * fix some inaccurate translation
>
> Link for V1:
> *https://lore.kernel.org/patchwork/patch/1405740/
> ---
> .../translations/zh_CN/dev-tools/gcov.rst | 279 ++++++++++++++++++
> .../translations/zh_CN/dev-tools/index.rst | 39 +++
> Documentation/translations/zh_CN/index.rst | 1 +
> 3 files changed, 319 insertions(+)
> create mode 100644 Documentation/translations/zh_CN/dev-tools/gcov.rst
> create mode 100644 Documentation/translations/zh_CN/dev-tools/index.rst
>
> diff --git a/Documentation/translations/zh_CN/dev-tools/gcov.rst b/Documentation/translations/zh_CN/dev-tools/gcov.rst
> new file mode 100644
> index 000000000000..e8ffb99b566d

Why replaced all ',' '。' with ',' '.' in zh_CN/dev-tools/gcov.rst ?
And also the columns in the v2 are much shorter than v1.
Please recover the above two points.

> --- /dev/null
> +++ b/Documentation/translations/zh_CN/dev-tools/gcov.rst
> @@ -0,0 +1,279 @@
> +.. SPDX-License-Identifier: GPL-2.0
> +
> +.. include:: ../disclaimer-zh_CN.rst
> +
> +:Original: :ref:`Documentation/dev-tools/gcov.rst <dev-tools_gcov>`

the original text have no article tag, simply use this
:Original: Documentation/dev-tools/gcov.rst

> +:Translator: 赵军奎 Bernard Zhao <bernard@vivo.com>
> +
> +.. _dev-tools_gcov:

Please remove above line, no need

> +
> +在Linux内核里使用gcov做代码覆盖率检查
> +====================================

Still got a lot of warning.
Please using monospaced font to fix this.
Build log:

/test/linux/Documentation/translations/zh_CN/dev-tools/gcov.rst:11: WARNING: Title underline too short.

在Linux内核里使用gcov做代码覆盖率检查
====================================
/test/linux/Documentation/translations/zh_CN/dev-tools/gcov.rst:110: WARNING: Title underline too short.
针对模块的统计
-------
/test/linux/Documentation/translations/zh_CN/dev-tools/gcov.rst:110: WARNING: Title underline too short.
针对模块的统计
-------
/test/linux/Documentation/translations/zh_CN/dev-tools/gcov.rst:154: WARNING: Block quote ends without a blank line; unexpected unindent.
/test/linux/Documentation/translations/zh_CN/dev-tools/gcov.rst:179: WARNING: Title underline too short.
关于编译器的注意事项
-----------------
/test/linux/Documentation/translations/zh_CN/dev-tools/gcov.rst:179: WARNING: Title underline too short.
关于编译器的注意事项
-----------------

> +
> +gcov是linux中已经集成的一个分析模块,该模块在内核中对
> +GCC的代码覆盖率统计提供了支持.
> +linux内核运行时的代码覆盖率数据会以gcov兼容的格式存储
> +在debug-fs中,可以通过gcov的“-o”选项(如下示例)获得
> +指定文件的代码运行覆盖率统计数据(需要跳转到内核编
> +译路径下并且要有root权限)::
> +
> + # cd /tmp/linux-out
> + # gcov -o /sys/kernel/debug/gcov/tmp/linux-out/kernel spinlock.c
> +
> +这将在当前目录中创建带有执行计数注释的源代码文件.
> +在获得这些统计文件后,可以使用图形化的gcov[1]前端工
> +具(比如lcov[2]),来实现自动化处理linux 内核的覆
> +盖率运行数据,同时生成易于阅读的HTML格式文件.

Sorry for the inconvenience,
I test again, found the url tags seems would not make namespace conflicts.
Thus,

在获得这些统计文件后,可以使用图形化的 gcov_ 前端工
具(比如 lcov_ ),来实现自动化处理linux 内核的覆

> +
> +可能的用途:
> +
> +* 调试(用来判断每一行的代码是否已经运行过)
> +* 测试改进(如何修改测试代码,尽可能地覆盖到没有运
> + 行过的代码)
> +* 内核配置优化(对于某一个选项配置,如果关联的代码
> + 从来没有运行过,是否还需要这个配置)
> +
> +[1]_gcov: https://gcc.gnu.org/onlinedocs/gcc/Gcov.html
> +[2]_lcov: http://ltp.sourceforge.net/coverage/lcov.php

.. _gcov: https://gcc.gnu.org/onlinedocs/gcc/Gcov.html
.. _lcov: http://ltp.sourceforge.net/coverage/lcov.php

> +
> +
> +准备
> +-----------
> +
> +内核打开如下配置::
> +
> + CONFIG_DEBUG_FS=y
> + CONFIG_GCOV_KERNEL=y
> +
> +获取整个内核的覆盖率数据,还需要打开::
> +
> + CONFIG_GCOV_PROFILE_ALL=y
> +
> +需要注意的是,整个内核开启覆盖率统计会造成内核镜像
> +文件尺寸的增大,同时内核运行的也会变慢一些.
> +另外,并不是所有的架构都支持整个内核开启覆盖率统计
> +
> +代码运行覆盖率数据只在debugfs挂载完成后才可以访问::
> +
> + mount -t debugfs none /sys/kernel/debug
> +
> +
> +客制化
> +-------------
> +
> +如果要单独针对某一个路径或者文件进行代码覆盖率统计
> +可以在内核相应路径的Makefile中增加如下的配置:
> +
> +- 单独统计单个文件(例如main.o)::
> +
> + GCOV_PROFILE_main.o := y
> +
> +- 单独统计某一个路径::
> +
> + GCOV_PROFILE := y
> +
> +如果要在整个内核的覆盖率统计(CONFIG_GCOV_PROFILE_ALL)
> +中单独排除某一个文件或者路径,可以使用如下的方法::
> +
> + GCOV_PROFILE_main.o := n
> +
> +和::
> +
> + GCOV_PROFILE := n
> +
> +此机制仅支持链接到内核镜像或编译为内核模块的文件.
> +
> +
> +相关文件
> +-------------
> +
> +gcov功能需要在debugfs中创建如下文件:
> +
> +``/sys/kernel/debug/gcov``
> + gcov相关功能的根路径
> +
> +``/sys/kernel/debug/gcov/reset``
> + 全局复位文件:向该文件写入数据后会将所有的gcov统计
> + 数据清0
> +
> +``/sys/kernel/debug/gcov/path/to/compile/dir/file.gcda``
> + gcov工具可以识别的覆盖率统计数据文件,向该文件写入
> + 数据后会将本文件的gcov统计数据清0
> +
> +``/sys/kernel/debug/gcov/path/to/compile/dir/file.gcno``
> + gcov工具需要的软连接文件(指向编译时生成的信息统
> + 计文件),这个文件是在gcc编译时如果配置了选项
> + “-ftest-coverage”时生成的.

``something`` is a code style mark, do not replace it with “”

``-ftest-coverage``

> +
> +
> +针对模块的统计
> +-------
> +
> +内核中的模块会动态的加载和卸载,模块卸载时对应的
> +数据会被清除掉.
> +gcov提供了一种机制,通过保留相关数据的副本来收集
> +这部分卸载模块的覆盖率数据.
> +模块卸载后这些备份数据在debugfs中会继续存在.
> +一旦这个模块重新加载,模块关联的运行统计会被初始
> +化成debugfs中备份的数据.
> +
> +可以通过对内核参数gcov_persist的修改来停用gcov
> +对模块的备份机制::
> +
> + gcov_persist = 0
> +
> +在运行时,用户还可以通过写入模块的数据文件或者写入
> +gcov复位文件来丢弃已卸载模块的数据.
> +
> +
> +分离的编译和运行设备
> +---------------------------------
> +
> +gcov的内核分析架构支持内核的编译和分析是在同一台
> +设备上,也可以编译和运行是在不同的设备上.
> +如果内核编译和运行是不同的设备,那么需要额外的准备
> +工作,这取决于gcov工具是在哪里使用的:
> +
> +a) 若gcov运行在测试设备上
> +
> + 测试设备上面gcov工具的版本必须要跟设备内核编
> + 译使用的gcc版本相兼容,同时下面的文件要从编译
> + 设备拷贝到测试设备上:
> +
> + 从源代码中:
> + - 所有的C文件和头文件
> +
> + 从编译目录中:
> + - 所有的C文件和头文件
> + - 所有的.gcda文件和.gcno文件
> + - 所有目录的链接
> +
> + 特别需要注意,测试机器上面的目录结构跟编译机器
> + 上面的目录机构必须完全一致.
> + 如果文件是软链接,需要替换成真正的目录文件
> + (这是由make的当前工作目录变量CURDIR引起的)
> +
> +b) 若gcov运行在编译设备上
> +
> + 测试用例运行结束后,如下的文件需要从测试设备中
> + 拷贝到编译设备上:
> +
> + 从sysfs中的gcov目录中:
> + - 所有的.gcda文件
> + - 所有的.gcno文件软链接
> +
> + 这些文件可以拷贝到编译设备的任意目录下,gcov使
> + 用-o选项指定拷贝的目录.
> +
> + 比如一个是示例的目录结构如下::
> +
> + /tmp/linux: 内核源码目录
> + /tmp/out: 内核编译文件路径(make O=指定)
> + /tmp/coverage: 从测试机器上面拷贝的数据文件路径
> +
> + [user@build] cd /tmp/out
> + [user@build] gcov -o /tmp/coverage/tmp/out/init main.c
> +
> +
> +关于编译器的注意事项
> +-----------------
> +
> +GCC和LLVM gcov工具不一定兼容.
> +如果编译器是GCC,使用gcov_[3]来处理.gcno和.gcda文件
> +如果是Clang编译器,则使用llvm-cov_[4].
> +

如果编译器是GCC,使用 gcov_ 来处理.gcno和.gcda文件
如果是Clang编译器,则使用 llvm-cov_ 。

> +[3]_gcov: https://gcc.gnu.org/onlinedocs/gcc/Gcov.html
> +[4]_llvm-cov: https://llvm.org/docs/CommandGuide/llvm-cov.html

.. _gcov: https://gcc.gnu.org/onlinedocs/gcc/Gcov.html
.. _llvm-cov: https://llvm.org/docs/CommandGuide/llvm-cov.html

> +
> +GCC和Clang gcov之间的版本差异由Kconfig处理的.
> +kconfig会根据编译工具链的检查自动选择合适的gcov格式.
> +
> +问题定位
> +---------------
> +
> +可能出现的问题1
> + 编译到链接阶段报错终止
> +
> +问题原因
> + 分析标志指定在了源文件但是没有链接到主内核
> + 或者客制化了链接程序
> +
> +解决方法
> + 通过在相应的Makefile中使用“GCOV_PROFILE := n”
> + 或者“GCOV_PROFILE_basename.o := n”来将链接报错的文件
> + 排除掉

same `` problem

``GCOV_PROFILE := n``
``GCOV_PROFILE_basename.o := n``

> +
> +可能出现的问题2
> + 从sysfs复制的文件显示为空或不完整
> +
> +问题原因
> + 由于seq_file的工作方式,某些工具(例如cp或tar)
> + 可能无法正确地从sysfs复制文件.
> +
> +解决方法
> + 使用“cat”读取“.gcda”文件,使用“cp -d”复制链接
> + 或者使用附录B中所示的机制.

same ``

``cat``
``.gcda``
``cp -d``

> +
> +
> +附录A::collect_on_build.sh
> +------------------------------
> +
> +用于在编译设备上收集覆盖率编译中间数据文件
> +的示例脚本
> +(如下6a示例)

actually is 上

> +
> +.. code-block:: sh
> +
> + #!/bin/bash
> +
> + KSRC=$1
> + KOBJ=$2
> + DEST=$3
> +
> + if [ -z "$KSRC" ] || [ -z "$KOBJ" ] || [ -z "$DEST" ]; then
> + echo "Usage: $0 <ksrc directory> <kobj directory> <output.tar.gz>" >&2
> + exit 1
> + fi
> +
> + KSRC=$(cd $KSRC; printf "all:\n\t@echo \${CURDIR}\n" | make -f -)
> + KOBJ=$(cd $KOBJ; printf "all:\n\t@echo \${CURDIR}\n" | make -f -)
> +
> + find $KSRC $KOBJ \( -name '*.gcno' -o -name '*.[ch]' -o -type l \) -a \
> + -perm /u+r,g+r | tar cfz $DEST -P -T -
> +
> + if [ $? -eq 0 ] ; then
> + echo "$DEST successfully created, copy to test system and unpack with:"
> + echo " tar xfz $DEST -P"
> + else
> + echo "Could not create file $DEST"
> + fi
> +
> +
> +附录B::collect_on_test.sh
> +-----------------------------
> +
> +用于在测试设备上收集覆盖里统计数据数据文件的示例脚本
> +(如下6b示例)

actually is 上

> +
> +.. code-block:: sh
> +
> + #!/bin/bash -e
> +
> + DEST=$1
> + GCDA=/sys/kernel/debug/gcov
> +
> + if [ -z "$DEST" ] ; then
> + echo "Usage: $0 <output.tar.gz>" >&2
> + exit 1
> + fi
> +
> + TEMPDIR=$(mktemp -d)
> + echo Collecting data..
> + find $GCDA -type d -exec mkdir -p $TEMPDIR/\{\} \;
> + find $GCDA -name '*.gcda' -exec sh -c 'cat < $0 > '$TEMPDIR'/$0' {} \;
> + find $GCDA -name '*.gcno' -exec sh -c 'cp -d $0 '$TEMPDIR'/$0' {} \;
> + tar czf $DEST -C $TEMPDIR sys
> + rm -rf $TEMPDIR
> +
> + echo "$DEST successfully created, copy to build system and unpack with:"
> + echo " tar xfz $DEST"
> diff --git a/Documentation/translations/zh_CN/dev-tools/index.rst b/Documentation/translations/zh_CN/dev-tools/index.rst
> new file mode 100644
> index 000000000000..0c6b5c2f4a7f
> --- /dev/null
> +++ b/Documentation/translations/zh_CN/dev-tools/index.rst
> @@ -0,0 +1,39 @@
> +.. SPDX-License-Identifier: GPL-2.0
> +
> +.. include:: ../disclaimer-zh_CN.rst
> +
> +:Original: :ref:`Documentation/dev-tools/gcov.rst <development_dev-tools_index>`

:Original: Documentation/dev-tools/gcov.rst

> +:Translator: 赵军奎 Bernard Zhao <bernard@vivo.com>
> +
> +.. _development_dev-tools_index:

remove above line

> +
> +================================
> +内核开发工具
> +================================
> +
> +本文档是有关内核开发工具文档的合集。
> +目前这些文档已经整理在一起,不需要再花费额外的精力。
> +欢迎任何补丁。
> +
> +.. class:: toc-title
> +
> + 目录
> +
> +.. toctree::
> + :maxdepth: 2
> +
> + gcov
> +
> +Todolist:
> +
> + coccinelle
> + sparse
> + kcov
> + kasan
> + ubsan
> + kmemleak
> + kcsan
> + gdb-kernel-debugging
> + kgdb
> + kselftest
> + kunit/index
> diff --git a/Documentation/translations/zh_CN/index.rst b/Documentation/translations/zh_CN/index.rst
> index be6f11176200..a5e483b0c7f2 100644
> --- a/Documentation/translations/zh_CN/index.rst
> +++ b/Documentation/translations/zh_CN/index.rst
> @@ -20,6 +20,7 @@
> process/index
> filesystems/index
> arm64/index
> + dev-tools/index
>
> 目录和表格
> ----------
> --
> 2.31.0

Thanks!

Wu X.C.

\
 
 \ /
  Last update: 2021-04-02 06:00    [W:0.083 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site