drm/amdkfd: Do not fail process debugfs setup on debugfs errors

debugfs is intended for debugging only, and failures to create debugfs
entries should not affect normal operation.

Remove the check for debugfs_create_dir() in kfd_debugfs_add_process().
If debugfs entries cannot be created, continue without them instead of
reporting an unnecessary error.

Fixes: 325de6bf3328 ("drm/amdkfd: expose pasid of secondary contexts by debugfs")
Reported-by: Dan Carpenter <error27@gmail.com>
Cc: Zhu Lingshan <lingshan.zhu@amd.com>
Cc: Felix Kuehling <felix.kuehling@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c b/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c
index 464836f2a53f9e0dd334fc1452d1936829f9536f..33e905d59dd958fe028e1033196a40a1c932b456 100644 (file)
--- a/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c
@@ -211,10 +211,6 @@ int kfd_debugfs_add_process(struct kfd_process *p)
                entry->proc_dentry = debugfs_create_dir(name,
                                                        primary_entry->proc_dentry);
        }
-       if (IS_ERR_OR_NULL(entry->proc_dentry)) {
-               ret = entry->proc_dentry ? PTR_ERR(entry->proc_dentry) : -ENOMEM;
-               goto err_free_entry;
-       }
 
        list_add(&entry->list, &procs);
        kfd_debugfs_create_pasid_files(p, entry->proc_dentry);