scsi: ufs: call single_release to avoid memory leak
Bug: 110044919
Test: Build
Change-Id: I9fa83b84cf52a54c24405f2416f8747a20901d3f
Signed-off-by: Wei Wang <wvw@google.com>
diff --git a/drivers/scsi/ufs/ufs-debugfs.c b/drivers/scsi/ufs/ufs-debugfs.c
index 4130f24..59f5779 100644
--- a/drivers/scsi/ufs/ufs-debugfs.c
+++ b/drivers/scsi/ufs/ufs-debugfs.c
@@ -366,6 +366,7 @@
.open = ufsdbg_err_inj_scenario_open,
.read = seq_read,
.write = ufsdbg_err_inj_scenario_write,
+ .release = single_release,
};
static int ufsdbg_err_inj_stats_read(struct seq_file *file, void *data)
@@ -407,6 +408,7 @@
.open = ufsdbg_err_inj_stats_open,
.read = seq_read,
.write = ufsdbg_err_inj_stats_write,
+ .release = single_release,
};
static void ufsdbg_setup_fault_injection(struct ufs_hba *hba)
@@ -593,6 +595,7 @@
.open = ufsdbg_tag_stats_open,
.read = seq_read,
.write = ufsdbg_tag_stats_write,
+ .release = single_release,
};
static int ufsdbg_query_stats_show(struct seq_file *file, void *data)
@@ -664,6 +667,7 @@
.open = ufsdbg_query_stats_open,
.read = seq_read,
.write = ufsdbg_query_stats_write,
+ .release = single_release,
};
static int ufsdbg_err_stats_show(struct seq_file *file, void *data)
@@ -768,6 +772,7 @@
.open = ufsdbg_err_stats_open,
.read = seq_read,
.write = ufsdbg_err_stats_write,
+ .release = single_release,
};
static int ufshcd_init_statistics(struct ufs_hba *hba)
@@ -847,6 +852,7 @@
static const struct file_operations ufsdbg_host_regs_fops = {
.open = ufsdbg_host_regs_open,
.read = seq_read,
+ .release = single_release,
};
static int ufsdbg_health_desc_show(struct seq_file *file, void *data)
@@ -1015,6 +1021,7 @@
static const struct file_operations ufsdbg_show_hba_fops = {
.open = ufsdbg_show_hba_open,
.read = seq_read,
+ .release = single_release,
};
static int ufsdbg_dump_device_desc_open(struct inode *inode, struct file *file)
@@ -1026,6 +1033,7 @@
static const struct file_operations ufsdbg_dump_device_desc = {
.open = ufsdbg_dump_device_desc_open,
.read = seq_read,
+ .release = single_release,
};
static int ufsdbg_power_mode_show(struct seq_file *file, void *data)
@@ -1264,6 +1272,7 @@
.open = ufsdbg_power_mode_open,
.read = seq_read,
.write = ufsdbg_power_mode_write,
+ .release = single_release,
};
static int ufsdbg_dme_read(void *data, u64 *attr_val, bool peer)
@@ -1444,6 +1453,7 @@
.open = ufsdbg_req_stats_open,
.read = seq_read,
.write = ufsdbg_req_stats_write,
+ .release = single_release,
};
static ssize_t ufsdbg_io_stats_write(struct file *filp,
@@ -1512,6 +1522,7 @@
.open = ufsdbg_io_stats_open,
.read = seq_read,
.write = ufsdbg_io_stats_write,
+ .release = single_release,
};
static int ufsdbg_reset_controller_show(struct seq_file *file, void *data)
@@ -1559,6 +1570,7 @@
.open = ufsdbg_reset_controller_open,
.read = seq_read,
.write = ufsdbg_reset_controller_write,
+ .release = single_release,
};
static int ufsdbg_clear_err_state(void *data, u64 val)
diff --git a/drivers/scsi/ufs/ufs-qcom-debugfs.c b/drivers/scsi/ufs/ufs-qcom-debugfs.c
index 4547a6d..a7cbce4 100644
--- a/drivers/scsi/ufs/ufs-qcom-debugfs.c
+++ b/drivers/scsi/ufs/ufs-qcom-debugfs.c
@@ -174,6 +174,7 @@
.open = ufs_qcom_dbg_testbus_cfg_open,
.read = seq_read,
.write = ufs_qcom_dbg_testbus_cfg_write,
+ .release = single_release,
};
static int ufs_qcom_dbg_testbus_bus_read(void *data, u64 *attr_val)
@@ -228,6 +229,7 @@
static const struct file_operations ufs_qcom_dbg_dbg_regs_desc = {
.open = ufs_qcom_dbg_dbg_regs_open,
.read = seq_read,
+ .release = single_release,
};
static int ufs_qcom_dbg_pm_qos_show(struct seq_file *file, void *data)
@@ -261,6 +263,7 @@
static const struct file_operations ufs_qcom_dbg_pm_qos_desc = {
.open = ufs_qcom_dbg_pm_qos_open,
.read = seq_read,
+ .release = single_release,
};
void ufs_qcom_dbg_add_debugfs(struct ufs_hba *hba, struct dentry *root)
diff --git a/drivers/scsi/ufs/ufs_test.c b/drivers/scsi/ufs/ufs_test.c
index e23dc3e8..0f3a490 100644
--- a/drivers/scsi/ufs/ufs_test.c
+++ b/drivers/scsi/ufs/ufs_test.c
@@ -78,6 +78,7 @@
.open = ufs_test_ ## test_name ## _open, \
.read = seq_read, \
.write = ufs_test_ ## test_name ## _write, \
+ .release = single_release, \
};
#define add_test(utd, test_name, upper_case_name) \
