diff options
author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-03-24 18:42:13 +0100 |
---|---|---|
committer | Adrian Bunk <bunk@stusta.de> | 2006-03-24 18:42:13 +0100 |
commit | c5d3237c2424c4a3cf69d33abc1f229943468367 (patch) | |
tree | 5bd2cfc657b48cfc9ca014586d33704fa643475b /fs/coda | |
parent | 88bcd51262ed45212d1b3a65abbac46eaf36bfeb (diff) | |
download | lwn-c5d3237c2424c4a3cf69d33abc1f229943468367.tar.gz lwn-c5d3237c2424c4a3cf69d33abc1f229943468367.zip |
BUG_ON() Conversion in fs/coda/
this changes if() BUG(); constructs to BUG_ON() which is
cleaner, contains unlikely() and can better optimized away.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Diffstat (limited to 'fs/coda')
-rw-r--r-- | fs/coda/cache.c | 2 | ||||
-rw-r--r-- | fs/coda/cnode.c | 3 |
2 files changed, 2 insertions, 3 deletions
diff --git a/fs/coda/cache.c b/fs/coda/cache.c index c607d923350a..5d0527133266 100644 --- a/fs/coda/cache.c +++ b/fs/coda/cache.c @@ -51,7 +51,7 @@ void coda_cache_clear_all(struct super_block *sb) struct coda_sb_info *sbi; sbi = coda_sbp(sb); - if (!sbi) BUG(); + BUG_ON(!sbi); atomic_inc(&permission_epoch); } diff --git a/fs/coda/cnode.c b/fs/coda/cnode.c index 23aeef5aa814..4c9fecbfa91f 100644 --- a/fs/coda/cnode.c +++ b/fs/coda/cnode.c @@ -120,8 +120,7 @@ void coda_replace_fid(struct inode *inode, struct CodaFid *oldfid, cii = ITOC(inode); - if (!coda_fideq(&cii->c_fid, oldfid)) - BUG(); + BUG_ON(!coda_fideq(&cii->c_fid, oldfid)); /* replace fid and rehash inode */ /* XXX we probably need to hold some lock here! */ |