Re: [PATCH 1/2] f2fs: enable inline_xattr by default

[Chao Yu]

Hi Jaegeuk,

On 2017/2/14 8:35, Jaegeuk Kim wrote:
> Hi Chao,
> 
> On 02/08, Chao Yu wrote:
>> In android, since SElinux is enable, security policy will be appliedd for
>> each file, it stores in inode as an xattr entry, so it will take one 4k
>> size node block additionally for each file.
>>
>> Let's enable inline_xattr by default in order to save storage space.
> 
> Then, we need to add "noinline_xattr".

OK, let me introduce the new option in another patch.

Thanks,

> 
> Thanks,
> 
>>
>> Signed-off-by: Chao Yu <yuchao0@xxxxxxxxxx>
>> ---
>>  fs/f2fs/super.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
>> index 68a555b014cf..e2f919d3096a 100644
>> --- a/fs/f2fs/super.c
>> +++ b/fs/f2fs/super.c
>> @@ -1026,6 +1026,7 @@ static void default_options(struct f2fs_sb_info *sbi)
>>  	sbi->active_logs = NR_CURSEG_TYPE;
>>  
>>  	set_opt(sbi, BG_GC);
>> +	set_opt(sbi, INLINE_XATTR);
>>  	set_opt(sbi, INLINE_DATA);
>>  	set_opt(sbi, INLINE_DENTRY);
>>  	set_opt(sbi, EXTENT_CACHE);
>> -- 
>> 2.8.2.295.g3f1c1d0
> 
> .
>