Looking at the commit, is there any reason that this has not been
accepted by upstream?

Noting comment by ryao on 2 Oct 2015:

"If I recall correctly, the root=ZFS=$DATASET syntax is myself and
either @dajhorn or @rlaager agreed to use to adopt as something of a
standard back in 2012. I would like to revisit it because the
root=ZFS=$DATASET syntax does not properly handle pool namespace
collisions. Ideally, I would like the boot loader to pass the
information about the pool by passing the pool's GUID and vdev
information (using by-path syntax) to the initramfs archive through the
kernel commandline so that we can handle spa namespace collisions
completely unambiguously. I have yet to make time to work out a syntax
that makes sense and implement the hooks required for it. When this is
done, I think the old syntax should remain around for backward
compatibility.

Also, I believe that it is possible to put the required logic into the
kernel by allowing us to detect the rootfs mount via current->fs ==
NULL, but I still need to verify this is NULL at the rootfs mount with
kgdb. This would hypothetically allow us to do pool import inside the
kernel using this information without an initramfs when copy-builtin is
used to build a kernel with ZFS support. It would also simplify
initramfs design in the normal case where one is loading modules by
allowing us to omit ZFS' userland binaries from the initramfs. All we
would need to do is use the standard mount command.

This is something of a tangent, but it is the direction that my thinking
on the boot process has taken as it has evolved."

So from what I understand, this fix is a partial solution because it
does not properly handle pool namespace collisions.


** Changed in: zfs-initramfs (Ubuntu)
       Status: Confirmed => In Progress

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1530953

Title:
  Support GRUB's native root=ZFS=

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/zfs-initramfs/+bug/1530953/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to