regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Norbert Preining]

Hi everyone,

(please Cc)

going to 2.6.38-rc1 I see the following regression: After reboot I
can easily adjust the backlight with the keys on my keyboard, but 
after a suspend-to-ram this does not work anymore.

This is a Sony laptop, and backlight is controlled by the video
acpi driver AFAIR.

Best wishes

Norbert
------------------------------------------------------------------------
Norbert Preining            preining@{jaist.ac.jp, logic.at, debian.org}
JAIST, Japan                                 TeX Live & Debian Developer
DSA: 0x09C5B094   fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
------------------------------------------------------------------------
AASLEAGH (n.)
A liqueur made only for drinking at the end of a revoltingly long
bottle party when all the drinkable drink has been drunk.
			--- Douglas Adams, The Meaning of Liff

Re: regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Eric Dumazet]

Le vendredi 21 janvier 2011 à 10:55 +0900, Norbert Preining a écrit :
> Hi everyone,
> 
> (please Cc)
> 
> going to 2.6.38-rc1 I see the following regression: After reboot I
> can easily adjust the backlight with the keys on my keyboard, but 
> after a suspend-to-ram this does not work anymore.
> 
> This is a Sony laptop, and backlight is controlled by the video
> acpi driver AFAIR.
> 

Hi Norbert

Maybe this is the problem Linus had to bisect, you could try his patch :

http://marc.info/?l=linux-netdev&m=129556877919576&w=2

Thanks !


--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Eric Dumazet]

Le vendredi 21 janvier 2011 à 05:11 +0100, Eric Dumazet a écrit :
> Le vendredi 21 janvier 2011 à 10:55 +0900, Norbert Preining a écrit :
> > Hi everyone,
> > 
> > (please Cc)
> > 
> > going to 2.6.38-rc1 I see the following regression: After reboot I
> > can easily adjust the backlight with the keys on my keyboard, but 
> > after a suspend-to-ram this does not work anymore.
> > 
> > This is a Sony laptop, and backlight is controlled by the video
> > acpi driver AFAIR.
> > 
> 
> Hi Norbert
> 
> Maybe this is the problem Linus had to bisect, you could try his patch :
> 
> http://marc.info/?l=linux-netdev&m=129556877919576&w=2
> 
> Thanks !
> 

Or just pull linux-2.6, since patch is now committed

commit 28e58ee8ce1f0e69c207f747b7b9054b071e328d
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date:   Thu Jan 20 16:21:59 2011 -0800

    Fix broken "pipe: use event aware wakeups" optimization
    
    Commit e462c448fdc8 ("pipe: use event aware wakeups") optimized the pipe
    event wakeup calls to avoid wakeups if the events do not match the
    requested set.
    
    However, the optimization was buggy, in that it didn't actually use the
    correct sets for the events: when we make room for more data to be
    written, the pipe poll() routine will return both the POLLOUT _and_
    POLLWRNORM bits.  Similarly for read.
    
    And most critically, when a pipe is released, that will potentially
    result in POLLHUP|POLLERR (depending on whether it was the last reader
    or writer), not just the regular POLLIN|POLLOUT.
    
    This bug showed itself as a hung gnome-screensaver-dialog process, stuck
    forever (or at least until it was poked by a signal or by being traced)
    in a poll() system call.
    
    Cc: Davide Libenzi <davidel@xmailserver.org>
    Cc: David S. Miller <davem@davemloft.net>
    Cc: Eric Dumazet <eric.dumazet@gmail.com>
    Cc: Jens Axboe <axboe@kernel.dk>
    Cc: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>


--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Norbert Preining]

Hi Eric, hi all

On Fr, 21 Jan 2011, Eric Dumazet wrote:
> Maybe this is the problem Linus had to bisect, you could try his patch :

> Or just pull linux-2.6, since patch is now committed
> 
> commit 28e58ee8ce1f0e69c207f747b7b9054b071e328d

I am now running on top of 8d99641f6c1a so I have this patch included,
but still see the same problem. Adjustment *before* suspend works,
afterwards does not.

Can someone give me a quick notive how to bisect only those commits
that touch the relevant parts? acpi?

Best wishes

Norbert
------------------------------------------------------------------------
Norbert Preining            preining@{jaist.ac.jp, logic.at, debian.org}
JAIST, Japan                                 TeX Live & Debian Developer
DSA: 0x09C5B094   fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
------------------------------------------------------------------------
POTT SHRIGLEY (n.)
Dried remains of a week-old casserole, eaten when extremely drunk at
two a.m.
			--- Douglas Adams, The Meaning of Liff

