From nobody Sun Feb 8 17:42:47 2026 Received: from mail-wm1-f45.google.com (mail-wm1-f45.google.com [209.85.128.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F08AE19D089 for ; Tue, 13 Aug 2024 10:51:53 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.45 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723546316; cv=none; b=aD0bWa4oDSkVMJBHkaSXKPnhaUjOncyfqSKh//QM+jx4ySqpFr1AtsGiGeDu6t+/CAazf4sFp3EFwVzuvaeENGxW8R1bPjTPYtZi1pQncaUvN0c9IUmsIpA3vvC1qGtd/IQMrk4QvsOW/IXC9qqg3HskzGv/BWqaBKfVzHT0tkQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1723546316; c=relaxed/simple; bh=XAkTXSUnO8yKhxoft8vmT45KV7rwTDV6cptm5NVJy0g=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=l9Zgm2+GL0mbStu7VQBGK7Sk0lGT2+dqB44RgpY8U/Qppi4Gbp2ib1+CMqz7Gk2l39ijAWEfS+uSXdBWizbRo/ScNJtJWWmhwOvUDTkzNJVnVrXAW6Rs9xQuxUKgqYXitfmgNJbeCfThSbk8kdNOPW9PDX89D1Y1+3rfxnJQVTQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=a2blu1h3; arc=none smtp.client-ip=209.85.128.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="a2blu1h3" Received: by mail-wm1-f45.google.com with SMTP id 5b1f17b1804b1-42817f1eb1fso37585065e9.1 for ; Tue, 13 Aug 2024 03:51:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1723546312; x=1724151112; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=2DZ0yno09DV6vF4OePQpExTeBzGL+DurmJi2vCnaEVY=; b=a2blu1h3Jef8yLbTvXN4Lx1tkS6AMTWawlxK8jiI+6a/V5qHN4dS+vjC/EYBVZAJ+C z1XhVEU40oZuIzvpBPlAZhbn7iydc7VIKKn8vdFgl3uc6Dd+enowRQvhYoIzSJznwdxH 5BMwZ6TGrz0Exst15X3n5G5VnyhuMFvH+5dBgVQkviARdNjO0slI8n7lSKWZADozaryl 1gT7z4Kn3PXuMcWFe6BWnsRE4NetB8Q/HA0gYYnjX442dLoI+/a/iP8dKKTDdu+MhuVR MUqdOKqmjzi2K8ffSuwmhWPNmyhShWjWomaJfdyUDYKxP0OSy0eaLEHG7g69K0WIgW1a 8l5A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1723546312; x=1724151112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=2DZ0yno09DV6vF4OePQpExTeBzGL+DurmJi2vCnaEVY=; b=Spds5qJ/l0WBb22OVNnJHaDqxyiJFbipyiCsdO+m3KvtXpYlkgfWVupJ3VbnT2m73E eFDbayCSPBpXbO1muFcZelpDDfOtvhSBYrvGoPMpTzYbCuStPLdtsRJWjw1Lxbfdqqch vbjHPStSgMlLZaYdGQeIxu+OGD6jScwzGbi0OhwAZI31z+Jk634W4BIgskW1TE8iG9d2 zHGSUGDmiY5fX79TlIHreRu7l6aFNiLmEaMe1vO5FXMPQ0JzkrAbg0GmXwIcjJG3n8Uu NM2RE6XHpz9FZUiohjv22v+CkEB3sq7I+GAbS69YwIy7jq6PCP/3kHEldKiXpUMoQj/R xGLQ== X-Forwarded-Encrypted: i=1; AJvYcCWUmhKeW3pQpxTsraSDhxtK0yKzbwKFP+95ksGBx+fYKizk80ilcmdW1fSkurxXdbmZHADiiMr1rBdByiQjvYTObdiewcJIWIKLUqsr X-Gm-Message-State: AOJu0Yxr9jRRN3U577ljqWPUaMAaYW51Gh2WEn7v+Apz57A7x/7PiTth WzhgtOP6/kqD0jKzNG8lUVvv80GfBtVMMtmXJi5FECBi0gWGobQG X-Google-Smtp-Source: AGHT+IGgpvIqHt0WTDhpBJPlq78QSS4s/Hy1j9s/Fr8VBNAtgOUXDZ38YXLRqoUGqRmktkNgk0w2DQ== X-Received: by 2002:a05:600c:1391:b0:426:6416:aa73 with SMTP id 5b1f17b1804b1-429d4828df2mr20461065e9.12.1723546312199; Tue, 13 Aug 2024 03:51:52 -0700 (PDT) Received: from fedora.. ([213.94.26.172]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-429d877e066sm17290785e9.1.2024.08.13.03.51.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Aug 2024 03:51:51 -0700 (PDT) From: =?UTF-8?q?Jos=C3=A9=20Exp=C3=B3sito?= To: rodrigosiqueiramelo@gmail.com Cc: melissa.srw@gmail.com, mairacanal@riseup.net, hamohammed.sa@gmail.com, daniel@ffwll.ch, maarten.lankhorst@linux.intel.com, mripard@kernel.org, tzimmermann@suse.de, airlied@gmail.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, louis.chauvet@bootlin.com, =?UTF-8?q?Jos=C3=A9=20Exp=C3=B3sito?= Subject: [RFC PATCH 11/17] drm/vkms: Add and remove VKMS instances via configfs Date: Tue, 13 Aug 2024 12:44:22 +0200 Message-ID: <20240813105134.17439-12-jose.exposito89@gmail.com> X-Mailer: git-send-email 2.46.0 In-Reply-To: <20240813105134.17439-1-jose.exposito89@gmail.com> References: <20240813105134.17439-1-jose.exposito89@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Allow to create, enable, disable and destroy VKMS instances using configfs. For the moment, add one primary plane, one CRC, one encoder and one connector until we add support to configure them via configfs. Signed-off-by: Jos=C3=A9 Exp=C3=B3sito --- Documentation/gpu/vkms.rst | 32 +++++ drivers/gpu/drm/vkms/Kconfig | 1 + drivers/gpu/drm/vkms/Makefile | 3 +- drivers/gpu/drm/vkms/vkms_configfs.c | 198 +++++++++++++++++++++++++++ drivers/gpu/drm/vkms/vkms_configfs.h | 9 ++ drivers/gpu/drm/vkms/vkms_drv.c | 19 ++- drivers/gpu/drm/vkms/vkms_drv.h | 4 + 7 files changed, 262 insertions(+), 4 deletions(-) create mode 100644 drivers/gpu/drm/vkms/vkms_configfs.c create mode 100644 drivers/gpu/drm/vkms/vkms_configfs.h diff --git a/Documentation/gpu/vkms.rst b/Documentation/gpu/vkms.rst index ba04ac7c2167..9895a9ae76f4 100644 --- a/Documentation/gpu/vkms.rst +++ b/Documentation/gpu/vkms.rst @@ -51,6 +51,38 @@ To disable the driver, use :: =20 sudo modprobe -r vkms =20 +Configuring With Configfs +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D + +It is possible to create and configure multiple VKMS instances via configf= s. + +Start by mounting configfs and loading VKMS:: + + sudo mount -t configfs none /config + sudo modprobe vkms + +Once VKMS is loaded, ``/config/vkms`` is created automatically. Each direc= tory +under ``/config/vkms`` represents a VKMS instance, create a new one:: + + sudo mkdir /config/vkms/my-vkms + +By default, the instance is disabled:: + + cat /config/vkms/my-vkms/enabled + 0 + +Once you are done configuring the VKMS instance, enable it:: + + echo "1" | sudo tee /config/vkms/my-vkms/enabled + +Finally, you can remove the VKMS instance disabling it:: + + echo "0" | sudo tee /config/vkms/my-vkms/enabled + +Or removing the top level directory:: + + sudo rmdir /config/vkms/my-vkms + Testing With IGT =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D =20 diff --git a/drivers/gpu/drm/vkms/Kconfig b/drivers/gpu/drm/vkms/Kconfig index b9ecdebecb0b..de218b21a579 100644 --- a/drivers/gpu/drm/vkms/Kconfig +++ b/drivers/gpu/drm/vkms/Kconfig @@ -6,6 +6,7 @@ config DRM_VKMS select DRM_KMS_HELPER select DRM_GEM_SHMEM_HELPER select CRC32 + select CONFIGFS_FS default n help Virtual Kernel Mode-Setting (VKMS) is used for testing or for diff --git a/drivers/gpu/drm/vkms/Makefile b/drivers/gpu/drm/vkms/Makefile index b371b5d70ee3..81c257d37689 100644 --- a/drivers/gpu/drm/vkms/Makefile +++ b/drivers/gpu/drm/vkms/Makefile @@ -7,6 +7,7 @@ vkms-y :=3D \ vkms_crtc.o \ vkms_composer.o \ vkms_writeback.o \ - vkms_config.o + vkms_config.o \ + vkms_configfs.o =20 obj-$(CONFIG_DRM_VKMS) +=3D vkms.o diff --git a/drivers/gpu/drm/vkms/vkms_configfs.c b/drivers/gpu/drm/vkms/vk= ms_configfs.c new file mode 100644 index 000000000000..3f25295f7788 --- /dev/null +++ b/drivers/gpu/drm/vkms/vkms_configfs.c @@ -0,0 +1,198 @@ +// SPDX-License-Identifier: GPL-2.0+ + +#include +#include +#include + +#include "vkms_drv.h" +#include "vkms_config.h" +#include "vkms_configfs.h" + +/* To avoid registering configfs more than once or unregistering on error = */ +static bool is_configfs_registered; + +/** + * struct vkms_configfs - Configfs configuration for a VKMS device + * + * @vkms_config: Configuration of the VKMS device + * @device_group: Top level configuration group that represents a VKMS dev= ice. + * Initialized when a new directory is created under "/config/vkms/" + * @lock: Lock used to project concurrent access to the configuration attr= ibutes + * @enabled: Protected by @lock. The device is created or destroyed when t= his + * option changes + */ +struct vkms_configfs { + struct vkms_config *vkms_config; + struct config_group device_group; + + /* protected by @lock */ + struct mutex lock; + bool enabled; +}; + +#define config_item_to_vkms_configfs(item) \ + container_of(to_config_group(item), struct vkms_configfs, device_group) + +static ssize_t device_enabled_show(struct config_item *item, char *page) +{ + struct vkms_configfs *configfs =3D config_item_to_vkms_configfs(item); + + return sprintf(page, "%d\n", configfs->enabled); +} + +static ssize_t device_enabled_store(struct config_item *item, const char *= page, + size_t count) +{ + struct vkms_configfs *configfs =3D config_item_to_vkms_configfs(item); + bool enabled; + int ret =3D 0; + + if (kstrtobool(page, &enabled)) + return -EINVAL; + + mutex_lock(&configfs->lock); + + if (!configfs->enabled && enabled) + ret =3D vkms_create(configfs->vkms_config); + else if (configfs->enabled && !enabled) + vkms_destroy(configfs->vkms_config); + + if (ret) + goto err_unlock; + + configfs->enabled =3D enabled; + + mutex_unlock(&configfs->lock); + + return (ssize_t)count; + +err_unlock: + mutex_unlock(&configfs->lock); + return ret; +} + +CONFIGFS_ATTR(device_, enabled); + +static struct configfs_attribute *device_group_attrs[] =3D { + &device_attr_enabled, + NULL, +}; + +static const struct config_item_type device_group_type =3D { + .ct_attrs =3D device_group_attrs, + .ct_owner =3D THIS_MODULE, +}; + +static struct config_group *make_device_group(struct config_group *group, + const char *name) +{ + struct vkms_configfs *configfs; + struct vkms_config_crtc *crtc_cfg =3D NULL; + struct vkms_config_encoder *encoder_cfg =3D NULL; + struct vkms_config_connector *connector_cfg =3D NULL; + char *config_name; + int ret; + + if (strcmp(name, DEFAULT_DEVICE_NAME) =3D=3D 0) + return ERR_PTR(-EINVAL); + + configfs =3D kzalloc(sizeof(*configfs), GFP_KERNEL); + if (!configfs) + return ERR_PTR(-ENOMEM); + + config_group_init_type_name(&configfs->device_group, name, &device_group_= type); + mutex_init(&configfs->lock); + + config_name =3D config_item_name(&configfs->device_group.cg_item); + configfs->vkms_config =3D vkms_config_create(config_name); + if (IS_ERR(configfs->vkms_config)) { + ret =3D PTR_ERR(configfs->vkms_config); + goto err_kfree; + } + + crtc_cfg =3D vkms_config_add_crtc(configfs->vkms_config, false, false); + if (IS_ERR(crtc_cfg)) { + ret =3D PTR_ERR(crtc_cfg); + goto err_kfree; + } + + encoder_cfg =3D vkms_config_add_encoder(configfs->vkms_config, BIT(0)); + if (IS_ERR(encoder_cfg)) { + ret =3D PTR_ERR(encoder_cfg); + goto err_kfree; + } + + connector_cfg =3D vkms_config_add_connector(configfs->vkms_config, BIT(0), + connector_status_connected); + if (IS_ERR(connector_cfg)) { + ret =3D PTR_ERR(connector_cfg); + goto err_kfree; + } + + return &configfs->device_group; + +err_kfree: + kfree(configfs); + kfree(crtc_cfg); + kfree(encoder_cfg); + kfree(connector_cfg); + return ERR_PTR(ret); +} + +static void drop_device_group(struct config_group *group, + struct config_item *item) +{ + struct vkms_configfs *configfs =3D config_item_to_vkms_configfs(item); + + mutex_lock(&configfs->lock); + + if (configfs->enabled) + vkms_destroy(configfs->vkms_config); + + kfree(configfs->vkms_config); + + mutex_unlock(&configfs->lock); + + kfree(configfs); +} + +static struct configfs_group_operations device_group_ops =3D { + .make_group =3D &make_device_group, + .drop_item =3D &drop_device_group, +}; + +static struct config_item_type vkms_type =3D { + .ct_group_ops =3D &device_group_ops, + .ct_owner =3D THIS_MODULE, +}; + +static struct configfs_subsystem vkms_subsys =3D { + .su_group =3D { + .cg_item =3D { + .ci_name =3D "vkms", + .ci_type =3D &vkms_type, + }, + }, + .su_mutex =3D __MUTEX_INITIALIZER(vkms_subsys.su_mutex), +}; + +int vkms_configfs_register(void) +{ + int ret; + + if (is_configfs_registered) + return 0; + + config_group_init(&vkms_subsys.su_group); + ret =3D configfs_register_subsystem(&vkms_subsys); + + is_configfs_registered =3D ret =3D=3D 0; + + return ret; +} + +void vkms_configfs_unregister(void) +{ + if (is_configfs_registered) + configfs_unregister_subsystem(&vkms_subsys); +} diff --git a/drivers/gpu/drm/vkms/vkms_configfs.h b/drivers/gpu/drm/vkms/vk= ms_configfs.h new file mode 100644 index 000000000000..b5d7fab877d5 --- /dev/null +++ b/drivers/gpu/drm/vkms/vkms_configfs.h @@ -0,0 +1,9 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ + +#ifndef _VKMS_CONFIGFS_H_ +#define _VKMS_CONFIGFS_H_ + +int vkms_configfs_register(void); +void vkms_configfs_unregister(void); + +#endif /* _VKMS_CONFIGFS_H_ */ diff --git a/drivers/gpu/drm/vkms/vkms_drv.c b/drivers/gpu/drm/vkms/vkms_dr= v.c index b0a079eb4598..f524a9550017 100644 --- a/drivers/gpu/drm/vkms/vkms_drv.c +++ b/drivers/gpu/drm/vkms/vkms_drv.c @@ -27,6 +27,7 @@ #include =20 #include "vkms_config.h" +#include "vkms_configfs.h" #include "vkms_drv.h" =20 #define DRIVER_NAME "vkms" @@ -157,7 +158,7 @@ static int vkms_modeset_init(struct vkms_device *vkmsde= v) return vkms_output_init(vkmsdev); } =20 -static int vkms_create(struct vkms_config *config) +int vkms_create(struct vkms_config *config) { int ret; struct platform_device *pdev; @@ -231,12 +232,22 @@ static int __init vkms_init(void) =20 ret =3D vkms_create(config); if (ret) - vkms_config_destroy(config); + goto err_kfree; =20 + ret =3D vkms_configfs_register(); + if (ret) + goto err_destroy; + + return 0; + +err_destroy: + vkms_destroy(config); +err_kfree: + vkms_config_destroy(config); return ret; } =20 -static void vkms_destroy(struct vkms_config *config) +void vkms_destroy(struct vkms_config *config) { struct platform_device *pdev; =20 @@ -257,6 +268,8 @@ static void vkms_destroy(struct vkms_config *config) =20 static void __exit vkms_exit(void) { + vkms_configfs_unregister(); + if (default_config->dev) vkms_destroy(default_config); =20 diff --git a/drivers/gpu/drm/vkms/vkms_drv.h b/drivers/gpu/drm/vkms/vkms_dr= v.h index 76394285dc68..eb32440e9f94 100644 --- a/drivers/gpu/drm/vkms/vkms_drv.h +++ b/drivers/gpu/drm/vkms/vkms_drv.h @@ -143,6 +143,10 @@ struct vkms_device { #define to_vkms_plane_state(target)\ container_of(target, struct vkms_plane_state, base.base) =20 +/* VKMS device */ +int vkms_create(struct vkms_config *config); +void vkms_destroy(struct vkms_config *config); + /* CRTC */ struct vkms_crtc *vkms_crtc_init(struct drm_device *dev, struct drm_plane *primary, --=20 2.46.0