zfs crypto discuss - Mar 2008 - [Bug 687] New: panic caused by ''zfs create -o encryption=on -o keyscope= dataset ...''

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

           Summary: panic caused by ''zfs create -o encryption=on -o
                    keyscope=dataset  ...''
    Classification: Development
           Product: zfs-crypto
           Version: unspecified
          Platform: i86pc/i386
        OS/Version: Solaris
            Status: NEW
          Severity: blocker
          Priority: P1
         Component: other
        AssignedTo: darrenm at opensolaris.org
        ReportedBy: hua.tang at sun.com
         QAContact: hua.tang at sun.com
                CC: zfs-crypto-discuss at opensolaris.org
   Estimated Hours: 0.0


Build: 2008-03-03

Command:
# zfs create -o encryption=on -o keyscope=dataset -o
keysource=passphrase,prompt mypool/fs1
Enter key: 
Re-enter key:


Stack info:
panic[cpu0]/thread=d0d7ede0: assertion failed: 0, file:
../../common/fs/zfs/zio_crypt.c, line: 657

d0d7ebac genunix:assfail+5a (f7417718, f7417a60,)
d0d7ec28 zfs:dsl_crypto_key_gen+126 (cbb7ee00, d10f1dd0,)
d0d7ec78 zfs:dmu_objset_create_sync+dd (cb2de200, d10f1d7c,)
d0d7ecb0 zfs:dsl_sync_task_group_sync+ec (d0b31000, c86b22e0)
d0d7ecfc zfs:dsl_pool_sync+f9 (c590b900, 3857, 0)
d0d7ed6c zfs:spa_sync+24c (cbdb1040, 3857, 0)
d0d7edc8 zfs:txg_sync_thread+25a (c590b900, 0)
d0d7edd8 unix:thread_start+8 ()

syncing file systems... done
dumping to /dev/dsk/c0t0d0s1, offset 107741184, content: kernel
100% done: 82273 pages dumped, compression ratio 3.63, dump succeeded
rebooting...

Core dump:
/var/crash/data3/*.0

-- 
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=687


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |CAUSEKNOWN




--- Comment #1 from Darren J Moffat <darrenm at opensolaris.org> 
2008-03-06 02:18:11 ---
This is intentional for just now because we don''t have the code
implemented to
get the key to the correct parts of the kernel side of ZFS to wrap the
generated key.

The panic is caused by an ASSERT(0) being triped.  The ASSERT(0) will be
removed and the correct code inserted once the ioctl can pass the data to the
kernel on dataset creation when keyscope=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=687


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |783




-- 
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=687


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=687


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 06:35:11 ---
Integrated first draft of implementation of this feature.  Has some issues but
no longer panics.

-- 
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=687


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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|FIXINSOURCE                 |CLOSED
         Resolution|                            |FIXED




--- Comment #3 from Darren J Moffat <darrenm at opensolaris.org> 
2008-04-02 06:27:28 ---
All seems to be working now.

-- 
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=687


Bug 687 depends on bug 783, which changed state.

Bug 783 Summary: keystatus property confused about persistence.
http://defect.opensolaris.org/bz/show_bug.cgi?id=783

           What    |Old Value                   |New Value
----------------------------------------------------------------------------
             Status|CAUSEKNOWN                  |CLOSED
         Resolution|                            |FIXED



-- 
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.