Page MenuHomeFreeBSD

Reduce contention on per-adapter lock.
ClosedPublic

Authored by jhb on Jun 25 2020, 9:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 24, 8:17 PM
Unknown Object (File)
Sun, Jul 12, 5:10 AM
Unknown Object (File)
Tue, Jul 7, 6:07 PM
Unknown Object (File)
Tue, Jul 7, 6:47 AM
Unknown Object (File)
Tue, Jul 7, 2:06 AM
Unknown Object (File)
Mon, Jul 6, 2:07 PM
Unknown Object (File)
Fri, Jul 3, 2:12 AM
Unknown Object (File)
Mon, Jun 29, 9:15 PM
Subscribers

Details

Summary
  • Move temporary sglists into the session structure and protect them with a per-session lock instead of a per-adapter lock.
  • Retire an unused session field, and move a debugging field under INVARIANTS to avoid using the session lock for completion handling when INVARIANTS isn't enabled.
  • Use counter_u64 for per-adapter statistics.

Note that this helps for cases where multiple sessions are used
(e.g. multiple IPsec SAs or multiple KTLS connections). It does not
help for workloads that use a single session (e.g. a single GELI
volume).

Test Plan
  • tested with KTLS RX where it made a marginal perf improvement (29 Gbps to 30-31 Gbps). However, in pmcstat callchains, lock_delay due to lock contention went from 6% of overall time (of which 82% was for the adapter lock in ccr_process) to 3% of overall time (of which 3% was for the per-session lock in ccr_process)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable