]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
drm/vc4: hvs: More logging for dlist generation
authorMaxime Ripard <mripard@kernel.org>
Fri, 21 Jun 2024 15:20:33 +0000 (16:20 +0100)
committerDave Stevenson <dave.stevenson@raspberrypi.com>
Mon, 9 Sep 2024 12:02:53 +0000 (13:02 +0100)
DLIST generation can get pretty tricky and there's not a lot of debug in
the driver to help. Let's add a few more to track the generated DLIST
size.

Signed-off-by: Maxime Ripard <mripard@kernel.org>
Reviewed-by: Maxime Ripard <mripard@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20240621152055.4180873-10-dave.stevenson@raspberrypi.com
Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/gpu/drm/vc4/vc4_hvs.c

index 32ecc26a749a12188353682e091e26ebcea188b6..e37851e13474a70c24b863765e28ca050810c4a8 100644 (file)
@@ -456,11 +456,21 @@ int vc4_hvs_atomic_check(struct drm_crtc *crtc, struct drm_atomic_state *state)
        if (hweight32(crtc_state->connector_mask) > 1)
                return -EINVAL;
 
-       drm_atomic_crtc_state_for_each_plane_state(plane, plane_state, crtc_state)
-               dlist_count += vc4_plane_dlist_size(plane_state);
+       drm_atomic_crtc_state_for_each_plane_state(plane, plane_state, crtc_state) {
+               u32 plane_dlist_count = vc4_plane_dlist_size(plane_state);
+
+               drm_dbg_driver(dev, "[CRTC:%d:%s] Found [PLANE:%d:%s] with DLIST size: %u\n",
+                              crtc->base.id, crtc->name,
+                              plane->base.id, plane->name,
+                              plane_dlist_count);
+
+               dlist_count += plane_dlist_count;
+       }
 
        dlist_count++; /* Account for SCALER_CTL0_END. */
 
+       drm_dbg_driver(dev, "[CRTC:%d:%s] Allocating DLIST block with size: %u\n",
+                      crtc->base.id, crtc->name, dlist_count);
        spin_lock_irqsave(&vc4->hvs->mm_lock, flags);
        ret = drm_mm_insert_node(&vc4->hvs->dlist_mm, &vc4_state->mm,
                                 dlist_count);