diff options
author | 2025-07-23 09:31:32 +0800 | |
---|---|---|
committer | 2025-08-12 14:49:14 +0200 | |
commit | dfbd535db74df0343ca39670e06326d7aee8c8f4 (patch) | |
tree | d97ce0f0d98fdb636fbba62dee08c0e0d68c3bc6 /rust/pin-init/internal/src | |
parent | HID: intel-thc-hid: intel-thc: Fix incorrect pointer arithmetic in I2C regs save (diff) | |
download | wireguard-linux-dfbd535db74df0343ca39670e06326d7aee8c8f4.tar.xz wireguard-linux-dfbd535db74df0343ca39670e06326d7aee8c8f4.zip |
HID: intel-ish-hid: Increase ISHTP resume ack timeout to 300ms
During s2idle suspend/resume testing on some systems, occasional several
tens of seconds delays were observed in HID sensor resume handling. Trace
analysis revealed repeated "link not ready" timeout errors during
set/get_report operations, which were traced to the
hid_ishtp_cl_resume_handler() timing out while waiting for the ISHTP
resume acknowledgment. The previous timeout was set to 50ms, which proved
insufficient on affected machines.
Empirical measurements on failing systems showed that the time from ISH
resume initiation to receiving the ISHTP resume ack could be as long as
180ms. As a result, the 50ms timeout caused failures.
To address this, increase the wait timeout for ISHTP resume ack from 50ms
to 300ms, providing a safer margin for slower hardware. Additionally, add
error logging when a timeout occurs to aid future debugging and issue
triage. No functional changes are made beyond the timeout adjustment and
improved error reporting.
Signed-off-by: Zhang Lixu <lixu.zhang@intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: Jiri Kosina <jkosina@suse.com>
Diffstat (limited to 'rust/pin-init/internal/src')
0 files changed, 0 insertions, 0 deletions