Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t v1 0/1] runner: Use BOOTTIME for time checks
@ 2025-03-28 16:33 Kamil Konieczny
  2025-03-28 16:33 ` [PATCH i-g-t v1 1/1] runner/executor: Use bootime " Kamil Konieczny
                   ` (5 more replies)
  0 siblings, 6 replies; 9+ messages in thread
From: Kamil Konieczny @ 2025-03-28 16:33 UTC (permalink / raw)
  To: igt-dev; +Cc: Kamil Konieczny, Ewelina Musial, Karol Krol, Petri Latvala

Instead of using igt_gettime() function, use local runner function
with CLOCK_BOOTTIME, which counts in also suspend time.

You could look into this here:

https://intel-gfx-ci.01.org/tree/intel-xe/xe-2730-873b1a50bb4394e95332cfa611aa6463de6b7cb0/shard-bmg-7/igt_runner14.txt

search for 2x-flip-vs-suspend and see difference:

[77.376851] [020/123] (932s left) kms_flip (2x-flip-vs-suspend)

[305.147183] Subtest 2x-flip-vs-suspend: SUCCESS (40.168s)
[314.469660] [021/123] (891s left) kms_addfb_basic (bad-pitch-999)

This shows test runs for around 932 - 891 = 40 seconds, while wall
time shows it took around 305 - 77 = 228 seconds, so it should had
less then 704s left.

Separate issue seen from this log is 9s difference until
next test run.

Link for igt_runner14.txt is available here:

https://intel-gfx-ci.01.org/tree/intel-xe/xe-2730-873b1a50bb4394e95332cfa611aa6463de6b7cb0/shard-bmg-7/igt@xe_pmu@engine-activity-idle@engine-drm_xe_engine_class_video_decode0.html

Cc: Ewelina Musial <ewelina.musial@intel.com>
Cc: Karol Krol <karol.krol@intel.com>
Cc: Petri Latvala <adrinael@adrinael.net>

Kamil Konieczny (1):
  runner/executor: Use bootime for time checks

 runner/executor.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

-- 
2.49.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2025-04-06 20:03 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-28 16:33 [PATCH i-g-t v1 0/1] runner: Use BOOTTIME for time checks Kamil Konieczny
2025-03-28 16:33 ` [PATCH i-g-t v1 1/1] runner/executor: Use bootime " Kamil Konieczny
2025-04-02 10:17   ` Piatkowski, Dominik Karol
2025-04-02 11:15   ` Peter Senna Tschudin
2025-03-29  2:42 ` ✓ Xe.CI.BAT: success for runner: Use BOOTTIME " Patchwork
2025-03-29  2:56 ` ✓ i915.CI.BAT: " Patchwork
2025-03-29  8:13 ` ✗ i915.CI.Full: failure " Patchwork
2025-03-29 21:16 ` ✗ Xe.CI.Full: " Patchwork
2025-04-06 20:03 ` Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox