Skip to content

Conversation

@fdamato
Copy link
Contributor

@fdamato fdamato commented Nov 6, 2025

| | | [1] SHA384 digest of FMC

Caliptra does not generate an Alias<sub>FMC</sub> CSR. Owners that wish to endorse Alias<sub>FMC</sub> must do so with proprietary flows.
| | | Owner Public Key Hash
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In 1.x, we included both the owner pub key hash from fuses and from the manifest (due to the fact this does not have to be fused). Should we be including both here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This measurement should only refer to the part state, so we shouldn`t capture what comes from the Manifest, which is endorsed by the Vendor/Owner key, whose digest is rooted in the fuse (which is captured here).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot about this piece of the manifest info. Are we going to still attest to both?

Copy link
Contributor Author

@fdamato fdamato Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For consistency with the conversation we had today, I would say let’s keep it for now. We’ll file a RFC to remove ‘non-fuse’ entries in a later release.


Caliptra does not generate an Alias<sub>FMC</sub> CSR. Owners that wish to endorse Alias<sub>FMC</sub> must do so with proprietary flows.
| | | Owner Public Key Hash
| | | PK Index (ECC/LMS)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we decided today we would include this in the vendor info because it is the vendor key index

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You’re correct. It has to go in the vendor tcbinfo. I’ll fix this.

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

Successfully merging this pull request may close these issues.

4 participants