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

[Allan Jude]

On 2016-05-31 14:44, Allan Jude wrote:
> On 2016-05-31 14:36, Nikolai Lifanov wrote:
>> On 05/31/2016 03:28, Jan Beich wrote:
>>> Allan Jude <allanjude at 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
> 

r301096 updates zpool-features(7) and zfs(8)

-- 
Allan Jude