Reviewed-by: Jacek Lawrynowicz <jacek.lawrynowicz@xxxxxxxxxxxxxxx>
On 24.05.2023 09:48, Stanislaw Gruszka wrote:
> Add initial debugfs support. Provide below functionality:
>
> - print buffer objects
> - print latest boot mode
> - trigger vpu engine reset
>
> Signed-off-by: Stanislaw Gruszka <stanislaw.gruszka@xxxxxxxxxxxxxxx>
> ---
> drivers/accel/ivpu/Makefile | 1 +
> drivers/accel/ivpu/ivpu_debugfs.c | 74 +++++++++++++++++++++++++++++++
> drivers/accel/ivpu/ivpu_debugfs.h | 13 ++++++
> drivers/accel/ivpu/ivpu_drv.c | 5 +++
> 4 files changed, 93 insertions(+)
> create mode 100644 drivers/accel/ivpu/ivpu_debugfs.c
> create mode 100644 drivers/accel/ivpu/ivpu_debugfs.h
>
> diff --git a/drivers/accel/ivpu/Makefile b/drivers/accel/ivpu/Makefile
> index 80f1fb3548ae..3ca2fb3936f6 100644
> --- a/drivers/accel/ivpu/Makefile
> +++ b/drivers/accel/ivpu/Makefile
> @@ -2,6 +2,7 @@
> # Copyright (C) 2023 Intel Corporation
>
> intel_vpu-y := \
> + ivpu_debugfs.o \
> ivpu_drv.o \
> ivpu_fw.o \
> ivpu_gem.o \
> diff --git a/drivers/accel/ivpu/ivpu_debugfs.c b/drivers/accel/ivpu/ivpu_debugfs.c
> new file mode 100644
> index 000000000000..df51ec008fb5
> --- /dev/null
> +++ b/drivers/accel/ivpu/ivpu_debugfs.c
> @@ -0,0 +1,74 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (C) 2020-2023 Intel Corporation
> + */
> +
> +#include <drm/drm_debugfs.h>
> +#include <drm/drm_file.h>
> +#include <drm/drm_print.h>
> +
> +#include <uapi/drm/ivpu_accel.h>
> +
> +#include "ivpu_debugfs.h"
> +#include "ivpu_drv.h"
> +#include "ivpu_gem.h"
> +#include "ivpu_jsm_msg.h"
> +#include "ivpu_pm.h"
> +
> +static int bo_list_show(struct seq_file *s, void *v)
> +{
> + struct drm_info_node *node = (struct drm_info_node *)s->private;
> + struct drm_printer p = drm_seq_file_printer(s);
> +
> + ivpu_bo_list(node->minor->dev, &p);
> +
> + return 0;
> +}
> +
> +static int last_bootmode_show(struct seq_file *s, void *v)
> +{
> + struct drm_info_node *node = (struct drm_info_node *)s->private;
> + struct ivpu_device *vdev = to_ivpu_device(node->minor->dev);
> +
> + seq_printf(s, "%s\n", (vdev->pm->is_warmboot) ? "warmboot" : "coldboot");
> +
> + return 0;
> +}
> +
> +static const struct drm_info_list vdev_debugfs_list[] = {
> + {"bo_list", bo_list_show, 0},
> + {"last_bootmode", last_bootmode_show, 0},
> +};
> +
> +static ssize_t
> +ivpu_reset_engine_fn(struct file *file, const char __user *user_buf, size_t size, loff_t *pos)
> +{
> + struct ivpu_device *vdev = file->private_data;
> +
> + if (!size)
> + return -EINVAL;
> +
> + if (ivpu_jsm_reset_engine(vdev, DRM_IVPU_ENGINE_COMPUTE))
> + return -ENODEV;
> + if (ivpu_jsm_reset_engine(vdev, DRM_IVPU_ENGINE_COPY))
> + return -ENODEV;
> +
> + return size;
> +}
> +
> +static const struct file_operations ivpu_reset_engine_fops = {
> + .owner = THIS_MODULE,
> + .open = simple_open,
> + .write = ivpu_reset_engine_fn,
> +};
> +
> +void ivpu_debugfs_init(struct drm_minor *minor)
> +{
> + struct ivpu_device *vdev = to_ivpu_device(minor->dev);
> +
> + drm_debugfs_create_files(vdev_debugfs_list, ARRAY_SIZE(vdev_debugfs_list),
> + minor->debugfs_root, minor);
> +
> + debugfs_create_file("reset_engine", 0200, minor->debugfs_root, vdev,
> + &ivpu_reset_engine_fops);
> +}
> diff --git a/drivers/accel/ivpu/ivpu_debugfs.h b/drivers/accel/ivpu/ivpu_debugfs.h
> new file mode 100644
> index 000000000000..78f80c1e00e4
> --- /dev/null
> +++ b/drivers/accel/ivpu/ivpu_debugfs.h
> @@ -0,0 +1,13 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +/*
> + * Copyright (C) 2020-2023 Intel Corporation
> + */
> +
> +#ifndef __IVPU_DEBUGFS_H__
> +#define __IVPU_DEBUGFS_H__
> +
> +struct drm_minor;
> +
> +void ivpu_debugfs_init(struct drm_minor *minor);
> +
> +#endif /* __IVPU_DEBUGFS_H__ */
> diff --git a/drivers/accel/ivpu/ivpu_drv.c b/drivers/accel/ivpu/ivpu_drv.c
> index 2df7643b843d..4c0345417c14 100644
> --- a/drivers/accel/ivpu/ivpu_drv.c
> +++ b/drivers/accel/ivpu/ivpu_drv.c
> @@ -14,6 +14,7 @@
> #include <drm/drm_prime.h>
>
> #include "vpu_boot_api.h"
> +#include "ivpu_debugfs.h"
> #include "ivpu_drv.h"
> #include "ivpu_fw.h"
> #include "ivpu_gem.h"
> @@ -378,6 +379,10 @@ static const struct drm_driver driver = {
> .gem_prime_import = ivpu_gem_prime_import,
> .gem_prime_mmap = drm_gem_prime_mmap,
>
> +#if defined(CONFIG_DEBUG_FS)
> + .debugfs_init = ivpu_debugfs_init,
> +#endif
> +
> .ioctls = ivpu_drm_ioctls,
> .num_ioctls = ARRAY_SIZE(ivpu_drm_ioctls),
> .fops = &ivpu_fops,
