On 03.02.2019 04:08, Eric Blau wrote:
I finally was able to complete the bisect on this issue. It seems that
this commit introduced the regression on my MacBook Pro:
cc8b329fef53c74a4abf98b0755b3832d572d6ce is the first bad commit
commit cc8b329fef53c74a4abf98b0755b3832d572d6ce
Author: Mathias Nyman <mathias.nyman@xxxxxxxxxxxxxxx>
Date: Thu Nov 15 11:38:41 2018 +0200
usb: xhci: Prevent bus suspend if a port connect change or polling
state is detected
commit 2f31a67f01a8beb22cae754c53522cb61a005750 upstream.
There's an additional fix for that patch, in 4.19 stable:
commit e13bfb357f5bc04f9e7ccff7d07770388062a8cc
Author: Mathias Nyman <mathias.nyman@xxxxxxxxxxxxxxx>
Date: Fri Dec 14 10:54:43 2018 +0200
xhci: Don't prevent USB2 bus suspend in state check intended for USB3 only
commit 45f750c16cae3625014c14c77bd9005eda975d35 upstream.
If you already have that fix then one of the USB ports may be in a odd state when suspending
Does adding dynamic debug for xhci_hub_control show any messages when suspending?
mount -t debugfs none /sys/kernel/debug
echo -n 'func xhci_bus_suspend =p' > /sys/kernel/debug/dynamic_debug/control
-Mathias
