Skip to content

feat(l2): integrate TDX as a prover #2777

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

Merged
merged 140 commits into from
May 22, 2025
Merged

feat(l2): integrate TDX as a prover #2777

merged 140 commits into from
May 22, 2025

Conversation

iovoid
Copy link
Contributor

@iovoid iovoid commented May 13, 2025

Motivation

In #2677 an example of a TDX-based prover was made. This uses the example code to add a prover.

Description

TDX is added as another prover, and made to use the same API

Copy link
Contributor

@LeanSerra LeanSerra left a comment

Choose a reason for hiding this comment

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

This PR is looking good however I think we could add to the docs the following to make it easier for people new to TDX to get up and running:

  • Detailed steps (with commands to run) on how to run the integration test in TDX capable hardware
  • Detailed steps (with an example) on how to update the RTMR/MRTD values as they change every time you rebuild the vm (mkosi vm -f).

@ManuelBilbao ManuelBilbao added the L2 Rollup client label May 20, 2025
@iovoid iovoid force-pushed the feat/tdx-integration branch from 7892adf to 438f818 Compare May 20, 2025 20:32
@iovoid iovoid added this pull request to the merge queue May 22, 2025
Merged via the queue into main with commit cdbe4c3 May 22, 2025
30 checks passed
@iovoid iovoid deleted the feat/tdx-integration branch May 22, 2025 18:08
mpaulucci added a commit that referenced this pull request May 23, 2025
**Motivation**

In #2677 an example of a TDX-based prover was made. This uses the
example code to add a prover.

**Description**

TDX is added as another prover, and made to use the same API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L2 Rollup client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants