drm/panfrost: Fix implicit sync
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 22 Jun 2021 16:55:02 +0000 (18:55 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 23 Jun 2021 19:16:41 +0000 (21:16 +0200)
commit7601d53c2c49e3a7e8150e8cf332b3c17943f75a
tree293377dd603d1424da92b5a74c9e7d6419dfcc71
parent7d7a0fc4a5cef5ce0eefbe16325ae419a6eb3d39
drm/panfrost: Fix implicit sync

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: Boris Brezillon <boris.brezillon@collabora.com>
Signed-off-by: Daniel 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
drivers/gpu/drm/panfrost/panfrost_job.c