Comment 23 for bug 1530953

Revision history for this message
Richard Laager (rlaager) wrote :

Attached is a debdiff relative to the alpha PPA. This is the last of the changes I have already implemented for the boot process.

It reworks the zfs initramfs script to more closely follow the upstream Debian (non-zfs) initramfs scripts. Instead of using a ZFS-specific /etc/default parameter to add a minimum delay to the boot process, it uses the rootdelay= kernel command-line parameter used in other Debian initramfs scripts, and makes it a maximum delay. This means that administrators who need to add a delay (which is not the default) can set a larger delay for safety without forcing that extra delay on every boot.

It uses the standard Debian panic function, which then honors the panic= kernel command-line option, which can be set to disable the initramfs panic shell for security reasons. It also follows the upstream goal of clearing all the environment variables which are set.

This is a net code reduction.

Here's a slightly different version of that explanation:
https://github.com/zfsonlinux/pkg-zfs/commit/50f30514c35d2ba2d915e0b327932b260eecce09

This change is explicitly and intentionally drops support for the rpool= and bootfs= parameters. I would like to see them gone before ZFS support hits an Ubuntu LTS release, because it'll be harder to drop that support later. If you would like to maintain support for rpool= and bootfs= but like the rest of the changes, I can update the script to that effect.