Page MenuHomeFreeBSD

ktls: Avoid wakeups and locking for synchronous callbacks
ClosedPublic

Authored by markj on Jan 16 2021, 6:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 11, 3:23 AM
Unknown Object (File)
Mon, Mar 11, 3:19 AM
Unknown Object (File)
Mon, Mar 11, 3:19 AM
Unknown Object (File)
Mon, Mar 11, 2:54 AM
Unknown Object (File)
Feb 12 2024, 12:13 AM
Unknown Object (File)
Jan 10 2024, 12:52 PM
Unknown Object (File)
Dec 22 2023, 10:02 PM
Unknown Object (File)
Dec 14 2023, 8:16 PM
Subscribers

Details

Summary

The KTLS crypto callback always locks the session to deliver a wakeup.
But if we're handling the operation synchronously this is wasted effort.
On an Ampere server this results in a large amount of sleepqueue lock
contention. Use CRYPTO_SESS_SYNC() to determine whether the operation
will be completed asynchronously or not, and select a callback
appropriately.

Diff Detail

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

Event Timeline

markj added reviewers: jhb, gallatin.
jhb added inline comments.
sys/opencrypto/ktls_ocf.c
141

These two statements can be moved up out of the loop.

This revision is now accepted and ready to land.Jan 19 2021, 7:48 PM
sys/opencrypto/ktls_ocf.c
141

I had written it that way originally, but I noticed that we handle EAGAIN (i.e., the crypto driver changed) by looping, in which case we should re-evaluate the flag. Does that not make sense for some reason?

sys/opencrypto/ktls_ocf.c
141

Oh, humm, ok. I think long term I want crypto_dispatch() to return EINPROGRESS when it doesn't complete the request synchronously, but this is fine for now.