[PATCH i-g-t] tests/intel/xe_drm_fdinfo: Extend mercy to the upper end
Lucas De Marchi
lucas.demarchi at intel.com
Mon Aug 26 13:03:50 UTC 2024
On Sat, Aug 24, 2024 at 02:24:37PM GMT, Lucas De Marchi wrote:
>When we are processing the fdinfo of each client, the gpu time is read
>first, and then later all the exec queues are accumulated. It's thus
>possible that the total gpu time is smaller than the time reported in
>the exec queues. A preemption in the middle of second sample would
>exaggerate the problem:
> total_cycles cycles
> s1: read gpu time |
> s1: read exec queues times | *
> | *
> .. | *
> | *
> s2: read gpu time | *
> -> prempted *
on a second thought, this reasoning seems wrong... because we do
call spin_sync_end() before s2. So delta for exec queue time
shouldn't be larger than the the one for gpu time (since in s1 we
got the gpu timestamp before).... need to check if the sync end is
actually doing its job, or if there's something else going on.
I couldn't actually reproduce this > 100% on my end.
Lucas De Marchi
More information about the igt-dev
mailing list