]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
s390/hypfs_diag: Remove unused dentry variable
authorMete Durlu <meted@linux.ibm.com>
Mon, 26 Aug 2024 12:22:48 +0000 (14:22 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Thu, 29 Aug 2024 20:56:33 +0000 (22:56 +0200)
Remove leftover dentry variable after hypfs refactoring.

Before 2fcb3686e160, hypfs_diag.c and other hypfs files were using
debugfs_create_file() explicitly for creating debugfs files and
were storing the returned pointer.

After the refactor, common debugfs file operations and also the
related dentry pointers have been moved into hypfs_dbfs.c and
redefined as new common mechanisms.
Therefore the dentry variable and the debugfs_remove() function
calls in hypfs_diag.c are now redundant.

Current code is not effected since the dentry pointer in
hypfs_diag is implicitly assigned to NULL and debugfs_remove()
returns without an error if the passed pointer is NULL.

Acked-by: Alexander Gordeev <agordeev@linux.ibm.com>
Signed-off-by: Mete Durlu <meted@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
arch/s390/hypfs/hypfs_diag.c

index 26a009f9c49e92de1d15669b40b0101a75a924d5..c8af67d2099417134cf2aabfd230bfd4e53ca40e 100644 (file)
@@ -29,8 +29,6 @@ static enum diag204_format diag204_info_type; /* used diag 204 data format */
 static void *diag204_buf;              /* 4K aligned buffer for diag204 data */
 static int diag204_buf_pages;          /* number of pages for diag204 data */
 
-static struct dentry *dbfs_d204_file;
-
 enum diag204_format diag204_get_info_type(void)
 {
        return diag204_info_type;
@@ -214,16 +212,13 @@ __init int hypfs_diag_init(void)
                hypfs_dbfs_create_file(&dbfs_file_d204);
 
        rc = hypfs_diag_fs_init();
-       if (rc) {
+       if (rc)
                pr_err("The hardware system does not provide all functions required by hypfs\n");
-               debugfs_remove(dbfs_d204_file);
-       }
        return rc;
 }
 
 void hypfs_diag_exit(void)
 {
-       debugfs_remove(dbfs_d204_file);
        hypfs_diag_fs_exit();
        diag204_free_buffer();
        hypfs_dbfs_remove_file(&dbfs_file_d204);