diff options
| author | Joe Perches <joe@perches.com> | 2013-06-04 19:39:15 -0400 |
|---|---|---|
| committer | Dave Kleikamp <dave.kleikamp@oracle.com> | 2013-06-05 15:47:19 -0400 |
| commit | eb8630d7d2fd13589e6a7a3ae2fe1f75f867fbed (patch) | |
| tree | 16a5040af9b8df061385de9e0ab8e13e51790ddf /fs/jfs/jfs_superblock.h | |
| parent | 21d1101f013c12d1dd78cfdf263d619c80975b47 (diff) | |
jfs: Update jfs_error
Use a more current logging style.
Add __printf format and argument verification.
Remove embedded function names from formats.
Add %pf, __builtin_return_address(0) to jfs_error.
Add newlines to formats for kernel style consistency.
(One format already had an erroneous newline)
Coalesce formats and align arguments.
Object size reduced ~1KiB.
$ size fs/jfs/built-in.o*
text data bss dec hex filename
201891 35488 63936 301315 49903 fs/jfs/built-in.o.new
202821 35488 64192 302501 49da5 fs/jfs/built-in.o.old
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
Diffstat (limited to 'fs/jfs/jfs_superblock.h')
| -rw-r--r-- | fs/jfs/jfs_superblock.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/jfs/jfs_superblock.h b/fs/jfs/jfs_superblock.h index 884fc21ab8ee..04847b8d3070 100644 --- a/fs/jfs/jfs_superblock.h +++ b/fs/jfs/jfs_superblock.h | |||
| @@ -108,6 +108,7 @@ struct jfs_superblock { | |||
| 108 | 108 | ||
| 109 | extern int readSuper(struct super_block *, struct buffer_head **); | 109 | extern int readSuper(struct super_block *, struct buffer_head **); |
| 110 | extern int updateSuper(struct super_block *, uint); | 110 | extern int updateSuper(struct super_block *, uint); |
| 111 | __printf(2, 3) | ||
| 111 | extern void jfs_error(struct super_block *, const char *, ...); | 112 | extern void jfs_error(struct super_block *, const char *, ...); |
| 112 | extern int jfs_mount(struct super_block *); | 113 | extern int jfs_mount(struct super_block *); |
| 113 | extern int jfs_mount_rw(struct super_block *, int); | 114 | extern int jfs_mount_rw(struct super_block *, int); |