Bug 10779 - drm_sysfs_suspend uses KERN_ERR in printk
drm_sysfs_suspend uses KERN_ERR in printk
Status: CLOSED CODE_FIX
Product: Power Management
Classification: Unclassified
Component: Hibernation/Suspend
All Linux
: P1 low
Assigned To: power-management_other
:
Depends on:
Blocks: 7216
  Show dependency treegraph
 
Reported: 2008-05-23 01:03 UTC by Chris Jones
Modified: 2008-09-23 03:50 UTC (History)
1 user (show)

See Also:
Kernel Version: 2.6.24-16-generic
Tree: Mainline
Regression: ---


Attachments

Description Chris Jones 2008-05-23 01:03:36 UTC
I am running Ubuntu 8.04's kernel and when I resume from S3 suspend, a single message is displayed on the console about drm_sysfs_suspend, which seems to come from the function:

static int drm_sysfs_suspend(struct device *dev, pm_message_t state)

This doesn't appear to be an error, so presumably this should be KERN_INFO, not KERN_ERR?

(less technical users often express surprise and concern when their laptop comes out of suspend and prints scary messages at them, even if it does actually work fine)
Comment 1 Jesse Barnes 2008-05-23 08:39:15 UTC
You're right, I'm not sure how that snuck in to the final upstream version.  Here's the fix (sending to Linus now).


diff --git a/drivers/char/drm/drm_sysfs.c b/drivers/char/drm/drm_sysfs.c
index 9a32169..af211a0 100644
--- a/drivers/char/drm/drm_sysfs.c
+++ b/drivers/char/drm/drm_sysfs.c
@@ -34,8 +34,6 @@ static int drm_sysfs_suspend(struct device *dev, pm_message_t
        struct drm_minor *drm_minor = to_drm_minor(dev);
        struct drm_device *drm_dev = drm_minor->dev;

-       printk(KERN_ERR "%s\n", __func__);
-
        if (drm_dev->driver->suspend)
                return drm_dev->driver->suspend(drm_dev, state);


Note You need to log in before you can comment on or make changes to this bug.