diff options
author | Eric Sandeen <sandeen@redhat.com> | 2010-02-15 14:19:27 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2010-02-15 14:19:27 -0500 |
commit | 12062dddda450976b129dcb1bacd91acaf4d8030 (patch) | |
tree | e64590b1147639cd3629f8a977b269410cd6bd13 /fs/ext4/super.c | |
parent | f710b4b96ba292dfed2153afc47e9063b0abfd89 (diff) |
ext4: move __func__ into a macro for ext4_warning, ext4_error
Just a pet peeve of mine; we had a mishash of calls with either __func__
or "function_name" and the latter tends to get out of sync.
I think it's easier to just hide the __func__ in a macro, and it'll
be consistent from then on.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/super.c')
-rw-r--r-- | fs/ext4/super.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 68a55dffb360..1c85bb67e6eb 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c | |||
@@ -333,7 +333,7 @@ static void ext4_handle_error(struct super_block *sb) | |||
333 | sb->s_id); | 333 | sb->s_id); |
334 | } | 334 | } |
335 | 335 | ||
336 | void ext4_error(struct super_block *sb, const char *function, | 336 | void __ext4_error(struct super_block *sb, const char *function, |
337 | const char *fmt, ...) | 337 | const char *fmt, ...) |
338 | { | 338 | { |
339 | va_list args; | 339 | va_list args; |
@@ -450,7 +450,7 @@ void ext4_msg (struct super_block * sb, const char *prefix, | |||
450 | va_end(args); | 450 | va_end(args); |
451 | } | 451 | } |
452 | 452 | ||
453 | void ext4_warning(struct super_block *sb, const char *function, | 453 | void __ext4_warning(struct super_block *sb, const char *function, |
454 | const char *fmt, ...) | 454 | const char *fmt, ...) |
455 | { | 455 | { |
456 | va_list args; | 456 | va_list args; |
@@ -507,7 +507,7 @@ void ext4_update_dynamic_rev(struct super_block *sb) | |||
507 | if (le32_to_cpu(es->s_rev_level) > EXT4_GOOD_OLD_REV) | 507 | if (le32_to_cpu(es->s_rev_level) > EXT4_GOOD_OLD_REV) |
508 | return; | 508 | return; |
509 | 509 | ||
510 | ext4_warning(sb, __func__, | 510 | ext4_warning(sb, |
511 | "updating to rev %d because of new feature flag, " | 511 | "updating to rev %d because of new feature flag, " |
512 | "running e2fsck is recommended", | 512 | "running e2fsck is recommended", |
513 | EXT4_DYNAMIC_REV); | 513 | EXT4_DYNAMIC_REV); |
@@ -3367,10 +3367,9 @@ static void ext4_clear_journal_err(struct super_block *sb, | |||
3367 | char nbuf[16]; | 3367 | char nbuf[16]; |
3368 | 3368 | ||
3369 | errstr = ext4_decode_error(sb, j_errno, nbuf); | 3369 | errstr = ext4_decode_error(sb, j_errno, nbuf); |
3370 | ext4_warning(sb, __func__, "Filesystem error recorded " | 3370 | ext4_warning(sb, "Filesystem error recorded " |
3371 | "from previous mount: %s", errstr); | 3371 | "from previous mount: %s", errstr); |
3372 | ext4_warning(sb, __func__, "Marking fs in need of " | 3372 | ext4_warning(sb, "Marking fs in need of filesystem check."); |
3373 | "filesystem check."); | ||
3374 | 3373 | ||
3375 | EXT4_SB(sb)->s_mount_state |= EXT4_ERROR_FS; | 3374 | EXT4_SB(sb)->s_mount_state |= EXT4_ERROR_FS; |
3376 | es->s_state |= cpu_to_le16(EXT4_ERROR_FS); | 3375 | es->s_state |= cpu_to_le16(EXT4_ERROR_FS); |