ARM: vexpress_defconfig: enable ACL and security labels for ext4
diff mbox

Message ID 1418383877-29685-1-git-send-email-l.stelmach@samsung.com
State New
Headers show

Commit Message

Łukasz Stelmach Dec. 12, 2014, 11:31 a.m. UTC
This patch enables security features of ext4 file-system required
by many conteporary user-space setups.

Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
---
 arch/arm/configs/vexpress_defconfig | 2 ++
 1 file changed, 2 insertions(+)

Comments

Arnd Bergmann Dec. 12, 2014, 11:38 a.m. UTC | #1
On Friday 12 December 2014 12:31:17 Łukasz Stelmach wrote:
> This patch enables security features of ext4 file-system required
> by many conteporary user-space setups.
> 
> Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
> ---
>  arch/arm/configs/vexpress_defconfig | 2 ++
>  1 file changed, 2 insertions(+)

The patch looks good, but should be addressed to the vexpress maintainers
(Liviu, Sudeep, Lorenzo)

	Arnd

> diff --git a/arch/arm/configs/vexpress_defconfig b/arch/arm/configs/vexpress_defconfig
> index f489fda..1d9dd6e 100644
> --- a/arch/arm/configs/vexpress_defconfig
> +++ b/arch/arm/configs/vexpress_defconfig
> @@ -138,6 +138,8 @@ CONFIG_EXT3_FS=y
>  # CONFIG_EXT3_DEFAULTS_TO_ORDERED is not set
>  # CONFIG_EXT3_FS_XATTR is not set
>  CONFIG_EXT4_FS=y
> +CONFIG_EXT4_FS_POSIX_ACL=y
> +CONFIG_EXT4_FS_SECURITY=y
>  CONFIG_VFAT_FS=y
>  CONFIG_TMPFS=y
>  CONFIG_JFFS2_FS=y
>
Liviu Dudau Dec. 12, 2014, 11:52 a.m. UTC | #2
On Fri, Dec 12, 2014 at 11:38:33AM +0000, Arnd Bergmann wrote:
> On Friday 12 December 2014 12:31:17 Łukasz Stelmach wrote:
> > This patch enables security features of ext4 file-system required
> > by many conteporary user-space setups.
> > 
> > Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
> > ---
> >  arch/arm/configs/vexpress_defconfig | 2 ++
> >  1 file changed, 2 insertions(+)
> 
> The patch looks good, but should be addressed to the vexpress maintainers
> (Liviu, Sudeep, Lorenzo)
> 
> 	Arnd

Looks good to me.

Acked-by: Liviu Dudau <Liviu.Dudau@arm.com>

> 
> > diff --git a/arch/arm/configs/vexpress_defconfig b/arch/arm/configs/vexpress_defconfig
> > index f489fda..1d9dd6e 100644
> > --- a/arch/arm/configs/vexpress_defconfig
> > +++ b/arch/arm/configs/vexpress_defconfig
> > @@ -138,6 +138,8 @@ CONFIG_EXT3_FS=y
> >  # CONFIG_EXT3_DEFAULTS_TO_ORDERED is not set
> >  # CONFIG_EXT3_FS_XATTR is not set
> >  CONFIG_EXT4_FS=y
> > +CONFIG_EXT4_FS_POSIX_ACL=y
> > +CONFIG_EXT4_FS_SECURITY=y
> >  CONFIG_VFAT_FS=y
> >  CONFIG_TMPFS=y
> >  CONFIG_JFFS2_FS=y
> > 
> 
>
Łukasz Stelmach Jan. 7, 2015, 10:27 a.m. UTC | #3
It was <2014-12-12 pią 12:38>, when Arnd Bergmann wrote:
> On Friday 12 December 2014 12:31:17 Łukasz Stelmach wrote:
>> This patch enables security features of ext4 file-system required
>> by many conteporary user-space setups.
>> 
>> Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
>> ---
>>  arch/arm/configs/vexpress_defconfig | 2 ++
>>  1 file changed, 2 insertions(+)
>
> The patch looks good, but should be addressed to the vexpress maintainers
> (Liviu, Sudeep, Lorenzo)
>
> 	Arnd

