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 Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1782177

Title:
  Screen sometimes jumps to full brightness on resume from suspend

Status in Linux:
  Incomplete
Status in linux package in Ubuntu:
  Incomplete

Bug description:
  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.

      $ 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. Seems like a regression
  from 16.04, which I ran on the same laptop without this issue.

To manage notifications about this bug go to:
https://bugs.launchpad.net/linux/+bug/1782177/+subscriptions

-- 
Mailing list: https://launchpad.net/~kernel-packages
Post to     : kernel-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kernel-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to