[PATCH] fs: grow_dev_page() failure path not a BUG()

From: Louis Huemiller
Date: Thu Oct 20 2011 - 18:07:23 EST


The function grow_dev_page() contains a label named failed which
is jumped to on allocation failure. When the system is low on
memory, which is not a bug, this path can be taken. This patch
removes the BUG() call, so that this path can be exercised for
proper operation on a debug kernel, without panicing the system.

Signed-off-by: Louis Huemiller <lhuemill@xxxxxxxxxx>
---
fs/buffer.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/fs/buffer.c b/fs/buffer.c
index 1a80b04..04781e6 100644
--- a/fs/buffer.c
+++ b/fs/buffer.c
@@ -1032,7 +1032,6 @@ grow_dev_page(struct block_device *bdev, sector_t block,
return page;

failed:
- BUG();
unlock_page(page);
page_cache_release(page);
return NULL;
--
1.7.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/