Re: [f2fs-dev] [PATCH 3/4] f2fs: avoid f2fs_lock_op in f2fs_write_begin

2015-12-24 Thread Jaegeuk Kim
Hi Chao, On Fri, Dec 25, 2015 at 09:18:06AM +0800, Chao Yu wrote: > Hi Jaegeuk, > > > -Original Message- > > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > > Sent: Friday, December 25, 2015 5:52 AM > > To: Chao Yu > > Cc: linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; > > l

Re: [f2fs-dev] [PATCH 4/4] f2fs: call f2fs_balance_fs only when node was changed

2015-12-24 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Chao Yu [mailto:chao2...@samsung.com] > Sent: Wednesday, December 23, 2015 5:47 PM > To: 'Jaegeuk Kim' > Cc: linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; > linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 4/4]

Re: [f2fs-dev] [PATCH 3/4] f2fs: avoid f2fs_lock_op in f2fs_write_begin

2015-12-24 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Friday, December 25, 2015 5:52 AM > To: Chao Yu > Cc: linux-fsde...@vger.kernel.org; linux-ker...@vger.kernel.org; > linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 3/4] f2fs: a

[f2fs-dev] [PATCH] f2fs: early check broken symlink length in the encrypted case

2015-12-24 Thread Jaegeuk Kim
If link is broken, its len is zero, and we don't need to move forward. Signed-off-by: Jaegeuk Kim --- fs/f2fs/namei.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c index fb41c80..5cc4128 100644 --- a/fs/f2fs/namei.c +++ b/fs/f2fs

Re: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-24 Thread Chao Yu
> -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Friday, December 25, 2015 4:16 AM > To: Chao Yu > Cc: linux-ker...@vger.kernel.org; linux-fsde...@vger.kernel.org; > linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH 2/4] f2fs: return early w

Re: [f2fs-dev] [PATCH 3/4] f2fs: avoid f2fs_lock_op in f2fs_write_begin

2015-12-24 Thread Jaegeuk Kim
Hi Chao, On Thu, Dec 24, 2015 at 12:33:50PM -0800, Jaegeuk Kim wrote: > Hi Chao, > > On Thu, Dec 24, 2015 at 01:50:32PM +0800, Chao Yu wrote: > > Hi Jaegeuk, > > > > > -Original Message- > > > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > > > Sent: Thursday, December 24, 2015 10:15 AM

Re: [f2fs-dev] [PATCH 4/5] f2fs: support data flush in ioctl

2015-12-24 Thread Jaegeuk Kim
Hi Chao, On Thu, Dec 24, 2015 at 06:10:25PM +0800, Chao Yu wrote: > Sometimes user want to sync all data belong to superblock into storage > for persistence, 'syncfs' syscall is one option, still f2fs supports > similar one through ioctl, difference is that sb releted kworker is > online for write

Re: [f2fs-dev] [PATCH 3/4] f2fs: avoid f2fs_lock_op in f2fs_write_begin

2015-12-24 Thread Jaegeuk Kim
Hi Chao, On Thu, Dec 24, 2015 at 01:50:32PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > > -Original Message- > > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > > Sent: Thursday, December 24, 2015 10:15 AM > > To: linux-ker...@vger.kernel.org; linux-fsde...@vger.kernel.org; > > linux-f2fs-dev

Re: [f2fs-dev] [PATCH 2/4] f2fs: return early when trying to read null nid

2015-12-24 Thread Jaegeuk Kim
On Thu, Dec 24, 2015 at 01:49:24PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > > -Original Message- > > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > > Sent: Thursday, December 24, 2015 10:15 AM > > To: linux-ker...@vger.kernel.org; linux-fsde...@vger.kernel.org; > > linux-f2fs-devel@lists.s

[f2fs-dev] [PATCH 5/5] f2fs: clean up f2fs_ioc_write_checkpoint

2015-12-24 Thread Chao Yu
Use f2fs_sync_fs to clean up codes in f2fs_ioc_write_checkpoint. Signed-off-by: Chao Yu --- fs/f2fs/file.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 91997a5..b75037b 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -1619,

[f2fs-dev] [PATCH 4/5] f2fs: support data flush in ioctl

2015-12-24 Thread Chao Yu
Sometimes user want to sync all data belong to superblock into storage for persistence, 'syncfs' syscall is one option, still f2fs supports similar one through ioctl, difference is that sb releted kworker is online for writebacking concurrently. Signed-off-by: Chao Yu --- fs/f2fs/file.c | 10 +++

[f2fs-dev] [PATCH 3/5] f2fs: report readonly status in ->fsync

2015-12-24 Thread Chao Yu
Report readonly status of filesystem during fsync. Signed-off-by: Chao Yu --- fs/f2fs/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 2d87a3c..a60d088 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -197,7 +197,7 @@ int f2fs_sy

[f2fs-dev] [PATCH 2/5] f2fs: let user being aware of IO error

2015-12-24 Thread Chao Yu
Sometimes we keep dumb when IO error occur in lower layer device, so user will not receive any error return value for some operation, but actually, the operation did not succeed. This sould be avoided, so this patch reports such kind of error to user. Signed-off-by: Chao Yu --- fs/f2fs/checkpoi

[f2fs-dev] [PATCH 1/5] f2fs: add missing f2fs_balance_fs in __recover_dot_dentries

2015-12-24 Thread Chao Yu
__recover_do_dentries will try to grab free space in storage, so fix to add missing f2fs_balance_fs here. Signed-off-by: Chao Yu --- fs/f2fs/namei.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c index e439f32..fb41c80 100644 --- a/fs/f2fs/namei.c +++ b/f