Re: [PATCH] block: fix aux stat accumulation destination

From: Yu Kuai
Date: Thu Dec 25 2025 - 11:45:03 EST


Hi,

在 2025/12/22 10:19, shechenglong 写道:
> Route bfqg_stats_add_aux() time accumulation into the destination
> stats object instead of the source, aligning with other stat fields.
>
> Signed-off-by: shechenglong <shechenglong@xxxxxxxxxxx>
> ---
> block/bfq-cgroup.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Please change the title and follow existing prefix:

block, bfq: fix aux stat accumulation destination

Otherwise, feel free to add:

Reviewed-by: Yu Kuai <yukuai@xxxxxxxxx>

>
> diff --git a/block/bfq-cgroup.c b/block/bfq-cgroup.c
> index 9fb9f3533150..6a75fe1c7a5c 100644
> --- a/block/bfq-cgroup.c
> +++ b/block/bfq-cgroup.c
> @@ -380,7 +380,7 @@ static void bfqg_stats_add_aux(struct bfqg_stats *to, struct bfqg_stats *from)
> blkg_rwstat_add_aux(&to->merged, &from->merged);
> blkg_rwstat_add_aux(&to->service_time, &from->service_time);
> blkg_rwstat_add_aux(&to->wait_time, &from->wait_time);
> - bfq_stat_add_aux(&from->time, &from->time);
> + bfq_stat_add_aux(&to->time, &from->time);
> bfq_stat_add_aux(&to->avg_queue_size_sum, &from->avg_queue_size_sum);
> bfq_stat_add_aux(&to->avg_queue_size_samples,
> &from->avg_queue_size_samples);

--
Thansk,
Kuai