Re: missing acpi backlight bisected to 9661e92c10

Re: missing acpi backlight bisected to 9661e92c10

[Matthew Garrett]

Sigh. Could you provide the output of lspci and acpidump?

-- 
Matthew Garrett | mjg59@srcf.ucam.org

Re: missing acpi backlight bisected to 9661e92c10

[Ali Gholami Rudi]

Matthew Garrett <mjg@redhat.com> wrote:
> Sigh. Could you provide the output of lspci and acpidump?

acpidump: https://bugzilla.kernel.org/attachment.cgi?id=24674

$ lspci
00:00.0 Host bridge: Intel Corporation Mobile GME965/GLE960 Memory Controller Hub (rev 0c)
00:02.0 VGA compatible controller: Intel Corporation Mobile GME965/GLE960 Integrated Graphics Controller (rev 0c)
00:02.1 Display controller: Intel Corporation Mobile GME965/GLE960 Integrated Graphics Controller (rev 0c)
00:1a.0 USB Controller: Intel Corporation 82801H (ICH8 Family) USB UHCI Controller #4 (rev 03)
00:1a.7 USB Controller: Intel Corporation 82801H (ICH8 Family) USB2 EHCI Controller #2 (rev 03)
00:1b.0 Audio device: Intel Corporation 82801H (ICH8 Family) HD Audio Controller (rev 03)
00:1c.0 PCI bridge: Intel Corporation 82801H (ICH8 Family) PCI Express Port 1 (rev 03)
00:1c.5 PCI bridge: Intel Corporation 82801H (ICH8 Family) PCI Express Port 6 (rev 03)
00:1d.0 USB Controller: Intel Corporation 82801H (ICH8 Family) USB UHCI Controller #1 (rev 03)
00:1d.1 USB Controller: Intel Corporation 82801H (ICH8 Family) USB UHCI Controller #2 (rev 03)
00:1d.2 USB Controller: Intel Corporation 82801H (ICH8 Family) USB UHCI Controller #3 (rev 03)
00:1d.7 USB Controller: Intel Corporation 82801H (ICH8 Family) USB2 EHCI Controller #1 (rev 03)
00:1e.0 PCI bridge: Intel Corporation 82801 Mobile PCI Bridge (rev f3)
00:1f.0 ISA bridge: Intel Corporation 82801HEM (ICH8M) LPC Interface Controller (rev 03)
00:1f.2 SATA controller: Intel Corporation 82801HBM/HEM (ICH8M/ICH8M-E) SATA AHCI Controller (rev 03)
30:00.0 Ethernet controller: Marvell Technology Group Ltd. Device 4357 (rev 10)

I don't know much about ACPI, but is acpidump's output supposed to
change?  The above dump has been taken almost a year ago (for
another ACPI bug) and now that I compare it with my current dump, I
get:

--- acpidump1	2011-08-11 18:16:37.509899444 +0430
+++ acpidump2	2011-08-11 18:03:33.778749912 +0430
@@ -5143,24 +5143,6 @@
   0060: 00 00 00 00 88 c4 7d 3f 00 00 00 00 2e c5 7d 3f  ......}?......}?
   0070: 00 00 00 00                                      ....
 
-FACP @ 0x3f7c8000
-  0000: 46 41 43 50 84 00 00 00 02 ff 48 50 20 20 20 20  FACP......HP    
-  0010: 33 30 38 41 20 20 20 20 02 00 00 00 48 50 20 20  308A    ....HP  
-  0020: 01 00 00 00 80 7d 7e 3f 48 83 7c 3f 00 00 09 00  .....}~?H.|?....
-  0030: b2 00 00 00 f1 f0 f2 80 00 10 00 00 00 00 00 00  ................
-  0040: 04 10 00 00 00 00 00 00 20 10 00 00 08 10 00 00  ........ .......
-  0050: 28 10 00 00 00 00 00 00 04 02 01 04 08 00 00 00  (...............
-  0060: 02 00 87 00 00 00 00 00 01 03 0d 00 32 13 00 00  ............2...
-  0070: a5 80 00 00 01 08 00 00 00 00 00 00 00 00 00 00  ................
-  0080: 00 00 00 00                                      ....
-
-RSDT @ 0x3f7c8178
-  0000: 52 53 44 54 4c 00 00 00 01 68 48 50 20 20 20 20  RSDTL....hHP    
-  0010: 33 30 38 41 20 20 20 20 20 09 07 28 48 50 20 20  308A     ..(HP  
-  0020: 01 00 00 00 00 80 7c 3f 38 82 7c 3f 70 82 7c 3f  ......|?8.|?p.|?
-  0030: d8 82 7c 3f 14 83 7c 3f 9f b3 7d 3f c7 b6 7d 3f  ..|?..|?..}?..}?
-  0040: 29 c2 7d 3f 88 c4 7d 3f 2e c5 7d 3f              ).}?..}?..}?
-
 RSD PTR @ 0xf6f60
   0000: 52 53 44 20 50 54 52 20 13 48 50 20 20 20 20 02  RSD PTR .HP    .
   0010: 78 81 7c 3f 24 00 00 00 c4 81 7c 3f 00 00 00 00  x.|?$.....|?....

	Ali

Re: missing acpi backlight bisected to 9661e92c10

[Matthew Garrett]

Well, that all looks fine. And I also can't see any way that this commit
could cause the backlight not to appear - all it does is set the parent 
if it's present. There's no new path that could cause it to return 
early. Does reverting this patch really make things work?

-- 
Matthew Garrett | mjg59@srcf.ucam.org

Re: missing acpi backlight bisected to 9661e92c10

[Ali Gholami Rudi]

Matthew Garrett <mjg@redhat.com> wrote:
> Well, that all looks fine. And I also can't see any way that this commit
> could cause the backlight not to appear - all it does is set the parent 
> if it's present. There's no new path that could cause it to return 
> early. Does reverting this patch really make things work?

Yes.  I'm running the patched kernel right now.

I can apply a debug patch on the unchanged kernel, if it helps?

	Ali

Re: missing acpi backlight bisected to 9661e92c10

[Matthew Garrett]

On Mon, Aug 22, 2011 at 10:25:24AM +0430, Ali Gholami Rudi wrote:

> Yet, /sys/devices/virtual/backlight/acpi_video0/brightness only
> appears after the revert.  Seems something changes its behavior
> if new_bd->dev.parent is not NULL in backlight_device_register().

Well, yes, if it has a parent it won't be under /sys/devices/virtual. 
Does it appear under /sys/class/backlight ?

-- 
Matthew Garrett | mjg59@srcf.ucam.org

Re: missing acpi backlight bisected to 9661e92c10

[Ali Gholami Rudi]

Matthew Garrett <mjg@redhat.com> wrote:
> 
> On Mon, Aug 22, 2011 at 10:25:24AM +0430, Ali Gholami Rudi wrote:
> 
> > Yet, /sys/devices/virtual/backlight/acpi_video0/brightness only
> > appears after the revert.  Seems something changes its behavior
> > if new_bd->dev.parent is not NULL in backlight_device_register().
> 
> Well, yes, if it has a parent it won't be under /sys/devices/virtual. 
> Does it appear under /sys/class/backlight ?

Yep!  Thought this is the 2.6.38-rc1 issue coming back (I really
thought I had searched /sys...).  Sorry for the noise.

Thanks,
Ali