<p><br>
On May 6, 2012 7:29 PM, "Tom Gundersen" <<a href="mailto:teg@jklm.no">teg@jklm.no</a>> wrote:<br>
><br>
> On Mon, May 7, 2012 at 1:23 AM, Dave Reisner <<a href="mailto:d@falconindy.com">d@falconindy.com</a>> wrote:<br>
> > On Sun, May 06, 2012 at 05:37:11PM -0500, Dan McGee wrote:<br>
> >> On Sun, May 6, 2012 at 9:59 AM, Tom Gundersen <<a href="mailto:teg@jklm.no">teg@jklm.no</a>> wrote:<br>
> >> > On Sun, May 6, 2012 at 4:54 PM, Tom Gundersen <<a href="mailto:teg@jklm.no">teg@jklm.no</a>> wrote:<br>
> >> >> This will allow us to tell how much time was spent in the initramfs,<br>
> >> >> which might be good for working on optimizations and making sure we<br>
> >> >> don't regress.<br>
> >> ><br>
> >> > If this is accepted, I suggest adding the following install hook (call<br>
> >> > it "timestamp" or something like that) to the systemd package:<br>
> >> ><br>
> ><br>
> > I'd call it systemd. It's unclear what's going to happen with early<br>
> > userspace a year from now wrt systemd possibly invading it in some way.<br>
> > We may as well start now with something more generically named to avoid<br>
> > having to possibly rename it at some point down the road.<br>
> ><br>
> >> ><br>
> >> ><br>
> >> > #!/bin/bash<br>
> >> ><br>
> >> > build() {<br>
> >> >    add_binary /usr/lib/systemd/systemd-timestamp /usr/bin/timestamp<br>
> >> I'm very confused as to why we would call the binary something<br>
> >> different in the initramfs. Shouldn't we not screw around with the<br>
> >> name?<br>
> ><br>
> > I'm inclined to agree with this. The namespacing should stay.<br>
><br>
> If you want to think of this as "a hook that does systemd related<br>
> things", then keeping the name makes the most sense. I was thinking it<br>
> would be "a hook that records the timestamp, we just happen to use the<br>
> systemd implementation" and that we'd add other hooks if we wanted<br>
> more systemd features in the future.<br>
><br>
> I'm fine with the way you and Dan suggest though. Do you want me to<br>
> resubmit, or will you just change it when you apply it?<br>
><br>
> -t</p>
<p>I'll fix it up when I apply it. I've got a few other minor nits to fix as well.</p>
<p>Just out of curiosity, is there anything else that reads RD_TIMESTAMP (like Auke's boot chart)? That's part of why I'm hesistant to generalize it as a timestamp hook.</p>
<p>d</p>