| Message ID | 20230515121256.14413-1-Sai.Sathujoda@toshiba-tsip.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Nobuhiro Iwamatsu |
| Headers | show |
| Series | [cip-kernel-config] 5.10.y-cip/arm : Add support for access control lists. | expand |
Hi! > From: Sai <Sai.Sathujoda@toshiba-tsip.com> > > These configs are required for acl package to work > which is used for IEC-62443-4-2 evaluation. > > Signed-off-by: Sai <Sai.Sathujoda@toshiba-tsip.com> Seems okay to me. We'll need to apply similar changes to 6.1. If we have ext4, do we need ext2 and ext3? IIRC ext4 can serve as both of those. Best regards, Pavel > +++ b/5.10.y-cip/arm/qemu_arm_defconfig > @@ -149,6 +149,8 @@ CONFIG_EXT2_FS=y > CONFIG_EXT2_FS_XATTR=y > CONFIG_EXT2_FS_POSIX_ACL=y > CONFIG_EXT3_FS=y > +CONFIG_EXT4_FS=y > +CONFIG_EXT4_FS_POSIX_ACL=y > +CONFIG_TMPFS_POSIX_ACL=y > CONFIG_AUTOFS4_FS=y > CONFIG_FUSE_FS=m > CONFIG_OVERLAY_FS=m
Hi, > -----Original Message----- > From: Sai.Sathujoda@toshiba-tsip.com <Sai.Sathujoda@toshiba-tsip.com> > Sent: Monday, May 15, 2023 9:13 PM > To: cip-dev@lists.cip-project.org; iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ > ＤＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu@toshiba.co.jp> > Cc: ashrith sai(ＴＳＩＰ) <Sai.Sathujoda@toshiba-tsip.com>; dinesh > kumar(ＴＳＩＰ TMIEC ODG Porting) <dinesh.kumar@toshiba-tsip.com> > Subject: [cip-kernel-config] 5.10.y-cip/arm : Add support for access control > lists. Please set a more descriptive subject. User don't know this is a change to ext4 and tmpfs. > > From: Sai <Sai.Sathujoda@toshiba-tsip.com> > > These configs are required for acl package to work which is used for > IEC-62443-4-2 evaluation. > > Signed-off-by: Sai <Sai.Sathujoda@toshiba-tsip.com> > --- > 5.10.y-cip/arm/qemu_arm_defconfig | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/5.10.y-cip/arm/qemu_arm_defconfig > b/5.10.y-cip/arm/qemu_arm_defconfig > index 58ca273..2992fea 100644 > --- a/5.10.y-cip/arm/qemu_arm_defconfig > +++ b/5.10.y-cip/arm/qemu_arm_defconfig > @@ -149,6 +149,8 @@ CONFIG_EXT2_FS=y > CONFIG_EXT2_FS_XATTR=y > CONFIG_EXT2_FS_POSIX_ACL=y > CONFIG_EXT3_FS=y > +CONFIG_EXT4_FS=y > +CONFIG_EXT4_FS_POSIX_ACL=y > +CONFIG_TMPFS_POSIX_ACL=y > CONFIG_AUTOFS4_FS=y > CONFIG_FUSE_FS=m > CONFIG_OVERLAY_FS=m > -- > 2.20.1 I could not apply. Please recheck this patch. Best regards, Nobuhiro
Hi, > -----Original Message----- > From: Pavel Machek <pavel@denx.de> > Sent: Tuesday, May 16, 2023 5:27 AM > To: cip-dev@lists.cip-project.org > Cc: iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ) > <nobuhiro1.iwamatsu@toshiba.co.jp>; ashrith sai(ＴＳＩＰ) > <Sai.Sathujoda@toshiba-tsip.com>; dinesh kumar(ＴＳＩＰ TMIEC ODG > Porting) <dinesh.kumar@toshiba-tsip.com> > Subject: Re: [cip-dev] [cip-kernel-config] 5.10.y-cip/arm : Add support for > access control lists. > > Hi! > > > From: Sai <Sai.Sathujoda@toshiba-tsip.com> > > > > These configs are required for acl package to work which is used for > > IEC-62443-4-2 evaluation. > > > > Signed-off-by: Sai <Sai.Sathujoda@toshiba-tsip.com> > > Seems okay to me. We'll need to apply similar changes to 6.1. > > If we have ext4, do we need ext2 and ext3? IIRC ext4 can serve as both of those. Agree. But IEC-62443-4-2 may require ext3, etc. I don't know the details of this... Or if we decide to only support ext4 as CIP/QEMU, I think we don't need to configure ext2/ext3. Best regards, Nobuhiro
Hi all, Based on the discussion, I got that we need to remove the below configs in 5.10 arm defconfig and we need to just include ext4 configs. Is that right ? CONFIG_EXT2_FS=y CONFIG_EXT2_FS_XATTR=y CONFIG_EXT2_FS_POSIX_ACL=y CONFIG_EXT3_FS=y Please confirm. Thanks and regards, Sai Ashrith (T S I P) -----Original Message----- From: iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu@toshiba.co.jp> Sent: Tuesday, May 16, 2023 5:27 AM To: Pavel Machek <pavel@denx.de>; cip-dev@lists.cip-project.org Cc: ashrith sai(ＴＳＩＰ) <Sai.Sathujoda@toshiba-tsip.com>; dinesh kumar(ＴＳＩＰ TMIEC ODG Porting) <dinesh.kumar@toshiba-tsip.com> Subject: RE: [cip-dev] [cip-kernel-config] 5.10.y-cip/arm : Add support for access control lists. Hi, > -----Original Message----- > From: Pavel Machek <pavel@denx.de> > Sent: Tuesday, May 16, 2023 5:27 AM > To: cip-dev@lists.cip-project.org > Cc: iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ) > <nobuhiro1.iwamatsu@toshiba.co.jp>; ashrith sai(ＴＳＩＰ) > <Sai.Sathujoda@toshiba-tsip.com>; dinesh kumar(ＴＳＩＰ TMIEC ODG > Porting) <dinesh.kumar@toshiba-tsip.com> > Subject: Re: [cip-dev] [cip-kernel-config] 5.10.y-cip/arm : Add > support for access control lists. > > Hi! > > > From: Sai <Sai.Sathujoda@toshiba-tsip.com> > > > > These configs are required for acl package to work which is used for > > IEC-62443-4-2 evaluation. > > > > Signed-off-by: Sai <Sai.Sathujoda@toshiba-tsip.com> > > Seems okay to me. We'll need to apply similar changes to 6.1. > > If we have ext4, do we need ext2 and ext3? IIRC ext4 can serve as both of those. Agree. But IEC-62443-4-2 may require ext3, etc. I don't know the details of this... Or if we decide to only support ext4 as CIP/QEMU, I think we don't need to configure ext2/ext3. Best regards, Nobuhiro
Hi all, I apologize for the confusion from my side. Actually my intention is to add the config for acl package in the IEC layer and it is not to introduce EXT4 file system config for armhf, it was added as a dependency. Currently I am checking with an acl config (CONFIG_EXT3_FS_POSIX_ACL) for the already present EXT3 file system configuration. I will re-share the patch once the outcome is positive. Thanks and regards, Sai Ashrith (T S I P) -----Original Message----- From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On Behalf Of sai.sathujoda@toshiba-tsip.com Sent: Tuesday, May 16, 2023 4:17 PM To: iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu@toshiba.co.jp>; pavel@denx.de; cip-dev@lists.cip-project.org Cc: dinesh kumar(ＴＳＩＰ TMIEC ODG Porting) <dinesh.kumar@toshiba-tsip.com> Subject: Re: [cip-dev] [cip-kernel-config] 5.10.y-cip/arm : Add support for access control lists. Hi all, Based on the discussion, I got that we need to remove the below configs in 5.10 arm defconfig and we need to just include ext4 configs. Is that right ? CONFIG_EXT2_FS=y CONFIG_EXT2_FS_XATTR=y CONFIG_EXT2_FS_POSIX_ACL=y CONFIG_EXT3_FS=y Please confirm. Thanks and regards, Sai Ashrith (T S I P) -----Original Message----- From: iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu@toshiba.co.jp> Sent: Tuesday, May 16, 2023 5:27 AM To: Pavel Machek <pavel@denx.de>; cip-dev@lists.cip-project.org Cc: ashrith sai(ＴＳＩＰ) <Sai.Sathujoda@toshiba-tsip.com>; dinesh kumar(ＴＳＩＰ TMIEC ODG Porting) <dinesh.kumar@toshiba-tsip.com> Subject: RE: [cip-dev] [cip-kernel-config] 5.10.y-cip/arm : Add support for access control lists. Hi, > -----Original Message----- > From: Pavel Machek <pavel@denx.de> > Sent: Tuesday, May 16, 2023 5:27 AM > To: cip-dev@lists.cip-project.org > Cc: iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□ＤＩＴ○ＯＳＴ) > <nobuhiro1.iwamatsu@toshiba.co.jp>; ashrith sai(ＴＳＩＰ) > <Sai.Sathujoda@toshiba-tsip.com>; dinesh kumar(ＴＳＩＰ TMIEC ODG > Porting) <dinesh.kumar@toshiba-tsip.com> > Subject: Re: [cip-dev] [cip-kernel-config] 5.10.y-cip/arm : Add > support for access control lists. > > Hi! > > > From: Sai <Sai.Sathujoda@toshiba-tsip.com> > > > > These configs are required for acl package to work which is used for > > IEC-62443-4-2 evaluation. > > > > Signed-off-by: Sai <Sai.Sathujoda@toshiba-tsip.com> > > Seems okay to me. We'll need to apply similar changes to 6.1. > > If we have ext4, do we need ext2 and ext3? IIRC ext4 can serve as both of those. Agree. But IEC-62443-4-2 may require ext3, etc. I don't know the details of this... Or if we decide to only support ext4 as CIP/QEMU, I think we don't need to configure ext2/ext3. Best regards, Nobuhiro
diff --git a/5.10.y-cip/arm/qemu_arm_defconfig b/5.10.y-cip/arm/qemu_arm_defconfig index 58ca273..2992fea 100644 --- a/5.10.y-cip/arm/qemu_arm_defconfig +++ b/5.10.y-cip/arm/qemu_arm_defconfig @@ -149,6 +149,8 @@ CONFIG_EXT2_FS=y CONFIG_EXT2_FS_XATTR=y CONFIG_EXT2_FS_POSIX_ACL=y CONFIG_EXT3_FS=y +CONFIG_EXT4_FS=y +CONFIG_EXT4_FS_POSIX_ACL=y +CONFIG_TMPFS_POSIX_ACL=y CONFIG_AUTOFS4_FS=y CONFIG_FUSE_FS=m CONFIG_OVERLAY_FS=m