Skip to content

[feat] Add Stable Owner Key derivation from HEK seed#3625

Merged
mhatrevi merged 11 commits into
chipsalliance:mainfrom
mhatrevi:vmhatre/stable-owner-key
May 6, 2026
Merged

[feat] Add Stable Owner Key derivation from HEK seed#3625
mhatrevi merged 11 commits into
chipsalliance:mainfrom
mhatrevi:vmhatre/stable-owner-key

Conversation

@mhatrevi
Copy link
Copy Markdown
Collaborator

ROM derives a Stable Owner Root Key from HEK seed via HKDF into KV15 during cold boot. CM_DERIVE_STABLE_KEY gains OwnerKey (0x3) which uses CMAC-KDF + HMAC-KDF with label "Stable Owner Key" to produce child keys as encrypted CMKs.

Both ROM firmware processor and runtime handle the new key type with identical derivation logic. Adds integration tests for all three key types (IDevId, LDevId, OwnerKey) and a separate test verifying different personalization seeds produce different keys.

ROM derives a Stable Owner Root Key from HEK seed via HKDF into KV15
during cold boot. CM_DERIVE_STABLE_KEY gains OwnerKey (0x3) which uses
CMAC-KDF + HMAC-KDF with label "Stable Owner Key" to produce child
keys as encrypted CMKs.

Both ROM firmware processor and runtime handle the new key type with
identical derivation logic. Adds integration tests for all three key
types (IDevId, LDevId, OwnerKey) and a separate test verifying
different personalization seeds produce different keys.
@mhatrevi
Copy link
Copy Markdown
Collaborator Author

mhatrevi commented Apr 16, 2026

Implements chipsalliance/Caliptra#657

swenson
swenson previously approved these changes Apr 16, 2026
Copy link
Copy Markdown
Collaborator

@swenson swenson left a comment

Choose a reason for hiding this comment

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

LGTM, one nit

Comment thread rom/dev/src/flow/cold_reset/idev_id.rs Outdated
Comment thread rom/dev/src/flow/cold_reset/idev_id.rs
Comment thread rom/dev/src/flow/cold_reset/idev_id.rs Outdated
Comment thread runtime/src/cryptographic_mailbox.rs
Comment thread rom/dev/src/flow/cold_reset/idev_id.rs Outdated
Comment thread runtime/src/cryptographic_mailbox.rs
Comment thread rom/dev/src/flow/cold_reset/idev_id.rs Outdated
@mhatrevi mhatrevi requested a review from jlmahowa-amd as a code owner April 24, 2026 00:52
Comment thread runtime/src/cryptographic_mailbox.rs Outdated
swenson
swenson previously approved these changes May 5, 2026
Copy link
Copy Markdown
Collaborator

@swenson swenson left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread drivers/src/soc_ifc.rs
Copy link
Copy Markdown
Collaborator

@swenson swenson left a comment

Choose a reason for hiding this comment

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

LGTM

@mhatrevi mhatrevi added this pull request to the merge queue May 6, 2026
Merged via the queue into chipsalliance:main with commit 24205a8 May 6, 2026
25 checks passed
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.

5 participants