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.

Sign up for GitHub

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

Jump to bottom

Inphi/datablobs #1

Draft
wants to merge 60 commits into
base: datablobs-base
Choose a base branch
from
Draft

Inphi/datablobs #1

wants to merge 60 commits into from

Conversation

Inphi
Copy link
Owner

@Inphi Inphi commented May 18, 2022

datablobs changes post-4f4622bc8b104fc87c5ed64ed86895fe118e1ea7

protolambda and others added 30 commits February 18, 2022 20:33
@protolambda
core/types: data blob transaction type and signer
92d7fa9
@protolambda
accounts,signer: blob tx signer support
562723e
@lightclient
params/conf: add sharding fork defn
6bc5155
@lightclient
all: add data hashes to evm msg
1d43675
@lightclient
core/vm: impl DATAHASH opcode
7c62e93
@lightclient
params: update gas per blob parameter to 120k
e552a93
@lightclient
core/vm: use correct gas price for DATAHASH
46a4b31
@lightclient
core/types/data_blob_tx: grab correct r,s values from signature
59dc43f
@lightclient
core/types/transaction_signing: use dank signer when needed
5e37223
@lightclient
core/types/transaction: grab data hashes from blob txs for message
e61cf36
@lightclient
core: add test for data blob txs
78187b8
@lightclient
all: fix intrinsic gas calls in various tests
89c1e27
@protolambda
Merge pull request ethereum#2 from lightclient/simple-blob
7a3a455 
Implement `DATAHASH` opcode and sharding fork
@protolambda
work in progress, tx wrapper for blob tx pool / journal / block build…
204c556 
…ing / engine API
@protolambda
core/types: implement data blob / kzg and lists ssz encoding
cbcdcdd
@asn-d6 @protolambda
Import go-kzg to geth
5b878b7
@asn-d6 @protolambda
Introduce dummy KZG trusted setup generator
dbfa618
@asn-d6 @protolambda
Introduce KZG crypto module (that implements the funcs from the mini-…
72eced9 
…danksharding EIP)
@asn-d6 @protolambda
Introduce unittests for KZG and the trusted setup
f83107f
@asn-d6 @protolambda
Add the trusted setup json data as a go file
f4f1230
@asn-d6 @protolambda
Introduce blobVerification precompile
9da9ab7
@asn-d6 @protolambda
Introduce blobVerification precompile test vector
b5c5515
@asn-d6 @protolambda
Introduce pointEvaluation precompile
b7db064
@asn-d6 @protolambda
Introduce pointEvaluation precompile test vector (happy case)
5e18fbd
@protolambda
eth/catalyst: GetBlobV1
cfb0440
@protolambda
go.mod,go.sum: update bls/kzg libs
b2d3882
@protolambda
go.mod,go.sum: update KZG lib
b1dc9bc
@protolambda
core/vm,crypto/kzg,tests: update data blob kzg to use From/To Compres…
ff1a74f 
…sed, and change snake_case to camelCase
@protolambda
core/types,crypto/kzg,signer: compute KZG commitment from raw blob data
21836f0
@asn-d6
Update test vectors for blobVerification/pointEvaluation
e0b5cde 
They broke when we changed blake2b to keccak in the versioned hash.
protolambda and others added 29 commits February 21, 2022 15:38
@protolambda
core/types: blob kzgs list ssz vector->list fix, fix blob tx encoding…
96bf205 
…/decoding
@protolambda
core/types: test dynamic fee and blob tx binary+json encoding
a38a121
@protolambda
core: don't reinject incomplete transactions on reorg
803f31e
@protolambda
core/types: encode/decode the block with transactions the minimal way
c971490
@protolambda @lightclient
core: consider blobs count for intrinsic gas check
d19e6ee 
Co-authored-by: lightclient <[email protected]>
@protolambda
Merge pull request ethereum#3 from asn-d6/prune-gokzg
a922276 
Prune gokzg from crypto/kzg
@protolambda
eth/catalyst: implement GetBlobsBundleV1 - EIP-4844 engine api extension
587a212
@protolambda
core/types: compute kzgs for blobs list
4b0c1cb
@protolambda
accounts,internal/ethapi: blob transaction api arguments
e9279d9
@protolambda
signer/core/apitypes: fix wrap-data NewTx option
5fa43da
@protolambda
core/types: fix marshal/unmarshal text Blob type, BLSFieldElement, KZ…
e539920 
…GCommitment
@asn-d6 @protolambda
Add functions to extract the crypto out of Blobs/KZGCommitment/BlobKzgs
184ba49
@asn-d6 @protolambda
Use optimized VerifyBlobs() function
6cd16f1
@asn-d6 @protolambda
Add unittest for VerifyBlobs() function
dafbb4d
@asn-d6 @protolambda
kzg: Rename CRS variables to match the EIP naming
0c236b0
@asn-d6 @protolambda
kzg: Document kzg code further
9f6e026
@protolambda
core/types: clean up blob and commitment parse methods, use ptr recei…
5009206 
…ver on array blob type
@protolambda
tests: blob test update, Parse blobs/commitments, fix mutation on copy
f80048e
@protolambda
params,core/types: remove duplicate kzg version param
d48a83e
@protolambda
core/vm,crypto/kzg,tests: remove versioned-hash func from crypto, app…
52abcb8 
…lication-layer detail, update tests. And make kzg tests run precompiles.
@protolambda
tests: benchmark kzg funcs
6e4051e
@protolambda
crypto/kzg: batch verification for multiple txs
db23602
@protolambda
core/types: support blobs batch-verify of multiple txs
d3ab1dd
@protolambda
core: refactor TxPool.add and .addTxsLocked to batch-verify txs
6f21779
@asn-d6
Update LatestSignerForChainID() to return a danksigner by default
bfe1f35
@asn-d6
Transaction inner size should be computed on-demand
7512b91 
We previously computed it in UnmarshalBinary() but in a wrong way (we considered the wrapped data as part of the inner
length)
@asn-d6
Include versioned hashes when creating a transaction using the JSON-R…
f87c6d6 
…PC API
@protolambda
Merge pull request ethereum#5 from asn-d6/eip4844-tooling
d6fb210 
Fix a bunch of bugs found during development of EIP-4844 tooling
@Inphi
Merge commit '4f4622bc8b104fc87c5ed64ed86895fe118e1ea7' into inphi/da…
351c452 
…tablobs
Inphi pushed a commit that referenced this pull request Jun 13, 2022
@mdehoog
Merge pull request #1 from Inphi/eip-4844-fix-blob-tx-enc
f15311b 
Use minimal transaction format as block codec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Inphi @protolambda @lightclient @asn-d6