Control: tag -1 pending

Thanks all for the patches and discussion, and sorry for not chiming in
earlier in the release cycle.

I now merged in Guilherme's patch modulo some minor fixes.  My first
reaction was this this was an “abuse” of initramfs-tools(7)'s interface
since it clearly state that passed local-top stage the root device node
is expected to be present (encouraging us to call the local-block/mdadm
from local-top/cryptroot, like we're already doing for local-block/lvm2),
however it also says later that if local-top fails to create the root
device node the local-block scripts will be called periodically.

So in the end I like Guilherme's approach to give up early at local-top
stage and let the local-block caller take care of the retry and polling
logic.  In fact on closer look it appears initramfs-tools(7) is now
discouraging to do any kind of polling: 
https://salsa.debian.org/kernel-team/initramfs-tools/-/commit/ab9130667d42d0532e8f2900fa3da280dfb61f03

This should allow further simplifying the boot script: no need to loop
up to $ROOTDELAY there.  And I believe we can even remove the LVM hack
for dm-crypt-over-LVM setups, too.

Thanks again!
-- 
Guilhem.

Attachment: signature.asc
Description: PGP signature

Reply via email to