diff options
| author | Lukas Czerner <lczerner@redhat.com> | 2013-01-29 03:21:12 +0100 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2013-01-29 03:21:12 +0100 |
| commit | b06acd38a44127b382fa53e49878f7a2b70af6f2 (patch) | |
| tree | 03fc90c10ddac5d83890286b52b66ef623a87a42 | |
| parent | ext4: remove unused variable flags (diff) | |
| download | linux-b06acd38a44127b382fa53e49878f7a2b70af6f2.tar.xz linux-b06acd38a44127b382fa53e49878f7a2b70af6f2.zip | |
ext4: remove explicit WARN_ON when ext4_map_blocks() fails
In two places we call WARN_ON() before we print out the debug message,
however we agreed that the WARN_ON() is unnecessary at those places so
remove them.
Also use ext4_warning() instead of ext4_msg() and printk().
Signed-off-by: Lukas Czerner <lczerner@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
| -rw-r--r-- | fs/ext4/extents.c | 24 |
1 files changed, 11 insertions, 13 deletions
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index 566c8f3789e1..db55c62e1f01 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -4462,11 +4462,11 @@ retry: ret = ext4_map_blocks(handle, inode, &map, flags); if (ret <= 0) { #ifdef EXT4FS_DEBUG - WARN_ON(ret <= 0); - printk(KERN_ERR "%s: ext4_ext_map_blocks " - "returned error inode#%lu, block=%u, " - "max_blocks=%u", __func__, - inode->i_ino, map.m_lblk, max_blocks); + ext4_warning(inode->i_sb, + "inode #%lu: block %u: len %u: " + "ext4_ext_map_blocks returned %d", + inode->i_ino, map.m_lblk, + map.m_len, ret); #endif ext4_mark_inode_dirty(handle, inode); ret2 = ext4_journal_stop(handle); @@ -4539,14 +4539,12 @@ int ext4_convert_unwritten_extents(struct inode *inode, loff_t offset, } ret = ext4_map_blocks(handle, inode, &map, EXT4_GET_BLOCKS_IO_CONVERT_EXT); - if (ret <= 0) { - WARN_ON(ret <= 0); - ext4_msg(inode->i_sb, KERN_ERR, - "%s:%d: inode #%lu: block %u: len %u: " - "ext4_ext_map_blocks returned %d", - __func__, __LINE__, inode->i_ino, map.m_lblk, - map.m_len, ret); - } + if (ret <= 0) + ext4_warning(inode->i_sb, + "inode #%lu: block %u: len %u: " + "ext4_ext_map_blocks returned %d", + inode->i_ino, map.m_lblk, + map.m_len, ret); ext4_mark_inode_dirty(handle, inode); ret2 = ext4_journal_stop(handle); if (ret <= 0 || ret2 ) |