Re: [RFC PATCH v10 3/4] blk-iocost: add iocost_ioc_tick tracepoint for per-period device summary

From: Tejun Heo

Date: Fri Oct 16 2026 - 13:55:20 EST


Hello, Tao.

On Sat, 03 Oct 2026 20:40:44 +0800, Tao Cui wrote:
> + * Periodic per-device summary, emitted once per period from the tail of
> + * ioc_timer_fn(). Unlike the state-change events above, this fires every

The event is emitted before the vrate adjustment and the period
transition, not from the tail.

> + /* whether the controller still has active iocgs */
> + __field(int, running)

nr_active counts every entry on ioc->active_iocgs, and ioc->lock is held
from the count to the emit point, so running always equals active != 0.
Maybe drop the field?

Thanks.

--
tejun