diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2006-05-05 16:59:11 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2006-05-05 16:59:11 -0400 |
commit | fd88de569b802c4a04aaa6ee74667775f4aed8c6 (patch) | |
tree | 1766c45303798bf289059afc8f117cf8bc784086 /fs/gfs2/inode.c | |
parent | 5bb76af1e089ac186c15c6aa792340d22b63d4b4 (diff) |
[GFS2] Readpages support
This adds readpages support (and also corrects a small bug in
the readpage error path at the same time). Hopefully this will
improve performance by allowing GFS to submit larger lumps of
I/O at a time.
In order to simplify the setting of BH_Boundary, it currently gets
set when we hit the end of a indirect pointer block. There is
always a boundary at this point with the current allocation code.
It doesn't get all the boundaries right though, so there is still
room for improvement in this.
See comments in fs/gfs2/ops_address.c for further information about
readpages with GFS2.
Signed-off-by: Steven Whitehouse
Diffstat (limited to 'fs/gfs2/inode.c')
-rw-r--r-- | fs/gfs2/inode.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c index 9084d6037a0c..0817f6ede188 100644 --- a/fs/gfs2/inode.c +++ b/fs/gfs2/inode.c | |||
@@ -1606,9 +1606,8 @@ int gfs2_glock_nq_atime(struct gfs2_holder *gh) | |||
1606 | curtime = get_seconds(); | 1606 | curtime = get_seconds(); |
1607 | if (curtime - ip->i_di.di_atime >= quantum) { | 1607 | if (curtime - ip->i_di.di_atime >= quantum) { |
1608 | gfs2_glock_dq(gh); | 1608 | gfs2_glock_dq(gh); |
1609 | gfs2_holder_reinit(LM_ST_EXCLUSIVE, | 1609 | gfs2_holder_reinit(LM_ST_EXCLUSIVE, gh->gh_flags & ~LM_FLAG_ANY, |
1610 | gh->gh_flags & ~LM_FLAG_ANY, | 1610 | gh); |
1611 | gh); | ||
1612 | error = gfs2_glock_nq(gh); | 1611 | error = gfs2_glock_nq(gh); |
1613 | if (error) | 1612 | if (error) |
1614 | return error; | 1613 | return error; |