diff options
author | Theodore Ts'o <tytso@mit.edu> | 2006-03-24 06:15:10 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-24 10:33:15 -0500 |
commit | 9b04c997b1120feefa1e6ee8e2902270bc055cd2 (patch) | |
tree | 09376b68d11ccce2a1ef53bac25a41a763ad36d0 /fs/super.c | |
parent | 6961ec8267d08e21011457b05d2263ec06bdcfe1 (diff) |
[PATCH] vfs: MS_VERBOSE should be MS_SILENT
The meaning of MS_VERBOSE is backwards; if the bit is set, it really means,
"don't be verbose". This is confusing and counter-intuitive.
In addition, there is also no way to set the MS_VERBOSE flag in the
mount(8) program in util-linux, but interesting, it does define options
which would do the right thing if MS_SILENT were defined, which
unfortunately we do not:
#ifdef MS_SILENT
{ "quiet", 0, 0, MS_SILENT }, /* be quiet */
{ "loud", 0, 1, MS_SILENT }, /* print out messages. */
#endif
So the obvious fix is to deprecate the use of MS_VERBOSE and replace it
with MS_SILENT.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/super.c')
-rw-r--r-- | fs/super.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/super.c b/fs/super.c index 425861cb1caa..37554b876182 100644 --- a/fs/super.c +++ b/fs/super.c | |||
@@ -712,7 +712,7 @@ struct super_block *get_sb_bdev(struct file_system_type *fs_type, | |||
712 | s->s_flags = flags; | 712 | s->s_flags = flags; |
713 | strlcpy(s->s_id, bdevname(bdev, b), sizeof(s->s_id)); | 713 | strlcpy(s->s_id, bdevname(bdev, b), sizeof(s->s_id)); |
714 | sb_set_blocksize(s, block_size(bdev)); | 714 | sb_set_blocksize(s, block_size(bdev)); |
715 | error = fill_super(s, data, flags & MS_VERBOSE ? 1 : 0); | 715 | error = fill_super(s, data, flags & MS_SILENT ? 1 : 0); |
716 | if (error) { | 716 | if (error) { |
717 | up_write(&s->s_umount); | 717 | up_write(&s->s_umount); |
718 | deactivate_super(s); | 718 | deactivate_super(s); |
@@ -756,7 +756,7 @@ struct super_block *get_sb_nodev(struct file_system_type *fs_type, | |||
756 | 756 | ||
757 | s->s_flags = flags; | 757 | s->s_flags = flags; |
758 | 758 | ||
759 | error = fill_super(s, data, flags & MS_VERBOSE ? 1 : 0); | 759 | error = fill_super(s, data, flags & MS_SILENT ? 1 : 0); |
760 | if (error) { | 760 | if (error) { |
761 | up_write(&s->s_umount); | 761 | up_write(&s->s_umount); |
762 | deactivate_super(s); | 762 | deactivate_super(s); |
@@ -785,7 +785,7 @@ struct super_block *get_sb_single(struct file_system_type *fs_type, | |||
785 | return s; | 785 | return s; |
786 | if (!s->s_root) { | 786 | if (!s->s_root) { |
787 | s->s_flags = flags; | 787 | s->s_flags = flags; |
788 | error = fill_super(s, data, flags & MS_VERBOSE ? 1 : 0); | 788 | error = fill_super(s, data, flags & MS_SILENT ? 1 : 0); |
789 | if (error) { | 789 | if (error) { |
790 | up_write(&s->s_umount); | 790 | up_write(&s->s_umount); |
791 | deactivate_super(s); | 791 | deactivate_super(s); |