panfrost: Don't cleanup the job if it was successfully queued
authorBoris Brezillon <boris.brezillon@collabora.com>
Tue, 31 Aug 2021 13:35:56 +0000 (15:35 +0200)
committerSteven Price <steven.price@arm.com>
Thu, 2 Sep 2021 09:11:58 +0000 (10:11 +0100)
commit771d2053d41ff4d7c8dded432182cd1807ddc204
tree8c7e74fb89dade6bed284b8f7d2badcc861910bb
parentc97f082c135210da06a82c373fb660f8bfd57d95
panfrost: Don't cleanup the job if it was successfully queued

The labels are misleading. Even though they are all prefixed with 'fail_'
the success case also takes that path, and we should definitely not
cleanup the job if it's been queued. While at it, let's rename those
labels so we don't do the same mistake again.

Fixes: 53516280cc38 ("drm/panfrost: use scheduler dependency tracking")
Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Steven Price <steven.price@arm.com>
Tested-by: Steven Price <steven.price@arm.com>
Signed-off-by: Steven Price <steven.price@arm.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210831133556.236984-1-boris.brezillon@collabora.com
Link: https://patchwork.freedesktop.org/patch/msgid/20210831133556.236984-1-boris.brezillon@collabora.com
drivers/gpu/drm/panfrost/panfrost_drv.c