Comment 9 for bug 1553503

Revision history for this message
groucho (grouchomarx-fr) wrote :

Thank you for your guidance.
Here is the result of the reverse bisect :

e2e407dc093f530b771ee8bf8fe1be41e3cea8b3 is the first bad commit
commit e2e407dc093f530b771ee8bf8fe1be41e3cea8b3
Author: Matt Roper <email address hidden>
Date: Mon Feb 8 11:05:28 2016 -0800

    drm/i915: Pretend cursor is always on for ILK-style WM calculations (v2)

    Due to our lack of two-step watermark programming, our driver has
    historically pretended that the cursor plane is always on for the
    purpose of watermark calculations; this helps avoid serious flickering
    when the cursor turns off/on (e.g., when the user moves the mouse
    pointer to a different screen). That workaround was accidentally
    dropped as we started working toward atomic watermark updates. Since we
    still aren't quite there yet with two-stage updates, we need to
    resurrect the workaround and treat the cursor as always active.

    v2: Tweak cursor width calculations slightly to more closely match the
        logic we used before the atomic overhaul began. (Ville)

    Cc: <email address hidden>
    Cc: <email address hidden>
    Cc: <email address hidden>
    Reported-by: <email address hidden>
    Reported-by: <email address hidden>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93892
    Fixes: 43d59eda1 ("drm/i915: Eliminate usage of plane_wm_parameters from ILK-style WM code (v2)")
    Signed-off-by: Matt Roper <email address hidden>
    Link: http://patchwork.freedesktop.<email address hidden>
    (cherry picked from commit b2435692dbb709d4c8ff3b2f2815c9b8423b72bb)
    Signed-off-by: Jani Nikula <email address hidden>
    Link: http://patchwork.freedesktop.org<email address hidden>

:040000 040000 545634aa608f59cd8959a6196761450afa57ee5b 7b3259c003cbd1e27e02c976d73af848f69d248c M drivers