...?
ping?

>> diff --git a/arch/arm/configs/vexpress_defconfig b/arch/arm/configs/vexpress_defconfig
>> index f489fda..1d9dd6e 100644
>> --- a/arch/arm/configs/vexpress_defconfig
>> +++ b/arch/arm/configs/vexpress_defconfig
>> @@ -138,6 +138,8 @@ CONFIG_EXT3_FS=y
>>  # CONFIG_EXT3_DEFAULTS_TO_ORDERED is not set
>>  # CONFIG_EXT3_FS_XATTR is not set
>>  CONFIG_EXT4_FS=y
>> +CONFIG_EXT4_FS_POSIX_ACL=y
>> +CONFIG_EXT4_FS_SECURITY=y
>>  CONFIG_VFAT_FS=y
>>  CONFIG_TMPFS=y
>>  CONFIG_JFFS2_FS=y
Lorenzo Pieralisi Jan. 7, 2015, 11:03 a.m. UTC | #4
On Wed, Jan 07, 2015 at 10:27:25AM +0000, Łukasz Stelmach wrote:
> It was <2014-12-12 pią 12:38>, when Arnd Bergmann wrote:
> > On Friday 12 December 2014 12:31:17 Łukasz Stelmach wrote:
> >> This patch enables security features of ext4 file-system required
> >> by many conteporary user-space setups.
> >> 
> >> Signed-off-by: Łukasz Stelmach <l.stelmach@samsung.com>
> >> ---
> >>  arch/arm/configs/vexpress_defconfig | 2 ++
> >>  1 file changed, 2 insertions(+)
> >
> > The patch looks good, but should be addressed to the vexpress maintainers
> > (Liviu, Sudeep, Lorenzo)
> >
> > 	Arnd
> 
> ...?
> ping?

http://lists.infradead.org/pipermail/linux-arm-kernel/2014-December/310300.html

Lorenzo

> >> diff --git a/arch/arm/configs/vexpress_defconfig b/arch/arm/configs/vexpress_defconfig
> >> index f489fda..1d9dd6e 100644
> >> --- a/arch/arm/configs/vexpress_defconfig
> >> +++ b/arch/arm/configs/vexpress_defconfig
> >> @@ -138,6 +138,8 @@ CONFIG_EXT3_FS=y
> >>  # CONFIG_EXT3_DEFAULTS_TO_ORDERED is not set
> >>  # CONFIG_EXT3_FS_XATTR is not set
> >>  CONFIG_EXT4_FS=y
> >> +CONFIG_EXT4_FS_POSIX_ACL=y
> >> +CONFIG_EXT4_FS_SECURITY=y
> >>  CONFIG_VFAT_FS=y
> >>  CONFIG_TMPFS=y
> >>  CONFIG_JFFS2_FS=y
> 
> -- 
> Łukasz Stelmach
> Samsung R&D Institute Poland
> Samsung Electronics

Patch
diff mbox

diff --git a/arch/arm/configs/vexpress_defconfig b/arch/arm/configs/vexpress_defconfig
index f489fda..1d9dd6e 100644
--- a/arch/arm/configs/vexpress_defconfig
+++ b/arch/arm/configs/vexpress_defconfig
@@ -138,6 +138,8 @@  CONFIG_EXT3_FS=y
 # CONFIG_EXT3_DEFAULTS_TO_ORDERED is not set
 # CONFIG_EXT3_FS_XATTR is not set
 CONFIG_EXT4_FS=y
+CONFIG_EXT4_FS_POSIX_ACL=y
+CONFIG_EXT4_FS_SECURITY=y
 CONFIG_VFAT_FS=y
 CONFIG_TMPFS=y
 CONFIG_JFFS2_FS=y