drm/vc4: add gem_info node via debugfs

The memps requires gem_info with gem_names to analyze graphics
shared memory, so this patch adds gem_info node via debugfs
interface.

Let's save pid/tgid in private file data only once when gem object is
created or prime_fd is imported and use them on gem_info. This is to
solve the wrong pid problem of gem_info created as different processes
share fd of drm device node.

Change-Id: Iaac2ccd63e124dd11a894f0e88ddbb533750f3d4
Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>

diff --git a/drivers/gpu/drm/vc4/vc4_bo.c b/drivers/gpu/drm/vc4/vc4_bo.c
index 55bbadc86a1ee45205360fb26e8525a8326db017..f25698dbe5af6454c8fb8dcaa0cee9b8cb8f5597 100644 (file)
--- a/drivers/gpu/drm/vc4/vc4_bo.c
+++ b/drivers/gpu/drm/vc4/vc4_bo.c
@@ -66,6 +66,76 @@ int vc4_bo_stats_debugfs(struct seq_file *m, void *unused)
 
        return 0;
 }
+
+struct vc4_gem_info_data {
+       struct drm_file *filp;
+       struct seq_file *m;
+};
+
+static int vc4_gem_one_info(int id, void *ptr, void *data)
+{
+       struct drm_gem_object *obj = (struct drm_gem_object *)ptr;
+       struct vc4_gem_info_data *gem_info_data = data;
+       struct drm_vc4_file_private *file_priv =
+               gem_info_data->filp->driver_priv;
+
+       if (!obj) {
+               DRM_ERROR("failed to get drm_gem_object\n");
+               return -EFAULT;
+       }
+
+       drm_gem_object_reference(obj);
+
+       seq_printf(gem_info_data->m,
+                       "%5d\t%5d\t%4d\t%4d\t\t%4d\t0x%08lx\t0x%x\t%4d\t%4d\t\t"
+                       "%4d\t\t0x%p\t%6d\n",
+                       file_priv->pid,
+                       file_priv->tgid,
+                       id,
+                       atomic_read(&obj->refcount.refcount) - 1,
+                       obj->handle_count,
+                       obj->size,
+                       0,
+                       0,
+                       obj->dma_buf ? 1 : 0,
+                       obj->import_attach ? 1 : 0,
+                       obj,
+                       obj->name);
+
+       drm_gem_object_unreference(obj);
+
+       return 0;
+}
+
+int vc4_debugfs_gem_info(struct seq_file *m, void *data)
+{
+       struct drm_info_node *node = (struct drm_info_node *)m->private;
+       struct drm_minor *minor = node->minor;
+       struct drm_device *drm_dev = minor->dev;
+       struct vc4_gem_info_data gem_info_data;
+       struct drm_file *filp;
+
+       gem_info_data.m = m;
+
+       seq_puts(gem_info_data.m,
+                       "pid\ttgid\thandle\trefcount\thcount\tsize\t\tflags\t"
+                       "pfnmap\texport_to_fd\timport_from_fd\tobj_addr\t\t"
+                       "name\n");
+
+       mutex_lock(&drm_dev->struct_mutex);
+       list_for_each_entry(filp, &drm_dev->filelist, lhead) {
+               gem_info_data.filp = filp;
+
+               spin_lock(&filp->table_lock);
+               idr_for_each(&filp->object_idr, vc4_gem_one_info,
+                               &gem_info_data);
+               spin_unlock(&filp->table_lock);
+       }
+       mutex_unlock(&drm_dev->struct_mutex);
+
+       return 0;
+}
+
 #endif
 
 static uint32_t bo_page_index(size_t size)
@@ -245,6 +315,25 @@ struct vc4_bo *vc4_bo_create(struct drm_device *dev, size_t unaligned_size,
        return to_vc4_bo(&cma_obj->base);
 }
 
+static void vc4_gem_register_pid(struct drm_file *file_priv)
+{
+       struct drm_vc4_file_private *driver_priv = file_priv->driver_priv;
+
+       if (!driver_priv->pid && !driver_priv->tgid) {
+               driver_priv->pid = task_pid_nr(current);
+               driver_priv->tgid = task_tgid_nr(current);
+       } else {
+               if (driver_priv->pid != task_pid_nr(current))
+                       DRM_DEBUG_KMS("wrong pid: %ld, %ld\n",
+                                       (unsigned long)driver_priv->pid,
+                                       (unsigned long)task_pid_nr(current));
+               if (driver_priv->tgid != task_tgid_nr(current))
+                       DRM_DEBUG_KMS("wrong tgid: %ld, %ld\n",
+                                       (unsigned long)driver_priv->tgid,
+                                       (unsigned long)task_tgid_nr(current));
+       }
+}
+
 int vc4_dumb_create(struct drm_file *file_priv,
                    struct drm_device *dev,
                    struct drm_mode_create_dumb *args)
@@ -266,6 +355,9 @@ int vc4_dumb_create(struct drm_file *file_priv,
        ret = drm_gem_handle_create(file_priv, &bo->base.base, &args->handle);
        drm_gem_object_unreference_unlocked(&bo->base.base);
 
+       if (!ret)
+               vc4_gem_register_pid(file_priv);
+
        return ret;
 }
 
@@ -379,6 +471,21 @@ vc4_prime_export(struct drm_device *dev, struct drm_gem_object *obj, int flags)
        return drm_gem_prime_export(dev, obj, flags);
 }
 
