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

[Alex Shi]

On Wed, Jan 19, 2022 at 11:28 AM teng sterling <sterlingteng@xxxxxxxxx> wrote:
>
> yong w <yongw.pur@xxxxxxxxx> 于2022年1月18日周二 20:38写道：
> >
> > Hello,
> > Is this patch OK?
> >
> > yong w <yongw.pur@xxxxxxxxx> 于2022年1月13日周四 22:56写道：
> > >
> > > Hello,
> > > teng sterling <sterlingteng@xxxxxxxxx> 于2022年1月13日周四 10:11写道：
> > > >
> > > > Hi Yong,
> > > > <yongw.pur@xxxxxxxxx> 于2022年1月13日周四 07:27写道：
> > > > >
> > > > > From: wangyong <wang.yong12@xxxxxxxxxx>
> > > > >
> > > > > 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@xxxxxxxxxx/
> > > > 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@xxxxxxxxxx>
> > > > > Reviewed-by: Yang Yang <yang.yang29@xxxxxxxxxx>
> > > > 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@xxxxxxxxxx>

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