eric kustarz wrote: > Darren J Moffat wrote: >> eric kustarz wrote: >> >>> Not sure if you've already posted this or not, but do you have a webrev? >> >> Yes it was in the original message: >> >> http://cr.grommit.com/~darrenm/zfs-crypto/ >> > > I didn't see anything obvious in the webrev, but have you tried to > temporarily make zio_decrypt_data() and zio_encrypt_data() no-ops to > rule out that new code and focus on the existing framework?
Silly me I should have noticed this myself :-) If you look at the source you will see that other than some of the types those functions are all #ifdef _KERNEL. Given that I can make exactly the same assert trip with ztest/zdb which uses the libzfs/libzpool builds of the uts/common/fs/zfs code I guess I've already done that. -- Darren J Moffat