+int vc4_drm_gem_prime_fd_to_handle(struct drm_device *dev,
+               struct drm_file *file_priv, int prime_fd, uint32_t *handle)
+{
+       int ret;
+
+       ret = drm_gem_prime_fd_to_handle(dev, file_priv, prime_fd, handle);
+       if (ret < 0)
+               goto out;
+
+       vc4_gem_register_pid(file_priv);
+
+out:
+       return ret;
+}
+
 int vc4_mmap(struct file *filp, struct vm_area_struct *vma)
 {
        struct drm_gem_object *gem_obj;
@@ -455,6 +562,9 @@ int vc4_create_bo_ioctl(struct drm_device *dev, void *data,
        ret = drm_gem_handle_create(file_priv, &bo->base.base, &args->handle);
        drm_gem_object_unreference_unlocked(&bo->base.base);
 
+       if (!ret)
+               vc4_gem_register_pid(file_priv);
+
        return ret;
 }
 
@@ -528,6 +638,9 @@ vc4_create_shader_bo_ioctl(struct drm_device *dev, void *data,
         */
        ret = drm_gem_handle_create(file_priv, &bo->base.base, &args->handle);
 
+       if (!ret)
+               vc4_gem_register_pid(file_priv);
+
  fail:
        drm_gem_object_unreference_unlocked(&bo->base.base);
 

diff --git a/drivers/gpu/drm/vc4/vc4_debugfs.c b/drivers/gpu/drm/vc4/vc4_debugfs.c
index 3ca476c6e0578e5fbe60d4fd623b09f8f937c0b7..91e957bd8f493a77f3a052bd95668038821903b6 100644 (file)
--- a/drivers/gpu/drm/vc4/vc4_debugfs.c
+++ b/drivers/gpu/drm/vc4/vc4_debugfs.c
@@ -27,6 +27,7 @@ static const struct drm_info_list vc4_debugfs_list[] = {
        {"crtc2_regs", vc4_crtc_debugfs_regs, 0, (void *)(uintptr_t)2},
        {"v3d_ident", vc4_v3d_debugfs_ident, 0},
        {"v3d_regs", vc4_v3d_debugfs_regs, 0},
+       {"gem_info", vc4_debugfs_gem_info, 0},
 };
 
 #define VC4_DEBUGFS_ENTRIES ARRAY_SIZE(vc4_debugfs_list)

diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c
index f528edc4b389e2e3843cd8be2faccf62947cbda4..e477119909412cc77c71c4b0face0c6ac8f5473e 100644 (file)
--- a/drivers/gpu/drm/vc4/vc4_drv.c
+++ b/drivers/gpu/drm/vc4/vc4_drv.c
@@ -93,6 +93,32 @@ static int vc4_get_param_ioctl(struct drm_device *dev, void *data,
        return 0;
 }
 
+static int vc4_open(struct drm_device *dev, struct drm_file *file)
+{
+       struct drm_vc4_file_private *file_priv;
+
+       DRM_DEBUG_DRIVER("%s\n", __FILE__);
+
+       file_priv = kzalloc(sizeof(*file_priv), GFP_KERNEL);
+       if (!file_priv)
+               return -ENOMEM;
+
+       file->driver_priv = file_priv;
+
+       return 0;
+}
+
+static void vc4_postclose(struct drm_device *dev, struct drm_file *file)
+{
+       DRM_DEBUG_DRIVER("%s\n", __FILE__);
+
+       if (!file->driver_priv)
+               return;
+
+       kfree(file->driver_priv);
+       file->driver_priv = NULL;
+}
+
 static void vc4_lastclose(struct drm_device *dev)
 {
        struct vc4_dev *vc4 = to_vc4_dev(dev);
@@ -135,6 +161,8 @@ static struct drm_driver vc4_drm_driver = {
                            DRIVER_HAVE_IRQ |
                            DRIVER_RENDER |
                            DRIVER_PRIME),
+       .open = vc4_open,
+       .postclose = vc4_postclose,
        .lastclose = vc4_lastclose,
        .irq_handler = vc4_irq,
        .irq_preinstall = vc4_irq_preinstall,
@@ -157,7 +185,7 @@ static struct drm_driver vc4_drm_driver = {
        .gem_vm_ops = &drm_gem_cma_vm_ops,
 
        .prime_handle_to_fd = drm_gem_prime_handle_to_fd,
-       .prime_fd_to_handle = drm_gem_prime_fd_to_handle,
+       .prime_fd_to_handle = vc4_drm_gem_prime_fd_to_handle,
        .gem_prime_import = drm_gem_prime_import,
        .gem_prime_export = vc4_prime_export,
        .gem_prime_get_sg_table = drm_gem_cma_prime_get_sg_table,

diff --git a/drivers/gpu/drm/vc4/vc4_drv.h b/drivers/gpu/drm/vc4/vc4_drv.h
index fcf1c45783736413c058a33f31e189ebaddb446e..705bcd663896fb27e3f8cb4d625084d9bc052049 100644 (file)
--- a/drivers/gpu/drm/vc4/vc4_drv.h
+++ b/drivers/gpu/drm/vc4/vc4_drv.h
@@ -577,3 +577,12 @@ bool vc4_check_tex_size(struct vc4_exec_info *exec,
 /* vc4_validate_shader.c */
 struct vc4_validated_shader_info *
 vc4_validate_shader(struct drm_gem_cma_object *shader_obj);
+
+struct drm_vc4_file_private {
+       pid_t                           pid;
+       pid_t                           tgid;
+};
+
+int vc4_debugfs_gem_info(struct seq_file *m, void *data);
+int vc4_drm_gem_prime_fd_to_handle(struct drm_device *dev,
+               struct drm_file *file_priv, int prime_fd, uint32_t *handle);