linux-kernel - Re: [PATCH] docs/zh_CN: Update zh_CN/accounting/delay-accounting.rst

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJy-Amne22+2cApoeEAGSc7d7ppSQja7pLX6SY_3+0wr8i3Asw@mail.gmail.com>
Date:   Wed, 19 Jan 2022 11:41:43 +0800
From:   Alex Shi <seakeel@...il.com>
To:     teng sterling <sterlingteng@...il.com>
Cc:     yong w <yongw.pur@...il.com>, Alex Shi <alexs@...nel.org>,
        Yanteng Si <siyanteng@...ngson.cn>,
        Jonathan Corbet <corbet@....net>,
        Linux Doc Mailing List <linux-doc@...r.kernel.org>,
        LKML <linux-kernel@...r.kernel.org>, wang.yong12@....com.cn,
        Yang Yang <yang.yang29@....com.cn>
Subject: Re: [PATCH] docs/zh_CN: Update zh_CN/accounting/delay-accounting.rst

On Wed, Jan 19, 2022 at 11:28 AM teng sterling <sterlingteng@...il.com> wrote:
>
> yong w <yongw.pur@...il.com> 于2022年1月18日周二 20:38写道：
> >
> > Hello,
> > Is this patch OK?
> >
> > yong w <yongw.pur@...il.com> 于2022年1月13日周四 22:56写道：
> > >
> > > Hello,
> > > teng sterling <sterlingteng@...il.com> 于2022年1月13日周四 10:11写道：
> > > >
> > > > Hi Yong,
> > > > <yongw.pur@...il.com> 于2022年1月13日周四 07:27写道：
> > > > >
> > > > > From: wangyong <wang.yong12@....com.cn>
> > > > >
> > > > > Update zh_CN/accounting/delay-accounting.rst.
> > > > > The document modification has been merged which refers to the following link:
> > > > > https://lore.kernel.org/all/1639583021-92977-1-git-send-email-wang.yong12@zte.com.cn/
> > > > The normal way to handle this is to add an appropriate Fixes tag.
> Since 4d3959d65968(Documentation/accounting/delay-accounting.rst: add
> thrashing page cache and direct compact) Modified the original
> document，
> It seems we forgot to update the Chinese documentation, let's update it now.
> > > >
> > > > By the way, this patch should be submitted together as part of that
> > > > patch series where you modified the original documentation. Also check
> > > > out the documentation for other languages and send it in the same way.
> > > > If you get stuck, simply --CC the maintainer is an elegant way to
> > > > handle it. >_<
> > > The previous patch has been merged into linux-next. There was no
> > > Chinese translation when I submitted it.
> > > At present, this document is only translated into Chinese, which was
> > > translated not long ago.
> > > So I update it in this patch.
> > > >
> > > > >
> > > > > Signed-off-by: wangyong <wang.yong12@....com.cn>
> > > > > Reviewed-by: Yang Yang <yang.yang29@....com.cn>
> > > > This looks like the first version of the patch and doesn't seem to
> > > > have been reviewed by anyone yet, so you can't sign it instead of
> > > > someone else.
> > > Yang Yang reviewed it offline.
> I'm not sure if this is appropriate, but lore is missing the review
> process and the mailing list developers don't know what's going on,
> which in a way makes it more difficult for future developers to learn.

Right. The better way is asking Yang Yang to give a reviewed-by in
mailing list next time. Otherwise, the private tag can't give a good
support.


for this patch. Reviewed-by: Alex Shi <alexs@...nel.org>

>
> Apart from the above，LGTM.
>
> Thanks,
> Yanteng
> > > >
> > > > Thanks,
> > > > Yanteng
> > > Thanks for your reply.