Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1576397 > unrolled thread

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

Started byChao Yu <yuchao0@huawei.com>
First post2017-02-08 10:50 +0100
Last post2017-02-15 03:30 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/2] f2fs: enable inline_xattr by default Chao Yu <yuchao0@huawei.com> - 2017-02-08 10:50 +0100
    Re: [PATCH 1/2] f2fs: enable inline_xattr by default Jaegeuk Kim <jaegeuk@kernel.org> - 2017-02-14 01:40 +0100
      Re: [PATCH 1/2] f2fs: enable inline_xattr by default Chao Yu <yuchao0@huawei.com> - 2017-02-15 03:30 +0100

#1576397 — [PATCH 1/2] f2fs: enable inline_xattr by default

FromChao Yu <yuchao0@huawei.com>
Date2017-02-08 10:50 +0100
Subject[PATCH 1/2] f2fs: enable inline_xattr by default
Message-ID<t8x2W-74c-11@gated-at.bofh.it>
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.

Signed-off-by: Chao Yu <yuchao0@huawei.com>
---
 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

[toc] | [next] | [standalone]


#1580166

FromJaegeuk Kim <jaegeuk@kernel.org>
Date2017-02-14 01:40 +0100
Message-ID<tazjX-2HA-7@gated-at.bofh.it>
In reply to#1576397
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".

Thanks,

> 
> Signed-off-by: Chao Yu <yuchao0@huawei.com>
> ---
>  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

[toc] | [prev] | [next] | [standalone]


#1581006

FromChao Yu <yuchao0@huawei.com>
Date2017-02-15 03:30 +0100
Message-ID<taXvX-1Eg-9@gated-at.bofh.it>
In reply to#1580166
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@huawei.com>
>> ---
>>  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
> 
> .
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web