mbox series

[SRU,B/OEM-B,0/3] screen displays abnormally on the lenovo M715 with the AMD GPU (Radeon Vega 8 Mobile, rev ca, 1002:15dd)

Message ID 1539060886-7450-1-git-send-email-hui.wang@canonical.com
Headers show
Series screen displays abnormally on the lenovo M715 with the AMD GPU (Radeon Vega 8 Mobile, rev ca, 1002:15dd) | expand

Message

Hui Wang Oct. 9, 2018, 4:54 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1791569

These patches are in the 4.16-rcN already, no need to send them to cosmic.

[Impact]
On the machine of Lenovo M715, there is an AMD GPU (1022:15dd rev ca), when it
switchs to amdgpufb, the screen displays abnormally, we tested with a couple monitors,
all of them can reproduce this issue.

[Fix]
Backported 3 patches from mainline kernel, all of them focus on the change of
disable_vga(). After applying these 3 patches, the issue disappears.


[Test Case]
boot the system, run glxgears, everything works well

[Regression Potential]
Very low, these patches come from upstream, and I have tested them on at least 6 different
lenovo machines and those machines have different AMD GPUs on them, all of them worked
as well as before.



Bhawanpreet Lakha (1):
  drm/amd/display: Fix takover from VGA mode

Clark Zheng (1):
  drm/amd/display: Refine disable VGA

Eric Yang (1):
  drm/amd/display: early return if not in vga mode in disable_vga

 drivers/gpu/drm/amd/display/dc/dce/dce_hwseq.h     | 17 ++++++++++++++-
 .../drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c  | 24 ++++++++++++++++++++++
 2 files changed, 40 insertions(+), 1 deletion(-)

Comments

Hui Wang Oct. 9, 2018, 5:06 a.m. UTC | #1
On 2018年10月09日 12:54, Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1791569
Sorry it should be: BugLink: https://bugs.launchpad.net/bugs/1796786

Other parts are fine. please review them.
>
> These patches are in the 4.16-rcN already, no need to send them to cosmic.
>
> [Impact]
> On the machine of Lenovo M715, there is an AMD GPU (1022:15dd rev ca), when it
> switchs to amdgpufb, the screen displays abnormally, we tested with a couple monitors,
> all of them can reproduce this issue.
>
> [Fix]
> Backported 3 patches from mainline kernel, all of them focus on the change of
> disable_vga(). After applying these 3 patches, the issue disappears.
>
>
> [Test Case]
> boot the system, run glxgears, everything works well
>
> [Regression Potential]
> Very low, these patches come from upstream, and I have tested them on at least 6 different
> lenovo machines and those machines have different AMD GPUs on them, all of them worked
> as well as before.
>
>
>
> Bhawanpreet Lakha (1):
>    drm/amd/display: Fix takover from VGA mode
>
> Clark Zheng (1):
>    drm/amd/display: Refine disable VGA
>
> Eric Yang (1):
>    drm/amd/display: early return if not in vga mode in disable_vga
>
>   drivers/gpu/drm/amd/display/dc/dce/dce_hwseq.h     | 17 ++++++++++++++-
>   .../drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c  | 24 ++++++++++++++++++++++
>   2 files changed, 40 insertions(+), 1 deletion(-)
>
Stefan Bader Oct. 9, 2018, 7:08 a.m. UTC | #2
On 09.10.2018 06:54, Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1791569
> 
> These patches are in the 4.16-rcN already, no need to send them to cosmic.
> 
> [Impact]
> On the machine of Lenovo M715, there is an AMD GPU (1022:15dd rev ca), when it
> switchs to amdgpufb, the screen displays abnormally, we tested with a couple monitors,
> all of them can reproduce this issue.
> 
> [Fix]
> Backported 3 patches from mainline kernel, all of them focus on the change of
> disable_vga(). After applying these 3 patches, the issue disappears.
> 
> 
> [Test Case]
> boot the system, run glxgears, everything works well
> 
> [Regression Potential]
> Very low, these patches come from upstream, and I have tested them on at least 6 different
> lenovo machines and those machines have different AMD GPUs on them, all of them worked
> as well as before.
> 
> 
> 
> Bhawanpreet Lakha (1):
>   drm/amd/display: Fix takover from VGA mode
> 
> Clark Zheng (1):
>   drm/amd/display: Refine disable VGA
> 
> Eric Yang (1):
>   drm/amd/display: early return if not in vga mode in disable_vga
> 
>  drivers/gpu/drm/amd/display/dc/dce/dce_hwseq.h     | 17 ++++++++++++++-
>  .../drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c  | 24 ++++++++++++++++++++++
>  2 files changed, 40 insertions(+), 1 deletion(-)
> 
The changes themselves are rather incomprehensible and thus not easy to look at.
However all for the same driver and some successful regression testing done. So
with the (->update buglink) buglink replaced by the one from this cover email:

