Page MenuHomeFreeBSD
Differential D25464

Remove CRYPTO_TIMING.
ClosedPublic
Actions

Authored by markj on Jun 26 2020, 2:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 16, 3:10 AM
Unknown Object (File)
Mon, Oct 13, 11:50 AM
Unknown Object (File)
Fri, Oct 10, 2:11 PM
Unknown Object (File)
Tue, Sep 30, 4:20 AM
Unknown Object (File)
Wed, Sep 24, 9:33 AM
Unknown Object (File)
Sat, Sep 20, 1:01 PM
Unknown Object (File)
Sep 16 2025, 4:21 AM
Unknown Object (File)
Sep 16 2025, 1:34 AM
View All 100 Files
Subscribers
imp

Details

Reviewers
jhb
cem
delphij
jmg
Commits
rG16ffd822c823: Remove CRYPTO_TIMING.
rS362801: Remove CRYPTO_TIMING.
Summary

It was added a very long time ago. It is single-threaded, so only
really useful for basic measurements, and in the meantime we've gotten
some more sophisticated profiling tools.

Diff Detail

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

Event Timeline

markj requested review of this revision.Jun 26 2020, 2:53 PM
markj created this revision.
Harbormaster completed remote builds in B31973: Diff 73692.Jun 26 2020, 2:53 PM
markj added reviewers: jhb, cem, delphij.Jun 26 2020, 2:54 PM
cem accepted this revision.Jun 26 2020, 4:19 PM
This revision is now accepted and ready to land.Jun 26 2020, 4:19 PM
delphij accepted this revision.Jun 26 2020, 4:20 PM
jhb accepted this revision.Jun 26 2020, 6:32 PM
jhb added inline comments.
sys/opencrypto/cryptodev.h
329 ↗(On Diff #73692)

Can this be removed as well?

markj updated this revision to Diff 73707.Jun 26 2020, 7:19 PM
markj marked an inline comment as done.

Remove the definition of struct cryptotstat as well.

This revision now requires review to proceed.Jun 26 2020, 7:19 PM
Harbormaster completed remote builds in B31987: Diff 73707.Jun 26 2020, 7:19 PM
jhb accepted this revision.Jun 26 2020, 7:28 PM
This revision is now accepted and ready to land.Jun 26 2020, 7:28 PM
Closed by commit rS362801: Remove CRYPTO_TIMING. (authored by markj). · Explain WhyJun 30 2020, 3:57 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rS362801: Remove CRYPTO_TIMING..
Herald added a reviewer: jmg. · View Herald TranscriptJun 30 2020, 3:57 PM
Herald added a subscriber: imp. · View Herald Transcript
mjg added a commit: rG16ffd822c823: Remove CRYPTO_TIMING..Sep 16 2021, 11:22 AM

Revision Contents
Changeset List

PathSize
head/
sys/
opencrypto/
81 lines
17 lines

Diff 73908

View Options

head/sys/opencrypto/crypto.c

Loading...
View Options

head/sys/opencrypto/cryptodev.h

Loading...