Differential D27068

Split logic to create new sessions into a separate function.
ClosedPublic
Actions

Authored by jhb on Nov 2 2020, 8:50 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Apr 27, 6:19 PM

	Unknown Object (File)
	Sat, Apr 27, 6:11 PM

	Unknown Object (File)
	Sat, Apr 27, 6:10 PM

	Unknown Object (File)
	Sat, Apr 27, 4:41 PM

	Unknown Object (File)
	Wed, Apr 24, 8:52 AM

	Unknown Object (File)
	Wed, Apr 24, 8:50 AM

	Unknown Object (File)
	Wed, Apr 24, 7:53 AM

	Unknown Object (File)
	Dec 23 2023, 1:51 AM

View All 29 Files

Subscribers

Details

Reviewers

markj
jmg

Commits

rS367409: Split logic to create new sessions into a separate function.

Summary

This simplifies cryptof_ioctl as it now a wrapper around functions that
contain the bulk of the per-ioctl logic.

Diff Detail

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 34583
Build 31674: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Nov 2 2020, 8:50 PM

jhb created this revision.

Harbormaster completed remote builds in B34583: Diff 79098.Nov 2 2020, 8:50 PM

jhb added a parent revision: D27066: Style fixes for function prototypes and definitions..Nov 2 2020, 8:59 PM

jhb added a child revision: D27069: Move cryptof_ioctl() below the routines it calls..

jhb added a child revision: D27070: Group session management routines together before first use..Nov 2 2020, 9:03 PM

markj accepted this revision.Nov 2 2020, 10:09 PM

markj added inline comments.

sys/opencrypto/cryptodev.c
431	At times I've wanted a CRYPTO_NULL or so for this case.
716	Is it worth keeping ARGSUSED and similar linter annotations? I usually delete them when I come across them.

This revision is now accepted and ready to land.Nov 2 2020, 10:09 PM

jhb added inline comments.Nov 5 2020, 11:46 PM

sys/opencrypto/cryptodev.c
431	Yeah. Unfortunately, CRYPTO_NULL_HMAC and CRYPTO_NULL_CBC get in the way. Really we should kill those though. I feel like IPsec, etc. should handle "null" ciphers explicitly. NULL_HMAC is especially egregious as it writes out a 96-bit zero hash that is IPsec-specific.
716	Hmm, probably not. I'll kill them in things I move.

Closed by commit rS367409: Split logic to create new sessions into a separate function. (authored by jhb). · Explain WhyNov 6 2020, 12:11 AM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS367409: Split logic to create new sessions into a separate function..

Herald added a reviewer: jmg. · View Herald TranscriptNov 6 2020, 12:11 AM

Herald added a subscriber: imp. · View Herald Transcript

Trimmed ARGSUSED in the followup commit that moves the function.

Revision Contents
Changeset List

Path

Size

sys/

opencrypto/

603 lines

Diff 79098

sys/opencrypto/cryptodev.c

Loading...