BACKPORT: PM / core: Add support to skip power management in device/driver model
authorSudeep Holla <sudeep.holla@arm.com>
Thu, 14 Feb 2019 18:29:10 +0000 (18:29 +0000)
committerNolen Johnson <johnsonnolen@gmail.com>
Thu, 3 Oct 2024 15:05:01 +0000 (11:05 -0400)
commit1067183e2071f5d377842a2452680394168bf6fb
treecf05bb7676f927f0e4cc0373ba853844935db905
parent12f82c4a59658475aec868e867b3761d018e83e2
BACKPORT: PM / core: Add support to skip power management in device/driver model

All device objects in the driver model contain fields that control the
handling of various power management activities. However, it's not
always useful. There are few instances where pseudo devices are added
to the model just to take advantage of many other features like
kobjects, udev events, and so on. One such example is cpu devices and
their caches.

The sysfs for the cpu caches are managed by adding devices with cpu
as the parent in cpu_device_create() when secondary cpu is brought
online. Generally when the secondary CPUs are hotplugged back in as part
of resume from suspend-to-ram, we call cpu_device_create() from the cpu
hotplug state machine while the cpu device associated with that CPU is
not yet ready to be resumed as the device_resume() call happens bit
later. It's not really needed to set the flag is_prepared for cpu
devices as they are mostly pseudo device and hotplug framework deals
with state machine and not managed through the cpu device.

This often results in annoying warning when resuming:
Enabling non-boot CPUs ...
CPU1: Booted secondary processor
 cache: parent cpu1 should not be sleeping
CPU1 is up
CPU2: Booted secondary processor
 cache: parent cpu2 should not be sleeping
CPU2 is up
.... and so on.

So in order to fix these kind of errors, we could just completely avoid
doing any power management related initialisations and operations if
they are not used by these devices.

Add no_pm flags to indicate that the device doesn't require any sort of
PM activities and all of them can be completely skipped. We can use the
same flag to also avoid adding not used *power* sysfs entries for these
devices. For now, lets use this for cpu cache devices.

Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Tested-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Albert I <kras@raphielgang.org>
Change-Id: Ia0fcfe47501397c8165a7760de0e4de0b0669699
drivers/base/cpu.c
drivers/base/power/main.c
drivers/base/power/sysfs.c
include/linux/device.h
include/linux/pm.h