zfs crypto discuss - Mar 2008 - [Bug 860] New: GRUB updates

http://defect.opensolaris.org/bz/show_bug.cgi?id=860

           Summary: GRUB updates
    Classification: Development
           Product: zfs-crypto
           Version: unspecified
          Platform: Other
        OS/Version: Solaris
            Status: NEW
          Severity: blocker
          Priority: P1
         Component: other
        AssignedTo: hua.tang at sun.com
        ReportedBy: darrenm at opensolaris.org
         QAContact: hua.tang at sun.com
                CC: zfs-crypto-discuss at opensolaris.org
   Estimated Hours: 0.0


Since this project adds a new ZFS version the GRUB zfs code needs to be aware
of this.

The ZFS boot team says:
-----
GRUB/zfs needs to grab a couple of files from

         rootpool dataset (menu.lst)
         rootpool/root_filesystem dataset (boot archive)

As long as GRUB can do that, then the only thing that I can think of for now
is to make sure the version is updated in
usr/src/grub/grub-0.95/stage2/zfs-include/zfs.h.
-----

Hopefully it is that easy.

The best, and hopefully easiest too, way to test this is to replace the ZFS
binaries (user, kernel, libs, utils, and updated grub bins) on an OpenSolaris
Developer Preview 2 release and check it still boots and we can still create
encrypted datasets.

I have a feeling there will be more than the single version change though since
the dnode_phys_t is changed for this project.

Other possible files that need changing are zio.h to add awareness of the new
checksum type - but that shouldn''t be needed since we don''t
support boot from a
dataset with the sha256+ccm checksum which can only happen for an encrypted
dataset.

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://defect.opensolaris.org/bz/show_bug.cgi?id=860


Darren J Moffat <darrenm at opensolaris.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|GRUB updates                |GRUB updates & zpool bootfs
                   |                            |not allowed
             Status|NEW                         |CAUSEKNOWN




--- Comment #1 from Darren J Moffat <darrenm at opensolaris.org> 
2008-03-27 05:13:33 ---
Add the version bump for the grub version of ZFS and added code to
spa_prop_validate to ensure the bootfs is not an encrypted dataset., fix needs
verification.

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://defect.opensolaris.org/bz/show_bug.cgi?id=860


Darren J Moffat <darrenm at opensolaris.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CAUSEKNOWN                  |FIXINPROGRESS




-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://defect.opensolaris.org/bz/show_bug.cgi?id=860


Darren J Moffat <darrenm at opensolaris.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|FIXINPROGRESS               |FIXINSOURCE




--- Comment #2 from Darren J Moffat <darrenm at opensolaris.org> 
2008-03-27 05:15:21 ---
Integrated fix, needs verification.

-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://defect.opensolaris.org/bz/show_bug.cgi?id=860


Danek Duvall <danek.duvall at sun.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|FIXINSOURCE                 |RESOLVED
         Resolution|                            |FIXINSOURCE




-- 
Configure bugmail: http://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.