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

Haowen Bai posted 2 patches 4 years ago
Only 1 patches received!
There is a newer version of this series
Documentation/translations/zh_CN/vm/page_owner.rst | 61 +++++++++++++++++++++-
Documentation/vm/page_owner.rst                    | 34 +++++++-----
2 files changed, 80 insertions(+), 15 deletions(-)
[PATCH 0/2] sync with original text Documentation/vm/page_owner.rst
Posted by Haowen Bai 4 years ago
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
Posted by yanteng si 4 years ago
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
>
Re: [PATCH 0/2] sync with original text Documentation/vm/page_owner.rst
Posted by baihaowen 4 years ago
在 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
Posted by Akira Yokosawa 4 years ago
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