Acked-by: Stefan Bader <stefan.bader@canonical.com>
Kleber Sacilotto de Souza Oct. 11, 2018, 12:09 p.m. UTC | #3
On 10/09/18 06:54, Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1791569
> 
> These patches are in the 4.16-rcN already, no need to send them to cosmic.
> 
> [Impact]
> On the machine of Lenovo M715, there is an AMD GPU (1022:15dd rev ca), when it
> switchs to amdgpufb, the screen displays abnormally, we tested with a couple monitors,
> all of them can reproduce this issue.
> 
> [Fix]
> Backported 3 patches from mainline kernel, all of them focus on the change of
> disable_vga(). After applying these 3 patches, the issue disappears.
> 
> 
> [Test Case]
> boot the system, run glxgears, everything works well
> 
> [Regression Potential]
> Very low, these patches come from upstream, and I have tested them on at least 6 different
> lenovo machines and those machines have different AMD GPUs on them, all of them worked
> as well as before.
> 
> 
> 
> Bhawanpreet Lakha (1):
>   drm/amd/display: Fix takover from VGA mode
> 
> Clark Zheng (1):
>   drm/amd/display: Refine disable VGA
> 
> Eric Yang (1):
>   drm/amd/display: early return if not in vga mode in disable_vga
> 
>  drivers/gpu/drm/amd/display/dc/dce/dce_hwseq.h     | 17 ++++++++++++++-
>  .../drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c  | 24 ++++++++++++++++++++++
>  2 files changed, 40 insertions(+), 1 deletion(-)
> 

All changes upstream, don't seem too intrusive and well tested for
regressions. So:

Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Khalid Elmously Oct. 22, 2018, 7:30 a.m. UTC | #4
On 2018-10-09 12:54:43 , Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1791569
> 
> These patches are in the 4.16-rcN already, no need to send them to cosmic.
> 
> [Impact]
> On the machine of Lenovo M715, there is an AMD GPU (1022:15dd rev ca), when it
> switchs to amdgpufb, the screen displays abnormally, we tested with a couple monitors,
> all of them can reproduce this issue.
> 
> [Fix]
> Backported 3 patches from mainline kernel, all of them focus on the change of
> disable_vga(). After applying these 3 patches, the issue disappears.
> 
> 
> [Test Case]
> boot the system, run glxgears, everything works well
> 
> [Regression Potential]
> Very low, these patches come from upstream, and I have tested them on at least 6 different
> lenovo machines and those machines have different AMD GPUs on them, all of them worked
> as well as before.
> 
> 
> 
> Bhawanpreet Lakha (1):
>   drm/amd/display: Fix takover from VGA mode
> 
> Clark Zheng (1):
>   drm/amd/display: Refine disable VGA
> 
> Eric Yang (1):
>   drm/amd/display: early return if not in vga mode in disable_vga
> 
>  drivers/gpu/drm/amd/display/dc/dce/dce_hwseq.h     | 17 ++++++++++++++-
>  .../drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c  | 24 ++++++++++++++++++++++
>  2 files changed, 40 insertions(+), 1 deletion(-)
> 
> -- 
> 2.7.4
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team