[PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[Haowen Bai]

This patches series will sync the translation of zh_CN as below:

Commit 8bd16bc0a081 ("tools/vm/page_owner_sort.c: support sorting blocks
by multiple keys") add sorting blocks by multiple keys usage
description.

Commit 78a0b94f3829 ("tools/vm/page_owner_sort.c: support for multi-value
selection in single argument") add multi-value selection in single
argument usage description.

Commit c89b3ad2dea2 ("doc/vm/page_owner.rst: remove content related to -c
option") remove  -c option usage description.

Commit 9c8a0a8e599f ("tools/vm/page_owner_sort.c: support for user-defined
culling rules") add user-defined culling rules usage description.

Commit 8ea8613a616a ("tools/vm/page_owner_sort.c: support for selecting by
PID, TGID or task command name") add selecting by PID, TGID or task
command name usage description.

Commit 194d52d771b8 ("tools/vm/page_owner_sort: support for sorting by
task command name") add sorting by task command name usage description.

But the original text table which is made of tab will display all
by one line. so we make a standard table for STANDARD FORMAT SPECIFIERS
description. I fix it up togeter by this patches series.


Haowen Bai (2):
  docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
  doc/vm/page_owner.rst: Fix table display confusion

 Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
 Documentation/vm/page_owner.rst                    | 34 +++++++-----
 2 files changed, 80 insertions(+), 15 deletions(-)

-- 
2.7.4

[PATCH 1/2] docs/zh_CN: sync with original text Documentation/vm/page_owner.rst

[Haowen Bai]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="y", Size: 4911 bytes --]

As the tools/vm/page_owner_sort added some feature and original text
updated, sync the translation of zh_CN as below.

Commit 8bd16bc0a081 ("tools/vm/page_owner_sort.c: support sorting blocks
by multiple keys") add sorting blocks by multiple keys usage
description.

Commit 78a0b94f3829 ("tools/vm/page_owner_sort.c: support for multi-value
selection in single argument") add multi-value selection in single
argument usage description.

Commit c89b3ad2dea2 ("doc/vm/page_owner.rst: remove content related to -c
option") remove  -c option usage description.

Commit 9c8a0a8e599f ("tools/vm/page_owner_sort.c: support for user-defined
culling rules") add user-defined culling rules usage description.

Commit 8ea8613a616a ("tools/vm/page_owner_sort.c: support for selecting by
PID, TGID or task command name") add selecting by PID, TGID or task
command name usage description.

Commit 194d52d771b8 ("tools/vm/page_owner_sort: support for sorting by
task command name") add sorting by task command name usage description.

Signed-off-by: Haowen Bai <baihaowen@meizu.com>
---
 Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
 1 file changed, 60 insertions(+), 1 deletion(-)

diff --git a/Documentation/translations/zh_CN/vm/page_owner.rst b/Documentation/translations/zh_CN/vm/page_owner.rst
index 9e951fabba9d..c692e81c724e 100644
--- a/Documentation/translations/zh_CN/vm/page_owner.rst
+++ b/Documentation/translations/zh_CN/vm/page_owner.rst
@@ -103,14 +103,73 @@ page owner在默认情况下是禁用的。所以，如果你想使用它，你
 		-m		按总内存排序
 		-p		按pid排序。
 		-P		按tgid排序。
+		-n		按任务名称排序。
 		-r		按内存释放时间排序。
 		-s		按堆栈跟踪排序。
 		-t		按时间排序（默认）。
+		--sort <order>	指定排序顺序。排序语法是 [+|-]key[,[+|-]key[,...]]. 从
+						**标准格式说明符** 部分选择一个键。"+" 是可选的，因为默认方向是增加数字或字典顺序。
+						允许混合使用缩写键和全称键。
+
+		Examples:
+				./page_owner_sort <input> <output> --sort=n,+pid,-tgid
+				./page_owner_sort <input> <output> --sort=at
 
    其它函数:
 
 	Cull:
-		-c		通过比较堆栈跟踪而不是总块来进行剔除。
+		--cull <rules>
+				指定筛选规则。筛选语法是 key[,key[,...]]。在**标准格式说明符**部分选择一个多字母键
+
+		<rules> 是逗号分隔列表形式的单个参数，它提供了一种指定单个筛选规则的方法。下面的**标准格式说明
+				符**部分描述了可识别的关键字。<rules> 可以由键 k1,k2, ... 顺序指定，如下面的
+				STANDARD SORT KEYS 部分所述。允许混合使用缩写形式和完整形式的键。
 
+		Examples:
+				./page_owner_sort <input> <output> --cull=stacktrace
+				./page_owner_sort <input> <output> --cull=st,pid,name
+				./page_owner_sort <input> <output> --cull=n,f
 	Filter:
 		-f		过滤掉内存已被释放的块的信息。
+
+	Select:
+		--pid <pidlist>		通过 pid 进行选择。这将选择进程 ID 号出现在 <pidlist> 中的块。
+		--tgid <tgidlist>	通过 tgid 进行选择。这将选择线程组 ID 号出现在 <tgidlist> 中的块。
+		--name <cmdlist>	按任务名称选择。这将选择任务名称出现在 <cmdlist> 中的块。
+
+		<pidlist>、<tgidlist>、<cmdlist>是逗号分隔列表形式的单参数，它提供了一种指定单个选择规则的方法。
+
+
+		Examples:
+				./page_owner_sort <input> <output> --pid=1
+				./page_owner_sort <input> <output> --tgid=1,2,3
+				./page_owner_sort <input> <output> --name name1,name2
+
+标准格式说明符
+==============
+
+--sort 选项:
+
+        ======          ==========      ===================
+        缩写键          全称键          描述
+        ======          ==========      ===================
+        p               pid             进程 ID
+        tg              tgid            线程组 ID
+        n               name            任务名称
+        st              stacktrace      页面分配的调用栈
+        T               txt             块的全文
+        ft              free_ts         页面被释放的时间戳
+        at              alloc_ts        页面分配的时间戳
+        ======          ==========      ===================
+
+--curl 选项:
+
+        ======          ==========      ==================
+        缩写键          全称键          描述
+        ======          ==========      ==================
+        p               pid             进程 ID
+        tg              tgid            线程组 ID
+        n               name            任务名称
+        f               free            该页面是否已被释放
+        st              stacktrace      页面分配的调用栈
+        ======          ==========      ==================
-- 
2.7.4

[PATCH 2/2] doc/vm/page_owner.rst: Fix table display confusion

[Haowen Bai]

After make htmldocs, the table which is made of tab will
display all by one line. so we make a standard table for STANDARD
FORMAT SPECIFIERS description.

Signed-off-by: Haowen Bai <baihaowen@meizu.com>
---
 Documentation/vm/page_owner.rst | 34 ++++++++++++++++++++--------------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/Documentation/vm/page_owner.rst b/Documentation/vm/page_owner.rst
index 422d5fa99e84..a8505e3a3044 100644
--- a/Documentation/vm/page_owner.rst
+++ b/Documentation/vm/page_owner.rst
@@ -174,22 +174,28 @@ STANDARD FORMAT SPECIFIERS
 
 For --sort option:
 
-	KEY		LONG		DESCRIPTION
-	p		pid		process ID
-	tg		tgid		thread group ID
-	n		name		task command name
-	st		stacktrace	stack trace of the page allocation
-	T		txt		full text of block
-	ft		free_ts		timestamp of the page when it was released
-	at		alloc_ts	timestamp of the page when it was allocated
+        ====            ==========      ============================================
+        KEY             LONG            DESCRIPTION
+        ====            ==========      ============================================
+        p               pid             process ID
+        tg              tgid            thread group ID
+        n               name            task command name
+        st              stacktrace      stack trace of the page allocation
+        T               txt             full text of block
+        ft              free_ts         timestamp of the page when it was released
+        at              alloc_ts        timestamp of the page when it was allocated
         ator            allocator       memory allocator for pages
+        ====            ==========      ============================================
 
 For --curl option:
 
-	KEY		LONG		DESCRIPTION
-	p		pid		process ID
-	tg		tgid		thread group ID
-	n		name		task command name
-	f		free		whether the page has been released or not
-	st		stacktrace	stack trace of the page allocation
+        ====            ===========     ============================================
+        KEY             LONG            DESCRIPTION
+        ====            ===========     ============================================
+        p               pid             process ID
+        tg              tgid            thread group ID
+        n               name            task command name
+        f               free            whether the page has been released or not
+        st              stacktrace      stack trace of the page allocation
         ator            allocator       memory allocator for pages
+        ====            ===========     ============================================
-- 
2.7.4

Re: [PATCH 1/2] docs/zh_CN: sync with original text Documentation/vm/page_owner.rst

[Jonathan Neuschäfer]

[-- Attachment #1: Type: text/plain, Size: 139 bytes --]

On Mon, Apr 25, 2022 at 03:32:10PM +0800, Haowen Bai wrote:
[...]
> +--curl 选项:

This should be --cull, not --curl.


Jonathan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[Jonathan Corbet]

Haowen Bai <baihaowen@meizu.com> writes:

> This patches series will sync the translation of zh_CN as below:

So I got two sets of these patches sent within minutes...so I'm not sure
what to do with them.  Why did you send them twice.

Additionally:

 - Please put a version number on your patch postings

 - A brief note (below the "---") line on what changed is helpful to
   reviewers and maintainers alike.

Thanks,

jon

[PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[Haowen Bai]

This patches series will sync the translation of zh_CN as below:

Commit 8bd16bc0a081 ("tools/vm/page_owner_sort.c: support sorting blocks
by multiple keys") add sorting blocks by multiple keys usage
description.

Commit 78a0b94f3829 ("tools/vm/page_owner_sort.c: support for multi-value
selection in single argument") add multi-value selection in single
argument usage description.

Commit c89b3ad2dea2 ("doc/vm/page_owner.rst: remove content related to -c
option") remove  -c option usage description.

Commit 9c8a0a8e599f ("tools/vm/page_owner_sort.c: support for user-defined
culling rules") add user-defined culling rules usage description.

Commit 8ea8613a616a ("tools/vm/page_owner_sort.c: support for selecting by
PID, TGID or task command name") add selecting by PID, TGID or task
command name usage description.

Commit 194d52d771b8 ("tools/vm/page_owner_sort: support for sorting by
task command name") add sorting by task command name usage description.

But the original text table which is made of tab will display all
by one line. so we make a standard table for STANDARD FORMAT SPECIFIERS
description. I fix it up togeter by this patches series.


Haowen Bai (2):
  docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
  doc/vm/page_owner.rst: Fix table display confusion

 Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
 Documentation/vm/page_owner.rst                    | 34 +++++++-----
 2 files changed, 80 insertions(+), 15 deletions(-)

-- 
2.7.4

Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[Akira Yokosawa]

Hi,

On Fri, 22 Apr 2022 09:40:39 +0800,
baihaowen wrote:
> 在 4/21/22 8:26 PM, yanteng si 写道:
[...]
>> I'm not sure why, your different versions of the patch are in one
>> thread and they look like this inside
>> <https://lore.kernel.org/linux-doc/>：
>>
>> [PATCH V3] docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
>>  2022-04-20 12:46 UTC  (22+ messages)
>>     ` [PATCH V4] "
>>             ` [PATCH V5] "
>>                   ` [PATCH V6] "
>>                     ` [PATCH V7] "
>>                       ` [PATCH V8] "
>>                           ` [PATCH 0/2] "
>>                             ` [PATCH 1/2] docs/zh_CN: "
>>                             ` [PATCH 2/2] doc/vm/page_owner.rst: Fix
>> table display confusion
> 
> I always reply mail using below method:
> 
>   git send-email \
>     --in-reply-to='CAEensMxtk8zJPAM-u7hvYkaNTBrVdmhKA7YzVsKmTw0TcOE=Pw@mail.gmail.com' \
>     --to=siyanteng01@gmail.com \
>     --cc=alexs@kernel.org \
>     --cc=baihaowen@meizu.com \
>     --cc=corbet@lwn.net \
>     --cc=linux-doc@vger.kernel.org \
>     --cc=linux-kernel@vger.kernel.org \
>     --cc=seakeel@gmail.com \
>     --cc=siyanteng@loongson.cn \
>     --subject='Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst' \
>     /path/to/YOUR_REPLY
> 
> link previous

You are not following the recommendation described in
Section "Explicit In-Reply-To headers" of
Documentation/process/submitting-patches.rst
(or "15) 明确回复邮件头(In-Reply-To)" in
Documentation/translations/zh_CN/process/submitting-patches.rst).

        Thanks, Akira

Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[baihaowen]

在 4/21/22 8:26 PM, yanteng si 写道:
> Haowen Bai <baihaowen@meizu.com> 于2022年4月20日周三 11:07写道：
>> This patches series will sync the translation of zh_CN as below:
>>
>> Commit 8bd16bc0a081 ("tools/vm/page_owner_sort.c: support sorting blocks
>> by multiple keys") add sorting blocks by multiple keys usage
>> description.
>>
>> Commit 78a0b94f3829 ("tools/vm/page_owner_sort.c: support for multi-value
>> selection in single argument") add multi-value selection in single
>> argument usage description.
>>
>> Commit c89b3ad2dea2 ("doc/vm/page_owner.rst: remove content related to -c
>> option") remove  -c option usage description.
>>
>> Commit 9c8a0a8e599f ("tools/vm/page_owner_sort.c: support for user-defined
>> culling rules") add user-defined culling rules usage description.
>>
>> Commit 8ea8613a616a ("tools/vm/page_owner_sort.c: support for selecting by
>> PID, TGID or task command name") add selecting by PID, TGID or task
>> command name usage description.
>>
>> Commit 194d52d771b8 ("tools/vm/page_owner_sort: support for sorting by
>> task command name") add sorting by task command name usage description.
>>
>> But the original text table which is made of tab will display all
>> by one line. so we make a standard table for STANDARD FORMAT SPECIFIERS
>> description. I fix it up togeter by this patches series.
>>
>>
>> Haowen Bai (2):
>>   docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
>>   doc/vm/page_owner.rst: Fix table display confusion
> hi haowen,
>
> I'm not sure why, your different versions of the patch are in one
> thread and they look like this inside
> <https://lore.kernel.org/linux-doc/>：
>
> [PATCH V3] docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
>  2022-04-20 12:46 UTC  (22+ messages)
>     ` [PATCH V4] "
>             ` [PATCH V5] "
>                   ` [PATCH V6] "
>                     ` [PATCH V7] "
>                       ` [PATCH V8] "
>                           ` [PATCH 0/2] "
>                             ` [PATCH 1/2] docs/zh_CN: "
>                             ` [PATCH 2/2] doc/vm/page_owner.rst: Fix
> table display confusion

I always reply mail using below method:

  git send-email \
    --in-reply-to='CAEensMxtk8zJPAM-u7hvYkaNTBrVdmhKA7YzVsKmTw0TcOE=Pw@mail.gmail.com' \
    --to=siyanteng01@gmail.com \
    --cc=alexs@kernel.org \
    --cc=baihaowen@meizu.com \
    --cc=corbet@lwn.net \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=seakeel@gmail.com \
    --cc=siyanteng@loongson.cn \
    --subject='Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst' \
    /path/to/YOUR_REPLY

link previous

> This will make trouble for the maintainer，Usually, they should look like this：
>
> [PATCH 0/2] docs/zh_CN: add hmm and numa translation
>  2022-04-21 11:59 UTC  (3+ messages)
> ` [PATCH 1/2] docs/zh_CN: add vm hmm translation
> ` [PATCH 2/2] docs/zh_CN: add vm numa translation
>
>
> Thanks,
> Yanteng
>
>>  Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
>>  Documentation/vm/page_owner.rst                    | 34 +++++++-----
>>  2 files changed, 80 insertions(+), 15 deletions(-)
>>
>> --
>> 2.7.4
>>

-- 
Haowen Bai

Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[yanteng si]

Haowen Bai <baihaowen@meizu.com> 于2022年4月20日周三 11:07写道：
>
> This patches series will sync the translation of zh_CN as below:
>
> Commit 8bd16bc0a081 ("tools/vm/page_owner_sort.c: support sorting blocks
> by multiple keys") add sorting blocks by multiple keys usage
> description.
>
> Commit 78a0b94f3829 ("tools/vm/page_owner_sort.c: support for multi-value
> selection in single argument") add multi-value selection in single
> argument usage description.
>
> Commit c89b3ad2dea2 ("doc/vm/page_owner.rst: remove content related to -c
> option") remove  -c option usage description.
>
> Commit 9c8a0a8e599f ("tools/vm/page_owner_sort.c: support for user-defined
> culling rules") add user-defined culling rules usage description.
>
> Commit 8ea8613a616a ("tools/vm/page_owner_sort.c: support for selecting by
> PID, TGID or task command name") add selecting by PID, TGID or task
> command name usage description.
>
> Commit 194d52d771b8 ("tools/vm/page_owner_sort: support for sorting by
> task command name") add sorting by task command name usage description.
>
> But the original text table which is made of tab will display all
> by one line. so we make a standard table for STANDARD FORMAT SPECIFIERS
> description. I fix it up togeter by this patches series.
>
>
> Haowen Bai (2):
>   docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
>   doc/vm/page_owner.rst: Fix table display confusion
hi haowen,

I'm not sure why, your different versions of the patch are in one
thread and they look like this inside
<https://lore.kernel.org/linux-doc/>：

[PATCH V3] docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
 2022-04-20 12:46 UTC  (22+ messages)
    ` [PATCH V4] "
            ` [PATCH V5] "
                  ` [PATCH V6] "
                    ` [PATCH V7] "
                      ` [PATCH V8] "
                          ` [PATCH 0/2] "
                            ` [PATCH 1/2] docs/zh_CN: "
                            ` [PATCH 2/2] doc/vm/page_owner.rst: Fix
table display confusion

This will make trouble for the maintainer，Usually, they should look like this：

[PATCH 0/2] docs/zh_CN: add hmm and numa translation
 2022-04-21 11:59 UTC  (3+ messages)
` [PATCH 1/2] docs/zh_CN: add vm hmm translation
` [PATCH 2/2] docs/zh_CN: add vm numa translation


Thanks,
Yanteng

>
>  Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
>  Documentation/vm/page_owner.rst                    | 34 +++++++-----
>  2 files changed, 80 insertions(+), 15 deletions(-)
>
> --
> 2.7.4
>

[PATCH 0/2] sync with original text Documentation/vm/page_owner.rst

[Haowen Bai]

This patches series will sync the translation of zh_CN as below:

Commit 8bd16bc0a081 ("tools/vm/page_owner_sort.c: support sorting blocks
by multiple keys") add sorting blocks by multiple keys usage
description.

Commit 78a0b94f3829 ("tools/vm/page_owner_sort.c: support for multi-value
selection in single argument") add multi-value selection in single
argument usage description.

Commit c89b3ad2dea2 ("doc/vm/page_owner.rst: remove content related to -c
option") remove  -c option usage description.

Commit 9c8a0a8e599f ("tools/vm/page_owner_sort.c: support for user-defined
culling rules") add user-defined culling rules usage description.

Commit 8ea8613a616a ("tools/vm/page_owner_sort.c: support for selecting by
PID, TGID or task command name") add selecting by PID, TGID or task
command name usage description.

Commit 194d52d771b8 ("tools/vm/page_owner_sort: support for sorting by
task command name") add sorting by task command name usage description.

But the original text table which is made of tab will display all
by one line. so we make a standard table for STANDARD FORMAT SPECIFIERS
description. I fix it up togeter by this patches series.


Haowen Bai (2):
  docs/zh_CN: sync with original text Documentation/vm/page_owner.rst
  doc/vm/page_owner.rst: Fix table display confusion

 Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
 Documentation/vm/page_owner.rst                    | 34 +++++++-----
 2 files changed, 80 insertions(+), 15 deletions(-)

-- 
2.7.4