diff options
author | Christoph Hellwig <hch@infradead.org> | 2011-08-23 04:28:07 -0400 |
---|---|---|
committer | Alex Elder <aelder@sgi.com> | 2011-10-11 22:15:00 -0400 |
commit | c2b006c1da1602551def200e4661535f02b82488 (patch) | |
tree | 6801f459bb218ed062d30e31eb92d3867ce8166a /fs/xfs/xfs_sync.c | |
parent | 61551f1ee536289084a4a8f1c4f187e2f371c440 (diff) |
xfs: let xfs_bwrite callers handle the xfs_buf_relse
Remove the xfs_buf_relse from xfs_bwrite and let the caller handle it to
mirror the delwri and read paths.
Also remove the mount pointer passed to xfs_bwrite, which is superflous now
that we have a mount pointer in the buftarg.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_sync.c')
-rw-r--r-- | fs/xfs/xfs_sync.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/xfs/xfs_sync.c b/fs/xfs/xfs_sync.c index 4604f90f86a3..90cc197e0433 100644 --- a/fs/xfs/xfs_sync.c +++ b/fs/xfs/xfs_sync.c | |||
@@ -322,6 +322,7 @@ xfs_sync_fsdata( | |||
322 | struct xfs_mount *mp) | 322 | struct xfs_mount *mp) |
323 | { | 323 | { |
324 | struct xfs_buf *bp; | 324 | struct xfs_buf *bp; |
325 | int error; | ||
325 | 326 | ||
326 | /* | 327 | /* |
327 | * If the buffer is pinned then push on the log so we won't get stuck | 328 | * If the buffer is pinned then push on the log so we won't get stuck |
@@ -334,8 +335,9 @@ xfs_sync_fsdata( | |||
334 | bp = xfs_getsb(mp, 0); | 335 | bp = xfs_getsb(mp, 0); |
335 | if (xfs_buf_ispinned(bp)) | 336 | if (xfs_buf_ispinned(bp)) |
336 | xfs_log_force(mp, 0); | 337 | xfs_log_force(mp, 0); |
337 | 338 | error = xfs_bwrite(bp); | |
338 | return xfs_bwrite(mp, bp); | 339 | xfs_buf_relse(bp); |
340 | return error; | ||
339 | } | 341 | } |
340 | 342 | ||
341 | /* | 343 | /* |