lkml.org 
[lkml]   [2007]   [Jan]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch] suspend/resume debugging: device filter
    Subject: [patch] suspend/resume debugging: device filter
    From: Ingo Molnar <mingo@elte.hu>

    this patch implements the /sys/power/filter attribute, which takes a
    string. If a device's name matches the filter string (exactly), then
    that device is excluded from suspend/resume.

    this can be helpful in a number of ways when debugging suspend and
    resume problems:

    - if CONFIG_DISABLE_CONSOLE_SUSPEND is used then the serial
    console is still suspended after which point there's no
    log output. Doing "echo serial > /sys/power/filter" keeps
    the serial port active, so any messages (and crash info)
    after that point is displayed.

    - if a device is suspected to be the reason of resume failure
    then it can be excluded via the filter. That device obviously
    wont work, but users can thus help us debug resume problems
    in combination with pm_trace, without having to hack the kernel.

    (note that you can obvious break suspend/resume via the filter, by
    excluding a vital device - so it is only to be used when suspend or
    resume is broken to begin with.)

    it might be better to do this centrally in sysfs, via a per-device
    attribute, to individually enable suspend and resume on a per device
    basis, but my sysfs-fu is not strong enough for that now ;-)

    Signed-off-by: Ingo Molnar <mingo@elte.hu>
    ---
    drivers/base/power/resume.c | 6 ++++
    drivers/base/power/suspend.c | 3 +-
    include/linux/resume-trace.h | 6 ++++
    kernel/power/main.c | 58 +++++++++++++++++++++++++++++++++++++++++++
    4 files changed, 72 insertions(+), 1 deletion(-)

    Index: linux/drivers/base/power/resume.c
    ===================================================================
    --- linux.orig/drivers/base/power/resume.c
    +++ linux/drivers/base/power/resume.c
    @@ -24,6 +24,9 @@ int resume_device(struct device * dev)
    {
    int error = 0;

    + if (power_filter(dev))
    + return 0;
    +
    TRACE_DEVICE(dev);
    TRACE_RESUME(0);
    down(&dev->sem);
    @@ -52,6 +55,9 @@ static int resume_device_early(struct de
    {
    int error = 0;

    + if (power_filter(dev))
    + return 0;
    +
    TRACE_DEVICE(dev);
    TRACE_RESUME(0);
    if (dev->bus && dev->bus->resume_early) {
    Index: linux/drivers/base/power/suspend.c
    ===================================================================
    --- linux.orig/drivers/base/power/suspend.c
    +++ linux/drivers/base/power/suspend.c
    @@ -10,6 +10,7 @@

    #include <linux/device.h>
    #include <linux/kallsyms.h>
    +#include <linux/resume-trace.h>
    #include <linux/pm.h>
    #include "../base.h"
    #include "power.h"
    @@ -78,7 +79,7 @@ int suspend_device(struct device * dev,
    suspend_report_result(dev->class->suspend, error);
    }

    - if (!error && dev->bus && dev->bus->suspend && !dev->power.power_state.event) {
    + if (!error && dev->bus && dev->bus->suspend && !dev->power.power_state.event && !power_filter(dev)) {
    dev_dbg(dev, "%s%s\n",
    suspend_verb(state.event),
    ((state.event == PM_EVENT_SUSPEND)
    Index: linux/include/linux/resume-trace.h
    ===================================================================
    --- linux.orig/include/linux/resume-trace.h
    +++ linux/include/linux/resume-trace.h
    @@ -9,6 +9,8 @@ struct device;
    extern void set_trace_device(struct device *);
    extern void generate_resume_trace(void *tracedata, unsigned int user);

    +extern int power_filter(struct device *dev);
    +
    #define TRACE_DEVICE(dev) set_trace_device(dev)
    #define TRACE_RESUME(user) do { \
    if (pm_trace_enabled) { \
    @@ -28,6 +30,10 @@ extern void generate_resume_trace(void *

    #define TRACE_DEVICE(dev) do { } while (0)
    #define TRACE_RESUME(dev) do { } while (0)
    +static inline int power_filter(struct device *dev)
    +{
    + return 0;
    +}

    #endif

    Index: linux/kernel/power/main.c
    ===================================================================
    --- linux.orig/kernel/power/main.c
    +++ linux/kernel/power/main.c
    @@ -15,6 +15,7 @@
    #include <linux/delay.h>
    #include <linux/errno.h>
    #include <linux/init.h>
    +#include <linux/device.h>
    #include <linux/pm.h>
    #include <linux/console.h>
    #include <linux/cpu.h>
    @@ -306,9 +307,66 @@ pm_trace_store(struct subsystem * subsys

    power_attr(pm_trace);

    +/**
    + * filter - exclude drivers from suspend and resume
    + *
    + * show() returns the current filter
    + *
    + * store() accepts a new filter (up to 128 chars long)
    + *
    + * Do "echo serial > /sys/power/filter" to exclude the
    + * serial driver from suspension - this can be useful to
    + * get kernel messages out after the serial console, or to
    + * see which device causes a resume failure.
    + */
    +
    +#define FILTER_LEN 128
    +
    +static char power_filter_str[FILTER_LEN+1] = "<none>";
    +
    +int power_filter(struct device *dev)
    +{
    + const char *str = dev_driver_string(dev);
    +
    + if (!strcmp(str, power_filter_str)) {
    + printk(KERN_INFO "power filter match for device: %s\n", str);
    + return 1;
    + }
    + return 0;
    +}
    +
    +static ssize_t filter_show(struct subsystem * subsys, char * buf)
    +{
    + char *s = buf;
    +
    + s += sprintf(s, "%s\n", power_filter_str);
    +
    + return (s - buf);
    +}
    +
    +static ssize_t
    +filter_store(struct subsystem * subsys, const char * buf, size_t n)
    +{
    + unsigned int len;
    +
    + strncpy(power_filter_str, buf, FILTER_LEN);
    +
    + len = strlen(power_filter_str);
    + /*
    + * Strip off any trailing '\n':
    + */
    + if (len && power_filter_str[len-1] == '\n')
    + power_filter_str[len-1] = 0;
    +
    + return len;
    +}
    +
    +power_attr(filter);
    +
    static struct attribute * g[] = {
    &state_attr.attr,
    &pm_trace_attr.attr,
    + &filter_attr.attr,
    NULL,
    };
    #else
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/
    \
     
     \ /
      Last update: 2007-01-25 12:35    [W:0.030 / U:31.136 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site