Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-14 Thread Paul Moore
On Tuesday, January 13, 2015 10:23:23 PM Mimi Zohar wrote: > I would assume only 'security.evm' is not portable as it attempts to > tightly bind the file metadata to the file data. Casey? Paul? [NOTE: Added the SELinux mailing list to the CC line.] The SELinux xattr should be portable assuming

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-14 Thread Mimi Zohar
On Tue, 2015-01-13 at 22:34 -0600, Rob Landley wrote: > > On 01/13/2015 09:23 PM, Mimi Zohar wrote: > > On Tue, 2015-01-13 at 15:42 -0600, Rob Landley wrote: > Then again if we add a new field right before the previous size then the > "treat it as 64 bits vs 2 32 bit ones" is an implementation

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-14 Thread Paul Moore
On Tuesday, January 13, 2015 10:23:23 PM Mimi Zohar wrote: I would assume only 'security.evm' is not portable as it attempts to tightly bind the file metadata to the file data. Casey? Paul? [NOTE: Added the SELinux mailing list to the CC line.] The SELinux xattr should be portable assuming

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-14 Thread Mimi Zohar
On Tue, 2015-01-13 at 22:34 -0600, Rob Landley wrote: On 01/13/2015 09:23 PM, Mimi Zohar wrote: On Tue, 2015-01-13 at 15:42 -0600, Rob Landley wrote: Then again if we add a new field right before the previous size then the treat it as 64 bits vs 2 32 bit ones is an implementation detail

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Rob Landley
On 01/13/2015 09:23 PM, Mimi Zohar wrote: > On Tue, 2015-01-13 at 15:42 -0600, Rob Landley wrote: >>> 4 bytes enough? > >> Eh, as long as we're breaking compatibility anyway, we might as well >> extend the file size. It's gzipped so the extra run of consecutive >> zeroes isn't really an issue,

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Mimi Zohar
On Tue, 2015-01-13 at 15:42 -0600, Rob Landley wrote: > > On 01/13/2015 02:20 PM, Mimi Zohar wrote: > > On Tue, 2015-01-13 at 12:48 -0600, Rob Landley wrote: > >> I note that there are two data formats of interest here: > >> > >> 1) the cpio file layout. > >> > >> 2) the list of files generated

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Rob Landley
On 01/13/2015 02:20 PM, Mimi Zohar wrote: > On Tue, 2015-01-13 at 12:48 -0600, Rob Landley wrote: >> I note that there are two data formats of interest here: >> >> 1) the cpio file layout. >> >> 2) the list of files generated by gen_initramfs_list.sh and consumed by >> gen_init_cpio. >> >> The

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Mimi Zohar
On Tue, 2015-01-13 at 12:48 -0600, Rob Landley wrote: > On 01/08/2015 04:08 PM, Mimi Zohar wrote: > > On Thu, 2015-01-08 at 12:19 -0600, Rob Landley wrote: > >> > >> But I am curious about how you propose to encode xattrs into the cpio > >> format. (Which Al Viro chose because it's _simple_.

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Rob Landley
On 01/08/2015 04:08 PM, Mimi Zohar wrote: > On Thu, 2015-01-08 at 12:19 -0600, Rob Landley wrote: >> >> But I am curious about how you propose to encode xattrs into the cpio >> format. (Which Al Viro chose because it's _simple_. There isn't really >> a >> controlling spec since Posix decided to

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Mimi Zohar
On Tue, 2015-01-13 at 15:42 -0600, Rob Landley wrote: On 01/13/2015 02:20 PM, Mimi Zohar wrote: On Tue, 2015-01-13 at 12:48 -0600, Rob Landley wrote: I note that there are two data formats of interest here: 1) the cpio file layout. 2) the list of files generated by

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Rob Landley
On 01/13/2015 09:23 PM, Mimi Zohar wrote: On Tue, 2015-01-13 at 15:42 -0600, Rob Landley wrote: 4 bytes enough? Eh, as long as we're breaking compatibility anyway, we might as well extend the file size. It's gzipped so the extra run of consecutive zeroes isn't really an issue, and if

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Rob Landley
On 01/08/2015 04:08 PM, Mimi Zohar wrote: On Thu, 2015-01-08 at 12:19 -0600, Rob Landley wrote: But I am curious about how you propose to encode xattrs into the cpio format. (Which Al Viro chose because it's _simple_. There isn't really a controlling spec since Posix decided to deprecated it

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Mimi Zohar
On Tue, 2015-01-13 at 12:48 -0600, Rob Landley wrote: On 01/08/2015 04:08 PM, Mimi Zohar wrote: On Thu, 2015-01-08 at 12:19 -0600, Rob Landley wrote: But I am curious about how you propose to encode xattrs into the cpio format. (Which Al Viro chose because it's _simple_. There isn't

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-13 Thread Rob Landley
On 01/13/2015 02:20 PM, Mimi Zohar wrote: On Tue, 2015-01-13 at 12:48 -0600, Rob Landley wrote: I note that there are two data formats of interest here: 1) the cpio file layout. 2) the list of files generated by gen_initramfs_list.sh and consumed by gen_init_cpio. The fact you're

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Mimi Zohar
On Thu, 2015-01-08 at 12:19 -0600, Rob Landley wrote: > > But I am curious about how you propose to encode xattrs into the cpio > format. (Which Al Viro chose because it's _simple_. There isn't really > a > controlling spec since Posix decided to deprecated it in 2001 and > yank > it from SUSv3

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Rob Landley
On 01/08/2015 09:13 AM, Mimi Zohar wrote: > On Thu, 2015-01-08 at 09:01 -0500, Josh Boyer wrote: >> On Wed, Jan 7, 2015 at 3:52 PM, Mimi Zohar wrote: >> That's pretty awkward. I think it highlights the major downside of >> this approach in that from a standard distro point of view this >>

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Mimi Zohar
On Thu, 2015-01-08 at 09:01 -0500, Josh Boyer wrote: > On Wed, Jan 7, 2015 at 3:52 PM, Mimi Zohar wrote: > > This patch modifies the gen_initramfs_list.sh script to include xattrs > > in the initramfs. > > > > Dracut creates the initramfs using the cpio tool on the system, not > > the kernel's

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Josh Boyer
On Wed, Jan 7, 2015 at 3:52 PM, Mimi Zohar wrote: > This patch modifies the gen_initramfs_list.sh script to include xattrs > in the initramfs. > > Dracut creates the initramfs using the cpio tool on the system, not > the kernel's gen_init_cpio script. The following commands, for example, > would

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Mimi Zohar
On Thu, 2015-01-08 at 12:19 -0600, Rob Landley wrote: But I am curious about how you propose to encode xattrs into the cpio format. (Which Al Viro chose because it's _simple_. There isn't really a controlling spec since Posix decided to deprecated it in 2001 and yank it from SUSv3 onwards.

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Josh Boyer
On Wed, Jan 7, 2015 at 3:52 PM, Mimi Zohar zo...@linux.vnet.ibm.com wrote: This patch modifies the gen_initramfs_list.sh script to include xattrs in the initramfs. Dracut creates the initramfs using the cpio tool on the system, not the kernel's gen_init_cpio script. The following commands,

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Rob Landley
On 01/08/2015 09:13 AM, Mimi Zohar wrote: On Thu, 2015-01-08 at 09:01 -0500, Josh Boyer wrote: On Wed, Jan 7, 2015 at 3:52 PM, Mimi Zohar zo...@linux.vnet.ibm.com wrote: That's pretty awkward. I think it highlights the major downside of this approach in that from a standard distro point of

Re: [RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-08 Thread Mimi Zohar
On Thu, 2015-01-08 at 09:01 -0500, Josh Boyer wrote: On Wed, Jan 7, 2015 at 3:52 PM, Mimi Zohar zo...@linux.vnet.ibm.com wrote: This patch modifies the gen_initramfs_list.sh script to include xattrs in the initramfs. Dracut creates the initramfs using the cpio tool on the system, not

[RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-07 Thread Mimi Zohar
This patch modifies the gen_initramfs_list.sh script to include xattrs in the initramfs. Dracut creates the initramfs using the cpio tool on the system, not the kernel's gen_init_cpio script. The following commands, for example, would create an initramfs containing xattrs. dracut -H -f

[RFC][PATCH 6/9] gen_initramfs_list.sh: include xattrs

2015-01-07 Thread Mimi Zohar
This patch modifies the gen_initramfs_list.sh script to include xattrs in the initramfs. Dracut creates the initramfs using the cpio tool on the system, not the kernel's gen_init_cpio script. The following commands, for example, would create an initramfs containing xattrs. dracut -H -f