The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Weijie Yuan <wy@wyuan.org>
To: Dongliang Mu <dzm91@hust.edu.cn>
Cc: Alex Shi <alexs@kernel.org>, Yanteng Si <si.yanteng@linux.dev>,
	Jonathan Corbet <corbet@lwn.net>,
	Shuah Khan <skhan@linuxfoundation.org>,
	Konstantin Ryabitsev <konstantin@linuxfoundation.org>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/4] Update maintainer-pgp-guide.rst Chinese translation
Date: Fri, 21 Aug 2026 22:01:57 +0800	[thread overview]
Message-ID: <aohaVdbMM2T871-s@wyuan.org> (raw)
In-Reply-To: <b7c913d1-578b-47c1-a2b7-c1420deb4d21@hust.edu.cn>

On Fri, Aug 21, 2026 at 09:45:26PM +0800, Dongliang Mu wrote:
> On 8/21/26 9:11 PM, Weijie Yuan wrote:
> > On Fri, Aug 21, 2026 at 08:57:46PM +0800, Dongliang Mu wrote:
> > > On 8/21/26 8:48 PM, Weijie Yuan wrote:
> > > > On Fri, Aug 21, 2026 at 08:17:06PM +0800, Dongliang Mu wrote:
> > > > [...]
> > > > > > (I definitely won't admit that the log file annoyed me a few times
> > > > > > because I accidentally included it with 'git add .' and then had to 'git
> > > > > > commit --amend' right after) ;-)
> > > > > My experience is to not use 'git add .'. In Linux, hidden files are not
> > > > > listed by default.
> > > > Yes, I was being lazy to type the full pach/full file name, oops.
> > > > 
> > > > While hidden files are not listed by default on Linux, thankfully the
> > > > kernel source tree ignores .* via .gitignore. But 'git add .' is indeed
> > > > a bad habit.
> > > > 
> > > > Btw, git commit -v, or setting commit.verbose = true, can really save
> > > > one at the last minute. (as mentioned in our how-to doc) And sadly I
> > > > actually happened to use another computer to write patches last night,
> > > > which I hadn't configured it. XD
> > > Actually these labor-tensive works should be done by LLM. This is one of the
> > > right directions of LLM usage.
> > > 
> > > You can write a simple skill to help you complete these works.
> > That makes sense. I'll give it a try. Still, given that the kernel
> > community tends to be a bit conservative, I've generally tried to do
> > things myself. Of course, the most important thing is that we supervise
> > what the LLM is doing and make sure it follows our actual intent rather
> > than hallucinating.
> 
> My experience is to always double check the result of LLM.
> 
> However, to be honest, translation is not a complicated task. Sometimes LLM
> proposes greater translation than me.

Aha, yes, I said that in [PATCH 1/4] too. ;-)

> And LLM is really good at capturing some common problems in the patchset.

Yes, agreed.

