diff options
author | Steven Whitehouse <swhiteho@redhat.com> | 2006-09-04 14:59:35 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2006-09-04 14:59:35 -0400 |
commit | 0bd5996a00346fee772cbdebc5666fd4e514089b (patch) | |
tree | 1ddcbd49888cb4eec7aa3f96693193c0fd1979a5 /fs | |
parent | dd538c832aaf8e35c46c98a825fa9dacee3cf226 (diff) |
[GFS2] Style changes in ops_address.c
As per the remainder of Jan Engelhardt's fourth email comments,
remove an cast thats not required. Also tidy up the "limit" code
in stuck_releasepage().
Cc: Jan Engelhardt <jengelh@linux01.gwdg.de>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/gfs2/ops_address.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/gfs2/ops_address.c b/fs/gfs2/ops_address.c index 96988ccf2087..d44d42fb4163 100644 --- a/fs/gfs2/ops_address.c +++ b/fs/gfs2/ops_address.c | |||
@@ -517,7 +517,7 @@ static int gfs2_commit_write(struct file *file, struct page *page, | |||
517 | 517 | ||
518 | kaddr = kmap_atomic(page, KM_USER0); | 518 | kaddr = kmap_atomic(page, KM_USER0); |
519 | memcpy(dibh->b_data + sizeof(struct gfs2_dinode) + from, | 519 | memcpy(dibh->b_data + sizeof(struct gfs2_dinode) + from, |
520 | (char *)kaddr + from, to - from); | 520 | kaddr + from, to - from); |
521 | kunmap_atomic(page, KM_USER0); | 521 | kunmap_atomic(page, KM_USER0); |
522 | 522 | ||
523 | SetPageUptodate(page); | 523 | SetPageUptodate(page); |
@@ -704,8 +704,9 @@ static void stuck_releasepage(struct buffer_head *bh) | |||
704 | struct gfs2_glock *gl; | 704 | struct gfs2_glock *gl; |
705 | static unsigned limit = 0; | 705 | static unsigned limit = 0; |
706 | 706 | ||
707 | if (limit++ > 3) | 707 | if (limit > 3) |
708 | return; | 708 | return; |
709 | limit++; | ||
709 | 710 | ||
710 | fs_warn(sdp, "stuck in gfs2_releasepage() %p\n", inode); | 711 | fs_warn(sdp, "stuck in gfs2_releasepage() %p\n", inode); |
711 | fs_warn(sdp, "blkno = %llu, bh->b_count = %d\n", | 712 | fs_warn(sdp, "blkno = %llu, bh->b_count = %d\n", |