• Daniel Vetter's avatar
    drm/panfrost: Fix implicit sync · 7601d53c
    Daniel Vetter authored
    Currently this has no practial relevance I think because there's not
    many who can pull off a setup with panfrost and another gpu in the
    same system. But the rules are that if you're setting an exclusive
    fence, indicating a gpu write access in the implicit fencing system,
    then you need to wait for all fences, not just the previous exclusive
    fence.
    
    panfrost against itself has no problem, because it always sets the
    exclusive fence (but that's probably something that will need to be
    fixed for vulkan and/or multi-engine gpus, or you'll suffer badly).
    Also no problem with that against display.
    
    With the prep work done to switch over to the dependency helpers this
    is now a oneliner.
    Reviewed-by: default avatarBoris Brezillon <boris.brezillon@collabora.com>
    Signed-off-by: default avatarDaniel Vetter <daniel.vetter@intel.com>
    Cc: Rob Herring <robh@kernel.org>
    Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com>
    Cc: Steven Price <steven.price@arm.com>
    Cc: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
    Cc: Sumit Semwal <sumit.semwal@linaro.org>
    Cc: "Christian König" <christian.koenig@amd.com>
    Cc: linux-media@vger.kernel.org
    Cc: linaro-mm-sig@lists.linaro.org
    Link: https://patchwork.freedesktop.org/patch/msgid/20210622165511.3169559-7-daniel.vetter@ffwll.ch
    7601d53c
panfrost_job.c 16.9 KB