Launchpad has imported 8 comments from the remote bug at https://bugs.freedesktop.org/show_bug.cgi?id=108703.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2018-11-10T01:18:16+00:00 Luke-schlather wrote: At night, I usually turn my brightness down as far as it will go using the brightness keys. This makes it dim, but visible and not off. When I walk away for a while, the screen goes to sleep, and when I come back and move the mouse, the screen jumps to full brightness, which is very unpleasant at night. Investigating, it seems that the brightness in intel_backlight is getting reset to 4437, but the one in acpi_video0 reads 0. This can be triggered by running xset dpms force standby. Here's the conflicting brightness values: $ for file in brightness max_brightness actual_brightness; > do > for directory in /sys/class/backlight/intel_backlight/ /sys/class/backlight/acpi_video0/ > do > echo $directory$file : $(cat $directory$file) > done > done /sys/class/backlight/intel_backlight/brightness : 4437 /sys/class/backlight/acpi_video0/brightness : 0 /sys/class/backlight/intel_backlight/max_brightness : 4437 /sys/class/backlight/acpi_video0/max_brightness : 15 /sys/class/backlight/intel_backlight/actual_brightness : 4437 /sys/class/backlight/acpi_video0/actual_brightness : 0 After I press the button to increase the brightness one notch, they are back in sync, and then I can turn it back down as well. $ for file in brightness max_brightness actual_brightness; > do > for directory in /sys/class/backlight/intel_backlight/ /sys/class/backlight/acpi_video0/ > do > echo $directory$file : $(cat $directory$file) > done > done /sys/class/backlight/intel_backlight/brightness : 53 /sys/class/backlight/acpi_video0/brightness : 1 /sys/class/backlight/intel_backlight/max_brightness : 4437 /sys/class/backlight/acpi_video0/max_brightness : 15 /sys/class/backlight/intel_backlight/actual_brightness : 53 /sys/class/backlight/acpi_video0/actual_brightness : 1 This is on a fresh install of Xubuntu 18.04 with default packages including the latest version of the i915 Intel Driver which is active. Seems like a regression from Xubuntu 16.04, which I ran on the same laptop without this issue. I initially filed this as a bug in XFCE power manager but after some troubleshooting there I can reproduce the same issue under Gnome as well. Original bug with troubleshooting: https://bugs.launchpad.net/ubuntu/+source/xfce4-power- manager/+bug/1782177 Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/7 ------------------------------------------------------------------------ On 2018-11-12T08:36:35+00:00 Lakshminarayana-vudum wrote: Luke, Can you set kernel parameters drm.debug=0x1e log_buf_len=4M and reboot the machine. Reproduce the issue and attach the dmesg log from boot. This way we know more information about the issue. BTW have you tried this issue with drm-tip? (https://cgit.freedesktop.org/drm-tip) Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/8 ------------------------------------------------------------------------ On 2018-11-13T18:51:59+00:00 Luke-schlather wrote: Created attachment 142454 DMESG output with drm.debug enabled Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/9 ------------------------------------------------------------------------ On 2018-11-14T08:01:53+00:00 Lakshminarayana-vudum wrote: Jani, any comments here? verifying with latest drm-tip will help here? Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/10 ------------------------------------------------------------------------ On 2018-11-14T08:30:47+00:00 Jani-nikula wrote: I don't think we have any fixes in drm-tip that should affect this; for that matter I don't recall changes in the area that should regress either. Backlight has been fairly solid for some years now. 1) Please paste the sysfs attributes *before* display gets disabled. 2) Is the behaviour reproducible if you don't go to minimum before disabling display? I.e. stay at the next higher level. Or, use # echo 1 > /sys/class/backlight/intel_backlight/brightness to stay above 0. We do have a check in place to crank the brightness to max if it's at the minimum upon encoder enable. However, that bit policy has been in place for ages. Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/11 ------------------------------------------------------------------------ On 2018-11-14T08:31:58+00:00 Jani-nikula wrote: (In reply to Jani Nikula from comment #4) > We do have a check in place to crank the brightness to max if it's at the > minimum upon encoder enable. However, that bit policy has been in place for > ages. Should be, "that bit of policy" Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/12 ------------------------------------------------------------------------ On 2018-11-14T13:04:07+00:00 Luke-schlather wrote: Yes, it only happens on the minimum brightness setting. This is what it looks like before standby to trigger the bug: /sys/class/backlight/intel_backlight/brightness : 0 /sys/class/backlight/acpi_video0/brightness : 0 /sys/class/backlight/intel_backlight/max_brightness : 4437 /sys/class/backlight/acpi_video0/max_brightness : 15 /sys/class/backlight/intel_backlight/actual_brightness : 0 /sys/class/backlight/acpi_video0/actual_brightness : 0 If I increase brightness to setting 1 the bug is not triggered. Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/13 ------------------------------------------------------------------------ On 2018-11-14T14:08:20+00:00 Jani-nikula wrote: I really can't think of any reason why this behaviour would have changed for you because of a kernel change. For 10 years now, since the introduction of i915 KMS support, we've had code in place to set the backlight to max if it's at 0 on panel enable. (In retrospect, that was not such a good policy decision, but I fear it's pretty much carved in stone now. Hindsight 20/20.) I suspect something changed in your userspace so that it now sets brightness to 0 rather than some non-zero low value. That something would typically truncate the 4438 levels provided by intel_backlight to a handful of discrete levels. The kernel change to nuke the policy would technically be as simple as diff --git a/drivers/gpu/drm/i915/intel_panel.c b/drivers/gpu/drm/i915/intel_panel.c index b6df63aa11e3..beb4801702d3 100644 --- a/drivers/gpu/drm/i915/intel_panel.c +++ b/drivers/gpu/drm/i915/intel_panel.c @@ -1104,8 +1104,8 @@ void intel_panel_enable_backlight(const struct intel_crtc_state *crtc_state, WARN_ON(panel->backlight.max == 0); - if (panel->backlight.level <= panel->backlight.min) { - panel->backlight.level = panel->backlight.max; + if (panel->backlight.level < panel->backlight.min) { + panel->backlight.level = panel->backlight.min; if (panel->backlight.device) panel->backlight.device->props.brightness = scale_hw_to_user(connector, but if there's any userspace around that relies on the behaviour, we'd have to revert back. Reply at: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1782177/comments/14 ** Changed in: linux Status: Unknown => Incomplete ** Changed in: linux Importance: Unknown => Medium -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1782177 Title: Screen sometimes jumps to full brightness on resume from suspend To manage notifications about this bug go to: https://bugs.launchpad.net/linux/+bug/1782177/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
