Subject : X200: Brightness broken since 2.6.29-rc4-58-g4c098bc Submitter : Nico Schottelius <nico-linux-20090213@schottelius.org> Date : 2009-02-13 9:33 References : http://marc.info/?l=linux-kernel&m=123451768406825&w=4 Handled-By : Len Brown <lenb@kernel.org> Notify-Also : Matthew Garrett <mjg@redhat.com> This entry is being used for tracking a regression from 2.6.28. Please don't close it until the problem is fixed in the mainline.
Hi, Nico Will you please attach the output of acpidump, lspci -vxxx? Thanks.
http://marc.info/?l=linux-kernel&m=123479975503827&w=2 e806b4957412bf472d826bd8cc571da041248799 is the first commit cc Eric
Do you mean the first bad one?
(In reply to comment #3) > Do you mean the first bad one? > right. commit e806b4957412bf472d826bd8cc571da041248799 Author: Eric Anholt <eric@anholt.net> Date: Thu Jan 22 09:56:58 2009 -0800 drm/i915: Suppress GEM teardown on X Server exit in KMS mode. Fixes hangs when starting X for the second time. Signed-off-by: Eric Anholt <eric@anholt.net> Signed-off-by: Dave Airlie <airlied@linux.ie> diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index debad5c..a590d61 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c @@ -3273,6 +3273,9 @@ i915_gem_lastclose(struct drm_device *dev) { int ret; + if (drm_core_check_feature(dev, DRIVER_MODESET)) + return; + ret = i915_gem_idle(dev); if (ret) DRM_ERROR("failed to idle hardware: %d\n", ret);
On Tuesday 24 February 2009, Nico Schottelius wrote: > Still existing. > > Rafael J. Wysocki [Mon, Feb 23, 2009 at 10:48:13PM +0100]: > > This message has been generated automatically as a part of a report > > of recent regressions. > > > > The following bug entry is on the current list of known regressions > > from 2.6.28. Please verify if it still should be listed and let me know > > (either way). > > > > > > Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=12705 > > Subject : X200: Brightness broken since 2.6.29-rc4-58-g4c098bc
On Wednesday 04 March 2009, Nico Schottelius wrote: > Rafael J. Wysocki [Tue, Mar 03, 2009 at 08:25:55PM +0100]: > > This message has been generated automatically as a part of a report > > of recent regressions. > > > > The following bug entry is on the current list of known regressions > > from 2.6.28. Please verify if it still should be listed and let me know > > (either way). > > Well, well... nobody really cares about this bug, so it's still > there. Seems this automatic reminder will be persistent until 2.6.30.
On Wednesday 04 March 2009, Zhang Rui wrote: > On Wed, 2009-03-04 at 03:22 +0800, Rafael J. Wysocki wrote: > > > Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=12705 > > Subject : X200: Brightness broken since 2.6.29-rc4-58-g4c098bc > > Submitter : Nico Schottelius <nico-linux-20090213@schottelius.org> > > Date : 2009-02-13 9:33 (19 days old) > > References : http://marc.info/?l=linux-kernel&m=123451768406825&w=4 > > Handled-By : Len Brown <lenb@kernel.org> > > > > e806b4957412bf472d826bd8cc571da041248799 is first bad commit commit e806b4957412bf472d826bd8cc571da041248799 Author: Eric Anholt <eric@anholt.net> Date: Thu Jan 22 09:56:58 2009 -0800 drm/i915: Suppress GEM teardown on X Server exit in KMS mode. Fixes hangs when starting X for the second time. Signed-off-by: Eric Anholt <eric@anholt.net> Signed-off-by: Dave Airlie <airlied@linux.ie> References : http://marc.info/?l=linux-kernel&m=123479975503827&w=2 First-Bad-Commit : e806b4957412bf472d826bd8cc571da041248799
Not-Handled-By : Len Brown <lenb@kernel.org> Handled-By : Eric Anholt <eric@anholt.net>
On Sunday 26 April 2009, Nico Schottelius wrote: > Rafael J. Wysocki [Sun, Apr 26, 2009 at 11:46:25AM +0200]: > > This message has been generated automatically as a part of a report > > of regressions introduced between 2.6.28 and 2.6.29. > > Sure, still broken. For some version it works, for most newer not, > as reported.
I'm running 2.6.30 (44b7532b8b464f606053562400719c9c21276037) on a X200 and writing on /proc/acpi/video/VID/LCD0/brightness works while on battery and AC. Using /sys/class/backlight/acpi_video0 also works. Nico, can you test with a newer version?
Can the reporter confirm this still exist in 2.6.30 or 31?
Closing stale bug