diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2007-11-08 09:25:12 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2008-01-25 03:07:52 -0500 |
commit | 2bcd610d2fdea608a8fdac32788fc35a32a2327c (patch) | |
tree | 5b3753ff18c1da54bb860dbd67211e6abea78ca7 /fs/gfs2/glock.c | |
parent | 8cbc4342478311c2a85260a7ca54d96cb7f71f7b (diff) |
[GFS2] Don't add glocks to the journal
The only reason for adding glocks to the journal was to keep track
of which locks required a log flush prior to release. We add a
flag to the glock to allow this check to be made in a simpler way.
This reduces the size of a glock (by 12 bytes on i386, 24 on x86_64)
and means that we can avoid extra work during the journal flush.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/glock.c')
-rw-r--r-- | fs/gfs2/glock.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 5fbd9d34ce23..d83df6888402 100644 --- a/fs/gfs2/glock.c +++ b/fs/gfs2/glock.c | |||
@@ -346,7 +346,6 @@ int gfs2_glock_get(struct gfs2_sbd *sdp, u64 number, | |||
346 | gl->gl_object = NULL; | 346 | gl->gl_object = NULL; |
347 | gl->gl_sbd = sdp; | 347 | gl->gl_sbd = sdp; |
348 | gl->gl_aspace = NULL; | 348 | gl->gl_aspace = NULL; |
349 | lops_init_le(&gl->gl_le, &gfs2_glock_lops); | ||
350 | INIT_DELAYED_WORK(&gl->gl_work, glock_work_func); | 349 | INIT_DELAYED_WORK(&gl->gl_work, glock_work_func); |
351 | 350 | ||
352 | /* If this glock protects actual on-disk data or metadata blocks, | 351 | /* If this glock protects actual on-disk data or metadata blocks, |
@@ -1900,8 +1899,6 @@ static int dump_glock(struct glock_iter *gi, struct gfs2_glock *gl) | |||
1900 | print_dbg(gi, " req_bh = %s\n", (gl->gl_req_bh) ? "yes" : "no"); | 1899 | print_dbg(gi, " req_bh = %s\n", (gl->gl_req_bh) ? "yes" : "no"); |
1901 | print_dbg(gi, " lvb_count = %d\n", atomic_read(&gl->gl_lvb_count)); | 1900 | print_dbg(gi, " lvb_count = %d\n", atomic_read(&gl->gl_lvb_count)); |
1902 | print_dbg(gi, " object = %s\n", (gl->gl_object) ? "yes" : "no"); | 1901 | print_dbg(gi, " object = %s\n", (gl->gl_object) ? "yes" : "no"); |
1903 | print_dbg(gi, " le = %s\n", | ||
1904 | (list_empty(&gl->gl_le.le_list)) ? "no" : "yes"); | ||
1905 | print_dbg(gi, " reclaim = %s\n", | 1902 | print_dbg(gi, " reclaim = %s\n", |
1906 | (list_empty(&gl->gl_reclaim)) ? "no" : "yes"); | 1903 | (list_empty(&gl->gl_reclaim)) ? "no" : "yes"); |
1907 | if (gl->gl_aspace) | 1904 | if (gl->gl_aspace) |