Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add kernel module(s) for TPM 2.0 #312

Open
sruehl opened this issue May 25, 2023 · 10 comments
Open

Add kernel module(s) for TPM 2.0 #312

sruehl opened this issue May 25, 2023 · 10 comments

Comments

@sruehl
Copy link

sruehl commented May 25, 2023

Copy&Paste from balena-os/balena-intel#218

We would like to have TPM 2.0 support in Balena on the Compulab Imx8 platform.

Use Case

We're using TPM 2.0 modules to sign tokens on the edge to identify edge devices to our cloud tier. Using the TPM is more secure than storing certs on the device because the private key never leaves the TPM.

Solution

See balena-os/balena-intel#218

Expected Interfaces

When TPM 2.0 support is present, we expect to see the following files:

/dev/tpm0
/dev/tpmrm
@sruehl
Copy link
Author

sruehl commented May 26, 2023

@vicgal can you help with this?

alexgg added a commit that referenced this issue May 30, 2023
When TPM support enabled the following devices are available:
/dev/tpm0
/dev/tpmrm0

Fixes: #312

Changelog-entry: Add TPM kernel module support
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg
Copy link
Contributor

alexgg commented May 30, 2023

hey @sruehl I have opened a PR - once it's built we will do some basic checks.

I don't think we have automated tests for this device type - we will have to add it to the automation so it can be released as we no longer have manual test processes.

alexgg added a commit that referenced this issue May 31, 2023
When TPM support enabled the following devices are available:
/dev/tpm0
/dev/tpmrm0

Fixes: #312

Changelog-entry: Add TPM kernel module support
Signed-off-by: Alex Gonzalez <[email protected]>
@acostach
Copy link
Contributor

acostach commented Jun 1, 2023

Hi @sruehl , do the /dev/tpm* nodes show up on your board with the reference Yocto Image from July 19, 2022 ? Note that you'll also have to flash the reference u-boot.

With that image on our iot-gate-imx8 unit there are no /dev/tpm nodes and the related device-tree nodes are disabled:

root@iot-gate-imx8:~# cat /sys/firmware/devicetree/base/soc@0/bus@30800000/spi@30830000/tpm@0/status
disabled
root@iot-gate-imx8:~# cat /sys/firmware/devicetree/base/soc@0/bus@30800000/spi@30840000/tpm@0/status
disabled

and looking at https://github.com/compulab-yokneam/meta-bsp-imx8mm/blob/d0846f32e434bb280b16ab01f04b70163fe81180/recipes-kernel/linux/compulab/imx8mm/0062-iot-gate-imx8-add-support-for-the-IE-TPM-module.patch it seems that they are not enabled by u-boot because the TPM module is not detected. Perhaps it's offered as an add-on board? Our unit is 4GB RAM | 32GB eMMC| WiFi + BT | Modem | FARS2 | FBRS2 | FCDIO | TET

@sruehl
Copy link
Author

sruehl commented Jun 1, 2023

Will check later...
In the meantime: @vraevsky do you have any idea why it would not visible?
On the product page for imx8 it says "TPM 2.0, implemented with Infineon SLB9670"

@sruehl
Copy link
Author

sruehl commented Jun 1, 2023

Oh I noticed you need the FATPM feature code to be able to use it... Need to check with my device if that is present

@sruehl
Copy link
Author

sruehl commented Jun 1, 2023

Ok my device says 4GB RAM | 32GB eMMC | WIFI+BT | Modem | FARS4 | FBCAN | TIC. So that would mean my device doesn't have the required feature for a TPM to be present :(.
That means only someone with FATPM could test that or someone on compulab. I opened a support Ticket with them to help out.

@acostach
Copy link
Contributor

acostach commented Jun 1, 2023

Thanks for the update @sruehl, please keep us posted on the testing progress. If you can get your hands on a unit with FATPM please do a local yocto build of the PR my colleague Alex raised and let us know if the enabled configs are sufficient or if we need to enable others for this particular device-type.

acostach pushed a commit that referenced this issue Jun 5, 2023
When TPM support enabled the following devices are available:
/dev/tpm0
/dev/tpmrm0

Fixes: #312

Changelog-entry: Add TPM kernel module support
Signed-off-by: Alex Gonzalez <[email protected]>
acostach pushed a commit that referenced this issue Jun 14, 2023
When TPM support enabled the following devices are available:
/dev/tpm0
/dev/tpmrm0

Fixes: #312

Changelog-entry: Add TPM kernel module support
Signed-off-by: Alex Gonzalez <[email protected]>
acostach pushed a commit that referenced this issue Jun 15, 2023
When TPM support enabled the following devices are available:
/dev/tpm0
/dev/tpmrm0

Fixes: #312

Changelog-entry: Add TPM kernel module support
Signed-off-by: Alex Gonzalez <[email protected]>
@otlaitil
Copy link

Hi, any update on this @acostach @sruehl ? I will be receiving https://shop-compulab.com/product/iot-gate-imx8plus-evaluation-kit/ shortly and can potentially do some testing.

Would this also enable secure boot and disk encryption for the device at some point?

@acostach
Copy link
Contributor

acostach commented Jul 29, 2024

I'll let @sruehl chip in if he got his hands on that specific hw configuration

@sruehl
Copy link
Author

sruehl commented Jul 30, 2024

@acostach no, sadly I didn't get my hands on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants