[PATCH v7 00/16] drm/vkms: Add configfs support

José Expósito posted 16 patches 1 month, 4 weeks ago
Documentation/gpu/vkms.rst                    | 100 ++-
drivers/gpu/drm/vkms/Kconfig                  |   1 +
drivers/gpu/drm/vkms/Makefile                 |   3 +-
drivers/gpu/drm/vkms/tests/vkms_config_test.c |  24 +
drivers/gpu/drm/vkms/vkms_config.c            |   8 +-
drivers/gpu/drm/vkms/vkms_config.h            |  26 +
drivers/gpu/drm/vkms/vkms_configfs.c          | 833 ++++++++++++++++++
drivers/gpu/drm/vkms/vkms_configfs.h          |   8 +
drivers/gpu/drm/vkms/vkms_connector.c         |  35 +
drivers/gpu/drm/vkms/vkms_connector.h         |   9 +
drivers/gpu/drm/vkms/vkms_drv.c               |  18 +-
drivers/gpu/drm/vkms/vkms_drv.h               |  20 +
12 files changed, 1072 insertions(+), 13 deletions(-)
create mode 100644 drivers/gpu/drm/vkms/vkms_configfs.c
create mode 100644 drivers/gpu/drm/vkms/vkms_configfs.h
[PATCH v7 00/16] drm/vkms: Add configfs support
Posted by José Expósito 1 month, 4 weeks ago
Hi everyone,

This series allow to configure one or more VKMS instances without having
to reload the driver using configfs.

The process of configuring a VKMS device is documented in "vkms.rst".

In addition, I created a CLI tool to easily control VKMS instances from the
command line: vkmsctl [1].

The series is structured in 3 blocks:

  - Patches 1..11: Basic device configuration. For simplicity, I kept the
    available options as minimal as possible.

  - Patches 12 and 13: New option to skip the default device creation and to-do
    cleanup.

  - Patches 14, 15 and 16: Allow to hot-plug and unplug connectors. This is not
    part of the minimal set of options, but I included in this series so it can
    be used as a template/example of how new configurations can be added.

Finally, the code is thoroughly tested by a collection of IGT tests [2].

Best wishes,
José Expósito

[1] https://github.com/JoseExposito/vkmsctl
[2] https://lore.kernel.org/igt-dev/20250807074550.6543-1-jose.exposito89@gmail.com/

Changes in v7:

  - Rebased on top of drm-misc-next
  - Fixed 2 documentation issues (thanks Harry!)
  - Added reviewed-by Harry and Luca, thanks a lot!
  - Link to v6: https://lore.kernel.org/dri-devel/20250901122541.9983-1-jose.exposito89@gmail.com/

Changes in v6:

  - No code changes, rebased on top of drm-misc-next
  - Added Tested-by: Mark Yacoub, who merged the series into the Android tree (thanks!)
    https://lore.kernel.org/dri-devel/CAC0gqY6ZH8h5aoNh31ck3dP6c3YYtfTRjJ47Obu6xSXSVXm5mA@mail.gmail.com/
  - Added a link in the cover letter the CLI to configure VKMS: vkmsctl
  - CCed more people to try to get the series merged
  - Link to v5: https://lore.kernel.org/dri-devel/20250507135431.53907-1-jose.exposito89@gmail.com/

Changes in v5:

  - No code changes, rebased on top of drm-misc-next
  - Added Reviewed-by tags, thanks Louis!
  - Link to v4: https://lore.kernel.org/dri-devel/20250407081425.6420-1-jose.exposito89@gmail.com/

Changes in v4:

  - No code changes, rebased on top of drm-misc-next
  - Since Louis and I worked on this together, set him as the author of some of
    the patches and me as co-developed-by to reflect this joint effort.
  - Link to v3: https://lore.kernel.org/all/20250307163353.5896-1-jose.exposito89@gmail.com/

Changes in v3:

  - Applied review comments by Louis Chauvet: (thanks!!)
    - Use scoped_guard() instead of guard(mutex)(...)
    - Fix a use-after-free error in the connector hot-plug code
  - Rebased on top of drm-misc-next
  - Link to v2: https://lore.kernel.org/all/20250225175936.7223-1-jose.exposito89@gmail.com/

Changes in v2:

  - Applied review comments by Louis Chauvet:
    - Use guard(mutex)(...) instead of lock/unlock
    - Return -EBUSY when trying to modify a enabled device
    - Move the connector hot-plug related patches to the end
  - Rebased on top of drm-misc-next
  - Link to v1: https://lore.kernel.org/dri-devel/20250218170808.9507-1-jose.exposito89@gmail.com/T/

José Expósito (6):
  drm/vkms: Expose device creation and destruction
  drm/vkms: Allow to configure the default device creation
  drm/vkms: Remove completed task from the TODO list
  drm/vkms: Allow to configure connector status
  drm/vkms: Allow to update the connector status
  drm/vkms: Allow to configure connector status via configfs

