Page MenuHomeFreeBSD

security/tpm2-abrmd: Enable load of tpm driver with devd rule
AcceptedPublic

Authored by luca.piccirillo_gmail.com on Aug 7 2023, 10:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 22, 2:40 PM
Unknown Object (File)
Sep 9 2025, 7:35 AM
Unknown Object (File)
Sep 4 2025, 12:56 PM
Unknown Object (File)
Sep 1 2025, 6:46 AM
Unknown Object (File)
Aug 31 2025, 1:10 AM
Unknown Object (File)
Aug 17 2025, 6:05 PM
Unknown Object (File)
Aug 3 2025, 3:26 PM
Unknown Object (File)
Aug 2 2025, 10:11 PM

Details

Summary

Replaces https://reviews.freebsd.org/D37577

As per GitHub PR https://github.com/freebsd/freebsd-ports/pull/166

This is done (and tested) using devd rules.

It was suggested to try devfs for this but it is not suitable for two main reasons:

  1. devd allows putting the rule in a separate file only inside this port package, otherwise a patch to system devfs is not comfortable because devd rule refers to _tss user/group created during pkg install
  2. for some reasons, I made a quick test on pfSense 2.7.0 (freebsd:14-current) with devfs.rules by setting a new system default ruleset and extending the previous default ruleset with a new extra rule for tpm. It didn't work until I restarted devfs despite tpm.ko was correctly loaded during boot with tpm_load="YES". Not sure if pfSense is doing strange things with devfs or this tpm driver couldn't work with devfs

I would be happy to get help on devfs if you know how to handle the non-existing _tss user, and how to make it as portable as a package.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

This revision is now accepted and ready to land.Sep 10 2023, 12:15 AM

Could you please rebase that on top of version 3.0.0 when that will be merged and resubmit.
D41803
Thanks!

Nothing needs to be rebased outside the version numbers. PORTREVISION increased from 0 to 1.

This revision now requires review to proceed.Sep 15 2023, 11:38 AM
This revision is now accepted and ready to land.Sep 15 2023, 5:28 PM