On Tue 25-05-10 16:49:09, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <Artem.Bityutskiy@xxxxxxxxx>
>
> ... use new VFS helpers instead.
Looks. OK. You can add
Acked-by: Jan Kara <jack@xxxxxxx>
Honza
> Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@xxxxxxxxx>
> Cc: Jan Kara <jack@xxxxxxx>
> ---
> fs/udf/super.c | 6 +++---
> fs/udf/udfdecl.h | 2 +-
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/fs/udf/super.c b/fs/udf/super.c
> index 1e4543c..998c911 100644
> --- a/fs/udf/super.c
> +++ b/fs/udf/super.c
> @@ -1940,7 +1940,7 @@ static int udf_fill_super(struct super_block *sb, void *options, int silent)
> sb->s_op = &udf_sb_ops;
> sb->s_export_op = &udf_export_ops;
> sb->dq_op = NULL;
> - sb->s_dirt = 0;
> + mark_sb_clean(sb);
> sb->s_magic = UDF_SUPER_MAGIC;
> sb->s_time_gran = 1000;
>
> @@ -2067,7 +2067,7 @@ static void udf_error(struct super_block *sb, const char *function,
>
> if (!(sb->s_flags & MS_RDONLY)) {
> /* mark sb error */
> - sb->s_dirt = 1;
> + mark_sb_dirty(sb);
> }
> va_start(args, fmt);
> vsnprintf(error_buf, sizeof(error_buf), fmt, args);
> @@ -2127,7 +2127,7 @@ static int udf_sync_fs(struct super_block *sb, int wait)
> * the buffer for IO
> */
> mark_buffer_dirty(sbi->s_lvid_bh);
> - sb->s_dirt = 0;
> + mark_sb_clean(sb);
> sbi->s_lvid_dirty = 0;
> }
> mutex_unlock(&sbi->s_alloc_mutex);
> diff --git a/fs/udf/udfdecl.h b/fs/udf/udfdecl.h
> index 9079ff7..9337e3f 100644
> --- a/fs/udf/udfdecl.h
> +++ b/fs/udf/udfdecl.h
> @@ -120,7 +120,7 @@ static inline void udf_updated_lvid(struct super_block *sb)
> WARN_ON_ONCE(((struct logicalVolIntegrityDesc *)
> bh->b_data)->integrityType !=
> cpu_to_le32(LVID_INTEGRITY_TYPE_OPEN));
> - sb->s_dirt = 1;
> + mark_sb_dirty(sb);
> UDF_SB(sb)->s_lvid_dirty = 1;
> }
>
> --
> 1.6.6.1
>
--
Jan Kara <jack@xxxxxxx>
SUSE Labs, CR
--
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
