Skip to content

Commit

Permalink
[GFS2] Style changes in ops_address.c
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Steven Whitehouse committed Sep 4, 2006
1 parent dd538c8 commit 0bd5996
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions fs/gfs2/ops_address.c
Original file line number Diff line number Diff line change
Expand Up @@ -517,7 +517,7 @@ static int gfs2_commit_write(struct file *file, struct page *page,

kaddr = kmap_atomic(page, KM_USER0);
memcpy(dibh->b_data + sizeof(struct gfs2_dinode) + from,
(char *)kaddr + from, to - from);
kaddr + from, to - from);
kunmap_atomic(page, KM_USER0);

SetPageUptodate(page);
Expand Down Expand Up @@ -704,8 +704,9 @@ static void stuck_releasepage(struct buffer_head *bh)
struct gfs2_glock *gl;
static unsigned limit = 0;

if (limit++ > 3)
if (limit > 3)
return;
limit++;

fs_warn(sdp, "stuck in gfs2_releasepage() %p\n", inode);
fs_warn(sdp, "blkno = %llu, bh->b_count = %d\n",
Expand Down

0 comments on commit 0bd5996

Please sign in to comment.