Re: [PATCH v8 3/7] docs/zh_CN: update DAMON index translation

[email protected]
Newsgroups dev.linux.lists.damon
Message-ID <[email protected]>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The Chinese documentation adds a translation and index entry for DAMON_STAT (`stat.rst`), but this feature and its corresponding English documentation do not exist in the mainline kernel.
--

--- Patch [3]: [PATCH v8 3/7] docs/zh_CN: update DAMON index translation ---
commit 21d805a6bcb98f821e0042d118c9ffbb71686e3a
Author: Doehyun Baek <[email protected]>

docs/zh_CN: update DAMON index translation

This commit updates the Chinese translation of the DAMON documentation index.
It also updates the translation up through commit c9380cdd949d.

> diff --git a/Documentation/translations/zh_CN/admin-guide/mm/damon/index.rst b/Documentation/translations/zh_CN/admin-guide/mm/damon/index.rst
> index 6f8676a50b381..77bbfa7c25dcb 100644
> --- a/Documentation/translations/zh_CN/admin-guide/mm/damon/index.rst
> +++ b/Documentation/translations/zh_CN/admin-guide/mm/damon/index.rst

[ ... ]

> @@ -23,7 +23,4 @@
>     usage
>     reclaim
>     lru_sort
> -
> -
> -
> -
> +   stat

[Severity: Low]
Does this mistakenly reference a document that doesn't exist in the mainline
kernel?

The English documentation doesn't appear to have a corresponding DAMON_STAT
feature or stat.rst file. Including it in the toctree might cause Sphinx
build warnings and mislead readers by describing an out-of-tree or
non-existent feature.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=3
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.