Page MenuHomeFreeBSD

Remove uses of CRIOGET in OCF tests after r368005.
ClosedPublic

Authored by jhb on Nov 25 2020, 1:13 AM.
Tags
None
Referenced Files
F102548879: D27367.diff
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
Subscribers

Details

Test Plan
  • still need to actually test these

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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

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
This revision was automatically updated to reflect the committed changes.
tools/tools/crypto/cryptokeytest.c
116 ↗(On Diff #79977)

We leak the fd now.

tools/tools/crypto/cryptokeytest.c
116 ↗(On Diff #79977)

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.

tools/tools/crypto/cryptokeytest.c
116 ↗(On Diff #79977)

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