<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 11, 2015 at 5:51 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"><span class="">On Thu, Mar 12, 2015 at 1:22 AM, Shawn Landden <<a href="mailto:shawn@churchofgit.com">shawn@churchofgit.com</a>> 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 greedy_realloc0()<br>
> is very messy to do correctly.<br>
><br>
> Take fopenat() from lsof.<br>
> Add opendirat()<br>
<br>
</span>We have that in util.c :: xopendirat()<br>
<span class=""><br>
> Future: generate BootLoaderSpec files for other kernel install locations<br>
<br>
</span>This approach duplicates, the potentially complex, boot manager kernel<br>
selection logic.<br>
<br></blockquote><br class=""><div>Hopefully it could be unified in some way. I find it disturbing that systemd will have a great deal of EFI-specific code.  </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">
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, but it<br>
generally does not sound right to duplicate the boot loader<br>
selection/discovery/enumeration logic here. I don't really think we<br>
should do that, or will be able to catch with the boot loader up here.<br>
<br>
Thanks,<br>
Kay<br>
<div class=""><div class="h5">_______________________________________________<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>