thunderbolt: Runtime PM activate both ends of the device link
authorMika Westerberg <mika.westerberg@linux.intel.com>
Sun, 14 Nov 2021 14:07:11 +0000 (16:07 +0200)
committerMika Westerberg <mika.westerberg@linux.intel.com>
Tue, 7 Dec 2021 12:18:33 +0000 (15:18 +0300)
commitf3380cac0c0b3a6f49ab161e2a057c363962f48d
tree8463632cd6f29bf04feeaf7d604615305bd0d673
parent19813551701d004b517534888aa4e2a62ca4488e
thunderbolt: Runtime PM activate both ends of the device link

If protocol tunnels are already up when the driver is loaded, for
instance if the boot firmware implements connection manager of its own,
runtime PM reference count of the consumer devices behind the tunnel
might have been increased already before the device link is created but
the supplier device runtime PM reference count is not. This leads to a
situation where the supplier (the Thunderbolt driver) can runtime
suspend even if it should not because the corresponding protocol tunnel
needs to be up causing the devices to be removed from the corresponding
native bus.

Prevent this from happening by making both sides of the link runtime PM
active briefly. The pm_runtime_put() for the consumer (PCIe
root/downstream port, xHCI) then allows it to runtime suspend again but
keeps the supplier runtime resumed the whole time it is runtime active.

Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/thunderbolt/acpi.c