linux-kernel - Re: [PATCH 0/3] drm/amdgpu: Fix suspend/resume issues with MST

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5db84bb9-6d2d-391d-3699-2331bdc75118@amd.com>
Date:   Tue, 8 Jan 2019 17:32:45 +0000
From:   "Wentland, Harry" <Harry.Wentland@....com>
To:     Lyude Paul <lyude@...hat.com>,
        "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "amd-gfx@...ts.freedesktop.org" <amd-gfx@...ts.freedesktop.org>
CC:     Sean Paul <sean@...rly.run>,
        "Zhou, David(ChunMing)" <David1.Zhou@....com>,
        Maxime Ripard <maxime.ripard@...tlin.com>,
        "Li, Sun peng (Leo)" <Sunpeng.Li@....com>,
        "Lakha, Bhawanpreet" <Bhawanpreet.Lakha@....com>,
        "Francis, David" <David.Francis@....com>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Kazlauskas, Nicholas" <Nicholas.Kazlauskas@....com>,
        David Airlie <airlied@...ux.ie>,
        "Zuo, Jerry" <Jerry.Zuo@....com>,
        "Cheng, Tony" <Tony.Cheng@....com>,
        Daniel Vetter <daniel@...ll.ch>,
        "S, Shirish" <Shirish.S@....com>,
        "Deucher, Alexander" <Alexander.Deucher@....com>,
        Mikita Lipski <mikita.lipski@....com>,
        "Koo, Anthony" <Anthony.Koo@....com>,
        "Koenig, Christian" <Christian.Koenig@....com>
Subject: Re: [PATCH 0/3] drm/amdgpu: Fix suspend/resume issues with MST

On 2019-01-07 7:56 p.m., Lyude Paul wrote:
> Fix the suspend/issues that Jerry Zuo found in amdgpu, and add some
> compiler warnings for drivers ignoring the return code of
> drm_dp_mst_topology_mgr_resume() to help ensure we don't need to fix
> this again in the future for someone else's driver.
> 
> Cc: Jerry Zuo <Jerry.Zuo@....com>

With the small change Daniel mentioned this series is
Reviewed-by: Harry Wentland <harry.wentland@....com>

Harry

> 
> Lyude Paul (3):
>   drm/amdgpu: Don't ignore rc from drm_dp_mst_topology_mgr_resume()
>   drm/amdgpu: Don't fail resume process if resuming atomic state fails
>   drm/dp_mst: Add __must_check to drm_dp_mst_topology_mgr_resume()
> 
>  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 37 +++++++++++++------
>  drivers/gpu/drm/drm_dp_mst_topology.c         |  3 +-
>  include/drm/drm_dp_mst_helper.h               |  3 +-
>  3 files changed, 29 insertions(+), 14 deletions(-)
>