Re: regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Zhang Rui]

how do you change the backlight? hotkeys?
does changing /sys/class/backlight/acpi_video0/brightness work for you?

thanks,
rui

On Fri, 2011-01-21 at 14:46 +0800, Norbert Preining wrote:
> Hi Eric, hi all
> 
> On Fr, 21 Jan 2011, Eric Dumazet wrote:
> > Maybe this is the problem Linus had to bisect, you could try his patch :
> 
> > Or just pull linux-2.6, since patch is now committed
> > 
> > commit 28e58ee8ce1f0e69c207f747b7b9054b071e328d
> 
> I am now running on top of 8d99641f6c1a so I have this patch included,
> but still see the same problem. Adjustment *before* suspend works,
> afterwards does not.
> 
> Can someone give me a quick notive how to bisect only those commits
> that touch the relevant parts? acpi?
> 
> Best wishes
> 
> Norbert
> ------------------------------------------------------------------------
> Norbert Preining            preining@{jaist.ac.jp, logic.at, debian.org}
> JAIST, Japan                                 TeX Live & Debian Developer
> DSA: 0x09C5B094   fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
> ------------------------------------------------------------------------
> POTT SHRIGLEY (n.)
> Dried remains of a week-old casserole, eaten when extremely drunk at
> two a.m.
> 			--- Douglas Adams, The Meaning of Liff
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Zhang Rui]

btw, please attach the acpidump of this laptop.

thanks,
rui

On Fri, 2011-01-21 at 15:28 +0800, Zhang Rui wrote:
> how do you change the backlight? hotkeys?
> does changing /sys/class/backlight/acpi_video0/brightness work for you?
> 
> thanks,
> rui
> 
> On Fri, 2011-01-21 at 14:46 +0800, Norbert Preining wrote:
> > Hi Eric, hi all
> > 
> > On Fr, 21 Jan 2011, Eric Dumazet wrote:
> > > Maybe this is the problem Linus had to bisect, you could try his patch :
> > 
> > > Or just pull linux-2.6, since patch is now committed
> > > 
> > > commit 28e58ee8ce1f0e69c207f747b7b9054b071e328d
> > 
> > I am now running on top of 8d99641f6c1a so I have this patch included,
> > but still see the same problem. Adjustment *before* suspend works,
> > afterwards does not.
> > 
> > Can someone give me a quick notive how to bisect only those commits
> > that touch the relevant parts? acpi?
> > 
> > Best wishes
> > 
> > Norbert
> > ------------------------------------------------------------------------
> > Norbert Preining            preining@{jaist.ac.jp, logic.at, debian.org}
> > JAIST, Japan                                 TeX Live & Debian Developer
> > DSA: 0x09C5B094   fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
> > ------------------------------------------------------------------------
> > POTT SHRIGLEY (n.)
> > Dried remains of a week-old casserole, eaten when extremely drunk at
> > two a.m.
> > 			--- Douglas Adams, The Meaning of Liff
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: regression 2.6.37 -> 2.6.38-rc1: after suspend backlight cannot be adjusted

[Norbert Preining]

[-- Attachment #1: Type: text/plain, Size: 1294 bytes --]

On Fr, 21 Jan 2011, Zhang Rui wrote:
> does changing /sys/class/backlight/acpi_video0/brightness work for you?

Interestingly, that is gone since 2.6.37.

A diff of the dmesg between 37 and 38-rc1 (+up to  git 8d99641f6c1a)
shows (besides other things):
(no idea if that is important)
-Switching to clocksource tsc
+Switching to clocksource hpet

and here it is:
-acpi device:01: registered as cooling_device2
-input: Video Bus as /devices/LNXSYSTM:00/device:00/PNP0A08:00/LNXVIDEO:00/input/input2
-ACPI: Video Device [OVGA] (multi-head: yes  rom: no  post: no)
+No ACPI video bus found

(also the cooling_device2 is gone)

Interestingly, adjustment via sony-laptop does work, but only 
at first boot, not after suspend?!?!

> btw, please attach the acpidump of this laptop.

Attached, in gzip.

Best

Norbert
------------------------------------------------------------------------
Norbert Preining            preining@{jaist.ac.jp, logic.at, debian.org}
JAIST, Japan                                 TeX Live & Debian Developer
DSA: 0x09C5B094   fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
------------------------------------------------------------------------
KERRY (n.)
The small twist of skin which separated each sausage on a string.
			--- Douglas Adams, The Meaning of Liff

[-- Attachment #2: sony-vgnz11.acpi.gz --]
[-- Type: application/octet-stream, Size: 63173 bytes --]