f2fs: record node block allocation in dnode_of_data

This patch introduces recording node block allocation in dnode_of_data.
This information helps to figure out whether any node block is allocated during
specific file operations.

Reviewed-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index cf0c9dda0365c1e18f7b93567993c2e34cb3493f..a7a9a05d012a6b03d4ebb5803ce39e4bc630053f 100644 (file)
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -226,6 +226,7 @@ void set_data_blkaddr(struct dnode_of_data *dn)
        addr_array = blkaddr_in_node(rn);
        addr_array[ofs_in_node] = cpu_to_le32(dn->data_blkaddr);
        set_page_dirty(node_page);
+       dn->node_changed = true;
 }
 
 int reserve_new_block(struct dnode_of_data *dn)

diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index 90fb970e2b98da615168ec62414d7cd9193e2422..3e4a60da408f5210240ee228f1d47f668fafcb10 100644 (file)
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -546,6 +546,7 @@ struct dnode_of_data {
        nid_t nid;                      /* node id of the direct node block */
        unsigned int ofs_in_node;       /* data offset in the node page */
        bool inode_page_locked;         /* inode page is locked or not */
+       bool node_changed;              /* is node block changed */
        block_t data_blkaddr;           /* block address of the node block */
 };
 

diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c
index 6cc8ac7e185a203981f1d0d7ca27a9b7405c891c..341de5d2353b3a3fb397d576d1c482d665ee023c 100644 (file)
--- a/fs/f2fs/node.c
+++ b/fs/f2fs/node.c
@@ -542,6 +542,7 @@ int get_dnode_of_data(struct dnode_of_data *dn, pgoff_t index, int mode)
 
                        set_nid(parent, offset[i - 1], nids[i], i == 1);
                        alloc_nid_done(sbi, nids[i]);
+                       dn->node_changed = true;
                        done = true;
                } else if (mode == LOOKUP_NODE_RA && i == level && level > 1) {
                        npage[i] = get_node_page_ra(parent, offset[i - 1]);
@@ -678,6 +679,7 @@ static int truncate_nodes(struct dnode_of_data *dn, unsigned int nofs,
                        if (ret < 0)
                                goto out_err;
                        set_nid(page, i, 0, false);
+                       dn->node_changed = true;
                }
        } else {
                child_nofs = nofs + ofs * (NIDS_PER_BLOCK + 1) + 1;
@@ -691,6 +693,7 @@ static int truncate_nodes(struct dnode_of_data *dn, unsigned int nofs,
                        ret = truncate_nodes(&rdn, child_nofs, 0, depth - 1);
                        if (ret == (NIDS_PER_BLOCK + 1)) {
                                set_nid(page, i, 0, false);
+                               dn->node_changed = true;
                                child_nofs += ret;
                        } else if (ret < 0 && ret != -ENOENT) {
                                goto out_err;
@@ -752,6 +755,7 @@ static int truncate_partial_nodes(struct dnode_of_data *dn,
                if (err < 0)
                        goto fail;
                set_nid(pages[idx], i, 0, false);
+               dn->node_changed = true;
        }
 
        if (offset[idx + 1] == 0) {
@@ -1153,6 +1157,7 @@ void sync_inode_page(struct dnode_of_data *dn)
        } else {
                update_inode_page(dn->inode);
        }
+       dn->node_changed = true;
 }
 
 int sync_node_pages(struct f2fs_sb_info *sbi, nid_t ino,