Page MenuHomeFreeBSD

[NEW PORT] security/highwayhash: Fast strong hash functions: SipHash/HighwayHash
ClosedPublic

Authored by ultima on Aug 12 2017, 4:32 AM.
Tags
None
Referenced Files
F102926327: D12002.id.diff
Mon, Nov 18, 8:15 PM
F102913162: D12002.diff
Mon, Nov 18, 4:29 PM
Unknown Object (File)
Mon, Nov 18, 7:56 AM
Unknown Object (File)
Mon, Nov 18, 7:49 AM
Unknown Object (File)
Mon, Nov 18, 7:28 AM
Unknown Object (File)
Tue, Oct 29, 4:27 AM
Unknown Object (File)
Tue, Oct 22, 12:54 PM
Unknown Object (File)
Oct 19 2024, 9:49 AM
Subscribers

Details

Summary

Strong (well-distributed and unpredictable) hashes:

  • Portable implementation of SipHash
  • HighwayHash, a 5x faster SIMD hash with security claims

WWW: https://github.com/google/highwayhash

PR\: 219232
Submitted by\: Yuri Victorovich (maintainer)
Reviewed by\: lifanov (mentor), matthew (mentor)
Approved by\: lifanov (mentor), matthew (mentor)
Differential Revision\: https://reviews.freebsd.org/DXXXXX

Test Plan

portlint:
looks fine.

poudriere:
103i386: is only for amd64 arm64 powerpc powerpc64, while you are running i386 (reason: Assembly is implemented only for specific architectures in highwayhash/tsc_timer.h)
103amd64
110i386: is only for amd64 arm64 powerpc powerpc64, while you are running i386 (reason: Assembly is implemented only for specific architectures in highwayhash/tsc_timer.h)
110amd64
12i386: is only for amd64 arm64 powerpc powerpc64, while you are running i386 (reason: Assembly is implemented only for specific architectures in highwayhash/tsc_timer.h)
12amd64

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

security/highwayhash/Makefile
27–33 ↗(On Diff #31976)

This is a bit messy, the maintainer has requested upstream to fix.

security/highwayhash/Makefile
27–33 ↗(On Diff #31976)

It's good when port maintainers upstream FreeBSD related changes -- here it would be good to get the content from patch-highwayhash_os__specific.cc upstreamed as well. If a change has been submitted upstream, mentioning that in the commit message is a good thing to do.

However, none of that should hold up getting the update into the tree. Showing that thousands of people are already using a patch is very persuasive to the upstream projects.

  • Added comment and link to upstream issue
This revision is now accepted and ready to land.Aug 13 2017, 12:37 PM
This revision was automatically updated to reflect the committed changes.