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
F132285970: D27367.id79977.diff
Wed, Oct 15, 1:01 PM
F132285967: D27367.id79978.diff
Wed, Oct 15, 1:01 PM
F132237612: D27367.id.diff
Wed, Oct 15, 2:05 AM
F132237611: D27367.id79977.diff
Wed, Oct 15, 2:05 AM
F132237606: D27367.id79978.diff
Wed, Oct 15, 2:05 AM
F132198316: D27367.diff
Tue, Oct 14, 5:11 PM
Unknown Object (File)
Sat, Sep 20, 3:14 PM
Unknown Object (File)
Sep 12 2025, 3:11 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.