jwasinger

Results 129 comments of jwasinger
trafficstars

whoops. pushed this to upstream instead of my personal repo.

Charts from a snap sync (LevelDB is orange, Pebble is green): ![image](https://user-images.githubusercontent.com/3411040/163356087-30b117c5-3478-46ab-a55d-829a5e49ab25.png) ![image](https://user-images.githubusercontent.com/3411040/163357374-a763bc44-3590-42c8-98bc-8ca93df5c6d1.png) ![image](https://user-images.githubusercontent.com/3411040/163357627-2bde8680-6295-4051-94aa-e45e3309ece3.png) ![image](https://user-images.githubusercontent.com/3411040/163358569-30fabaad-53df-4f59-b16d-6d4ed624e98a.png) Pebble disk usage: ``` 311.7G /datadir/geth/geth/chaindata/ancient 503.6G /datadir/geth/geth/chaindata 230.3M /datadir/geth/geth/ethash 670.1M /datadir/geth/geth/triecache 4.4M /datadir/geth/geth/nodes 504.5G...

Actually they have responded https://github.com/cockroachdb/pebble/pull/1628#pullrequestreview-1026664054 .

A note on sealing (and block import) implementation: The block coinbase field contains the vote (or is zeroed). During contract execution, it must be set to the author of the...

My point was mainly to illustrate that you need to set the `COINBASE` to a different value when executing contracts, and reset it afterwards to the vote value.

Good point. Done https://github.com/ethereum/EIPs/pull/1846

I'm not sure. I don't think it can be tested for unless we want to write unit test cases for hard-coded sentinel contract bytecodes.... If we had more detailed output...

I built a new version of the sentinel contract from source and tried to test a simple contract deployment. This bug persists. My rustc version: ``` nightly-x86_64-unknown-linux-gnu (default) rustc 1.38.0-nightly...

``` Executing message in Hera Metering (input 120 bytes)... Executing message in Hera Executing with wabt... Module failed to load. Metering done (output 0 bytes, used 9223372036854775807 gas) with code=1...