From 5372ad1ebff628b353e623bf9456102da6188952 Mon Sep 17 00:00:00 2001 From: Theodore Ts'o Date: Tue, 29 Jun 2010 11:07:07 -0400 Subject: [PATCH] --- yaml --- r: 206293 b: refs/heads/master c: c67d859e39896e4286249da89c4ca0ef8bd949cb h: refs/heads/master i: 206291: d8b667d93d54cfb6238ef904f8cf41a59e8e2b8f v: v3 --- [refs] | 2 +- trunk/fs/ext4/ext4.h | 4 +++- trunk/fs/ext4/ext4_jbd2.c | 4 ++-- trunk/fs/ext4/super.c | 11 +++++------ 4 files changed, 11 insertions(+), 10 deletions(-) diff --git a/[refs] b/[refs] index 823a3ba41a5f..5be646d62e85 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 4a9cdec73f79b2858e9ecf0b6cfac7f6b200bf3a +refs/heads/master: c67d859e39896e4286249da89c4ca0ef8bd949cb diff --git a/trunk/fs/ext4/ext4.h b/trunk/fs/ext4/ext4.h index 3864a2775458..d8ec824f5a25 100644 --- a/trunk/fs/ext4/ext4.h +++ b/trunk/fs/ext4/ext4.h @@ -1629,8 +1629,10 @@ extern void ext4_error_inode(const char *, struct inode *, const char *, ...) extern void ext4_error_file(const char *, struct file *, const char *, ...) __attribute__ ((format (printf, 3, 4))); extern void __ext4_std_error(struct super_block *, const char *, int); -extern void ext4_abort(struct super_block *, const char *, const char *, ...) +extern void __ext4_abort(struct super_block *, const char *, const char *, ...) __attribute__ ((format (printf, 3, 4))); +#define ext4_abort(sb, message...) __ext4_abort(sb, __func__, \ + ## message) extern void __ext4_warning(struct super_block *, const char *, const char *, ...) __attribute__ ((format (printf, 3, 4))); diff --git a/trunk/fs/ext4/ext4_jbd2.c b/trunk/fs/ext4/ext4_jbd2.c index cfd27b38fa15..df26a015d043 100644 --- a/trunk/fs/ext4/ext4_jbd2.c +++ b/trunk/fs/ext4/ext4_jbd2.c @@ -93,8 +93,8 @@ int __ext4_forget(const char *where, handle_t *handle, int is_metadata, err = jbd2_journal_revoke(handle, blocknr, bh); if (err) { ext4_journal_abort_handle(where, __func__, bh, handle, err); - ext4_abort(inode->i_sb, __func__, - "error %d when attempting revoke", err); + __ext4_abort(inode->i_sb, where, + "error %d when attempting revoke", err); } BUFFER_TRACE(bh, "exit"); return err; diff --git a/trunk/fs/ext4/super.c b/trunk/fs/ext4/super.c index 422a4ce66778..11441e483b62 100644 --- a/trunk/fs/ext4/super.c +++ b/trunk/fs/ext4/super.c @@ -248,7 +248,7 @@ handle_t *ext4_journal_start_sb(struct super_block *sb, int nblocks) journal = EXT4_SB(sb)->s_journal; if (journal) { if (is_journal_aborted(journal)) { - ext4_abort(sb, __func__, "Detected aborted journal"); + ext4_abort(sb, "Detected aborted journal"); return ERR_PTR(-EROFS); } return jbd2_journal_start(journal, nblocks); @@ -464,8 +464,8 @@ void __ext4_std_error(struct super_block *sb, const char *function, int errno) * case we take the easy way out and panic immediately. */ -void ext4_abort(struct super_block *sb, const char *function, - const char *fmt, ...) +void __ext4_abort(struct super_block *sb, const char *function, + const char *fmt, ...) { va_list args; @@ -660,8 +660,7 @@ static void ext4_put_super(struct super_block *sb) err = jbd2_journal_destroy(sbi->s_journal); sbi->s_journal = NULL; if (err < 0) - ext4_abort(sb, __func__, - "Couldn't clean up the journal"); + ext4_abort(sb, "Couldn't clean up the journal"); } ext4_release_system_zone(sb); @@ -3605,7 +3604,7 @@ static int ext4_remount(struct super_block *sb, int *flags, char *data) } if (sbi->s_mount_flags & EXT4_MF_FS_ABORTED) - ext4_abort(sb, __func__, "Abort forced by user"); + ext4_abort(sb, "Abort forced by user"); sb->s_flags = (sb->s_flags & ~MS_POSIXACL) | (test_opt(sb, POSIX_ACL) ? MS_POSIXACL : 0);