[f2fs-dev] [PATCH v2] f2fs: fix to truncate inline data past EOF

2015-03-09 Thread Chao Yu
Previously if inode is with inline data, we will try to invalid partial inline data in page #0 when we truncate size of inode in truncate_partial_data_page(). And then we set page #0 to dirty, after this we can synchronize inode page with page #0 at -writepage(). But sometimes we will fail to

Re: [f2fs-dev] [PATCH] f2fs: fix to truncate inline data past EOF

2015-03-09 Thread Jaegeuk Kim
Hi Chao, On Tue, Mar 10, 2015 at 10:02:46AM +0800, Chao Yu wrote: Hi Jaegeuk, [snip] +static int truncate_partial_data_page(struct inode *inode, u64 from, bool force) { unsigned offset = from (PAGE_CACHE_SIZE - 1); struct page *page; - if

Re: [f2fs-dev] [PATCH] f2fs: fix to truncate inline data past EOF

2015-03-09 Thread Chao Yu
Hi Jaegeuk, -Original Message- From: Jaegeuk Kim [mailto:jaeg...@kernel.org] Sent: Monday, March 09, 2015 11:49 AM To: Chao Yu Cc: 'Changman Lee'; linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org Subject: Re: [PATCH] f2fs: fix to truncate inline data past EOF

Re: [f2fs-dev] [PATCH 3/3] f2fs:use enum dentrt_ptr type to replace constant use

2015-03-09 Thread Chao Yu
Hi Yuan, -Original Message- From: Yuan Zhong [mailto:yuan.mark.zh...@samsung.com] Sent: Saturday, March 07, 2015 6:10 PM To: Jaegeuk Kim Cc: linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net Subject: [f2fs-dev] [PATCH 3/3] f2fs:use

Re: [f2fs-dev] [PATCH 1/3 v2] f2fs:remove unnecessary condition judgment

2015-03-09 Thread Chao Yu
-Original Message- From: Yuan Zhong [mailto:yuan.mark.zh...@samsung.com] Sent: Monday, March 09, 2015 10:44 AM To: Jaegeuk Kim; ??? Cc: linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net Subject: [f2fs-dev] [PATCH 1/3 v2] f2fs:remove

[f2fs-dev] [PATCH] f2fs: fix to calculate max length of contiguous free slots correctly

2015-03-09 Thread Chao Yu
When lookuping for creating, we will try to record the level of current dentry hash table if current dentry has enough contiguous slots for storing name of new file which will be created later, this can save our lookup time when add a link into parent dir. But currently in find_target_dentry, our

Re: [f2fs-dev] [PATCH 2/3] f2fs: fix get stale meta pages when build free nids

2015-03-09 Thread Chao Yu
Hi Wanpeng, -Original Message- From: Wanpeng Li [mailto:wanpeng...@linux.intel.com] Sent: Monday, March 09, 2015 12:25 PM To: Jaegeuk Kim Cc: Changman Lee; Chao Yu; linux-f2fs-devel@lists.sourceforge.net; linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; Wanpeng Li