diff options
| author | Christian König <ckoenig.leichtzumerken@gmail.com> | 2023-08-29 13:01:14 +0200 |
|---|---|---|
| committer | Christian König <christian.koenig@amd.com> | 2023-09-01 08:53:05 +0200 |
| commit | ec9c7073bb082412a49466059053ace537c1a30d (patch) | |
| tree | 2a24e3269e0e16f52a562cd5fa180a36a59294ff /include | |
| parent | 0b30d57acafcaa5374756d314ee54f80d0bcc860 (diff) | |
drm/debugfs: remove dev->debugfs_list and debugfs_mutex v2
The mutex was completely pointless in the first place since any
parallel adding of files to this list would result in random
behavior since the list is filled and consumed multiple times.
Completely drop that approach and just create the files directly but
return -ENODEV while opening the file when the minors are not
registered yet.
v2: rebase on debugfs directory rework, limit access before minors are
registered.
Signed-off-by: Christian König <christian.koenig@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230829110115.3442-5-christian.koenig@amd.com
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/drm/drm_device.h | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/include/drm/drm_device.h b/include/drm/drm_device.h index 3cf12b14232d..c490977ee250 100644 --- a/include/drm/drm_device.h +++ b/include/drm/drm_device.h @@ -318,21 +318,6 @@ struct drm_device { */ struct dentry *debugfs_root; - /** - * @debugfs_mutex: - * - * Protects &debugfs_list access. - */ - struct mutex debugfs_mutex; - - /** - * @debugfs_list: - * - * List of debugfs files to be created by the DRM device. The files - * must be added during drm_dev_register(). - */ - struct list_head debugfs_list; - /* Everything below here is for legacy driver, never use! */ /* private: */ #if IS_ENABLED(CONFIG_DRM_LEGACY) |