Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t v2 0/8] tests/core_hotunplug: Make it fail only once
@ 2024-08-05 14:37 Kamil Konieczny
  2024-08-05 14:37 ` [PATCH i-g-t v2 1/8] lib/drmtest: add function for retriving chipset Kamil Konieczny
                   ` (11 more replies)
  0 siblings, 12 replies; 17+ messages in thread
From: Kamil Konieczny @ 2024-08-05 14:37 UTC (permalink / raw)
  To: igt-dev
  Cc: Kamil Konieczny, Andrzej Hajda, Janusz Krzysztofik,
	Zbigniew Kempczyński

Test core_hotunplug has some checks outside of subtest which
could turn it into a failed subtest. When it do a fail, then every
other fixture will also fail and repeats printing same logs. Changed
that into check performed only once when it is really needed, e.g. only
after an actual subtest was done.

Added also some helpers in drmtest lib to print what driver we are testing
and make sure we reload the same, not some other.

v1 was sent here: https://patchwork.freedesktop.org/series/134626/

v2: Added few more prints into subtest to better see after which step
  it started failing, changes series title

Cc: Andrzej Hajda <andrzej.hajda@intel.com>
Cc: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: "Zbigniew Kempczyński" <zbigniew.kempczynski@intel.com>

Kamil Konieczny (8):
  lib/drmtest: add function for retriving chipset
  tests/core_hotunplug: set known chipset before tests
  lib/drmtest: Allow to get drm device name
  tests/core_hotunplug: Open the same driver
  tests/core_hotunplug: Fix device close
  tests/core_hotunplug: Skip if no render available
  tests/core_hotunplug: Check health once after subtest
  tests/core_hotunplug: Print info between steps

 lib/drmtest.c          |  35 +++++++++-
 lib/drmtest.h          |   4 ++
 tests/core_hotunplug.c | 153 +++++++++++++++++++++++++++--------------
 3 files changed, 141 insertions(+), 51 deletions(-)

-- 
2.43.0


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

end of thread, other threads:[~2024-08-07 12:58 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-05 14:37 [PATCH i-g-t v2 0/8] tests/core_hotunplug: Make it fail only once Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 1/8] lib/drmtest: add function for retriving chipset Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 2/8] tests/core_hotunplug: set known chipset before tests Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 3/8] lib/drmtest: Allow to get drm device name Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 4/8] tests/core_hotunplug: Open the same driver Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 5/8] tests/core_hotunplug: Fix device close Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 6/8] tests/core_hotunplug: Skip if no render available Kamil Konieczny
2024-08-05 14:37 ` [PATCH i-g-t v2 7/8] tests/core_hotunplug: Check health once after subtest Kamil Konieczny
2024-08-07 12:26   ` Andrzej Hajda
2024-08-05 14:37 ` [PATCH i-g-t v2 8/8] tests/core_hotunplug: Print info between steps Kamil Konieczny
2024-08-07 12:58   ` Andrzej Hajda
2024-08-05 15:20 ` ✓ CI.xeBAT: success for tests/core_hotunplug: Make it fail only once Patchwork
2024-08-05 15:31 ` ✓ Fi.CI.BAT: " Patchwork
2024-08-05 16:25 ` ✗ CI.xeFULL: failure " Patchwork
2024-08-05 18:45   ` Kamil Konieczny
2024-08-05 23:51 ` ✗ Fi.CI.IGT: " Patchwork
2024-08-06 12:51   ` Kamil Konieczny

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