Page MenuHomeFreeBSD

cryptosoft: Stop single-threading requests within a session.
ClosedPublic

Authored by jhb on Dec 7 2021, 12:37 AM.
Tags
None
Referenced Files
F86930593: D33317.diff
Thu, Jun 27, 12:44 PM
Unknown Object (File)
Apr 14 2024, 2:53 AM
Unknown Object (File)
Jan 30 2024, 2:43 PM
Unknown Object (File)
Jan 17 2024, 7:31 AM
Unknown Object (File)
Dec 30 2023, 6:20 PM
Unknown Object (File)
Dec 22 2023, 3:16 PM
Unknown Object (File)
Dec 20 2023, 3:29 AM
Unknown Object (File)
Nov 30 2023, 5:10 AM
Subscribers

Details

Summary

All of the request handlers no longer modify session state, so remove
the mutex limiting operations to one per session. In addition, change
the pointer to the session state passed to process callbacks to const.

Suggested by: mjg
Sponsored by: The FreeBSD Foundation

Diff Detail

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

Event Timeline

jhb requested review of this revision.Dec 7 2021, 12:37 AM
markj added inline comments.
sys/opencrypto/cryptosoft.c
79

No need to break the line, I think.

910

I think this can stay as one line.

This revision is now accepted and ready to land.Dec 7 2021, 1:18 AM
sys/opencrypto/cryptosoft.c
79

I think it crossed 80 cols in my editor, but I'll double check.

Thank you!

Is this mfcable to stable/12 by any chance?

In D33317#754176, @mjg wrote:

Thank you!

Is this mfcable to stable/12 by any chance?

Hmmm, I can maybe see about merging to 13. OCF in 12 is quite a bit different. It could perhaps be reimplemented there, but there might be quite a few conflicts.

In that case I would not bother.

jhb marked 2 inline comments as done.Dec 9 2021, 7:28 PM