RFC: Device tree patch (support for partial tree suspend/resume)
Subsystems (4)
Hi.
As mentioned in a previous email on LKML, here's a patch I've written which adds support for suspending and resuming portions of the device tree. It's a simple extension of the current scheme: imagine the dpm_active, dpm_off and dpm_irq_off lists as representing not necessarily the whole tree but possibly just a portion of it. The current device_suspend, device_resume... functions are retained, and operate on this 'default_device_tree'. At runtime, however, new trees can be created, devices and their ancestors moved to the new trees, suspended and resumed separately to the main tree and then merged back into the main tree when done with. New *_tree functions allow the current suspend/resume/powerdown/powerup operations to be applied to these dynamically created trees.
That's what the following patch does.
I've tested it using suspend2 and then pulled out the relevant portion. If you want some sample usage, just say the word!
Regards,
Nigel
drivers/base/power/main.c +19 -5
@@ -4,6 +4,9 @@* Copyright (c) 2003 Patrick Mochel * Copyright (c) 2003 Open Source Development Lab * +* Partial tree additions +* Copyright (c) 2004 Nigel Cunningham +* * This file is released under the GPLv2 * *
@@ -23,10 +26,18 @@#include <linux/device.h> #include "power.h" -LIST_HEAD (dpm_active); -LIST_HEAD (dpm_off); -LIST_HEAD (dpm_off_irq); - +struct partial_device_tree default_device_tree = +{ + .dpm_active =LIST_HEAD_INIT (default_device_tree.dpm_active), + .dpm_off =LIST_HEAD_INIT (default_device_tree.dpm_off), + .dpm_off_irq =LIST_HEAD_INIT (default_device_tree.dpm_off_irq), +}; +EXPORT_SYMBOL (default_device_tree); + +/* +* One mutex for all trees because we can be moving items +* between trees. +*/ DECLARE_MUTEX (dpm_sem);/*
@@ -76,7 +87,9 @@ dev->bus ? dev->bus->name : "No Bus", dev->kobj.name);atomic_set (&dev->power.pm_users,0 );down (&dpm_sem); -list_add_tail (&dev->power.entry, &dpm_active); +list_add_tail (&dev->power.entry, &default_device_tree.dpm_active); + dev->current_list =DEVICE_LIST_DPM_ACTIVE ; + dev->tree = &default_device_tree;device_pm_set_parent (dev, dev->parent);if ((error =dpm_sysfs_add (dev)))list_del (&dev->power.entry);
@@ -92,6 +105,7 @@dpm_sysfs_remove (dev);device_pm_release (dev->power.pm_parent);list_del (&dev->power.entry); + dev->current_list =DEVICE_LIST_NONE ;up (&dpm_sem); }
drivers/base/power/Makefile +1 -1
@@ -1,5 +1,5 @@ obj-y := shutdown.o -obj-$(CONFIG_PM) += main.o suspend.o resume.o runtime.o sysfs.o +obj-$(CONFIG_PM) += main.o suspend.o resume.o runtime.o sysfs.o tree.o ifeq ($(CONFIG_DEBUG_DRIVER),y) EXTRA_CFLAGS += -DDEBUG
drivers/base/power/power.h +17 -3
@@ -30,10 +30,22 @@/* * The PM lists. */ -extern struct list_head dpm_active; -extern struct list_head dpm_off; -extern struct list_head dpm_off_irq; +struct partial_device_tree +{ +struct list_head dpm_active; +struct list_head dpm_off; +struct list_head dpm_off_irq; +}; + +enum { +DEVICE_LIST_NONE , +DEVICE_LIST_DPM_ACTIVE , +DEVICE_LIST_DPM_OFF , +DEVICE_LIST_DPM_OFF_IRQ , +}; + +extern struct partial_device_tree default_device_tree;static inline struct dev_pm_info *to_pm_info (struct list_head * entry) {
@@ -59,7 +71,9 @@* resume.c */ +extern void dpm_resume_tree (struct partial_device_tree * tree);extern void dpm_resume (void ); +extern void dpm_power_up_tree (struct partial_device_tree * tree);extern void dpm_power_up (void );extern int resume_device (struct device *);
drivers/base/power/resume.c +35 -9
84 lines not on this page · raw message
drivers/base/power/suspend.c +22 -11
@@ -51,7 +51,7 @@/** -* device_suspend - Save state and stop all devices in system. +* device_suspend_tree - Save state and stop all devices in system. * @state: Power state to put each device in. * * Walk the dpm_active list, call ->suspend() for each device, and move
@@ -60,7 +60,7 @@* the device to the dpm_off list. If it returns -EAGAIN, we move it to * the dpm_off_irq list. If we get a different error, try and back out. * -* If we hit a failure with any of the devices, call device_resume() +* If we hit a failure with any of the devices, call device_resume_tree() * above to bring the suspended devices back to life. * * Note this function leaves dpm_sem held to
@@ -70,22 +70,24 @@* */ -int device_suspend (u32 state) +int device_suspend_tree (u32 state,struct partial_device_tree * tree) {int error =0 ;down (&dpm_sem); -while (!list_empty (&dpm_active)) { -struct list_head * entry = dpm_active.prev; +while (!list_empty (&tree->dpm_active)) { +struct list_head * entry = tree->dpm_active.prev;struct device * dev =to_device (entry); error =suspend_device (dev, state);if (!error) {list_del (&dev->power.entry); -list_add (&dev->power.entry, &dpm_off); +list_add (&dev->power.entry, &tree->dpm_off); + dev->current_list =DEVICE_LIST_DPM_OFF ; }else if (error == -EAGAIN ) {list_del (&dev->power.entry); -list_add (&dev->power.entry, &dpm_off_irq); +list_add (&dev->power.entry, &tree->dpm_off_irq); + dev->current_list =DEVICE_LIST_DPM_OFF_IRQ ; }else {printk (KERN_ERR "Could not suspend device %s: " "error %d\n" ,kobject_name (&dev->kobj), error);
@@ -96,10 +98,15 @@up (&dpm_sem);return error; Error: -dpm_resume (); +dpm_resume_tree (tree); goto Done; } +EXPORT_SYMBOL (device_suspend_tree); +int device_suspend (u32 state) +{ +return device_suspend_tree (state, &default_device_tree); +}EXPORT_SYMBOL (device_suspend);
@@ -112,12 +119,12 @@* done, power down system devices. */ -int device_power_down (u32 state) +int device_power_down_tree (u32 state,struct partial_device_tree * tree) {int error =0 ;struct device * dev; -list_for_each_entry_reverse (dev, &dpm_off_irq, power.entry) { +list_for_each_entry_reverse (dev, &tree->dpm_off_irq, power.entry) {if ((error =suspend_device (dev, state)))break ; }
@@ -131,6 +138,10 @@dpm_power_up (); goto Done; } +EXPORT_SYMBOL (device_power_down_tree); +int device_power_down (u32 state) +{ +return device_power_down_tree (state, &default_device_tree); +}EXPORT_SYMBOL (device_power_down); -
drivers/base/power/tree.c +105 -0
105 lines not on this page · raw message
include/linux/device.h +5 -0
@@ -289,6 +289,11 @@override */ void (*release)(struct device * dev); + +struct partial_device_tree * tree;/* Which tree of devices this +device is in */ +int current_list;/* Which list within the tree the +device is on (speeds moving) */ }; static inline struct device *
include/linux/pm.h +12 -2
@@ -241,12 +241,22 @@extern void device_pm_set_parent (struct device * dev,struct device * parent); +struct partial_device_tree ; +extern struct partial_device_tree default_device_tree; +extern int device_suspend (u32 state); +extern int device_suspend_tree (u32 state,struct partial_device_tree * tree);extern int device_power_down (u32 state); +extern int device_power_down_tree (u32 state,struct partial_device_tree * tree);extern void device_power_up (void ); +extern void device_power_up_tree (struct partial_device_tree * tree);extern void device_resume (void ); - - +extern void device_resume_tree (struct partial_device_tree * tree); +extern void device_merge_tree (struct partial_device_tree * source, +struct partial_device_tree * dest); +extern void device_switch_trees (struct device * dev,struct partial_device_tree * tree); +extern struct partial_device_tree *device_create_tree (void ); +extern void device_destroy_tree (struct partial_device_tree * tree);#endif /* __KERNEL__ */ #endif /* _LINUX_PM_H */