f2fs_convert_inline_page introduce what read_inline_data
already does for copying out the inline data from inode_page.
We can use read_inline_data instead to simplify the code.

Signed-off-by: Shawn Lin <shawn....@rock-chips.com>

---

Changes in v2:
- rework the commit msg to fit the changes
- further simplify the code suggested by Chao Yu

 fs/f2fs/inline.c | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)

diff --git a/fs/f2fs/inline.c b/fs/f2fs/inline.c
index d27347e..3e3bc26 100644
--- a/fs/f2fs/inline.c
+++ b/fs/f2fs/inline.c
@@ -105,7 +105,6 @@ int f2fs_read_inline_data(struct inode *inode, struct page 
*page)
 
 int f2fs_convert_inline_page(struct dnode_of_data *dn, struct page *page)
 {
-       void *src_addr, *dst_addr;
        struct f2fs_io_info fio = {
                .sbi = F2FS_I_SB(dn->inode),
                .type = DATA,
@@ -115,8 +114,6 @@ int f2fs_convert_inline_page(struct dnode_of_data *dn, 
struct page *page)
        };
        int dirty, err;
 
-       f2fs_bug_on(F2FS_I_SB(dn->inode), page->index);
-
        if (!f2fs_exist_data(dn->inode))
                goto clear_out;
 
@@ -125,19 +122,8 @@ int f2fs_convert_inline_page(struct dnode_of_data *dn, 
struct page *page)
                return err;
 
        f2fs_bug_on(F2FS_P_SB(page), PageWriteback(page));
-       if (PageUptodate(page))
-               goto no_update;
 
-       zero_user_segment(page, MAX_INLINE_DATA, PAGE_CACHE_SIZE);
-
-       /* Copy the whole inline data block */
-       src_addr = inline_data_addr(dn->inode_page);
-       dst_addr = kmap_atomic(page);
-       memcpy(dst_addr, src_addr, MAX_INLINE_DATA);
-       flush_dcache_page(page);
-       kunmap_atomic(dst_addr);
-       SetPageUptodate(page);
-no_update:
+       read_inline_data(page, dn->inode_page);
        set_page_dirty(page);
 
        /* clear dirty state */
-- 
2.3.7



------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to