[Intel-gfx] [RFC PATCH 157/162] drm/i915: Improve accuracy of eviction stats
Chris Wilson
chris at chris-wilson.co.uk
Fri Nov 27 14:40:28 UTC 2020
Quoting Matthew Auld (2020-11-27 12:07:13)
> From: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
>
> Current code uses jiffie time to do the accounting and then does:
>
> diff = jiffies - start;
> msec = diff * 1000 / HZ;
> ...
> atomic_long_add(msec, &i915->time_swap_out_ms);
>
> If we assume jiffie can be as non-granular as 10ms and that the current
> accounting records all evictions faster than one jiffie as infinite speed,
> we can end up over-estimating the reported eviction throughput.
>
> Fix this by accumulating ktime_t and only dividing to more user friendly
> granularity at presentation time (debugfs read).
>
> At the same time consolidate the code a bit and convert from multiple
> atomics to single seqlock per stat.
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> Cc: CQ Tang <cq.tang at intel.com>
> Cc: Sudeep Dutt <sudeep.dutt at intel.com>
> Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
A lot of effort to fix up patches after the fact, might as well make it
a real PMU interface.
-Chris
More information about the Intel-gfx
mailing list