diff options
author | Mark Fasheh <mfasheh@suse.com> | 2010-04-05 21:17:13 -0400 |
---|---|---|
committer | Joel Becker <joel.becker@oracle.com> | 2010-05-05 21:18:06 -0400 |
commit | 73c8a80003d13be54e2309865030404441075182 (patch) | |
tree | 9b5a47939c22838133f46ea6d207254aa68d177b /fs/ocfs2/localalloc.h | |
parent | a57c8fd2ad238258cc983049008aea5f985804b2 (diff) |
ocfs2: clean up localalloc mount option size parsing
This patch pulls the local alloc sizing code into localalloc.c and provides
a callout to it from ocfs2_fill_super(). Behavior is essentially unchanged
except that I correctly calculate the maximum local alloc size. The old code
in ocfs2_parse_options() calculated the max size as:
ocfs2_local_alloc_size(sb) * 8
which is correct, in bits. Unfortunately though the option passed in is in
megabytes. Ultimately, this bug made no real difference - the shrink code
would catch a too-large size and bring it down to something reasonable.
Still, it's less than efficient as-is.
Signed-off-by: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Diffstat (limited to 'fs/ocfs2/localalloc.h')
-rw-r--r-- | fs/ocfs2/localalloc.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/ocfs2/localalloc.h b/fs/ocfs2/localalloc.h index ac5ea9f86653..04195c67f7c1 100644 --- a/fs/ocfs2/localalloc.h +++ b/fs/ocfs2/localalloc.h | |||
@@ -30,6 +30,8 @@ int ocfs2_load_local_alloc(struct ocfs2_super *osb); | |||
30 | 30 | ||
31 | void ocfs2_shutdown_local_alloc(struct ocfs2_super *osb); | 31 | void ocfs2_shutdown_local_alloc(struct ocfs2_super *osb); |
32 | 32 | ||
33 | void ocfs2_la_set_sizes(struct ocfs2_super *osb, int requested_mb); | ||
34 | |||
33 | int ocfs2_begin_local_alloc_recovery(struct ocfs2_super *osb, | 35 | int ocfs2_begin_local_alloc_recovery(struct ocfs2_super *osb, |
34 | int node_num, | 36 | int node_num, |
35 | struct ocfs2_dinode **alloc_copy); | 37 | struct ocfs2_dinode **alloc_copy); |