On 03/07/26 12:33, Maíra Canal wrote:
Being brief, this small series collects a correctness fix (#1) reported by
Sashiko, a error-path fix (#2) reported by Sashiko, and a cleanup (#3)
found while reading the code.
The first patch is the only notable one: a v3d submission expands into a
chain of jobs (e.g. BIN + RENDER + CACHE CLEAN), but the BO list was only
ever attached to the last job. Since implicit synchronization is added
per-job and gated on job->bo_count, every job before the last one silently
skipped implicit dependencies. In practice the RENDER job could read a
buffer object while another context was still writing it. It's fixed by
looking up the BOs onto every job that references them.
Best regards,
- Maíra
To: Melissa Wen <[email protected]>
To: Iago Toral Quiroga <[email protected]>
To: David Airlie <[email protected]>
To: Simona Vetter <[email protected]>
Cc: [email protected]
Cc: [email protected]
v1 -> v2:
https://lore.kernel.org/r/[email protected]
- [1/3] Don't attach implicit dependencies to BIN jobs (Iago Toral)
- [2/3, 3/3] Add Iago's R-b (Iago Toral)
v2 -> v3:
https://lore.kernel.org/r/[email protected]
- [1/3] Add a comment clarifing why we don't attach BOs to BIN jobs (Iago Toral)
- [1/3] Add Iago's R-b (Iago Toral)
---
Maíra Canal (3):
drm/v3d: Associate BOs with every job that accesses them
drm/v3d: Reject invalid indirect BO handle in indirect CSD setup
drm/v3d: Use write_to_buffer() helper in performance query copy
drivers/gpu/drm/v3d/v3d_sched.c | 11 ++--------
drivers/gpu/drm/v3d/v3d_submit.c | 44 +++++++++++++++++++++++-----------------
2 files changed, 27 insertions(+), 28 deletions(-)
Patches #2 and #3 were applied separately to drm-misc-fixes and drm-
misc-next, respectively, as I still need to address an issue pointed
by Sashiko in PATCH #1.
Best Regards,
- Maíra
---
base-commit: 903f8773ee96c5dc5fb9aec65227f39fd3e7a1dc
change-id: 20260609-v3d-cpu-job-fixes-fbe2051bee3a
