From: Chris Fries <c.fr...@motorola.com>
Panic loops while running LTP fsstress has been able to get
a disk into two different panic loops from dec_valid_node_count.
f2fs.h:714 BUG_ON(sbi->total_valid_node_count < count);
Once, it happens during recovery itself, and the disk would cause
a panic every time it mounted.
Another time, it would happen during garbage collection, so the disk
would cause a panic within 200 seconds of mounting.
Removing this BUG_ON hasn't shown any side effects, so let's take it
out and monitor.
Signed-off-by: Chris Fries <c.fr...@motorola.com>
---
fs/f2fs/f2fs.h | 18 +++++++++++++++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index e80a87c..b8e9679 100644
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -722,9 +722,21 @@ static inline void dec_valid_node_count(struct
f2fs_sb_info *sbi,
{
spin_lock(&sbi->stat_lock);
- BUG_ON(sbi->total_valid_block_count < count);
- BUG_ON(sbi->total_valid_node_count < count);
- BUG_ON(inode->i_blocks < count);
+ if (sbi->total_valid_block_count < count) {
+ WARN(1, "F2FS: total_valid_block_count too small- %d vs %d\n",
+ (unsigned int)sbi->total_valid_block_count, count);
+ count = sbi->total_valid_block_count;
+ }
+ if (sbi->total_valid_node_count < count) {
+ WARN(1, "F2FS: total_valid_node_count too small- %d vs %d\n",
+ sbi->total_valid_node_count, count);
+ count = sbi->total_valid_node_count;
+ }
+ if (inode->i_blocks < count) {
+ WARN(1, "F2FS: inode->i_blocks too small - %d vs %d\n",
+ (unsigned int)inode->i_blocks, count);
+ count = sbi->total_valid_node_count;
+ }
inode->i_blocks -= count;
sbi->total_valid_node_count -= count;
--
1.8.0
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
