Skip to content

SYSTEM transactions in the HUB #159

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

Open
wants to merge 66 commits into
base: main
Choose a base branch
from

Conversation

OlivierBBB
Copy link
Collaborator

@OlivierBBB OlivierBBB commented Apr 17, 2025

This PR contains a major overhaul of the HUB module's low level constraints. This is required in order to properly deal with system transactions along side user transactions.

TODOs:

  • replace ABS_TX_NUM by the appropriate transaction number, likely a mixture of
    • TOTL_TRANSACTION_NUMBER
    • USER_TRANSACTION_NUMBER
  • in particular update lookups
    • the columns may have changed
    • the selectors are different now e.g. for HUB -> TXN_DATA we should now use USER ∙ TXN
  • rename TXN_DATA to USER_TXN_DATA
  • re-insert and update spec of SYST_TXN_DATA
  • add the spec for
    • TX_SKIP ∧ SYSI
      • bla ∧ TXN ∧ EIP_4788_⚜️
      • bla ∧ TXN ∧ EIP_2935_⚜️
    • TX_SKIP ∧ USER
    • TX_SKIP ∧ SYSF
    • TX_FINL (we need to finish on a CON row)

Both EIP_4788 and EIP_2935 will be of the form

TXN
ACC
STO
CON

The TX_SKIP ∧ USER case will be

TXN
ACC
ACC
ACC
CON

The TX_SKIP ∧ SYSF case will be

TXN
CON

@OlivierBBB OlivierBBB added hardfork cancun EIP's for the Cancun hardfork London to Prague London to Pectra arithmetization labels Apr 17, 2025
@OlivierBBB OlivierBBB self-assigned this Apr 17, 2025
Copy link

cla-assistant bot commented Apr 17, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
Copy link

cla-assistant bot commented Apr 17, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@OlivierBBB OlivierBBB added this to the Cancun milestone Apr 17, 2025
@OlivierBBB OlivierBBB linked an issue Apr 17, 2025 that may be closed by this pull request
@OlivierBBB OlivierBBB changed the title EIP-4788: Beacon block root in the EVM v2 Clean SYSTEM TRANSACTIONS in the HUB Apr 19, 2025
@OlivierBBB OlivierBBB changed the title SYSTEM TRANSACTIONS in the HUB SYSTEM transactions in the HUB Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arithmetization hardfork cancun EIP's for the Cancun hardfork London to Prague London to Pectra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EIP-4788: Beacon block root in the EVM
1 participant