diff options
author | Christoph Hellwig <hch@infradead.org> | 2008-10-30 02:55:08 -0400 |
---|---|---|
committer | Lachlan McIlroy <lachlan@sgi.com> | 2008-10-30 02:55:08 -0400 |
commit | 1ec7944beb6f3c29f1e58a66422130133727e9e1 (patch) | |
tree | e71281cbfe81e427a0a307db696778af89210472 /fs/xfs/linux-2.6/xfs_super.c | |
parent | 469fc23d5dd172665c69099db776c39bb1b70f92 (diff) |
[XFS] fix biosize option
iosizelog shouldn't be the same as iosize but the logarithm of it. Then
again the current biosize option doesn't make much sense to me as it
doesn't set the preferred I/O size as mentioned in the comment next to it
but rather the allocation size and thus is identical to the allocsize
option (except for the missing logarithm). It's also not documented in
Documentation/filesystems/xfs.txt or the mount manpage.
SGI-PV: 987246
SGI-Modid: xfs-linux-melb:xfs-kern:32373a
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Signed-off-by: Donald Douwsma <donaldd@sgi.com>
Signed-off-by: David Chinner <david@fromorbit.com>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Diffstat (limited to 'fs/xfs/linux-2.6/xfs_super.c')
-rw-r--r-- | fs/xfs/linux-2.6/xfs_super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/linux-2.6/xfs_super.c b/fs/xfs/linux-2.6/xfs_super.c index 62d485888496..c3d004bc4621 100644 --- a/fs/xfs/linux-2.6/xfs_super.c +++ b/fs/xfs/linux-2.6/xfs_super.c | |||
@@ -270,7 +270,7 @@ xfs_parseargs( | |||
270 | return EINVAL; | 270 | return EINVAL; |
271 | } | 271 | } |
272 | iosize = simple_strtoul(value, &eov, 10); | 272 | iosize = simple_strtoul(value, &eov, 10); |
273 | iosizelog = (uint8_t) iosize; | 273 | iosizelog = ffs(iosize) - 1; |
274 | } else if (!strcmp(this_char, MNTOPT_ALLOCSIZE)) { | 274 | } else if (!strcmp(this_char, MNTOPT_ALLOCSIZE)) { |
275 | if (!value || !*value) { | 275 | if (!value || !*value) { |
276 | cmn_err(CE_WARN, | 276 | cmn_err(CE_WARN, |