On Tue, 25 Aug 2009 22:23:05 GMT
[email protected] wrote:

> http://bugzilla.kernel.org/show_bug.cgi?id=14060
> 
>            Summary: oops: sysfs_remove_link and i915
>            Product: Drivers
>            Version: 2.5
>     Kernel Version: 2.6.31-rc7
>           Platform: All
>         OS/Version: Linux
>               Tree: Mainline
>             Status: NEW
>           Severity: normal
>           Priority: P1
>          Component: Video(DRI)
>         AssignedTo: [email protected]
>         ReportedBy: [email protected]
>             Blocks: 13615
>         Regression: Yes
> 
> 
> Subject    : oops: sysfs_remove_link, i915 and 2.6.31-rc7
> Submitter  : Dominik Brodowski <[email protected]>
> Date       : 2009-08-22 5:48
> References : http://marc.info/?l=linux-kernel&m=125092139113955&w=4
> 
> This entry is being used for tracking a regression from 2.6.30.
> Please don't close it until the problem is fixed in the mainline.

Fixed by this patch?

-- 
Jesse Barnes, Intel Open Source Technology Center

diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
index 8851315..60ea984 100644
--- a/drivers/acpi/video.c
+++ b/drivers/acpi/video.c
@@ -2004,8 +2004,11 @@ static int acpi_video_bus_put_one_device(struct 
acpi_video_device *device)
        status = acpi_remove_notify_handler(device->dev->handle,
                                            ACPI_DEVICE_NOTIFY,
                                            acpi_video_device_notify);
-       sysfs_remove_link(&device->backlight->dev.kobj, "device");
-       backlight_device_unregister(device->backlight);
+       if (device->backlight) {
+               sysfs_remove_link(&device->backlight->dev.kobj, "device");
+               backlight_device_unregister(device->backlight);
+               device->backlight = NULL;
+       }
        if (device->cdev) {
                sysfs_remove_link(&device->dev->dev.kobj,
                                  "thermal_cooling");

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to