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.
prev parent 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