Commit 958f4425 authored by Dan Moulding's avatar Dan Moulding Committed by Christian König

drm: ttm: Don't bail from ttm_global_init if debugfs_create_dir fails

In 69de4421 ("drm/ttm: Initialize debugfs from
ttm_global_init()"), ttm_global_init was changed so that if creation
of the debugfs global root directory fails, ttm_global_init will bail
out early and return an error, leading to initialization failure of
DRM drivers. However, not every system will be using debugfs. On such
a system, debugfs directory creation can be expected to fail, but DRM
drivers must still be usable. This changes it so that if creation of
TTM's debugfs root directory fails, then no biggie: keep calm and
carry on.

Fixes: 69de4421 ("drm/ttm: Initialize debugfs from ttm_global_init()")
Signed-off-by: default avatarDan Moulding <dmoulding@me.com>
Tested-by: default avatarHuacai Chen <chenhuacai@loongson.cn>
Reviewed-by: default avatarHuang Rui <ray.huang@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210810195906.22220-2-dmoulding@me.comSigned-off-by: default avatarChristian König <christian.koenig@amd.com>
parent bf33677a
...@@ -78,9 +78,7 @@ static int ttm_global_init(void) ...@@ -78,9 +78,7 @@ static int ttm_global_init(void)
ttm_debugfs_root = debugfs_create_dir("ttm", NULL); ttm_debugfs_root = debugfs_create_dir("ttm", NULL);
if (IS_ERR(ttm_debugfs_root)) { if (IS_ERR(ttm_debugfs_root)) {
ret = PTR_ERR(ttm_debugfs_root);
ttm_debugfs_root = NULL; ttm_debugfs_root = NULL;
goto out;
} }
/* Limit the number of pages in the pool to about 50% of the total /* Limit the number of pages in the pool to about 50% of the total
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment