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
Unknown Object (File)
Mon, Jun 30, 5:15 AM
Unknown Object (File)
Thu, Jun 26, 3:56 PM
Unknown Object (File)
Sat, Jun 21, 12:32 AM
Unknown Object (File)
Fri, Jun 20, 9:56 PM
Unknown Object (File)
Tue, Jun 17, 6:17 PM
Unknown Object (File)
Mon, Jun 16, 12:25 PM
Unknown Object (File)
Mon, Jun 16, 5:48 AM
Unknown Object (File)
Sat, Jun 14, 1:25 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 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