diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2013-09-27 07:49:33 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2013-09-27 07:49:33 -0400 |
commit | af5c269799feaef110e59ce55b497cdd08712b0c (patch) | |
tree | c5aad271b31ba67c5eef857b927ed2dc45186176 /fs/gfs2/inode.c | |
parent | 5ca1db41ecdeb0358b968265fadb755213558a85 (diff) |
GFS2: Clean up reservation removal
The reservation for an inode should be cleared when it is truncated so
that we can start again at a different offset for future allocations.
We could try and do better than that, by resetting the search based on
where the truncation started from, but this is only a first step.
In addition, there are three callers of gfs2_rs_delete() but only one
of those should really be testing the value of i_writecount. While
we get away with that in the other cases currently, I think it would
be better if we made that test specific to the one case which
requires it.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/inode.c')
-rw-r--r-- | fs/gfs2/inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index cd58611912f5..4b79c19100d2 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c | |||
@@ -711,7 +711,7 @@ fail_gunlock2: | |||
711 | fail_free_inode: | 711 | fail_free_inode: |
712 | if (ip->i_gl) | 712 | if (ip->i_gl) |
713 | gfs2_glock_put(ip->i_gl); | 713 | gfs2_glock_put(ip->i_gl); |
714 | gfs2_rs_delete(ip); | 714 | gfs2_rs_delete(ip, NULL); |
715 | free_inode_nonrcu(inode); | 715 | free_inode_nonrcu(inode); |
716 | inode = NULL; | 716 | inode = NULL; |
717 | fail_gunlock: | 717 | fail_gunlock: |