<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 11, 2015 at 6:24 PM, Kay Sievers <span dir="ltr"><<a href="mailto:kay@vrfy.org" target="_blank">kay@vrfy.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Thu, Mar 12, 2015 at 2:07 AM, Shawn Landden <<a href="mailto:shawn@churchofgit.com">shawn@churchofgit.com</a>> wrote:<br>
> On Wed, Mar 11, 2015 at 5:51 PM, Kay Sievers <<a href="mailto:kay@vrfy.org">kay@vrfy.org</a>> wrote:<br>
>><br>
>> On Thu, Mar 12, 2015 at 1:22 AM, Shawn Landden <<a href="mailto:shawn@churchofgit.com">shawn@churchofgit.com</a>><br>
>> wrote:<br>
>> > Still use helper when Xen Dom0, to avoid duplicating some hairy code.<br>
>> ><br>
>> > I think the rbtree version was far more understandable as<br>
>> > greedy_realloc0()<br>
>> > is very messy to do correctly.<br>
>> ><br>
>> > Take fopenat() from lsof.<br>
>> > Add opendirat()<br>
>><br>
>> We have that in util.c :: xopendirat()<br>
>><br>
>> > Future: generate BootLoaderSpec files for other kernel install locations<br>
>><br>
>> This approach duplicates, the potentially complex, boot manager kernel<br>
>> selection logic.<br>
>><br>
>> The recent systemd-boot boot loader and efi stub loader which carries<br>
>> the kernel, the cmdline, the initrd in one single EFI binary will also<br>
>> not use any boot loader snippets, it will be discovered by the loader<br>
>> itself, which parses the PE/COFF files and looks for specific content.<br>
>><br>
>> The snippets are meant to unify the boot loader *configuration*, but<br>
>> they do not mean that every bootable kernel will or should have one.<br>
>> There might be many ways for kernels to be found by the boot loader,<br>
>> the snippets are just one source for that.<br>
>><br>
>> I'm not sure what exact problem this patch tries to solve,<br>
><br>
> rebooting with kexec is faster than a full reboot. Currently we do not<br>
> support kexec very well. Lennart asked for something like this, but perhaps<br>
> we no longer want to support kexec loading?<br>
<br>
</div></div>I kind of miss a description what this change is supposed to support<br>
in the end. It can't be described with "replacing a call to a binary".<br>
<br>
Automatic searching and deciding what kernel to boot, and how to<br>
search for these kernels, and how all that should continue to work<br>
reliably in the longer run, while the boot loaders underneath improve<br>
and change; that is something we should define before and have a clear<br>
idea, before we copy only parts of that logic from the current tools.<br>
<div class=""><div class="h5"><br></div></div></blockquote><div>I thought you wrote a specification? <a href="http://freedesktop.org/wiki/Specifications/BootLoaderSpec/">freedesktop.org/wiki/Specifications/BootLoaderSpec/</a></div><div><br></div><div>Anyways, you should at least merge this patch:</div><div><pre style="color:rgb(0,0,0);word-wrap:break-word;white-space:pre-wrap"><a href="mailto:1423865887-1507-1-git-send-email-shawn@churchofgit.com">1423865887-1507-1-git-send-email-shawn@churchofgit.com</a></pre></div><div> (<a href="https://www.mail-archive.com/systemd-devel@lists.freedesktop.org/msg28005.html">https://www.mail-archive.com/systemd-devel@lists.freedesktop.org/msg28005.html</a>)</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">
Thanks,<br>
Kay<br>
_______________________________________________<br>
systemd-devel mailing list<br>
<a href="mailto:systemd-devel@lists.freedesktop.org">systemd-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Shawn Landden</div>
</div></div>