summaryrefslogtreecommitdiffstats
path: root/fs/gfs2
diff options
context:
space:
mode:
authorBob Peterson <rpeterso@redhat.com>2012-08-28 08:45:56 -0400
committerSteven Whitehouse <swhiteho@redhat.com>2012-09-24 10:47:23 +0100
commit0688a5ecea61a36ba12d17a18ab9f8712145cfa2 (patch)
treea33dcb7283fc30e334b4d9c2ba7b9f3513d0eedd /fs/gfs2
parentc743ffd09fa7d3464c6f74767a3ae2ca5dc3ebf7 (diff)
downloadop-kernel-dev-0688a5ecea61a36ba12d17a18ab9f8712145cfa2.zip
op-kernel-dev-0688a5ecea61a36ba12d17a18ab9f8712145cfa2.tar.gz
GFS2: Stop block extents at the end of bitmaps
This patch stops multiple block allocations if a nonzero return code is received from gfs2_rbm_from_block. Without this patch, if enough pressure is put on the file system, you get a kernel warning quickly followed by: BUG: unable to handle kernel NULL pointer dereference at (null) IP: [<ffffffffa04f47e8>] gfs2_alloc_blocks+0x2c8/0x880 [gfs2] With this patch, things run normally. Signed-off-by: Bob Peterson <rpeterso@redhat.com> Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2')
-rw-r--r--fs/gfs2/rgrp.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c
index 8869541..defb826 100644
--- a/fs/gfs2/rgrp.c
+++ b/fs/gfs2/rgrp.c
@@ -1834,8 +1834,7 @@ static void gfs2_alloc_extent(const struct gfs2_rbm *rbm, bool dinode,
block++;
while (*n < elen) {
ret = gfs2_rbm_from_block(&pos, block);
- WARN_ON(ret);
- if (gfs2_testbit(&pos) != GFS2_BLKST_FREE)
+ if (ret || gfs2_testbit(&pos) != GFS2_BLKST_FREE)
break;
gfs2_trans_add_bh(pos.rgd->rd_gl, pos.bi->bi_bh, 1);
gfs2_setbit(&pos, true, GFS2_BLKST_USED);
OpenPOWER on IntegriCloud