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)
Thu, Apr 18, 2:08 AM
Unknown Object (File)
Feb 28 2024, 9:48 PM
Unknown Object (File)
Feb 20 2024, 2:40 PM
Unknown Object (File)
Feb 18 2024, 11:37 PM
Unknown Object (File)
Dec 23 2023, 3:17 AM
Unknown Object (File)
Sep 10 2023, 5:36 PM
Unknown Object (File)
Sep 8 2023, 7:11 PM
Unknown Object (File)
Jun 29 2023, 3:00 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