Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-26 Thread Taras Kondratiuk
Quoting Rob Landley (2018-01-25 18:40:54) > On 01/24/2018 09:27 PM, Taras Kondratiuk wrote: > > diff --git a/Documentation/early-userspace/buffer-format.txt > > b/Documentation/early-userspace/buffer-format.txt > > index e1fd7f9dad16..d818df4f72dc 100644 > > --- a/Documentation/early-userspace/buf

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-26 Thread Henrique de Moraes Holschuh
On Fri, 26 Jan 2018, Victor Kamensky wrote: > On Fri, 26 Jan 2018, Henrique de Moraes Holschuh wrote: > > On Thu, 25 Jan 2018, Rob Landley wrote: > > > That said, I don't think -h newcx should emit (or recognize) the > > > "TRAILER!!!1!" entry. That's kinda silly in-band signaling for 2018: > > >

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-26 Thread Victor Kamensky
On Fri, 26 Jan 2018, Henrique de Moraes Holschuh wrote: On Thu, 25 Jan 2018, Rob Landley wrote: That said, I don't think -h newcx should emit (or recognize) the "TRAILER!!!1!" entry. That's kinda silly in-band signaling for 2018: files have a length, pipes provide EOF, and each cpiox entry st

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-26 Thread Henrique de Moraes Holschuh
On Thu, 25 Jan 2018, Rob Landley wrote: > That said, I don't think -h newcx should emit (or recognize) the > "TRAILER!!!1!" entry. That's kinda silly in-band signaling for 2018: > files have a length, pipes provide EOF, and each cpiox entry starts with > 6 bytes of c_magic anyway. (I stopped toybox

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-26 Thread Arnd Bergmann
On Fri, Jan 26, 2018 at 3:39 AM, Rob Landley wrote: > The problem with 1 second timestamps was you honestly could confuse > "make" about which file was newer once an exec() could complete in the > same second having done real work. That was the motivating issue causing > the change, going to nano

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-25 Thread Rob Landley
On 01/24/2018 09:27 PM, Taras Kondratiuk wrote: > diff --git a/Documentation/early-userspace/buffer-format.txt > b/Documentation/early-userspace/buffer-format.txt > index e1fd7f9dad16..d818df4f72dc 100644 > --- a/Documentation/early-userspace/buffer-format.txt > +++ b/Documentation/early-userspace

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-25 Thread Rob Landley
On 01/25/2018 03:29 AM, Arnd Bergmann wrote: > On Thu, Jan 25, 2018 at 4:27 AM, Taras Kondratiuk wrote: >> Many of the Linux security/integrity features are dependent on file >> metadata, stored as extended attributes (xattrs), for making decisions. >> These features need to be initialized during

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-25 Thread Taras Kondratiuk
Quoting Arnd Bergmann (2018-01-25 13:02:49) > On Thu, Jan 25, 2018 at 9:26 PM, Taras Kondratiuk wrote: > > > For initramfs nanoseconds field can be ignored during > > unpacking. > > That sounds like a pointless microoptimization. Most likely we won't ever > need the nanoseconds in the initramfs,

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-25 Thread Arnd Bergmann
On Thu, Jan 25, 2018 at 9:26 PM, Taras Kondratiuk wrote: > Quoting Arnd Bergmann (2018-01-25 01:29:12) >> On Thu, Jan 25, 2018 at 4:27 AM, Taras Kondratiuk wrote: > > We can match statx(2) by having 64 bits for seconds plus 32 bits for > nanoseconds. Ok. > For initramfs nanoseconds field can be

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-25 Thread Taras Kondratiuk
Quoting Arnd Bergmann (2018-01-25 01:29:12) > On Thu, Jan 25, 2018 at 4:27 AM, Taras Kondratiuk wrote: > > Many of the Linux security/integrity features are dependent on file > > metadata, stored as extended attributes (xattrs), for making decisions. > > These features need to be initialized durin

Re: [PATCH v2 01/15] Documentation: add newcx initramfs format description

2018-01-25 Thread Arnd Bergmann
On Thu, Jan 25, 2018 at 4:27 AM, Taras Kondratiuk wrote: > Many of the Linux security/integrity features are dependent on file > metadata, stored as extended attributes (xattrs), for making decisions. > These features need to be initialized during initcall and enabled as > early as possible for co