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)
	Wed, Nov 13, 10:13 PM

	Unknown Object (File)
	Mon, Nov 11, 5:17 PM

	Unknown Object (File)
	Thu, Oct 31, 11:45 AM

	Unknown Object (File)
	Thu, Oct 24, 8:55 PM

	Unknown Object (File)
	Oct 3 2024, 8:29 PM

	Unknown Object (File)
	Oct 1 2024, 5:30 PM

	Unknown Object (File)
	Oct 1 2024, 2:32 PM

	Unknown Object (File)
	Sep 27 2024, 5:56 AM

View All 61 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...