Re: [RFC PATCH 9/9] Docs/mm/damon/design: clarify bp is basis point

[Randy Dunlap]

On 9/12/26 1:08 PM, SJ Park wrote:
> DAMON design document uses "bp" for "basis point" in multiple places.
> Because it is not clearly mentioned, it is difficult to understand what
> "bp" stands for.  Add the clarification.
> 
> Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Closes: https://lore.kernel.org/107dc6ba-697e-4b25-ba3e-8ce2499cac9a@xxxxxxxxxxxxx
> Signed-off-by: SJ Park <sj@xxxxxxxxxx>

Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
Thanks.

> ---
>  Documentation/mm/damon/design.rst | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/mm/damon/design.rst b/Documentation/mm/damon/design.rst
> index 0a86792f90a18..e82390e77a70a 100644
> --- a/Documentation/mm/damon/design.rst
> +++ b/Documentation/mm/damon/design.rst
> @@ -445,7 +445,7 @@ users to set the aimed amount of access events to observe via DAMON within
>  given time interval.  The target can be specified by the user as a ratio of
>  DAMON-observed access events to the theoretical maximum amount of the events
>  (``access_bp``) that measured within a given number of aggregations
> -(``aggrs``).
> +(``aggrs``).  The ratio is in basis point (bp or 1/10,000).
>  
>  The DAMON-observed access events are calculated in byte granularity based on
>  DAMON :ref:`region assumption <damon_design_region_based_sampling>`.  For
> @@ -717,7 +717,8 @@ mechanism tries to make ``current_value`` of ``target_metric`` be same to
>    in microseconds that measured from last quota reset to next quota reset.
>    DAMOS does the measurement on its own, so only ``target_value`` need to be
>    set by users at the initial time.  In other words, DAMOS does self-feedback.
> -- ``node_mem_used_bp``: Specific NUMA node's used memory ratio in bp (1/10,000).
> +- ``node_mem_used_bp``: Specific NUMA node's used memory ratio in basis point
> +  (bp or 1/10,000).
>  - ``node_mem_free_bp``: Specific NUMA node's free memory ratio in bp (1/10,000).
>  - ``node_memcg_used_bp``: Specific cgroup's node used memory ratio for a
>    specific NUMA node, in bp (1/10,000).

-- 
~Randy