> > > > > > > > Btw I've been trying to know more about Sphinx and kernel-docs, hope my
> > > > > > > > mistake won't happen again.
> > > > > > > That change can fix problem. The underlying problem is RST syntax. Maybe you
> > > > > > > can have a look at this or query it in the LLM.
> > > > > > Absolutely. I did have relatively little experience writing rst before.
> > > > > > But that's definitely no excuse - I'll try to pick it up quickly.
> > > > > LLM is really useful to capture some syntax and semantics issues. Therefore,
> > > > > before submitting patch, use LLM to scan problem in this issue.
> > > > Got it, thanks.
> > > > 
> > > > > I am writing a draft skill for reviewing patches of kernel Chinese
> > > > > documentation.
> > > > Nice! I've been trying "b4 review tui" these days and I also made some
> > > > modifications to its pre-configured prompt to fit our specific
> > > > workflows:
> > > > 
> > > >     https://git.kernel.org/pub/scm/utils/b4/b4.git/tree/misc/agent-reviewer.md
> > > > 
> > > > b4 review tui 'agents' - AI-assisted review:
> > > > 
> > > >     https://b4.docs.kernel.org/en/latest/reviewer/getting-started.html#ai-assisted-review
> > > > 
> > > > Sometimes my agent makes false alarms, but at other times it does manage
> > > > to identify the issues that we have overlooked. This is probably also
> > > > one of the reasons why the community is gradually becoming more
> > > > accepting of LLMs.
> > > Ah, this is interesting.
> > > 
> > > My skill is to use LLM to search the patch name, use b4 to apply patches and
> > > finally ask LLM to refer to how-to.rst and review the patch or patchset.
> > As Chen-Yu mentioned before, I´ve basically trained myself to find
> > almost any email in the lore archives manually. ;-)
> > 
> > And yes, every time before I send a patch, I have an LLM check whether
> > it follows the instructions in the how-to.rst.
> 
> Please take a look at my review:
> 
> https://lore.kernel.org/all/b45c6389-a048-4a94-8649-ab0c5a80fac2@hust.edu.cn/
> 
> The first comment is from Codex + ChatGPT.
> 
> And my skill complains the following warning:
> 
> https://lore.kernel.org/all/7724c093-cb99-490d-b632-1381e16962ff@hust.edu.cn/

I'm looking at right now ;-) because I'm at the "blame" line already.

> > 
> > > And I think we could try to add our subsystem to sashiko. What we need to do
> > > is to write some subsystem-related prompts.
> > I was thinking about this a few days ago. I'd actually planned to talk
> > to the Sashiko maintainer about it first, but I haven´t gotten around
> > to it yet. As far as I know, though, Sashiko can currently only be
> > enabled based on mailing lists; it can't filter or match patches
> > specifically for the Chinese translation subsystem.
> > 
> > Overall, though, I´m in favor of enabling Sashiko for us. Having an
> > extra pair of automated eyes certainly doesn't hurt. That said, some
> > settings may be worth discussing - for example, whether the Sashiko bot
> > should be allowed to send review comments directly to the original patch
> > thread.
> 
> Maybe we can do some tricks in the prompt. docs/zh_CN: is an obvious signal
> for us.

Yes, I'll take a look at it later.

      reply	other threads:[~2026-08-21 14:02 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-20 17:13 [PATCH 0/4] Update maintainer-pgp-guide.rst Chinese translation Weijie Yuan
2026-08-20 17:13 ` [PATCH 1/4] docs/zh_CN: update maintainer-pgp-guide translation Weijie Yuan
2026-08-20 17:13 ` [PATCH 2/4] docs/zh_CN: clarify free Nitrokey requirements Weijie Yuan
2026-08-20 17:13 ` [PATCH 3/4] docs/zh_CN: update maintainer-pgp-guide kernel.org link Weijie Yuan
2026-08-20 17:13 ` [PATCH 4/4] docs/zh_CN: add kernel.org trust repository reference Weijie Yuan
2026-08-20 17:38   ` Weijie Yuan
2026-08-21  4:14     ` Dongliang Mu
2026-08-21  5:14       ` Weijie Yuan
2026-08-21  6:54 ` [PATCH 0/4] Update maintainer-pgp-guide.rst Chinese translation Dongliang Mu
2026-08-21  7:22   ` Weijie Yuan
2026-08-21 11:18     ` Dongliang Mu
2026-08-21 11:56       ` Weijie Yuan
2026-08-21 12:17         ` Dongliang Mu
2026-08-21 12:48           ` Weijie Yuan
2026-08-21 12:57             ` Dongliang Mu
2026-08-21 13:11               ` Weijie Yuan
2026-08-21 13:45                 ` Dongliang Mu
2026-08-21 14:01                   ` Weijie Yuan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aohaVdbMM2T871-s@wyuan.org \
    --to=wy@wyuan.org \
    --cc=alexs@kernel.org \
    --cc=corbet@lwn.net \
    --cc=dzm91@hust.edu.cn \
    --cc=konstantin@linuxfoundation.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=si.yanteng@linux.dev \
    --cc=skhan@linuxfoundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox