| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |\
| |
| | |
consensus, core, tests: implement Metropolis EIP 649
|
| | | |
|
| |\ \
| | |
| | | |
core/types: encode receipt status in PostState field
|
| |/ /
| |
| |
| |
| |
| |
| | |
This fixes a regression where the new Failed field in ReceiptForStorage
rejected previously stored receipts. Fix it by removing the new field
and store status in the PostState field. This also removes massive RLP
hackery around the status field.
|
| | | |
|
| | | |
|
| |\ \
| |/
|/| |
rlp: fix decoding long strings into RawValue types
|
| |/ |
|
| |\
| |
| | |
core: add status as a consensus field in receipt
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
* cmd/evm, core/vm, internal/ethapi: Add 'err' to tracer interface CaptureEnd
* cmd/evm: fix nullpointer when there is no error
|
| |\ \
| | |
| | | |
node: fix instance dir locking and improve error message
|
| | | |
| | |
| | |
| | |
| | |
| | | |
The lock file was ineffective because opening leveldb storage in
read-only mode doesn't really take the lock. Fix it by including a
dedicated flock library (which is actually split out of goleveldb).
|
| |\ \ \
| | | |
| | | | |
metrics: change MetricsEnabledFlag to be const
|
| | | | | |
|
| |\ \ \ \
| |_|_|/
|/| | | |
tests: enable Byzantium state tests for CI
|
| |/ / / |
|
| |/ /
| |
| | |
Signed-off-by: Ti Zhou <tizhou1986@gmail.com>
|
| |\ \
| | |
| | | |
vendor: update USB HID lib (fix macOS crash)
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | | |
core/vm: implement REVERT metropolis opcode
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* whisper: Fix spelling and grammar in error
* whisper: Fix grammar in comments
|
| |\ \ \
| | | |
| | | | |
README: add missing full stop
|
| |/ / /
| | |
| | | |
missing period at line 119
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
eth: send but not announce block to peers if propagate is true
|
| | | | | |
|
| |\ \ \ \
| |_|_|/
|/| | | |
cmd/puppeth: support blacklisting malicious IPs on ethstats
|
| |/ / / |
|
| |\ \ \
| |_|/
|/| | |
core/vm, crypto/bn256: fix bn256 use and pairing corner case
|
| |/ / |
|
| |\ \
| | |
| | | |
core/vm: implement RETURNDATA metropolis opcodes
|
| | | | |
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | | |
core/vm: implement metropolis static call opcode
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* Fix STATICCALL so it is able to call precompiles too
* Fix write detection to use the correct value argument of CALL
* Fix write protection to ignore the value in CALLCODE
|
| | | | |
|
| |/ / |
|
| |\ \
| | |
| | | |
core/vm: metropolis precompiles
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | | |
* Update modexp gas calculation to new version
* Fix modexp modulo 0 special case to return zero
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
swarm: fix megacheck warnings
|
| | |/ / |
|
| |\ \ \
| |/ /
|/| | |
accounts, console, internal: support trezor hardware wallet
|
| | | | |
|
| | | | |
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
tests: update tests, use blockchain test "network" field
|
| | |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Blockchain tests now include the "network" which determines the chain
config to use. Remove config matching based on test name and share the
name-to-config index with state tests.
Byzantium/Constantinople tests are still skipped because most of them
fail anyway.
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* core: Remove unused `state` parameter to `NewEnv`.
`cfg.State` is used instead.
* core/vm/runtime: remove unused import
|
| | | | |
|
| |/ / |
|
| | |
| |
| |
| |
| |
| |
| |
| | |
* eth/downloader: fix race in downloadTesterPeer
Signed-off-by: Ivan Daniluk <ivan.daniluk@gmail.com>
* eth/downloader: minor datarace fix cleanup
|
| | |
| |
| |
| |
| |
| | |
* contracts: fix megacheck errors
* contracts: drop useless sleep, lets see what breaks
|
| | |
| |
| |
| |
| |
| | |
* les: fix megacheck warnings
* les: fixed testGetProofs
|
| | |
| |
| |
| |
| |
| | |
* whisper: fix megacheck warnings
* whisper/whisperv5: regenerate json codec to fix unused override type
|
| |\ \ |
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* crypto: fix megacheck warnings
* crypto/ecies: remove ASN.1 support
|
| |\ \ \
| | | |
| | | | |
core: fix txpool journal and test races
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* ethdb: add basic sanity test
* ethdb: test MemDatabase
* ethdb: add parallel tests
|
| | | | | |
|
| |\ \ \ \
| |/ / /
|/| | | |
eth: fix megacheck warnings
|
| |/ / / |
|
| | | | |
|
| | | | |
|
| |\ \ \
| |/ /
|/| | |
internal/build: add GoTool and document why it uses GOROOT
|
| | | | |
|
| |\ \ \
| |/ /
|/| | |
core: fix blockchain goroutine leaks in tests
|
| |/ / |
|
| |\ \
| | |
| | | |
consensus: fix megacheck warnings
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* cmd: fix megacheck warnings
* cmd: revert time.Until changes, keep readFloat
|
| |\ \ \
| | | |
| | | | |
miner: fix megacheck warnings
|
| | |/ / |
|
| |\ \ \
| | | |
| | | | |
node: fix megacheck warnings
|
| | |/ / |
|
| |\ \ \
| | | |
| | | | |
ethstats: fix megacheck warnings
|
| | |/ / |
|
| |\ \ \
| | | |
| | | | |
log: fix megacheck warnings
|
| | |/ / |
|
| |\ \ \
| | | |
| | | | |
console: fix megacheck warnings
|
| | |/ / |
|
| |\ \ \
| | | |
| | | | |
common: fix megacheck warnings
|
| | |/ / |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
* accounts: fix megacheck warnings
* accounts: don't modify abi in favor of full cleanup
|
| |\ \ \
| |/ /
|/| | |
core: bump timeout test to avoid flakyness on overloaded ci
|
| | | | |
|
| |\ \ \
| |/ /
|/| | |
build: drop yakkety builds (launchpad end of life)
|
| |/ / |
|
| |\ \
| | |
| | | |
node: fix doc typo
|
| |/ / |
|
| |\ \
| | |
| | | |
cmd/puppeth: remove wrapping loop in single reads
|
| | | | |
|
| |\ \ \
| |/ /
|/| | |
Makefile: call shell function to get pwd
|
| |/ / |
|
| | | |
|
| |\ \
| | |
| | | |
cmd: add makecache cmd, use caches during import cmd
|
| | | | |
|
| |/ / |
|
| | |
| |
| |
| | |
There is currently no simple way to obtain the network ID from a Client.
This adds a NetworkID method that wraps the net_version JSON-RPC call.
|
| | |
| |
| |
| |
| | |
Previously, NewManifest was asynchronous so subsequent code which tried
to use the returned manifest could error as the manifest was not yet
persisted.
|
| | | |
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
This patch updates the Address type in common/types.go so that the Hex
function provides an EIP55-compliant output string. The implementation is pretty lightweight;
on my laptop the benchmark gives 1100ns/op, with the majority of that value due to the Keccak hash.
|
| | | | |
|
| | | | |
|
| | |/ |
|
| | | |
|
| | | |
|
| | | |
|
| |\ \
| | |
| | | |
mobile: add RinkebyGenesis method
|
| |/ / |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* core: reduce txpool event loop goroutines and sync structs
* cmd, core, eth: journal local transactions to disk
* core: journal replacement pending transactions too
* core: separate transaction journal from pool
|
| |\ \
| | |
| | | |
trie: typo in comment
|
| | | | |
|
| |\ \ \
| |/ /
|/| | |
core/vm/runtime: fix evm command to use --gasprice flag value
|
| |/ / |
|
| | |
| |
| |
| | |
(#14806)
|
| | |
| |
| |
| | |
As per https://golang.org/s/generatedcode. This will allow other tools
such as golint to properly ignore the files.
|
| | | |
|
| | |
| |
| |
| |
| |
| | |
* node: Rename TrusterNodes
* node: Rename TrusterNodes
|
| |\ \
| | |
| | | |
cmd/faucet: ignore whitespace in gist content
|
| | | | |
|
| |\ \ \
| |/ /
|/| | |
cmd/puppeth: limit cotnainers to 10MB logs
|
| |/ / |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* mobile: don't retain transient []byte in CallMsg.SetData
Go mobile doesn't copy []byte parameters, for performance and to allow
writes to the byte array be reflected in the native byte array.
Unfortunately, that means []byte arguments are only valid during the
call it is being passed into.
CallMsg.SetData retains such a byte array. Copy it instead
Fixes #14675
* mobile: copy all []byte arguments from gomobile
To avoid subtle errors when accidentially retaining an otherwise
transient byte slice coming from gomobile, copy all byte slices before
use.
* mobile: replace copySlice with common.CopyBytes
|
| | |
| |
| |
| |
| |
| | |
* mobile: Use EIP155Signer - closes #14762
* mobile: Correctly fall back on HomesteadSigner when no chainID is passed in
|
| |\ \
| | |
| | | |
README: get rid of the non-existent disasm command
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
consensus/ethash, core: implement Metropolis EIP 100
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This PR polishes the EIP 100 difficulty adjustment algorithm
to match the same mechanisms as the Homestead was implemented
to keep the code uniform. It also avoids a few memory allocs
by reusing big1 and big2, pulling it out of the common package
and into ethash.
The commit also fixes chain maker to forward the uncle hash
when creating a simulated chain (it wasn't needed until now
so we just skipped a copy there).
|
| | | | | |
|
| | | | |
| | | |
| | | | |
Implements ethereum/EIPs#98
|
| | |_|/
|/| | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* core: remove redundant storage of transactions and receipts
* core, eth, internal: new transaction schema usage polishes
* eth: implement upgrade mechanism for db deduplication
* core, eth: drop old sequential key db upgrader
* eth: close last iterator on successful db upgrage
* core: prefix the lookup entries to make their purpose clearer
|
| | |/
|/| |
|
| | | |
|
| | | |
|
| |\ \
| | |
| | | |
dockerignore: ignore all git metadata and all tests
|
| |/ / |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Tests are now included as a submodule. This should make updating easier
and removes ~60MB of JSON data from the working copy.
State tests are replaced by General State Tests, which run the same test
with multiple fork configurations.
With the new test runner, consensus tests are run as subtests by walking
json files. Many hex issues have been fixed upstream since the last
update and most custom parsing code is replaced by existing JSON hex
types. Tests can now be marked as 'expected failures', ensuring that
fixes for those tests will trigger an update to test configuration. The
new test runner also supports parallel execution and the -short flag.
|
| | | |
|
| | |
| |
| |
| |
| | |
As stated in the documentation, this method should be called traceBlockFromFile
and not traceBlockByFile. Previously this would result in a 'The method ... does
not exist/is not available' error.
|
| |\ \
| | |
| | | |
Txpool localaccounts
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The commit reworks the transaction pool queue limitation tests
to cater for testing local accounts, also testing the nolocal flag.
In addition, it also fixes a panic if local transactions exceeded
the global queue allowance (no accounts left to drop from) and also
fixes queue eviction to operate on all accounts, not just the one
being updated.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | |/ |
|
| | | |
|
| |\ \
| | |
| | | |
Refactor downloader to use interfaces instead of callbacks
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| |\ \ \
| |_|/
|/| | |
params: remove redundant consts, disable metro on AllProtocolChanges
|
| |/ / |
|
| |\ \
| | |
| | | |
cmd/swarm: Exit if --ethapi is set
|
| |/ /
| |
| |
| |
| |
| |
| |
| | |
The previous attempt to use --ethapi as a fallback if --ens-api is not
set does not work because --ens-api has a default value, and also
setting --ens-api to "" is the suggested way to disable ENS lookups.
Signed-off-by: Lewis Marshall <lewis@lmars.net>
|
| |\ \
| | |
| | | |
cmd/swarm: Support using Mainnet for resolving ENS names
|
| | | |
| | |
| | |
| | | |
Signed-off-by: Lewis Marshall <lewis@lmars.net>
|
| |\ \ \
| |_|/
|/| | |
Fix error when reporting numer of txs in imported blocks
|
| |/ / |
|
| |\ \
| | |
| | | |
core/vm: fix overflow in gas calculation formula
|
| | | | |
|
| | | | |
|
| |/ / |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
With this commit, core/state's access to the underlying key/value database is
mediated through an interface. Database errors are tracked in StateDB and
returned by CommitTo or the new Error method.
Motivation for this change: We can remove the light client's duplicated copy of
core/state. The light client now supports node iteration, so tracing and storage
enumeration can work with the light client (not implemented in this commit).
|
| |\ \
| | |
| | | |
common/hexutil: wrap errors in json.UnmarshalTypeError
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
This adds type and struct field context to error messages.
Instead of "hex string of odd length" users will now see "json: cannot
unmarshal hex string of odd length into Go struct field SendTxArgs.from
of type common.Address".
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* accounts/abi: reorganizing package and some notes and a quick correction of name.
Signed-off-by: RJ Catalano <rj@monax.io>
get rid of some imports
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: move file names
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fix boolean decode function
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fix for the array set and for creating a bool
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: be very very very correct
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fix up error message and variable names
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: take out unnecessary argument in pack method
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: add bool unpack test and add a panic to readBool function
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fix panic message
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: change from panic to basic error
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fix nil to false
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fill out type regex tests and fill with the correct type for integers
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: move packNumbers into pack.go.
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: separation of the testing suite into appropriately named files.
Signed-off-by: RJ Catalano <rj@monax.io>
* account/abi: change to hex string tests.
Signed-off-by: RJ Catalano <rj@monax.io>
* account/abi: fix up rest of tests to hex
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: declare bool at the package level
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: use errors package in the error file.
Signed-off-by: RJ Catalano <rj@monax.io>
* accounts/abi: fix ugly hack and fix error type declaration.
Signed-off-by: RJ Catalano <rj@monax.io>
|
| | | | |
|
| |\ \ \
| |_|/
|/| | |
cmd/puppeth: fix key reuse during faucet deploys
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
whisperv5: integrate whisper and implement API
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | | |
core: remove unused events
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | | |
Demo of the issue: https://play.golang.org/p/EeTLFfppqC
|
| |\ \ \ \ \
| | | | | |
| | | | | | |
swarm/storage: remove panic on invalid chunk
|
| | | |_|/ /
| |/| | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* Makefile: add make swarm command
* Makefile: minor code formatting polishes
|
| |/ / / /
| | | |
| | | | |
README: change heading to "Go Ethereum"
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | | |
consensus/clique: choose valid votes
|
| | | | | | |
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | | |
eth: gracefully error if database cannot be opened
|
| | | | | | | |
|
| | | |_|_|/
| |/| | | |
|
| |\ \ \ \ \
| |/ / / /
|/| | | | |
travis.yml, cmd/swarm: fix Travis CI build
|
| | | | | | |
|
| |/ / / / |
|
| |\ \ \ \
| | | | |
| | | | | |
core: add testcase for txpool
|
| | | | | | |
|
| | | | | | |
|
| |\ \ \ \ \
| | | | | |
| | | | | | |
cmd/geth: corrently init gas price for CLI CPU mining
|
| | |/ / / / |
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* eth/downloader: separate state sync from queue
Scheduling of state node downloads hogged the downloader queue lock when
new requests were scheduled. This caused timeouts for other requests.
With this change, state sync is fully independent of all other downloads
and doesn't involve the queue at all.
State sync is started and checked on in processContent. This is slightly
awkward because processContent doesn't have a select loop. Instead, the
queue is closed by an auxiliary goroutine when state sync fails. We
tried several alternatives to this but settled on the current approach
because it's the least amount of change overall.
Handling of the pivot block has changed slightly: the queue previously
prevented import of pivot block receipts before the state of the pivot
block was available. In this commit, the receipt will be imported before
the state. This causes an annoyance where the pivot block is committed
as fast block head even when state downloads fail. Stay tuned for more
updates in this area ;)
* eth/downloader: remove cancelTimeout channel
* eth/downloader: retry state requests on timeout
* eth/downloader: improve comment
* eth/downloader: mark peers idle when state sync is done
* eth/downloader: move pivot block splitting to processContent
This change also ensures that pivot block receipts aren't imported
before the pivot block itself.
* eth/downloader: limit state node retries
* eth/downloader: improve state node error handling and retry check
* eth/downloader: remove maxStateNodeRetries
It fails the sync too much.
* eth/downloader: remove last use of cancelCh in statesync.go
Fixes TestDeliverHeadersHang*Fast and (hopefully)
the weird cancellation behaviour at the end of fast sync.
* eth/downloader: fix leak in runStateSync
* eth/downloader: don't run processFullSyncContent in LightSync mode
* eth/downloader: improve comments
* eth/downloader: fix vet, megacheck
* eth/downloader: remove unrequested tasks anyway
* eth/downloader, trie: various polishes around duplicate items
This commit explicitly tracks duplicate and unexpected state
delieveries done against a trie Sync structure, also adding
there to import info logs.
The commit moves the db batch used to commit trie changes one
level deeper so its flushed after every node insertion. This
is needed to avoid a lot of duplicate retrievals caused by
inconsistencies between Sync internals and database. A better
approach is to track not-yet-written states in trie.Sync and
flush on commit, but I'm focuing on correctness first now.
The commit fixes a regression around pivot block fail count.
The counter previously was reset to 1 if and only if a sync
cycle progressed (inserted at least 1 entry to the database).
The current code reset it already if a node was delivered,
which is not stong enough, because unless it ends up written
to disk, an attacker can just loop and attack ad infinitum.
The commit also fixes a regression around state deliveries
and timeouts. The old downloader tracked if a delivery is
stale (none of the deliveries were requestedt), in which
case it didn't mark the node idle and did not send further
requests, since it signals a past timeout. The current code
did mark it idle even on stale deliveries, which eventually
caused two requests to be in flight at the same time, making
the deliveries always stale and mass duplicating retrievals
between multiple peers.
* eth/downloader: fix state request leak
This commit fixes the hang seen sometimes while doing the state
sync. The cause of the hang was a rare combination of events:
request state data from peer, peer drops and reconnects almost
immediately. This caused a new download task to be assigned to
the peer, overwriting the old one still waiting for a timeout,
which in turned leaked the requests out, never to be retried.
The fix is to ensure that a task assignment moves any pending
one back into the retry queue.
The commit also fixes a regression with peer dropping due to
stalls. The current code considered a peer stalling if they
timed out delivering 1 item. However, the downloader never
requests only one, the minimum is 2 (attempt to fine tune
estimated latency/bandwidth). The fix is simply to drop if
a timeout is detected at 2 items.
Apart from the above bugfixes, the commit contains some code
polishes I made while debugging the hang.
* core, eth, trie: support batched trie sync db writes
* trie: rename SyncMemCache to syncMemBatch
|
| |\ \ \ \
| | | | |
| | | | | |
consensus/clique: fix typo and don't need to add snapshot into recents again
|
| | | |/ /
| |/| | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | | |
swarm/fuse: simplify externalUnmount, use subtests
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The code looked for /usr/bin/diskutil on darwin, but it's actually
located in /usr/sbin. Fix that by not specifying the absolute path.
Also remove weird timeout construction and extra whitespace.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This commit does various code refactorings:
- generalizes and moves the request retrieval/timeout/resend logic out of LesOdr
(will be used by a subsequent PR)
- reworks the peer management logic so that all services can register with
peerSet to get notified about added/dropped peers (also gets rid of the ugly
getAllPeers callback in requestDistributor)
- moves peerSet, LesOdr, requestDistributor and retrieveManager initialization
out of ProtocolManager because I believe they do not really belong there and the
whole init process was ugly and ad-hoc
|
| |/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
As per #14661 TransactionByHash always returns false for pending.
This uses blockNumber rather than blockHash to ensure that it returns
the correct value for pending and will not suffer side-effects if
eth_getTransactionByHash is fixed in future.
|
| |/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* ethdb: remove Set
Set deadlocks immediately and isn't part of the Database interface.
* trie: add Err to Iterator
This is useful for testing because the underlying NodeIterator doesn't
need to be kept in a separate variable just to get the error.
* trie: add LeafKey to iterator, panic when not at leaf
LeafKey is useful for callers that can't interpret Path.
* trie: retry failed seek/peek in iterator Next
Instead of failing iteration irrecoverably, make it so Next retries the
pending seek or peek every time.
Smaller changes in this commit make this easier to test:
* The iterator previously returned from Next on encountering a hash
node. This caused it to visit the same path twice.
* Path returned nibbles with terminator symbol for valueNode attached
to fullNode, but removed it for valueNode attached to shortNode. Now
the terminator is always present. This makes Path unique to each node
and simplifies Leaf.
* trie: add Path to MissingNodeError
The light client trie iterator needs to know the path of the node that's
missing so it can retrieve a proof for it. NodeIterator.Path is not
sufficient because it is updated when the node is resolved and actually
visited by the iterator.
Also remove unused fields. They were added a long time ago before we
knew which fields would be needed for the light client.
|
| |\ \ \
| | | |
| | | | |
cmd/geth: fixed a minor typo in the comments
|
| |/ / / |
|
| | | | |
|
| |\ \ \
| | | |
| | | | |
core/vm: improve opByte
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
endianness
|
| | |/ / |
|
| |\ \ \
| | | |
| | | | |
mobile: use EIP155 signer for determining sender
|
