Differential D27367

Remove uses of CRIOGET in OCF tests after r368005.
ClosedPublic
Actions

Authored by jhb on Nov 25 2020, 1:13 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jul 4, 12:02 PM

	Unknown Object (File)
	Thu, Jun 26, 1:35 AM

	Unknown Object (File)
	Tue, Jun 24, 8:49 PM

	Unknown Object (File)
	Tue, Jun 24, 12:52 PM

	Unknown Object (File)
	Fri, Jun 20, 3:37 AM

	Unknown Object (File)
	Wed, Jun 18, 9:31 PM

	Unknown Object (File)
	Wed, Jun 18, 7:33 AM

	Unknown Object (File)
	Sun, Jun 15, 8:58 PM

View All 96 Files

Subscribers

Details

Reviewers

cem
jmg

Commits

rS368008: Remove uses of CRIOGET in OCF tests after r368005.

Test Plan

still need to actually test these

Diff Detail

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 35028
Build 32013: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Nov 25 2020, 1:13 AM

jhb created this revision.

Harbormaster completed remote builds in B35028: Diff 79977.Nov 25 2020, 1:13 AM

Passes running tests. I'm going to merge this to unbreak the build.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 25 2020, 1:31 AM

Closed by commit rS368008: Remove uses of CRIOGET in OCF tests after r368005. (authored by jhb). · Explain Why

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rS368008: Remove uses of CRIOGET in OCF tests after r368005..

Herald added a reviewer: jmg. · View Herald TranscriptNov 25 2020, 1:31 AM

Herald added a subscriber: imp. · View Herald Transcript

markj added inline comments.Nov 25 2020, 1:36 AM

tools/tools/crypto/cryptokeytest.c
116	We leak the fd now.

Lgtm.

jhb added inline comments.Nov 25 2020, 9:58 PM

tools/tools/crypto/cryptokeytest.c
116	devcrypto() allocates a single static fd for /dev/crypto that it leaks until process exit by design. This just reuses that. I went for a smaller diff than replacing ioctl() calls, though perhaps there is only one ioctl() call so just using devcrypto() directly for the ioctl() would have been fine.

markj added inline comments.Nov 25 2020, 10:00 PM

tools/tools/crypto/cryptokeytest.c
116	Sorry, I missed that devcrypto() caches the open fd. I thought we were opening /dev/crypto anew upon each call.

Revision Contents
Changeset List

Path

Size

tests/

sys/

opencrypto/

13 lines

13 lines

poly1305_test.c

13 lines

tools/

tools/

crypto/

cryptokeytest.c

5 lines

16 lines

Diff 79977

tests/sys/opencrypto/blake2_test.c

Loading...

tests/sys/opencrypto/cryptodev.py

Loading...

tests/sys/opencrypto/poly1305_test.c

Loading...

tools/tools/crypto/cryptokeytest.c

Loading...

tools/tools/crypto/cryptotest.c

Loading...