[PATCH i-g-t 0/3] Drop device scan cache logic
Zbigniew Kempczyński
zbigniew.kempczynski at intel.com
Thu Jan 23 09:22:07 UTC 2025
In this series I haven't added limiting fetching attributes - after
reading udevadm-info.c I'm not sure it makes sense to limit it. Udevadm
in general does same things current igt_device_scan does, especially
it also skips some attributes. Additionally it skips attributes which
are binaries but I've some resistence to do this similar as strlen()
on blob is imo risky and it may result in segfault. I may send attrs
limiting in separate patch after this series will be reviewed (and
hopefully merged).
Series addresses main concern - stale device cache. Now rescans are
unconditional what prevents from opening old device after operations
on module.
Test added which checks device scanning in multithreading code likely
is not needed for merge. Adding to BAT should reveal early potential
problems with the series.
Cc: Lucas De Marchi <lucas.demarchi at intel.com>
Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
Zbigniew Kempczyński (3):
lib/igt_device_scan: drop device scan cache logic
lib/igt_device_scan: change device list variable visibility
tests/xe_intel_bb: check multithreading rescans won't segfault
benchmarks/gem_wsim.c | 2 +-
lib/drmtest.c | 2 +-
lib/igt_device_scan.c | 20 +++-------
lib/igt_device_scan.h | 2 +-
lib/igt_multigpu.c | 2 +-
tests/core_hotunplug.c | 2 +-
tests/device_reset.c | 2 +-
tests/intel-ci/xe-fast-feedback.testlist | 1 +
tests/intel/i915_suspend.c | 4 +-
tests/intel/xe_intel_bb.c | 47 ++++++++++++++++++++++++
tools/intel_gpu_top.c | 2 +-
tools/intel_pm_rpm.c | 2 +-
tools/lsgpu.c | 2 +-
13 files changed, 64 insertions(+), 26 deletions(-)
--
2.34.1
More information about the igt-dev
mailing list