Page MenuHomeFreeBSD

tcp: Use jenkins_hash32() in hostcache
ClosedPublic

Authored by rscheff on Mar 31 2021, 4:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 12, 4:16 AM
Unknown Object (File)
Tue, Dec 3, 4:06 AM
Unknown Object (File)
Sun, Dec 1, 3:12 AM
Unknown Object (File)
Fri, Nov 29, 8:08 AM
Unknown Object (File)
Mon, Nov 25, 4:00 PM
Unknown Object (File)
Mon, Nov 25, 5:47 AM
Unknown Object (File)
Sun, Nov 24, 9:03 PM
Unknown Object (File)
Sun, Nov 24, 7:47 PM
Subscribers

Details

Summary

While investigating the reported hostcache
issues and adding a histogram to look at the
distribution of used hashbuckets, and
bucketdepth, I found that the exteremly
fast XOR "hash" would result in a rather
skewed utilization of the hostcache, for
randomly selected remote IP addresses.

As other parts of the base tcp stack (eg.
tcp fastopen) already use jenkins_hash32,
and the properties appear reasonably good,
switching to use that.

Test Plan

Anecdotal evidence (single trial, using
approximately 1000 randomly selected hosts,
overall, but ~400 for XOR and ~600 for Jenkins):

Unused Hash buckets: XOR ~51%, Jenkins ~35%
Average Bucket Depth: XOR ~2.44, Jenkins ~1.54

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ae added inline comments.
sys/netinet/tcp_hostcache.c
222

Maybe would it be better to use full IPv6 address to compute hash?

ae added inline comments.
sys/netinet/tcp_hostcache.c
222

ah, sorry for the noise :)

transport call: initialize salt randomly, and change it when purging, when triggereing purge with "2" instead of "1".

  • initialize the hash salt with a random number
  • reseed the salt when purging all entries with "2"
  • adding man entries for all hostcache related sysctls
This revision is now accepted and ready to land.Apr 8 2021, 5:58 PM
This revision now requires review to proceed.Apr 8 2021, 6:25 PM
This revision is now accepted and ready to land.Apr 8 2021, 6:31 PM
This revision was automatically updated to reflect the committed changes.