Louis Chauvet (10):
  drm/vkms: Add and remove VKMS instances via configfs
  drm/vkms: Allow to configure multiple planes via configfs
  drm/vkms: Allow to configure the plane type via configfs
  drm/vkms: Allow to configure multiple CRTCs via configfs
  drm/vkms: Allow to configure CRTC writeback support via configfs
  drm/vkms: Allow to attach planes and CRTCs via configfs
  drm/vkms: Allow to configure multiple encoders via configfs
  drm/vkms: Allow to attach encoders and CRTCs via configfs
  drm/vkms: Allow to configure multiple connectors via configfs
  drm/vkms: Allow to attach connectors and encoders via configfs

 Documentation/gpu/vkms.rst                    | 100 ++-
 drivers/gpu/drm/vkms/Kconfig                  |   1 +
 drivers/gpu/drm/vkms/Makefile                 |   3 +-
 drivers/gpu/drm/vkms/tests/vkms_config_test.c |  24 +
 drivers/gpu/drm/vkms/vkms_config.c            |   8 +-
 drivers/gpu/drm/vkms/vkms_config.h            |  26 +
 drivers/gpu/drm/vkms/vkms_configfs.c          | 833 ++++++++++++++++++
 drivers/gpu/drm/vkms/vkms_configfs.h          |   8 +
 drivers/gpu/drm/vkms/vkms_connector.c         |  35 +
 drivers/gpu/drm/vkms/vkms_connector.h         |   9 +
 drivers/gpu/drm/vkms/vkms_drv.c               |  18 +-
 drivers/gpu/drm/vkms/vkms_drv.h               |  20 +
 12 files changed, 1072 insertions(+), 13 deletions(-)
 create mode 100644 drivers/gpu/drm/vkms/vkms_configfs.c
 create mode 100644 drivers/gpu/drm/vkms/vkms_configfs.h


base-commit: b291e4f1a4951204ce858cd01801291d34962a33
-- 
2.51.0

Re: [PATCH v7 00/16] drm/vkms: Add configfs support
Posted by Luca Ceresoli 1 month, 3 weeks ago
On Thu, 16 Oct 2025 19:56:02 +0200, José Expósito wrote:
> This series allow to configure one or more VKMS instances without having
> to reload the driver using configfs.
> 
> The process of configuring a VKMS device is documented in "vkms.rst".
> 
> In addition, I created a CLI tool to easily control VKMS instances from the
> command line: vkmsctl [1].
> 
> [...]

Applied, thanks!

[01/16] drm/vkms: Expose device creation and destruction
        commit: 7965d1c5354a868063f61eb101f892480ede892f
[02/16] drm/vkms: Add and remove VKMS instances via configfs
        commit: 13fc9b9745cc5dbf38c4d559114cf98f8179b95f
[03/16] drm/vkms: Allow to configure multiple planes via configfs
        commit: 2f1734ba271bb98d582b02f72d77d1c7710a8d7a
[04/16] drm/vkms: Allow to configure the plane type via configfs
        commit: 187bc30625f3e0ee8d0b3694592c4b8ff771c845
[05/16] drm/vkms: Allow to configure multiple CRTCs via configfs
        commit: 3e4d5b30d2b262c6db84773cafe9097f7ec61ff5
[06/16] drm/vkms: Allow to configure CRTC writeback support via configfs
        commit: ee5c2c7d4bb6998ff11778436acbdc3154ce74ef
[07/16] drm/vkms: Allow to attach planes and CRTCs via configfs
        commit: 95fa73787a7947a8fdfbb1ad310c3f11b26065d3
[08/16] drm/vkms: Allow to configure multiple encoders via configfs
        commit: 67d8cf92e13ec05e64745ae7b63545de5d8e867a
[09/16] drm/vkms: Allow to attach encoders and CRTCs via configfs
        commit: fad1138b2377aa094b1c80fad852eadbcf3a85c3
[10/16] drm/vkms: Allow to configure multiple connectors via configfs
        commit: 272acbca96a3c6f43414e10d433befe2bb906d7c
[11/16] drm/vkms: Allow to attach connectors and encoders via configfs
        commit: 64229b846a7e5b54cc076092475280888f74c92c
[12/16] drm/vkms: Allow to configure the default device creation
        commit: 8c29107a6119289b57d2a80a9db849c4bda66a74
[13/16] drm/vkms: Remove completed task from the TODO list
        commit: 085dadb3101d7c37d8296580c946f5f1a2153628
[14/16] drm/vkms: Allow to configure connector status
        commit: 6f00987f5ce39bfbc01af7230d3939149b006f74
[15/16] drm/vkms: Allow to update the connector status
        commit: 466f43885ac0b75efe8107d82f7c60d7daf64f54
[16/16] drm/vkms: Allow to configure connector status via configfs
        commit: f97180f094aed00bae9776f00ae61c7c020b4d79

Best regards,
-- 
Luca Ceresoli <luca.ceresoli@bootlin.com>