Re: [PATCH v7 7/7] docs/zh_CN: update DAMON LRU sort 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:
- [Medium] The Chinese translation introduces documentation for module parameters and warnings that do not exist in the kernel source code or the English documentation.
--

--- Patch [7]: [PATCH v7 7/7] docs/zh_CN: update DAMON LRU sort translation ---
commit adc70b54833d5e4ddeaaa02dcb9d816dc74cba6a
Author: Doehyun Baek <[email protected]>

docs/zh_CN: update DAMON LRU sort translation

Update the Chinese translation of .../admin-guide/mm/damon/lru_sort.rst.

Update the translation through commit c9380cdd949d
("Docs/{admin-guide,mm}/damon: fix DAMON documentation details").

> diff --git a/Documentation/translations/zh_CN/admin-guide/mm/damon/lru_sort.rst b/Documentation/translations/zh_CN/admin-guide/mm/damon/lru_sort.rst
> index 03d33c710604c..6af0dd59ef5db 100644
> --- a/Documentation/translations/zh_CN/admin-guide/mm/damon/lru_sort.rst
> +++ b/Documentation/translations/zh_CN/admin-guide/mm/damon/lru_sort.rst

[ ... ]

> @@ -71,6 +71,41 @@ commit_inputs
>  ,DAMON_LRU_SORT会再次读取除了 ``enabled`` 之外的参数。读取完成后,这个参数会被
>  设置为 ``N`` 。如果在读取时发现有无效参数,DAMON_LRU_SORT会被关闭。
>  
> +一旦向该参数写入 ``Y``,用户在再次读取 ``commit_inputs`` 返回 ``N`` 之前,
> +不得写入任何参数。如果用户违反该规则,内核可能表现出未定义行为。

[Severity: Medium]
Does this introduce a warning about undefined behavior that does not exist in
the upstream English documentation?

> +
> +active_mem_bp
> +-------------

[Severity: Medium]
Is active_mem_bp a valid parameter for damon_lru_sort? It appears this might
be documenting a parameter that does not exist in the mainline kernel source
or the English documentation.

> +
> +期望的活跃内存与[非]活跃内存比率,单位为 bp(1/10,000)。
> +
> +在保持其他配额设置的上限的同时,DAMON_LRU_SORT 会自动增减配额的有效水平,
> +目标是让热/冷内存的 LRU [降低]优先级处理产生该活跃内存与[非]活跃内存比率。
> +值为零表示禁用该自动调优功能。
> +
> +默认禁用。
> +
> +autotune_monitoring_intervals
> +-----------------------------

[Severity: Medium]
Does the autotune_monitoring_intervals parameter exist for damon_lru_sort?

> +
> +如果该参数设置为 ``Y``,DAMON_LRU_SORT 会自动调优 DAMON 的采样和聚集间隔。
> +自动调优的目标是在每个 DAMON 快照中捕获有意义数量的访问事件,同时将采样
> +间隔限制在最小 5 毫秒、最大 10 秒。将其设置为 ``N`` 会禁用自动调优。
> +
> +默认禁用。
> +
> +filter_young_pages
> +------------------

[Severity: Medium]
Like the parameters above, does filter_young_pages exist in the mainline
implementation for damon_lru_sort?

> +
> +相应地为 LRU [降低]优先级处理过滤[非]年轻页。
> +
> +如果设置该参数,则在每次 LRU [降低]优先级处理操作前再次检查页级访问

[ ... ]

> @@ -176,14 +213,34 @@ monitor_region_start
>  
>  目标内存区域的起始物理地址。
>  
> -DAMON_LRU_SORT要处理的目标内存区域的起始物理地址。默认,使用系统最大内存。
> +DAMON_LRU_SORT要处理的目标内存区域的起始物理地址。默认,使用系统的整个物理内存。
>  
>  monitor_region_end
>  ------------------
>  
>  目标内存区域的结束物理地址。
>  
> -DAMON_LRU_SORT要处理的目标内存区域的结束物理地址。默认,使用系统最大内存。
> +DAMON_LRU_SORT要处理的目标内存区域的结束物理地址。默认,使用系统的整个物理内存。
> +
> +addr_unit
> +---------
> +
> +内存地址和字节数的缩放因子。
> +
> +该参数用于设置和获取 DAMON_RECLAIM 的 DAMON 实例的 :ref:`地址单位 <damon_design_addr_unit_zh_CN>` 参数。

[Severity: Medium]
Is addr_unit a valid parameter for damon_lru_sort? Also, should this
reference DAMON_LRU_SORT instead of DAMON_RECLAIM?

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=7
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.