diff options
author | Joel Becker <joel.becker@oracle.com> | 2009-02-13 06:45:49 -0500 |
---|---|---|
committer | Joel Becker <joel.becker@oracle.com> | 2009-09-04 19:08:12 -0400 |
commit | a1cf076ba93f9fdf3eb4195f9f43d1e7cb7550f2 (patch) | |
tree | 27fbf5b5b5b87e3d427d9deee06ec882370d1d1a /fs | |
parent | f3868d0fa2e20d923087a8296fda47b0afe7f9ba (diff) |
ocfs2: __ocfs2_mark_extent_written() doesn't need struct inode.
We only allow unwritten extents on data, so the toplevel
ocfs2_mark_extent_written() can use an inode all it wants. But the
subfunction isn't even using the inode argument.
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ocfs2/alloc.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/fs/ocfs2/alloc.c b/fs/ocfs2/alloc.c index 7b0f2cd9f66f..4488685a7022 100644 --- a/fs/ocfs2/alloc.c +++ b/fs/ocfs2/alloc.c | |||
@@ -5026,9 +5026,8 @@ out: | |||
5026 | * have been brought into cache (and pinned via the journal), so the | 5026 | * have been brought into cache (and pinned via the journal), so the |
5027 | * extra overhead is not expressed in terms of disk reads. | 5027 | * extra overhead is not expressed in terms of disk reads. |
5028 | */ | 5028 | */ |
5029 | static int __ocfs2_mark_extent_written(struct inode *inode, | 5029 | static int __ocfs2_mark_extent_written(handle_t *handle, |
5030 | struct ocfs2_extent_tree *et, | 5030 | struct ocfs2_extent_tree *et, |
5031 | handle_t *handle, | ||
5032 | struct ocfs2_path *path, | 5031 | struct ocfs2_path *path, |
5033 | int split_index, | 5032 | int split_index, |
5034 | struct ocfs2_extent_rec *split_rec, | 5033 | struct ocfs2_extent_rec *split_rec, |
@@ -5062,7 +5061,7 @@ static int __ocfs2_mark_extent_written(struct inode *inode, | |||
5062 | 5061 | ||
5063 | /* | 5062 | /* |
5064 | * The core merge / split code wants to know how much room is | 5063 | * The core merge / split code wants to know how much room is |
5065 | * left in this inodes allocation tree, so we pass the | 5064 | * left in this allocation tree, so we pass the |
5066 | * rightmost extent list. | 5065 | * rightmost extent list. |
5067 | */ | 5066 | */ |
5068 | if (path->p_tree_depth) { | 5067 | if (path->p_tree_depth) { |
@@ -5185,7 +5184,7 @@ int ocfs2_mark_extent_written(struct inode *inode, | |||
5185 | split_rec.e_flags = path_leaf_el(left_path)->l_recs[index].e_flags; | 5184 | split_rec.e_flags = path_leaf_el(left_path)->l_recs[index].e_flags; |
5186 | split_rec.e_flags &= ~OCFS2_EXT_UNWRITTEN; | 5185 | split_rec.e_flags &= ~OCFS2_EXT_UNWRITTEN; |
5187 | 5186 | ||
5188 | ret = __ocfs2_mark_extent_written(inode, et, handle, left_path, | 5187 | ret = __ocfs2_mark_extent_written(handle, et, left_path, |
5189 | index, &split_rec, meta_ac, | 5188 | index, &split_rec, meta_ac, |
5190 | dealloc); | 5189 | dealloc); |
5191 | if (ret) | 5190 | if (ret) |