| To: | Eric Sandeen <sandeen@xxxxxxxxxx>, xfs-oss <xfs@xxxxxxxxxxx> |
|---|---|
| Subject: | [PATCH 4/4] xfs: remove rbpp check from xfs_rtmodify_summary_int |
| From: | Eric Sandeen <sandeen@xxxxxxxxxxx> |
| Date: | Thu, 21 Aug 2014 20:03:19 -0500 |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <53F6942B.80808@xxxxxxxxxx> |
| References: | <53F6942B.80808@xxxxxxxxxx> |
rbpp is always passed into xfs_rtmodify_summary
and xfs_rtget_summary, so there is no need to
test for it in xfs_rtmodify_summary_int.
Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
---
diff --git a/fs/xfs/libxfs/xfs_rtbitmap.c b/fs/xfs/libxfs/xfs_rtbitmap.c
index 50e3b93..7c818f1 100644
--- a/fs/xfs/libxfs/xfs_rtbitmap.c
+++ b/fs/xfs/libxfs/xfs_rtbitmap.c
@@ -460,7 +460,7 @@ xfs_rtmodify_summary_int(
/*
* If we have an old buffer, and the block number matches, use that.
*/
- if (rbpp && *rbpp && *rsb == sb)
+ if (*rbpp && *rsb == sb)
bp = *rbpp;
/*
* Otherwise we have to get the buffer.
@@ -469,7 +469,7 @@ xfs_rtmodify_summary_int(
/*
* If there was an old one, get rid of it first.
*/
- if (rbpp && *rbpp)
+ if (*rbpp)
xfs_trans_brelse(tp, *rbpp);
error = xfs_rtbuf_get(mp, tp, sb, 1, &bp);
if (error) {
@@ -478,10 +478,8 @@ xfs_rtmodify_summary_int(
/*
* Remember this buffer and block for the next call.
*/
- if (rbpp) {
- *rbpp = bp;
- *rsb = sb;
- }
+ *rbpp = bp;
+ *rsb = sb;
}
/*
* Point to the summary information, modify/log it, and/or copy it out.
@@ -493,14 +491,8 @@ xfs_rtmodify_summary_int(
*sp += delta;
xfs_trans_log_buf(tp, bp, first, first + sizeof(*sp) - 1);
}
- if (sum) {
- /*
- * Drop the buffer if we're not asked to remember it.
- */
- if (!rbpp)
- xfs_trans_brelse(tp, bp);
+ if (sum)
*sum = *sp;
- }
return 0;
}
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 3/4] xfs: combine xfs_rtmodify_summary and xfs_rtget_summary, Eric Sandeen |
|---|---|
| Next by Date: | Re: [PATCH V2] xfs: combine xfs_seek_hole & xfs_seek_data, Brian Foster |
| Previous by Thread: | Re: [PATCH 3/4] xfs: combine xfs_rtmodify_summary and xfs_rtget_summary, Eric Sandeen |
| Next by Thread: | Re: [PATCH 4/4] xfs: remove rbpp check from xfs_rtmodify_summary_int, Brian Foster |
| Indexes: | [Date] [Thread] [Top] [All Lists] |