forked from marfrit/marfrit-packages
marfrit
5e68aec2e9
Replace the hand-rolled draft patches with the proper git-format-patch output. The new files apply cleanly via git am against unmodified Linux 6.12 mainline, verified by reset-and-apply roundtrip on /tmp/hantro-src (the local sparse checkout used during the chromium-fourier campaign). All kernel API calls also sanity-checked against the real include/linux/dma-fence.h and include/linux/dma-resv.h signatures: - dma_fence_init(fence, ops, lock, context, seqno) — argument list matches our call exactly - dma_resv_add_fence(obj, fence, usage) — DMA_RESV_USAGE_WRITE enum value confirmed present - dma_fence_signal, dma_fence_set_error, dma_fence_get, dma_fence_put, dma_fence_context_alloc — all present and correctly used - dma_resv_lock(obj, NULL), dma_resv_unlock — present, correctly paired README updated to reflect the post-verification status. Remaining gates before sending to linux-media are now: full-tree compile test (needs complete kernel checkout, hours of work), boot test on ohm (needs patched kernel build), and the runtime A/B (install patched kernel + uninstall kwin-fourier — chrome should still play 1080p30 because the fence is now real). Cover letter blurb filled in with the full motivation, test setup, and review-question list.
49 lines
1.9 KiB
Diff
49 lines
1.9 KiB
Diff
From 1a1619ab9ad3583842fbfffc649d7662619fb73b Mon Sep 17 00:00:00 2001
|
|
From: Markus Fritsche <mfritsche@reauktion.de>
|
|
Date: Tue, 28 Apr 2026 19:23:51 +0000
|
|
Subject: [PATCH 3/3] media: rockchip-rga: attach dma_resv release fence at
|
|
buf_queue
|
|
|
|
Opt the Rockchip RGA driver into the new vb2 release-fence helper.
|
|
|
|
Same shape as the hantro patch: rga_buf_queue enqueues the buffer
|
|
in the driver's m2m queue via v4l2_m2m_buf_queue and additionally
|
|
attaches a release fence to each plane's dmabuf->resv via
|
|
vb2_buffer_attach_release_fence(). vb2_buffer_done signals the
|
|
fence when RGA completes the M2M operation.
|
|
|
|
Userspace consumers of RGA-produced dmabufs (image-processing
|
|
pipelines, screen-rotation servers, gstreamer flows on Rockchip
|
|
boards) get spec-clean implicit-sync semantics, matching what
|
|
hantro now does in the same patch series.
|
|
|
|
Signed-off-by: Markus Fritsche <mfritsche@reauktion.de>
|
|
---
|
|
drivers/media/platform/rockchip/rga/rga-buf.c | 10 ++++++++++
|
|
1 file changed, 10 insertions(+)
|
|
|
|
diff --git a/drivers/media/platform/rockchip/rga/rga-buf.c b/drivers/media/platform/rockchip/rga/rga-buf.c
|
|
index 70808049d..5557ca632 100644
|
|
--- a/drivers/media/platform/rockchip/rga/rga-buf.c
|
|
+++ b/drivers/media/platform/rockchip/rga/rga-buf.c
|
|
@@ -153,6 +153,16 @@ static void rga_buf_queue(struct vb2_buffer *vb)
|
|
struct rga_ctx *ctx = vb2_get_drv_priv(vb->vb2_queue);
|
|
|
|
v4l2_m2m_buf_queue(ctx->fh.m2m_ctx, vbuf);
|
|
+
|
|
+ /*
|
|
+ * Opt in to vb2's dma_resv release-fence path so userspace
|
|
+ * consumers of RGA-produced dmabufs get a real producer fence
|
|
+ * to wait on instead of the dma_buf core's stub fence. See
|
|
+ * the leading patch in this series for rationale. Best-effort:
|
|
+ * fence-allocation failure means we lose implicit-sync
|
|
+ * precision but the m2m operation itself proceeds normally.
|
|
+ */
|
|
+ (void)vb2_buffer_attach_release_fence(vb);
|
|
}
|
|
|
|
static void rga_buf_cleanup(struct vb2_buffer *vb)
|
|
--
|
|
2.47.3
|
|
|