Skip to contentSkip to conversation

ext4: Change all super.c messages to print the device

PATCH In review Eric Sandeen 3 participants 3 messages May 20, 2009 – Jun 4, 2009 ext4 file system

lore
Share & tools
lore

Eric Sandeen

author
#0 lore raw

PATCH 1/1 ext4: Change all super.c messages to print the device

Do the same for ext4 as Andi did for ext3:

"This patch changes ext4 super.c to include the device name with all warning/error messages, by using a new utility function ext4_msg. It's a rather large patch, but very mechanic. I left debug printks alone."

Cc Andi Kleen <[email protected]>
Signed-off-by Eric Sandeen <[email protected]>

---

fs/ext4/ext4.h +2 -0
Index: linux-2.6/fs/ext4/ext4.h
===================================================================
@@ -1123,6 +1123,8 @@ extern void ext4_abort(struct super_bloc
 	__attribute__ ((format (printf, 3, 4)));
 extern void ext4_warning(struct super_block *, const char *, const char *, ...)
 	__attribute__ ((format (printf, 3, 4)));
+extern void ext4_msg(struct super_block *, const char *, const char *, ...)
+	__attribute__ ((format (printf, 3, 4)));
 extern void ext4_grp_locked_error(struct super_block *, ext4_group_t,
 				const char *, const char *, ...)
 	__attribute__ ((format (printf, 4, 5)));
fs/ext4/super.c +221 -218

1,057 lines not on this page · raw message

maintainer
#1 lore raw
Quoted 6 lines · Eric Sandeen

Do the same for ext4 as Andi did for ext3:

"This patch changes ext4 super.c to include the device name with all warning/error messages, by using a new utility function ext4_msg. It's a rather large patch, but very mechanic. I left debug printks alone."

I've added this to the ext4 patch queue.

- Ted

#2 lore raw
Quoted 9 lines · Theodore Tso
Quoted 6 lines · Eric Sandeen

Do the same for ext4 as Andi did for ext3:

"This patch changes ext4 super.c to include the device name with all warning/error messages, by using a new utility function ext4_msg. It's a rather large patch, but very mechanic. I left debug printks alone."

I've added this to the ext4 patch queue.

And how about the ext3 patch?

-Andi

Signature · 1 line
[email protected] -- Speaking for myself only.