Page MenuHomeFreeBSD

bhyve/tpm: create crb thread for sending tpm commands
ClosedPublic

Authored by corvink on Jun 7 2023, 12:05 PM.
Tags
None
Referenced Files
F107017986: D40458.id122925.diff
Thu, Jan 9, 1:17 AM
Unknown Object (File)
Sat, Dec 28, 9:25 AM
Unknown Object (File)
Mon, Dec 16, 3:48 PM
Unknown Object (File)
Dec 10 2024, 6:53 AM
Unknown Object (File)
Nov 14 2024, 12:40 AM
Unknown Object (File)
Nov 5 2024, 2:41 PM
Unknown Object (File)
Oct 22 2024, 1:28 PM
Unknown Object (File)
Oct 16 2024, 9:51 PM
Subscribers

Details

Summary

Commands send to a tpm are very slow. They can take up to several
seconds for completion. For that reason, create a thread which issues
the commands to the tpm device.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 52135
Build 49026: arc lint + arc unit

Event Timeline

usr.sbin/bhyve/tpm_intf_crb.c
214
296

The mutex and condvar don't get destroyed anywhere.

310

Why not just write pthread_set_name_np(crb->thread, "tpm_intf_crb");?

331

Don't you need to wake up the thread here?

  • destroy cond and mutex
  • wakeup thread on destroy
markj added inline comments.
usr.sbin/bhyve/tpm_intf_crb.c
292

warn/warnx/warc/err/errx/errc always append a newline.

292

pthread_* returns an error number upon failure, so you can get a better error message with warnc(error, "%s: failed to init mutex", __func__);. Same below.

This revision is now accepted and ready to land.Jun 19 2023, 7:36 PM