udf: Convert udf_mkdir() to new directory iteration code
authorJan Kara <jack@suse.cz>
Thu, 6 Oct 2022 12:22:33 +0000 (14:22 +0200)
committerJan Kara <jack@suse.cz>
Mon, 9 Jan 2023 09:39:52 +0000 (10:39 +0100)
Convert udf_mkdir() to new directory iteration code.

Signed-off-by: Jan Kara <jack@suse.cz>
fs/udf/namei.c

index 788ac6c..c012f2a 100644 (file)
@@ -928,8 +928,7 @@ static int udf_mkdir(struct user_namespace *mnt_userns, struct inode *dir,
                     struct dentry *dentry, umode_t mode)
 {
        struct inode *inode;
-       struct udf_fileident_bh fibh;
-       struct fileIdentDesc cfi, *fi;
+       struct udf_fileident_iter iter;
        int err;
        struct udf_inode_info *dinfo = UDF_I(dir);
        struct udf_inode_info *iinfo;
@@ -941,47 +940,42 @@ static int udf_mkdir(struct user_namespace *mnt_userns, struct inode *dir,
        iinfo = UDF_I(inode);
        inode->i_op = &udf_dir_inode_operations;
        inode->i_fop = &udf_dir_operations;
-       fi = udf_add_entry(inode, NULL, &fibh, &cfi, &err);
-       if (!fi) {
-               inode_dec_link_count(inode);
+       err = udf_fiiter_add_entry(inode, NULL, &iter);
+       if (err) {
+               clear_nlink(inode);
                discard_new_inode(inode);
-               goto out;
+               return err;
        }
        set_nlink(inode, 2);
-       cfi.icb.extLength = cpu_to_le32(inode->i_sb->s_blocksize);
-       cfi.icb.extLocation = cpu_to_lelb(dinfo->i_location);
-       *(__le32 *)((struct allocDescImpUse *)cfi.icb.impUse)->impUse =
+       iter.fi.icb.extLength = cpu_to_le32(inode->i_sb->s_blocksize);
+       iter.fi.icb.extLocation = cpu_to_lelb(dinfo->i_location);
+       *(__le32 *)((struct allocDescImpUse *)iter.fi.icb.impUse)->impUse =
                cpu_to_le32(dinfo->i_unique & 0x00000000FFFFFFFFUL);
-       cfi.fileCharacteristics =
+       iter.fi.fileCharacteristics =
                        FID_FILE_CHAR_DIRECTORY | FID_FILE_CHAR_PARENT;
-       udf_write_fi(inode, &cfi, fi, &fibh, NULL, NULL);
-       brelse(fibh.sbh);
+       udf_fiiter_write_fi(&iter, NULL);
+       udf_fiiter_release(&iter);
        mark_inode_dirty(inode);
 
-       fi = udf_add_entry(dir, dentry, &fibh, &cfi, &err);
-       if (!fi) {
+       err = udf_fiiter_add_entry(dir, dentry, &iter);
+       if (err) {
                clear_nlink(inode);
-               mark_inode_dirty(inode);
                discard_new_inode(inode);
-               goto out;
+               return err;
        }
-       cfi.icb.extLength = cpu_to_le32(inode->i_sb->s_blocksize);
-       cfi.icb.extLocation = cpu_to_lelb(iinfo->i_location);
-       *(__le32 *)((struct allocDescImpUse *)cfi.icb.impUse)->impUse =
+       iter.fi.icb.extLength = cpu_to_le32(inode->i_sb->s_blocksize);
+       iter.fi.icb.extLocation = cpu_to_lelb(iinfo->i_location);
+       *(__le32 *)((struct allocDescImpUse *)iter.fi.icb.impUse)->impUse =
                cpu_to_le32(iinfo->i_unique & 0x00000000FFFFFFFFUL);
-       cfi.fileCharacteristics |= FID_FILE_CHAR_DIRECTORY;
-       udf_write_fi(dir, &cfi, fi, &fibh, NULL, NULL);
+       iter.fi.fileCharacteristics |= FID_FILE_CHAR_DIRECTORY;
+       udf_fiiter_write_fi(&iter, NULL);
+       udf_fiiter_release(&iter);
        inc_nlink(dir);
        dir->i_ctime = dir->i_mtime = current_time(dir);
        mark_inode_dirty(dir);
        d_instantiate_new(dentry, inode);
-       if (fibh.sbh != fibh.ebh)
-               brelse(fibh.ebh);
-       brelse(fibh.sbh);
-       err = 0;
 
-out:
-       return err;
+       return 0;
 }
 
 static int empty_dir(struct inode *dir)