Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 31 May 2016 14:44:11 -0400
From:      Allan Jude <allanjude@freebsd.org>
To:        Nikolai Lifanov <lifanov@mail.lifanov.com>
Cc:        svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r301010 - in head/sys: cddl/contrib/opensolaris/common/zfs cddl/contrib/opensolaris/uts/common cddl/contrib/opensolaris/uts/common/fs/zfs cddl/contrib/opensolaris/uts/common/fs/zfs/sys ...
Message-ID:  <778aec32-af48-4837-5790-5a92bbb5765a@freebsd.org>
In-Reply-To: <b44cc14e-ea6c-1851-41d1-f6b45de8850c@mail.lifanov.com>
References:  <201605310412.u4V4CEh4021513@repo.freebsd.org> <inxu-sn3p-wny@vfemail.net> <b44cc14e-ea6c-1851-41d1-f6b45de8850c@mail.lifanov.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2016-05-31 14:36, Nikolai Lifanov wrote:
> On 05/31/2016 03:28, Jan Beich wrote:
>> Allan Jude <allanjude@FreeBSD.org> writes:
>>
>>> Author: allanjude
>>> Date: Tue May 31 04:12:14 2016
>>> New Revision: 301010
>>> URL: https://svnweb.freebsd.org/changeset/base/301010
>>>
>>> Log:
>>>   Connect the SHA-512t256 and Skein hashing algorithms to ZFS
>>>
>>>   Support for the new hashing algorithms in ZFS was introduced in r289422
>>>   However it was disconnected because FreeBSD lacked implementations of
>>>   SHA-512 (truncated to 256 bits), and Skein.
>>>
>>>   These implementations were introduced in r300921 and r300966 respectively
>>>
>>>   This commit connects them to ZFS and enabled these new checksum algorithms
>>>
>>>   This new algorithms are not supported by the boot blocks, so do not use them
>>>   on your root dataset if you boot from ZFS.
>>
>> Can you document the feature and booting caveat in zpool-features(7) manpage?
>> And Illumos seems to limit booting support to pools vs. datasets.
>>
>
> +1. Also, zfs(8) needs to be updated.
>
>>   Booting off of pools using skein is NOT supported -- any attempt to
>>   enable skein on a root pool will fail with an error.
>>
>> https://illumos.org/man/5/zpool-features
>>
>
>

Yeah, i thought these were included in the original MFV, I'll add them

-- 
Allan Jude



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?778aec32-af48-4837-5790-5a92bbb5765a>