diff options
| author | Roel Kluin <12o3l@tiscali.nl> | 2008-04-17 17:25:37 +0200 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2008-04-18 08:36:09 +0100 |
| commit | 62be1f71677c53d5e51223807a06ac9052f49b0f (patch) | |
| tree | 954b987466f9616934364b716d7fb93dcba12b46 /fs/gfs2/log.c | |
| parent | 16c5f06f15ad4e5a5d6e90b78ffb1ac14319e445 (diff) | |
| download | linux-62be1f71677c53d5e51223807a06ac9052f49b0f.tar.gz linux-62be1f71677c53d5e51223807a06ac9052f49b0f.tar.bz2 linux-62be1f71677c53d5e51223807a06ac9052f49b0f.zip | |
[GFS2] fix assertion in log_refund()
since unsigned, unused >= 0 is always true.
Signed-off-by: Roel Kluin <12o3l@tiscali.nl>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/log.c')
| -rw-r--r-- | fs/gfs2/log.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/log.c b/fs/gfs2/log.c index b335304fc5d6..548264b1836d 100644 --- a/fs/gfs2/log.c +++ b/fs/gfs2/log.c @@ -769,8 +769,8 @@ static void log_refund(struct gfs2_sbd *sdp, struct gfs2_trans *tr) sdp->sd_log_commited_revoke += tr->tr_num_revoke - tr->tr_num_revoke_rm; gfs2_assert_withdraw(sdp, ((int)sdp->sd_log_commited_revoke) >= 0); reserved = calc_reserved(sdp); + gfs2_assert_withdraw(sdp, sdp->sd_log_blks_reserved + tr->tr_reserved >= reserved); unused = sdp->sd_log_blks_reserved - reserved + tr->tr_reserved; - gfs2_assert_withdraw(sdp, unused >= 0); atomic_add(unused, &sdp->sd_log_blks_free); gfs2_assert_withdraw(sdp, atomic_read(&sdp->sd_log_blks_free) <= sdp->sd_jdesc->jd_blocks); |