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
F105262490: D33317.diff
Sat, Dec 14, 5:02 AM
Unknown Object (File)
Mon, Dec 2, 6:51 AM
Unknown Object (File)
Mon, Dec 2, 6:50 AM
Unknown Object (File)
Mon, Dec 2, 6:49 AM
Unknown Object (File)
Mon, Dec 2, 6:27 AM
Unknown Object (File)
Fri, Nov 22, 11:33 PM
Unknown Object (File)
Fri, Nov 15, 9:05 PM
Unknown Object (File)
Nov 8 2024, 2:26 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
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 43208
Build 40096: arc lint + arc unit

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.

911

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