summaryrefslogtreecommitdiff
path: root/tests/device_reset.c
AgeCommit message (Collapse)Author
2021-11-10tests/device_reset: remove artificial timeoutMarcin Bernatowicz
Do not use artificial timeout. Let the test fail or finish. Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com> Acked-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> Acked-by: Zbigniew KempczyƄski <zbigniew.kempczynski@intel.com>
2021-10-08tests/core_hotunplug: Drop log level for preventively unloading snd driverJeevan B
change igt_warn to igt_info when unloading the snd module before unbinding i915 until WA is fixed. As this "todo reminder" is flagged as BAT failure and showing up in all sorts of top-bugs lists for various platforms. v2: Update commit message Signed-off-by: Jeevan B <jeevan.b@intel.com> Reviewed-by: Uma Shankar <uma.shankar@intel.com>
2021-06-29tests/device_reset: Unload snd driver before i915 unbindUma Shankar
Unload the snd module before unbinding i915. Audio holds a wakeref which triggers a warning otherwise, resulting in below warning and test failure. Currently HSW/BDW and DG1 are the platforms affected, can be extended to other platforms as well. <4> [137.001006] ------------[ cut here ]------------ <4> [137.001010] i915 0000:00:02.0: i915 raw-wakerefs=1 wakelocks=1 on cleanup <4> [137.001076] WARNING: CPU: 0 PID: 1417 at drivers/gpu/drm/i915/intel_runtime_pm.c:619 intel_runtime_pm_driver_release+0x56/0x60 [i915] <4> [137.001078] Modules linked in: snd_hda_intel i915 snd_hda_codec_hdmi mei_hdcp intel_pmt_telemetry intel_pmt_core x86_pkg_temp_thermal coretemp smsc75xx crct10dif_pclmul usbnet crc32_pclmul mii ghash_clmulni_intel kvm_intel e1000e snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core ptp pps_core mei_me snd_pcm mei prime_numbers intel_pmt [last unloaded: i915] <4> [137.001095] CPU: 0 PID: 1417 Comm: kworker/u16:7 Tainted: G U 5.9.0-g79478e23b1878-DII_3204+ #1 <4> [137.001097] Hardware name: Intel Corporation Tiger Lake Client Platform/TigerLake U DDR4 SODIMM RVP, BIOS TGLSFWI1.R00.3197.A00.2005110542 05/11/2020 <4> [137.001102] Workqueue: events_unbound async_run_entry_fn <4> [137.001140] RIP: 0010:intel_runtime_pm_driver_release+0x56/0x60 [i915] <4> [137.001142] Code: fd 10 4c 8b 67 50 4d 85 e4 75 03 4c 8b 27 e8 91 59 58 e1 45 89 e8 89 e9 4c 89 e2 48 89 c6 48 c7 c7 b0 f3 48 a0 e8 55 25 ef e0 <0f> 0b eb b5 66 0f 1f 44 00 00 48 8b 87 88 45 ff ff b9 02 00 00 00 <4> [137.001144] RSP: 0018:ffffc900007dbd68 EFLAGS: 00010286 <4> [137.001147] RAX: 0000000000000000 RBX: ffff88847338bea8 RCX: 0000000000000001 <4> [137.001148] RDX: 0000000080000001 RSI: ffffffff823efa86 RDI: 00000000ffffffff <4> [137.001150] RBP: 0000000000000001 R08: 0000000000000001 R09: 0000000000000001 <4> [137.001152] R10: 000000009bda34df R11: 00000000e2a8a89a R12: ffff88849b209880 <4> [137.001153] R13: 0000000000000001 R14: ffff88847338bea8 R15: ffff88847338fcc0 <4> [137.001155] FS: 0000000000000000(0000) GS:ffff8884a0600000(0000) knlGS:0000000000000000 <4> [137.001157] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 <4> [137.001159] CR2: 00007fc03597dd88 CR3: 0000000006610005 CR4: 0000000000770ef0 <4> [137.001160] PKRU: 55555554 <4> [137.001162] Call Trace: <4> [137.001199] i915_drm_suspend_late+0x102/0x120 [i915] <4> [137.001204] ? pci_pm_poweroff_late+0x30/0x30 <4> [137.001209] dpm_run_callback+0x61/0x270 <4> [137.001214] __device_suspend_late+0x8b/0x180 <4> [137.001217] async_suspend_late+0x15/0x90 <4> [137.001220] async_run_entry_fn+0x34/0x160 <4> [137.001224] process_one_work+0x26c/0x5c0 <4> [137.001231] worker_thread+0x37/0x380 <4> [137.001235] ? process_one_work+0x5c0/0x5c0 <4> [137.001238] kthread+0x149/0x170 <4> [137.001241] ? kthread_park+0x80/0x80 <4> [137.001246] ret_from_fork+0x1f/0x30 <4> [137.001256] irq event stamp: 2329 Cc: Kai Vehmanen <kai.vehmanen@linux.intel.com> Cc: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> Signed-off-by: Uma Shankar <uma.shankar@intel.com> Acked-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
2020-12-15tests/device_reset: check for i915 in unbind_reset_rebind()Jeremy Cline
Commit 265f1d4a5a14 ("tests/device_reset: Work around for driver unbind issue with audio") introduced an assertion on is_i915_device() by way of intel_get_drm_devid(). Since this is a work-around for an i915-specific issue, guard it with a check on the device type so the test runs on other devices. Signed-off-by: Jeremy Cline <jcline@redhat.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2020-09-14tests/device_reset: Work around for driver unbind issue with audioUma Shankar
Unbinding the i915 driver on some Haswell and Broadwell platforms with Azalia audio results in a kernel WARNING on "i915 raw-wakerefs=1 wakelocks=1 on cleanup". The issue can be worked around by manually enabling runtime power management for the conflicting audio adapter. Use that method but also display a warning to preserve visibility of the issue. Also tag the workaround with a FIXME comment. Suggested-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> Signed-off-by: Uma Shankar <uma.shankar@intel.com> Reviewed-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
2020-07-23tests/device_reset: Test device sysfs resetMarcin Bernatowicz
Device reset is initiated by writing "1" to reset sysfs file, which should initiate PCI device Function Level Reset if supported by device. Test scenarios: 1. unbind driver from device, initiate sysfs reset, rebind driver to device 2. device reset with bound driver v2: removed unbind-rebind (duplicates core_hotunplug@unbind-rebind) added healthcheck to each test (Janusz) v3: after review corrections (renamed sysfs_fds to device_fds, corrected not closed file descriptor, removed variable length array) v4: updated description (Martin), snprintf corrections (Janusz), reset-bound added to blacklist as it hangs several gens of hosts (Tomi) Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz@intel.com> Reviewed-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>