diff options
author | Theodore Ts'o <tytso@mit.edu> | 2013-02-14 15:51:58 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2013-02-14 15:51:58 -0500 |
commit | 01a523eb51cb505a4bc1eaffeeccd2527d6ab619 (patch) | |
tree | 0e7bacf5b07dbf1e133c2f2f8d925a55e24d4146 /fs/ext4/inode.c | |
parent | 8de5c325b4ee7d5b23b95edd28b81c9a2a9f427f (diff) |
ext4: add debugging context for warning in ext4_da_update_reserve_space()
Print some additional debugging context to hopefully help to debug a
warning which is getting triggered by xfstests #74.
Also remove extraneous newlines from when printk's were converted to
ext4_warning() and ext4_msg().
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/inode.c')
-rw-r--r-- | fs/ext4/inode.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index b85d5dae726b..c4e9177f60c6 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c | |||
@@ -352,10 +352,12 @@ void ext4_da_update_reserve_space(struct inode *inode, | |||
352 | } | 352 | } |
353 | 353 | ||
354 | if (unlikely(ei->i_allocated_meta_blocks > ei->i_reserved_meta_blocks)) { | 354 | if (unlikely(ei->i_allocated_meta_blocks > ei->i_reserved_meta_blocks)) { |
355 | ext4_warning(inode->i_sb, "%s: ino %lu, allocated %d " | 355 | ext4_warning(inode->i_sb, "ino %lu, allocated %d " |
356 | "with only %d reserved metadata blocks\n", __func__, | 356 | "with only %d reserved metadata blocks " |
357 | inode->i_ino, ei->i_allocated_meta_blocks, | 357 | "(releasing %d blocks with reserved %d data blocks)", |
358 | ei->i_reserved_meta_blocks); | 358 | inode->i_ino, ei->i_allocated_meta_blocks, |
359 | ei->i_reserved_meta_blocks, used, | ||
360 | ei->i_reserved_data_blocks); | ||
359 | WARN_ON(1); | 361 | WARN_ON(1); |
360 | ei->i_allocated_meta_blocks = ei->i_reserved_meta_blocks; | 362 | ei->i_allocated_meta_blocks = ei->i_reserved_meta_blocks; |
361 | } | 363 | } |
@@ -1609,7 +1611,7 @@ static void mpage_da_map_and_submit(struct mpage_da_data *mpd) | |||
1609 | (unsigned long long) next, | 1611 | (unsigned long long) next, |
1610 | mpd->b_size >> mpd->inode->i_blkbits, err); | 1612 | mpd->b_size >> mpd->inode->i_blkbits, err); |
1611 | ext4_msg(sb, KERN_CRIT, | 1613 | ext4_msg(sb, KERN_CRIT, |
1612 | "This should not happen!! Data will be lost\n"); | 1614 | "This should not happen!! Data will be lost"); |
1613 | if (err == -ENOSPC) | 1615 | if (err == -ENOSPC) |
1614 | ext4_print_free_blocks(mpd->inode); | 1616 | ext4_print_free_blocks(mpd->inode); |
1615 | } | 1617 | } |