Page MenuHomeFreeBSD

ZFS doesn't use skein assembly
Needs ReviewPublic

Authored by gldisater_gmail.com on Aug 15 2019, 12:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 8 2024, 5:33 PM
Unknown Object (File)
Jan 6 2024, 11:55 PM
Unknown Object (File)
Dec 20 2023, 8:35 AM
Unknown Object (File)
Dec 10 2023, 7:37 PM
Unknown Object (File)
Oct 10 2023, 5:30 PM
Unknown Object (File)
Sep 6 2023, 4:31 AM
Unknown Object (File)
Aug 14 2023, 6:53 PM
Unknown Object (File)
Aug 14 2023, 8:56 AM
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

ZFS can use skein as a checksum algorithm. But is currently not using assembly implementation, only the C implementation. Removal of these lines allows ZFS to use the crypto module's assembly implementation for an improvement in ZFS performances and cpu usage when skein is the selected checksum.

Other options would be to duplicate the Makefile lines from the crypto module to include the assembly file in ZFS' compilation, or to inline the assembly implementation into the C file.

Test Plan

Created a FreeBSD bhyve with only one vcpu and copied 600GB of data into a skein enabled dataset. The scrub performance was 3h34m without this change, and 2h:43m with this change.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

I can't help with the review itself, but the least I can do for this excellent idea is to remind you that you'll want to add some reviewers so that your code can be reviewed. :)

I think you need to actually add them as reviewers?