From: Artem Bityutskiy <artem.bityutskiy@xxxxxxxxxxxxxxx> This is a preparation patch which introduces a 'fat_mark_fsinfo_dirty()' helper function which just sets the 's_dirt' flag to 1. I'll add more code to this helper in one of the next patches, and this is why I did not mark it as 'inline'. Signed-off-by: Artem Bityutskiy <artem.bityutskiy@xxxxxxxxxxxxxxx> --- fs/fat/fatent.c | 13 +++++++++---- 1 files changed, 9 insertions(+), 4 deletions(-) diff --git a/fs/fat/fatent.c b/fs/fat/fatent.c index 2e81ac0..153c053 100644 --- a/fs/fat/fatent.c +++ b/fs/fat/fatent.c @@ -308,6 +308,11 @@ void fat_ent_access_init(struct super_block *sb) } } +static void fat_mark_fsinfo_dirty(struct super_block *sb) +{ + sb->s_dirt = 1; +} + static inline int fat_ent_update_ptr(struct super_block *sb, struct fat_entry *fatent, int offset, sector_t blocknr) @@ -498,7 +503,7 @@ int fat_alloc_clusters(struct inode *inode, int *cluster, int nr_cluster) sbi->prev_free = entry; if (sbi->free_clusters != -1) sbi->free_clusters--; - sb->s_dirt = 1; + fat_mark_fsinfo_dirty(sb); cluster[idx_clus] = entry; idx_clus++; @@ -520,7 +525,7 @@ int fat_alloc_clusters(struct inode *inode, int *cluster, int nr_cluster) /* Couldn't allocate the free entries */ sbi->free_clusters = 0; sbi->free_clus_valid = 1; - sb->s_dirt = 1; + fat_mark_fsinfo_dirty(sb); err = -ENOSPC; out: @@ -587,7 +592,7 @@ int fat_free_clusters(struct inode *inode, int cluster) ops->ent_put(&fatent, FAT_ENT_FREE); if (sbi->free_clusters != -1) { sbi->free_clusters++; - sb->s_dirt = 1; + fat_mark_fsinfo_dirty(sb); } if (nr_bhs + fatent.nr_bhs > MAX_BUF_PER_PAGE) { @@ -677,7 +682,7 @@ int fat_count_free_clusters(struct super_block *sb) } sbi->free_clusters = free; sbi->free_clus_valid = 1; - sb->s_dirt = 1; + fat_mark_fsinfo_dirty(sb); fatent_brelse(&fatent); out: unlock_fat(sbi); -- 1.7.7.6 -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html