diff options
Diffstat (limited to 'meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2795-amdkfd-no-need-to-check-return-value-of-debugfs_crea.patch')
| -rw-r--r-- | meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2795-amdkfd-no-need-to-check-return-value-of-debugfs_crea.patch | 79 |
1 files changed, 79 insertions, 0 deletions
diff --git a/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2795-amdkfd-no-need-to-check-return-value-of-debugfs_crea.patch b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2795-amdkfd-no-need-to-check-return-value-of-debugfs_crea.patch new file mode 100644 index 00000000..c92e1f2b --- /dev/null +++ b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.19.8/2795-amdkfd-no-need-to-check-return-value-of-debugfs_crea.patch @@ -0,0 +1,79 @@ +From a18591ff06fe7a26d30c2653ad54572fa89aaff5 Mon Sep 17 00:00:00 2001 +From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +Date: Thu, 13 Jun 2019 15:19:45 +0200 +Subject: [PATCH 2795/2940] amdkfd: no need to check return value of + debugfs_create functions +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +When calling debugfs functions, there is no need to ever check the +return value. The function can work or not, but the code logic should +never do something different based on this. + +Cc: Oded Gabbay <oded.gabbay@gmail.com> +Cc: Alex Deucher <alexander.deucher@amd.com> +Cc: "Christian König" <christian.koenig@amd.com> +Cc: "David (ChunMing) Zhou" <David1.Zhou@amd.com> +Cc: David Airlie <airlied@linux.ie> +Cc: Daniel Vetter <daniel@ffwll.ch> +Cc: dri-devel@lists.freedesktop.org +Cc: amd-gfx@lists.freedesktop.org +Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> +Signed-off-by: Alex Deucher <alexander.deucher@amd.com> +--- + drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c | 36 ++++++------------------ + 1 file changed, 8 insertions(+), 28 deletions(-) + +diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c b/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c +index ab37d36d9cd6..15c523027285 100644 +--- a/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c ++++ b/drivers/gpu/drm/amd/amdkfd/kfd_debugfs.c +@@ -85,36 +85,16 @@ static const struct file_operations kfd_debugfs_hang_hws_fops = { + + void kfd_debugfs_init(void) + { +- struct dentry *ent; +- + debugfs_root = debugfs_create_dir("kfd", NULL); +- if (!debugfs_root || debugfs_root == ERR_PTR(-ENODEV)) { +- pr_warn("Failed to create kfd debugfs dir\n"); +- return; +- } +- +- ent = debugfs_create_file("mqds", S_IFREG | 0444, debugfs_root, +- kfd_debugfs_mqds_by_process, +- &kfd_debugfs_fops); +- if (!ent) +- pr_warn("Failed to create mqds in kfd debugfs\n"); +- +- ent = debugfs_create_file("hqds", S_IFREG | 0444, debugfs_root, +- kfd_debugfs_hqds_by_device, +- &kfd_debugfs_fops); +- if (!ent) +- pr_warn("Failed to create hqds in kfd debugfs\n"); +- +- ent = debugfs_create_file("rls", S_IFREG | 0444, debugfs_root, +- kfd_debugfs_rls_by_device, +- &kfd_debugfs_fops); +- +- ent = debugfs_create_file("hang_hws", S_IFREG | 0644, debugfs_root, +- NULL, +- &kfd_debugfs_hang_hws_fops); + +- if (!ent) +- pr_warn("Failed to create rls in kfd debugfs\n"); ++ debugfs_create_file("mqds", S_IFREG | 0444, debugfs_root, ++ kfd_debugfs_mqds_by_process, &kfd_debugfs_fops); ++ debugfs_create_file("hqds", S_IFREG | 0444, debugfs_root, ++ kfd_debugfs_hqds_by_device, &kfd_debugfs_fops); ++ debugfs_create_file("rls", S_IFREG | 0444, debugfs_root, ++ kfd_debugfs_rls_by_device, &kfd_debugfs_fops); ++ debugfs_create_file("hang_hws", S_IFREG | 0644, debugfs_root, ++ NULL, &kfd_debugfs_hang_hws_fops); + } + + void kfd_debugfs_fini(void) +-- +2.17.1 + |