Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

v1.1: Maintain sysvar balances for consistent market cap. #9942

Merged
solana-grimes merged 1 commit into
solana-labs:v1.1from
ryoqun:sysvar-balance-for-capitalization-v1.1
May 8, 2020
Merged

v1.1: Maintain sysvar balances for consistent market cap. #9942
solana-grimes merged 1 commit into
solana-labs:v1.1from
ryoqun:sysvar-balance-for-capitalization-v1.1

Conversation

@ryoqun
Copy link
Copy Markdown
Contributor

@ryoqun ryoqun commented May 8, 2020

manual back-port of #9936 without gating logics for the v1.1.x / tds

@ryoqun ryoqun requested a review from mvines May 8, 2020 15:19
@ryoqun
Copy link
Copy Markdown
Contributor Author

ryoqun commented May 8, 2020

Well, the sad fact is that there is already unknown discrepancy of capitalization in tds:

$ solana-ledger-tool --ledger tds/ capitalization
[2020-05-08T15:04:35.503514089Z INFO  solana_ledger::blockstore] Maximum open file descriptors: 65000
[2020-05-08T15:04:35.503564117Z INFO  solana_ledger::blockstore] Opening database at "/home/ryoqun/work/solana/solana/5D1f/rocksdb"
[2020-05-08T15:04:35.536950989Z INFO  solana_ledger::blockstore] "/home/ryoqun/work/solana/solana/5D1f/rocksdb" open took 33ms
[2020-05-08T15:04:35.537036937Z INFO  solana_ledger::bank_forks_utils] Initializing snapshot path: "/home/ryoqun/work/solana/solana/5D1f/snapshot"
[2020-05-08T15:04:35.541898229Z INFO  solana_ledger::bank_forks_utils] Loading snapshot package: "/home/ryoqun/work/solana/solana/5D1f/snapshot-12822928-81cubJjBXGxcgXiGzUM5nA6iRHtHfvHh3oKemjt1Biip.tar.bz2"
[2020-05-08T15:05:55.229727741Z INFO  solana_ledger::snapshot_utils] snapshot untar took 79.7s
[2020-05-08T15:05:55.230065904Z INFO  solana_ledger::snapshot_utils] snapshot version: 1.1.0
[2020-05-08T15:05:55.230174752Z INFO  solana_ledger::snapshot_utils] Loading bank from "/home/ryoqun/work/solana/solana/5D1f/snapshot/.tmpWXnvlj/snapshots/12822928/12822928"
[2020-05-08T15:05:55.336373995Z INFO  solana_ledger::snapshot_utils] Rebuilding accounts...
[2020-05-08T15:06:06.501652974Z INFO  solana_ledger::snapshot_utils] Rebuilding status cache...
[2020-05-08T15:06:06.709094295Z INFO  solana_ledger::snapshot_utils] Loaded bank for slot: 12822928
[2020-05-08T15:06:06.711176929Z INFO  solana_runtime::accounts_db] total_stores: 13820, newest_slot: 12822928, oldest_slot: 0, max_slot: 9090444 (num=22), min_slot: 11894848 (num=1)
[2020-05-08T15:06:06.711275067Z INFO  solana_metrics::metrics] metrics disabled: SOLANA_METRICS_CONFIG: environment variable not found
[2020-05-08T15:06:06.711480279Z INFO  solana_metrics::metrics] datapoint: accounts_db-stores total_count=13820i
[2020-05-08T15:06:10.486085498Z INFO  solana_runtime::accounts_db] scan took 568us merge took 1us accumulate took 453us
[2020-05-08T15:06:10.486229375Z INFO  solana_runtime::bank] bank frozen: 12822928 hash: HqiVtkpdNq1se4wGkMhKFqBpYTrovJMEVBZrjrZ2igcp accounts_delta: 6E3XZdecUQbx3M97D57yZy9tHceckfCJYr4j5UuwwCpa signature_count: 194 last_blockhash: DQnhS2caVbVrmeZrorNhwynBSDaEGTTWXPtniCJVhKZM capitalization: 999995461971326667
[2020-05-08T15:06:10.486521200Z INFO  solana_runtime::bank] accounts hash slot: 12822928 stats: BankHashStats { num_removed_accounts: 144, num_added_accounts: 0, num_lamports_stored: 63087448494382, total_data_len: 161372, num_executable_accounts: 0 }
[2020-05-08T15:06:10.486579152Z INFO  solana_ledger::snapshot_utils] bank rebuild from snapshot took 15.3s
[2020-05-08T15:06:10.494902642Z INFO  solana_ledger::blockstore_processor] processing ledger from slot 12822928...
[2020-05-08T15:06:11.817136278Z INFO  solana_ledger::blockstore_processor] ledger processed in 1320ms. 91 MB allocated. 1 fork at 12822928, with 1 frozen bank
thread 'main' panicked at 'Capitalization mismatch!?: 999995461971326667 != 999995461874499147', ledger-tool/src/main.rs:1122:25

@ryoqun ryoqun added the automerge Merge this Pull Request automatically once CI passes label May 8, 2020
@solana-grimes solana-grimes merged commit cfdc0eb into solana-labs:v1.1 May 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants