Repository files (latest commit first)

Filename	Latest commit message	Latest commit date
Tony Lindgren 6c799f6c74 PM: runtime: Fix unpaired parent child_count for force_resume ... commit c745253e2a691a40c66790defe85c104a887e14a upstream. As pm_runtime_need_not_resume() relies also on usage_count, it can return a different value in pm_runtime_force_suspend() compared to when called in pm_runtime_force_resume(). Different return values can happen if anything calls PM runtime functions in between, and causes the parent child_count to increase on every resume. So far I've seen the issue only for omapdrm that does complicated things with PM runtime calls during system suspend for legacy reasons: omap_atomic_commit_tail() for omapdrm.0 dispc_runtime_get() wakes up 58000000.dss as it's the dispc parent dispc_runtime_resume() rpm_resume() increases parent child_count dispc_runtime_put() won't idle, PM runtime suspend blocked pm_runtime_force_suspend() for 58000000.dss, !pm_runtime_need_not_resume() __update_runtime_status() system suspended pm_runtime_force_resume() for 58000000.dss, pm_runtime_need_not_resume() pm_runtime_enable() only called because of pm_runtime_need_not_resume() omap_atomic_commit_tail() for omapdrm.0 dispc_runtime_get() wakes up 58000000.dss as it's the dispc parent dispc_runtime_resume() rpm_resume() increases parent child_count dispc_runtime_put() won't idle, PM runtime suspend blocked ... rpm_suspend for 58000000.dss but parent child_count is now unbalanced Let's fix the issue by adding a flag for needs_force_resume and use it in pm_runtime_force_resume() instead of pm_runtime_need_not_resume(). Additionally omapdrm system suspend could be simplified later on to avoid lots of unnecessary PM runtime calls and the complexity it adds. The driver can just use internal functions that are shared between the PM runtime and system suspend related functions. Fixes: 4918e1f87c ("PM / runtime: Rework pm_runtime_force_suspend/resume()") Signed-off-by: Tony Lindgren <tony@atomide.com> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org> Tested-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Cc: 4.16+ <stable@vger.kernel.org> # 4.16+ Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2021-05-19 10:08:20 +02:00
..
firmware_loader	firmware_loader: fix memory leak for paged buffer	2020-09-23 12:40:34 +02:00
power	PM: runtime: Fix unpaired parent child_count for force_resume	2021-05-19 10:08:20 +02:00
regmap	regmap: set debugfs_name to NULL after it is freed	2021-05-14 09:44:17 +02:00
test
arch_topology.c	cpu-topology: Fix the potential data corruption	2020-10-01 13:17:40 +02:00
attribute_container.c
base.h
bus.c
cacheinfo.c
class.c
component.c	component: Silence bind error on -EPROBE_DEFER	2020-05-27 17:46:31 +02:00
container.c
core.c	driver core: Extend device_is_dependent()	2021-01-27 11:47:51 +01:00
cpu.c	x86/speculation: Add Special Register Buffer Data Sampling (SRBDS) mitigation	2020-06-10 20:24:57 +02:00
dd.c	driver core: Fix locking bug in deferred_probe_timeout_work_func()	2021-04-16 11:46:38 +02:00
devcon.c
devcoredump.c
devres.c
devtmpfs.c
driver.c
firmware.c
hypervisor.c
init.c
isa.c
Kconfig
Makefile
map.c
memory.c
module.c
node.c	node: fix device cleanups in error handling code	2021-05-14 09:44:22 +02:00
pinctrl.c
platform-msi.c
platform.c	drivers: base: Fix NULL pointer exception in __platform_driver_probe() if a driver developer is foolish	2020-06-24 17:50:24 +02:00
property.c
soc.c
swnode.c	software node: Fix node registration	2021-03-17 17:03:48 +01:00
syscore.c
topology.c
transport_class.c