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

[zero zkEVM] - Remove post-state execution bits #1255

Closed
Nashtare opened this issue Oct 1, 2024 · 1 comment
Closed

[zero zkEVM] - Remove post-state execution bits #1255

Nashtare opened this issue Oct 1, 2024 · 1 comment
Labels
zero zkEVM Ticket specific to Polygon Zero zkEVM

Comments

@Nashtare
Copy link

Nashtare commented Oct 1, 2024

This piece of execution below is not needed for zero zkEVM in the way it handles block hashes consistency, and would need to be removed to match the expected prover state:

var l2InfoRoot *libcommon.Hash
if chainConfig.IsForkID7Etrog(blockNum) {
l2InfoRoot, err = blockinfo.BuildBlockInfoTree(
&header.Coinbase,
header.Number.Uint64(),
header.Time,
blockGasLimit,
*usedGas,
*ger,
*l1Blockhash,
*prevBlockRoot,
&txInfos,
)
if err != nil {
return nil, err
}
}
ibs.PostExecuteStateSet(chainConfig, block.NumberU64(), l2InfoRoot)

@Nashtare Nashtare added the zero zkEVM Ticket specific to Polygon Zero zkEVM label Oct 1, 2024
@mandrigin mandrigin added this to the v2.61.x (zero prover) milestone Oct 30, 2024
@rachit77
Copy link

rachit77 commented Nov 9, 2024

This PR which is already merged fixes this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
zero zkEVM Ticket specific to Polygon Zero zkEVM
Projects
None yet
Development

No branches or pull requests

4 participants