f2fs: should put the dnode when NEW_ADDR is detected
authorJaegeuk Kim <jaegeuk.kim@samsung.com>
Thu, 26 Dec 2013 07:55:22 +0000 (16:55 +0900)
committerJaegeuk Kim <jaegeuk.kim@samsung.com>
Thu, 26 Dec 2013 11:33:06 +0000 (20:33 +0900)
When get_dnode_of_data() in get_data_block() returns a successful dnode, we
should put the dnode.
But, previously, if its data block address is equal to NEW_ADDR, we didn't do
that, resulting in a deadlock condition.
So, this patch splits original error conditions with this case, and then calls
f2fs_put_dnode before finishing the function.

Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
fs/f2fs/data.c

index 0879d2aa97e60d9821d411356fd39d97ab0f8f0b..991e36835df89368e0daed6990b070a167f2283e 100644 (file)
@@ -626,11 +626,13 @@ static int get_data_block(struct inode *inode, sector_t iblock,
        /* When reading holes, we need its node page */
        set_new_dnode(&dn, inode, NULL, NULL, 0);
        err = get_dnode_of_data(&dn, pgofs, mode);
-       if (err || dn.data_blkaddr == NEW_ADDR) {
+       if (err) {
                if (err == -ENOENT)
                        err = 0;
                goto unlock_out;
        }
+       if (dn.data_blkaddr == NEW_ADDR)
+               goto put_out;
 
        if (dn.data_blkaddr != NULL_ADDR) {
                map_bh(bh_result, inode->i_sb, dn.data_blkaddr);
@@ -659,11 +661,14 @@ get_next:
 
                set_new_dnode(&dn, inode, NULL, NULL, 0);
                err = get_dnode_of_data(&dn, pgofs, mode);
-               if (err || dn.data_blkaddr == NEW_ADDR) {
+               if (err) {
                        if (err == -ENOENT)
                                err = 0;
                        goto unlock_out;
                }
+               if (dn.data_blkaddr == NEW_ADDR)
+                       goto put_out;
+
                end_offset = IS_INODE(dn.node_page) ?
                        ADDRS_PER_INODE(F2FS_I(inode)) : ADDRS_PER_BLOCK;
        }