Clean up the test code, add some new basic subtests, then unblock
unbind test variants.
No incompletes / aborts nor subsequently run test issues have been
reported by Trybot. The hotrebind-lateclose subtest fails on a so far
unidentified driver sysfs issue but the device is fully recovered and
left in a usable state. Perceived Haswell/Broadwell issue with audio
power management has been worked around and its potential occurrence
is reported as an IGT warning.
Series changelog:
v2: New patch "Un-blocklist *bind* subtests added.
v3: Patch "Follow failed subtests with healthcheck" renamed to "Recover
from subtest failures".
- a new patche "Clean up device open error handling" added, an old
patch "Fix missing newline" obsoleted by the new one dropped,
- other new patches added:
- "Let the driver time out essential sysfs operations",
- "More thorough i915 healthcheck and recovery",
- a patch "Add 'lateclose before restore' variants" from another
series included.
v4: Optional patch "Duplicate debug messages in dmesg" from another
series included.
v5: New patch added with Haswell audio related kernel warning worked
around and replaced with an IGT warning to preserve visibility of
the issue.
v6: New patch added for also checking health of render device nodes,
- new patch added with proper handling of health check before late
close,
- inclusion of unbind-rebind scenario to BAT scope proposed.
@Michał: Since some patch updates are trivial, I've preserved your
v1/v2 Reviewd-by: except for patches with non-trivial changes, where I
marked your R-b as v1/v2 applicable. Please have a look and confirm if
you are still OK with them.
@Tvrtko: As I already asked before, please support my attempt to remove
the unbind test variants from the blocklist.
@Petri, @Martin: Assuming CI results will be as good as those obtained
on Trybot, please give me your green light for merging this series if
you have no objections.
Thanks,
Janusz
Janusz Krzysztofik (24):
tests/core_hotunplug: Use igt_assert_fd()
tests/core_hotunplug: Constify dev_bus_addr string
tests/core_hotunplug: Clean up device open error handling
tests/core_hotunplug: Consolidate duplicated debug messages
tests/core_hotunplug: Assert successful device filter application
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: Recover from subtest failures
tests/core_hotunplug: Fail subtests on device close errors
tests/core_hotunplug: Let the driver time out essential sysfs
operations
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: Also check health of render device node
tests/core_hotunplug: More thorough i915 healthcheck and recovery
tests/core_hotunplug: Add 'lateclose before restore' variants
tests/core_hotunplug: Check health both before and after late close
tests/core_hotunplug: HSW/BDW audio issue workaround
tests/core_hotunplug: Duplicate debug messages in dmesg
tests/core_hotunplug: Un-blocklist *bind* subtests
tests/core_hotunplug: Add unbind-rebind subtest to BAT scope
tests/core_hotunplug.c | 560 ++++++++++++++++++++------
tests/intel-ci/blacklist.txt | 2 +-
tests/intel-ci/fast-feedback.testlist | 1 +
3 files changed, 431 insertions(+), 132 deletions(-)
--
2.21.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
