Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-21 Thread Namjae Jeon
> > Sorry for late. I was very busy. > > Could you tell me if it happens difference between xattr and i_mode, > what will you do? First of all, I want to know which case make mismatching permission between xattr and i_mode. And when we call chmod, inode is locked in sys_chmod. If so, inode->i_mode

Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-20 Thread Changman Lee
On 금, 2013-06-14 at 13:20 +0900, Namjae Jeon wrote: > 2013/6/11, Namjae Jeon : > > 2013/6/11, Changman Lee : > >> On 화, 2013-06-11 at 07:57 +0900, Namjae Jeon wrote: > >>> 2013/6/10, Changman Lee : > >>> > Hello, Namjae > >>> Hi. Changman. > >>> > > >>> > If using ACL, whenever i_mode is changed we

Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-13 Thread Namjae Jeon
2013/6/11, Namjae Jeon : > 2013/6/11, Changman Lee : >> On 화, 2013-06-11 at 07:57 +0900, Namjae Jeon wrote: >>> 2013/6/10, Changman Lee : >>> > Hello, Namjae >>> Hi. Changman. >>> > >>> > If using ACL, whenever i_mode is changed we should update acl_mode >>> > which >>> > is written to xattr block,

Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-11 Thread Namjae Jeon
2013/6/11, Changman Lee : > On 화, 2013-06-11 at 07:57 +0900, Namjae Jeon wrote: >> 2013/6/10, Changman Lee : >> > Hello, Namjae >> Hi. Changman. >> > >> > If using ACL, whenever i_mode is changed we should update acl_mode >> > which >> > is written to xattr block, too. And vice versa. >> > Because

Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-10 Thread Changman Lee
On 화, 2013-06-11 at 07:57 +0900, Namjae Jeon wrote: > 2013/6/10, Changman Lee : > > Hello, Namjae > Hi. Changman. > > > > If using ACL, whenever i_mode is changed we should update acl_mode which > > is written to xattr block, too. And vice versa. > > Because update_inode() is called at any reason a

Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-10 Thread Namjae Jeon
2013/6/10, Changman Lee : > Hello, Namjae Hi. Changman. > > If using ACL, whenever i_mode is changed we should update acl_mode which > is written to xattr block, too. And vice versa. > Because update_inode() is called at any reason and anytime, so we should > sync both the moment xattr is written.

Re: [f2fs-dev] [PATCH 1/4] f2fs: reorganize the f2fs_setattr() function.

2013-06-09 Thread Changman Lee
Hello, Namjae If using ACL, whenever i_mode is changed we should update acl_mode which is written to xattr block, too. And vice versa. Because update_inode() is called at any reason and anytime, so we should sync both the moment xattr is written. We don't hope that only i_mode is written to disk a