Received-SPF: pass (google.com: domain of linux-kernel+bounces-128811-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <d6d188a3-5604-4848-8b45-a7aff2535611@rockwork.org>
Date: Wed, 3 Apr 2024 06:58:13 +0800
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] docs/zh_CN: core-api: Update translation of workqueue.rst
 to 6.9-rc1
To: Yanteng Si <siyanteng@loongson.cn>, alexs@kernel.org, corbet@lwn.net
Cc: zhoubinbin@loongson.cn, tj@kernel.org, wangjinchao@xfusion.com,
 linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20240331172116.257780-1-rockrush@rockwork.org>
 <2932e9f7-5d5f-4331-8043-7aa85e9044a4@loongson.cn>
Content-Language: en-US
From: Xingyou Chen <rockrush@rockwork.org>
In-Reply-To: <2932e9f7-5d5f-4331-8043-7aa85e9044a4@loongson.cn>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

On 4/2/24 19:47, Yanteng Si wrote:
> Hi xingyou,
> 
> 在 2024/4/1 01:21, Xingyou Chen 写道:
>> Significant changes have been made to workqueue, and there are staging
>> works transferring from tasklet, while the current translation doesn't
>> include description around WQ_BH, an update seems to be helpful.
>>
>> Synchronize translation from upstream commit 3bc1e711c26b
>> ("workqueue: Don't implicitly make UNBOUND workqueues w/ 
>> @max_active==1 ordered")
> It seems that you cannot pass the build test. You need to find a way to 
> silence these warnings
There are some "Title underline too short" warnings, I'll send an 
updated one.
> 
> 
> /home/siyanteng/doc/linux/Documentation/core-api/workqueue:776: 
> ./include/linux/workqueue.h:493: WARNING: Inline literal start-string 
> without end-string
It's "%WQ_*" in workqueue.h, and didn't found note on this labeling rule 
after quick search, I'll do an in depth reading around reStructuredText 
and Sphinx. We may simply remove "%" here, or protect asterisk as "%WQ_\*".
> 
> ...
> 
> 
> You need to follow the steps below:
> 
> $ ./scripts/sphinx-pre-install
> 
>       install them.
> 
> $ make cleandocs
> 
> $ make htmldocs
> 
>      silence warnings
> 
> 
> 
> Thanks,
> 
> Yanteng
>