El vie, 09-01-2026 a las 15:35 -0300, Maíra Canal escribió:
(...)
> @@ -126,9 +126,9 @@ v3d_reset(struct v3d_dev *v3d)
> {
> struct drm_device *dev = &v3d->drm;
>
> - DRM_DEV_ERROR(dev->dev, "Resetting GPU for hang.\n");
> - DRM_DEV_ERROR(dev->dev, "V3D_ERR_STAT: 0x%08x\n",
> - V3D_CORE_READ(0, V3D_ERR_STAT));
> + drm_warn(dev, "Resetting GPU for hang.\n");
> + drm_warn(dev, "V3D_ERR_STAT: 0x%08x\n", V3D_CORE_READ(0,
> V3D_ERR_STAT));
> +
These look like they should be drm_err, no?
(...)
> static int
> -v3d_get_multisync_post_deps(struct drm_file *file_priv,
> +v3d_get_multisync_post_deps(struct v3d_dev *v3d,
Same comment as in the previous patch. Again, not necessarily against
it.
> + struct drm_file *file_priv,
> struct v3d_submit_ext *se,
> u32 count, u64 handles)
> {
> @@ -346,7 +347,7 @@ v3d_get_multisync_post_deps(struct drm_file
> *file_priv,
>
> if (copy_from_user(&out, post_deps++, sizeof(out)))
> {
> ret = -EFAULT;
> - DRM_DEBUG("Failed to copy post dep
> handles\n");
> + drm_dbg(&v3d->drm, "Failed to copy post dep
> handles\n");
I think we have a lot of these kind of things as dbg that sould
probably be err, no? Not sure if that is very relevant though, but we
can fix that separately if we want to.
Iago
> goto fail;
> }
>
