Re: [f2fs-dev] [PATCH 1/2] f2fs: reorganize __f2fs_add_link

2015-07-28 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Saturday, July 25, 2015 12:11 AM > To: Chao Yu > Cc: linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org > Subject: Re: [PATCH 1/2] f2fs: reorganize __f2fs_add_link > > Hi Chao, > >

Re: [f2fs-dev] [PATCH 1/2] f2fs: reorganize __f2fs_add_link

2015-07-24 Thread Jaegeuk Kim
Hi Chao, BTW, isn't there any problem on update_inode/mark_inode_dirty stuffs? And, is there a hole to write uncompleted node pages unnecessarily? On Fri, Jul 24, 2015 at 08:52:00AM -0700, Jaegeuk Kim wrote: > Hi Chao, > > On Fri, Jul 24, 2015 at 06:31:16PM +0800, Chao Yu wrote: > > Hi Jaegeuk,

Re: [f2fs-dev] [PATCH 1/2] f2fs: reorganize __f2fs_add_link

2015-07-24 Thread Jaegeuk Kim
Hi Chao, On Fri, Jul 24, 2015 at 06:31:16PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > > -Original Message- > > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > > Sent: Friday, July 24, 2015 2:15 AM > > To: Chao Yu > > Cc: linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org >

Re: [f2fs-dev] [PATCH 1/2] f2fs: reorganize __f2fs_add_link

2015-07-24 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Friday, July 24, 2015 2:15 AM > To: Chao Yu > Cc: linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org > Subject: Re: [PATCH 1/2] f2fs: reorganize __f2fs_add_link > > Hi Chao, > > On

Re: [f2fs-dev] [PATCH 1/2] f2fs: reorganize __f2fs_add_link

2015-07-23 Thread Jaegeuk Kim
Hi Chao, On Wed, Jul 22, 2015 at 06:21:52PM +0800, Chao Yu wrote: > Firstly, spliting init_inode_metadata into two parts as below since they > are relatively independent. > 1) init_inode_metadata: init datas belong to newly created inode; > 2) update_inode_metadata: update inode info for linking i

[f2fs-dev] [PATCH 1/2] f2fs: reorganize __f2fs_add_link

2015-07-22 Thread Chao Yu
Firstly, spliting init_inode_metadata into two parts as below since they are relatively independent. 1) init_inode_metadata: init datas belong to newly created inode; 2) update_inode_metadata: update inode info for linking inode to new dentry. Secondly, move init_inode_metadata to the front of __f