Clean up the test code and unblock unbind variants. @Michał: Since most v2 updates are trivial, I've preserved your Reviewd-by: except for patch 11/16 "Follow failed subtests with health checks" - please have a look and confirm if you are still OK with it. Thanks, Janusz Janusz Krzysztofik (16): tests/core_hotunplug: Use igt_assert_fd() tests/core_hotunplug: Constify dev_bus_addr string tests/core_hotunplug: Consolidate duplicated debug messages tests/core_hotunplug: Assert successful device filter application tests/core_hotunplug: Fix missing newline tests/core_hotunplug: Maintain a single data structure instance tests/core_hotunplug: Pass errors via a data structure field tests/core_hotunplug: Handle device close errors tests/core_hotunplug: Prepare invariant data once per test run tests/core_hotunplug: Skip selectively on sysfs close errors tests/core_hotunplug: Follow failed subtests with health checks tests/core_hotunplug: Fail subtests on device close errors tests/core_hotunplug: Process return values of sysfs operations tests/core_hotunplug: Assert expected device presence/absence tests/core_hotunplug: Explicitly ignore unused return values tests/core_hotunplug: Un-blocklist *unbind* subtests tests/core_hotunplug.c | 334 +++++++++++++++++++++-------------- tests/intel-ci/blacklist.txt | 2 +- 2 files changed, 204 insertions(+), 132 deletions(-) -- 2.21.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx