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
F115668354: D40458.diff
Sat, Apr 26, 8:40 PM
Unknown Object (File)
Thu, Apr 24, 1:14 AM
Unknown Object (File)
Mon, Apr 21, 10:42 PM
Unknown Object (File)
Mon, Apr 21, 8:59 PM
Unknown Object (File)
Mon, Apr 21, 12:07 PM
Unknown Object (File)
Mon, Apr 14, 9:38 AM
Unknown Object (File)
Mon, Apr 14, 5:13 AM
Unknown Object (File)
Mon, Apr 14, 1:33 AM
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 51950
Build 48841: 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");?

334

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