]> git.baikalelectronics.ru Git - kernel.git/commit
drm/v3d: fix sched job resources cleanup when a job is aborted
authorMelissa Wen <mwen@igalia.com>
Thu, 16 Sep 2021 21:27:26 +0000 (22:27 +0100)
committerMelissa Wen <melissa.srw@gmail.com>
Sun, 19 Sep 2021 19:40:23 +0000 (20:40 +0100)
commitf2043ca1d4fac18987490cb60a4ebabffde4b96e
treed0ed1f415cd1deadf51c5e4f09092959dc3fd297
parent23585f091e62863cbf04b55f6d57c292596d6e17
drm/v3d: fix sched job resources cleanup when a job is aborted

In a cl submission, when bin job initialization fails, sched job resources
were already allocated for the render job. At this point,
drm_sched_job_init(render) was done in v3d_job_init but the render job is
aborted before drm_sched_job_arm (in v3d_job_push) happens; therefore, not
only v3d_job_put but also drm_sched_job_cleanup should be called (by
v3d_job_cleanup). A similar issue is addressed for csd and tfu submissions.

The issue was noticed from a review by Iago Toral in a patch that touches
the same part of the code.

Fixes: a74e570ec3b86 ("drm/v3d: Move drm_sched_job_init to v3d_job_init")
Signed-off-by: Melissa Wen <mwen@igalia.com>
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
Signed-off-by: Melissa Wen <melissa.srw@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210916212726.2u2psq2egwy2mdva@mail.igalia.com
drivers/gpu/drm/v3d/v3d_gem.c