udf: Drop udf_sync_inode()

The only place using udf_sync_inode() is now inode_getblk() for flushing
IS_SYNC inodes after write and page_mkwrite allocating blocks. For write
the flushing is actually taken care of by generic_write_sync() so it
isn't needed here. For page_mkwrite it does have effect however none of
the other filesystems seem to bother with flushing IS_SYNC inode on page
fault and properly synchronizing such writeback with standard inode
writeback would be slightly complex due to locking constraints. So just
drop IS_SYNC inode handling from inode_getblk().

Signed-off-by: Jan Kara <jack@suse.cz>
Link: https://patch.msgid.link/20260727104923.3828017-32-jack@suse.cz
Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>

diff --git a/fs/udf/inode.c b/fs/udf/inode.c
index 600705f5edf9eae0b5721c3afe2fce53276d6983..c751a02d865c342d59a4552c000a67d1973c4f47 100644 (file)
--- a/fs/udf/inode.c
+++ b/fs/udf/inode.c
@@ -52,7 +52,6 @@ struct udf_map_rq;
 
 static umode_t udf_convert_permissions(struct fileEntry *);
 static int udf_update_inode(struct inode *, int);
-static int udf_sync_inode(struct inode *inode);
 static int udf_alloc_i_data(struct inode *inode, size_t size);
 static int inode_getblk(struct inode *inode, struct udf_map_rq *map);
 static int udf_insert_aext(struct inode *, struct extent_position,
@@ -938,10 +937,7 @@ static int inode_getblk(struct inode *inode, struct udf_map_rq *map)
        iinfo->i_next_alloc_goal = newblocknum + 1;
        inode_set_ctime_current(inode);
 
-       if (IS_SYNC(inode))
-               udf_sync_inode(inode);
-       else
-               mark_inode_dirty(inode);
+       mark_inode_dirty(inode);
        ret = 0;
 out_free:
        brelse(prev_epos.bh);
@@ -1733,11 +1729,6 @@ out:
        return err;
 }
 
-static int udf_sync_inode(struct inode *inode)
-{
-       return udf_update_inode(inode, 1);
-}
-
 static void udf_adjust_time(struct udf_inode_info *iinfo, struct timespec64 time)
 {
        if (iinfo->i_crtime.tv_sec > time.tv_sec ||