exfat: rename exfat_free_dentry_set() to exfat_put_dentry_set()
authorYuezhang Mo <Yuezhang.Mo@sony.com>
Thu, 17 Mar 2022 10:12:40 +0000 (18:12 +0800)
committerNamjae Jeon <linkinjeon@kernel.org>
Mon, 12 Dec 2022 02:02:49 +0000 (11:02 +0900)
Since struct exfat_entry_set_cache is allocated from stack,
no need to free, so rename exfat_free_dentry_set() to
exfat_put_dentry_set(). After renaming, the new function pair
is exfat_get_dentry_set()/exfat_put_dentry_set().

Signed-off-by: Yuezhang Mo <Yuezhang.Mo@sony.com>
Reviewed-by: Andy Wu <Andy.Wu@sony.com>
Reviewed-by: Aoyama Wataru <wataru.aoyama@sony.com>
Reviewed-by: Sungjong Seo <sj1557.seo@samsung.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
fs/exfat/dir.c
fs/exfat/exfat_fs.h
fs/exfat/inode.c
fs/exfat/namei.c

index a3fb609..a9a0b3e 100644 (file)
@@ -55,7 +55,7 @@ static void exfat_get_uniname_from_ext_entry(struct super_block *sb,
                uniname += EXFAT_FILE_NAME_LEN;
        }
 
-       exfat_free_dentry_set(&es, false);
+       exfat_put_dentry_set(&es, false);
 }
 
 /* read a directory entry from the opened directory */
@@ -602,7 +602,7 @@ void exfat_update_dir_chksum_with_entry_set(struct exfat_entry_set_cache *es)
        es->modified = true;
 }
 
-int exfat_free_dentry_set(struct exfat_entry_set_cache *es, int sync)
+int exfat_put_dentry_set(struct exfat_entry_set_cache *es, int sync)
 {
        int i, err = 0;
 
@@ -860,7 +860,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
 
        ep = exfat_get_dentry_cached(es, 0);
        if (!exfat_validate_entry(exfat_get_entry_type(ep), &mode))
-               goto free_es;
+               goto put_es;
 
        num_entries = type == ES_ALL_ENTRIES ?
                ep->dentry.file.num_ext + 1 : type;
@@ -882,7 +882,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
                        if (p_dir->flags == ALLOC_NO_FAT_CHAIN)
                                clu++;
                        else if (exfat_get_next_cluster(sb, &clu))
-                               goto free_es;
+                               goto put_es;
                        sec = exfat_cluster_to_sector(sbi, clu);
                } else {
                        sec++;
@@ -890,7 +890,7 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
 
                bh = sb_bread(sb, sec);
                if (!bh)
-                       goto free_es;
+                       goto put_es;
                es->bh[es->num_bh++] = bh;
        }
 
@@ -898,12 +898,12 @@ int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
        for (i = 1; i < num_entries; i++) {
                ep = exfat_get_dentry_cached(es, i);
                if (!exfat_validate_entry(exfat_get_entry_type(ep), &mode))
-                       goto free_es;
+                       goto put_es;
        }
        return 0;
 
-free_es:
-       exfat_free_dentry_set(es, false);
+put_es:
+       exfat_put_dentry_set(es, false);
        return -EIO;
 }
 
index 2ffe579..324acc5 100644 (file)
@@ -493,7 +493,7 @@ struct exfat_dentry *exfat_get_dentry_cached(struct exfat_entry_set_cache *es,
 int exfat_get_dentry_set(struct exfat_entry_set_cache *es,
                struct super_block *sb, struct exfat_chain *p_dir, int entry,
                unsigned int type);
-int exfat_free_dentry_set(struct exfat_entry_set_cache *es, int sync);
+int exfat_put_dentry_set(struct exfat_entry_set_cache *es, int sync);
 int exfat_count_dir_entries(struct super_block *sb, struct exfat_chain *p_dir);
 
 /* inode.c */
index cdcf037..a84eae7 100644 (file)
@@ -83,7 +83,7 @@ int __exfat_write_inode(struct inode *inode, int sync)
        }
 
        exfat_update_dir_chksum_with_entry_set(&es);
-       return exfat_free_dentry_set(&es, sync);
+       return exfat_put_dentry_set(&es, sync);
 }
 
 int exfat_write_inode(struct inode *inode, struct writeback_control *wbc)
index 8d72527..57510d7 100644 (file)
@@ -676,7 +676,7 @@ static int exfat_find(struct inode *dir, struct qstr *qname,
                             ep->dentry.file.access_time,
                             ep->dentry.file.access_date,
                             0);
-       exfat_free_dentry_set(&es, false);
+       exfat_put_dentry_set(&es, false);
 
        if (ei->start_clu == EXFAT_FREE_CLUSTER) {
                exfat_fs_error(sb,