lkml.org 
[lkml]   [2021]   [Jul]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.13 549/800] drm/msm/disp/dpu1: avoid perf update in frame done event
    Date
    From: Krishna Manikandan <mkrishn@codeaurora.org>

    [ Upstream commit a1f2ba60eace242fd034173db3762f342a824a2e ]

    Crtc perf update from frame event work can result in
    wrong bandwidth and clock update from dpu if the work
    is scheduled after the swap state has happened.

    Avoid such issues by moving perf update to complete
    commit once the frame is accepted by the hardware.

    Fixes: a29c8c024165 ("drm/msm/disp/dpu1: fix display underruns during modeset")
    Signed-off-by: Krishna Manikandan <mkrishn@codeaurora.org>
    Tested-by: Douglas Anderson <dianders@chromium.org>
    Link: https://lore.kernel.org/r/1622092076-5100-1-git-send-email-mkrishn@codeaurora.org
    Signed-off-by: Rob Clark <robdclark@chromium.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 4 +---
    1 file changed, 1 insertion(+), 3 deletions(-)

    diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
    index 18bc76b7f1a3..4523d6ba891b 100644
    --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
    +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
    @@ -407,9 +407,6 @@ static void dpu_crtc_frame_event_work(struct kthread_work *work)
    fevent->event);
    }

    - if (fevent->event & DPU_ENCODER_FRAME_EVENT_DONE)
    - dpu_core_perf_crtc_update(crtc, 0, false);
    -
    if (fevent->event & (DPU_ENCODER_FRAME_EVENT_DONE
    | DPU_ENCODER_FRAME_EVENT_ERROR))
    frame_done = true;
    @@ -477,6 +474,7 @@ static void dpu_crtc_frame_event_cb(void *data, u32 event)
    void dpu_crtc_complete_commit(struct drm_crtc *crtc)
    {
    trace_dpu_crtc_complete_commit(DRMID(crtc));
    + dpu_core_perf_crtc_update(crtc, 0, false);
    _dpu_crtc_complete_flip(crtc);
    }

    --
    2.30.2


    \
     
     \ /
      Last update: 2021-07-12 10:46    [W:4.042 / U:2.148 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site