diff options
| author | Steven Whitehouse <swhiteho@redhat.com> | 2012-04-19 10:38:50 +0100 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2012-04-24 16:44:38 +0100 |
| commit | 144a4c2ff75552e2aabb9256ca0ec5e277097153 (patch) | |
| tree | aa27e93cc135ba15eb3fe17e732ed74b1a7b8886 /fs/gfs2/log.c | |
| parent | 4306629e1c0fd098d52f499e5a60aaa03e30df0d (diff) | |
| download | op-kernel-dev-144a4c2ff75552e2aabb9256ca0ec5e277097153.zip op-kernel-dev-144a4c2ff75552e2aabb9256ca0ec5e277097153.tar.gz | |
GFS2: Log code fixes
This patch removes a log lock from around atomic operation where
it is not needed, removes an unused variable, and also changes
a void pointer used incorrectly to a struct page pointer.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/log.c')
| -rw-r--r-- | fs/gfs2/log.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/gfs2/log.c b/fs/gfs2/log.c index 2307218..db9cb18 100644 --- a/fs/gfs2/log.c +++ b/fs/gfs2/log.c @@ -643,10 +643,8 @@ void gfs2_log_flush(struct gfs2_sbd *sdp, struct gfs2_glock *gl) if (sdp->sd_log_head != sdp->sd_log_flush_head) { log_write_header(sdp, 0); } else if (sdp->sd_log_tail != current_tail(sdp) && !sdp->sd_log_idle){ - gfs2_log_lock(sdp); atomic_dec(&sdp->sd_log_blks_free); /* Adjust for unreserved buffer */ trace_gfs2_log_blocks(sdp, -1); - gfs2_log_unlock(sdp); log_write_header(sdp, 0); } lops_after_commit(sdp, ai); @@ -797,11 +795,9 @@ int gfs2_logd(void *data) struct gfs2_sbd *sdp = data; unsigned long t = 1; DEFINE_WAIT(wait); - unsigned preflush; while (!kthread_should_stop()) { - preflush = atomic_read(&sdp->sd_log_pinned); if (gfs2_jrnl_flush_reqd(sdp) || t == 0) { gfs2_ail1_empty(sdp); gfs2_log_flush(sdp, NULL); |
