2021-07-09 12:55:31 -07:00
|
|
|
//! `cost_tracker` keeps tracking transaction cost per chained accounts as well as for entire block
|
2021-07-06 08:41:25 -07:00
|
|
|
//! The main functions are:
|
2021-10-19 12:37:33 -07:00
|
|
|
//! - would_transaction_fit(&tx_cost), immutable function to test if tx with tx_cost would fit into current block
|
|
|
|
//! - add_transaction_cost(&tx_cost), mutable function to accumulate tx_cost to tracker.
|
2021-06-01 07:16:17 -07:00
|
|
|
//!
|
2021-12-03 09:00:31 -08:00
|
|
|
use {
|
|
|
|
crate::{block_cost_limits::*, cost_model::TransactionCost},
|
|
|
|
solana_sdk::{clock::Slot, pubkey::Pubkey, transaction::SanitizedTransaction},
|
|
|
|
std::collections::HashMap,
|
|
|
|
};
|
2021-06-01 07:16:17 -07:00
|
|
|
|
2021-06-28 19:34:04 -07:00
|
|
|
const WRITABLE_ACCOUNTS_PER_BLOCK: usize = 512;
|
|
|
|
|
2022-03-13 08:58:57 -07:00
|
|
|
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
|
2021-10-19 12:37:33 -07:00
|
|
|
pub enum CostTrackerError {
|
|
|
|
/// would exceed block max limit
|
|
|
|
WouldExceedBlockMaxLimit,
|
|
|
|
|
2022-01-12 21:27:19 -08:00
|
|
|
/// would exceed vote max limit
|
|
|
|
WouldExceedVoteMaxLimit,
|
|
|
|
|
2021-10-19 12:37:33 -07:00
|
|
|
/// would exceed account max limit
|
|
|
|
WouldExceedAccountMaxLimit,
|
2021-12-12 12:57:18 -08:00
|
|
|
|
2022-03-13 08:58:57 -07:00
|
|
|
/// would exceed account data block limit
|
|
|
|
WouldExceedAccountDataBlockLimit,
|
|
|
|
|
|
|
|
/// would exceed account data total limit
|
|
|
|
WouldExceedAccountDataTotalLimit,
|
2021-10-19 12:37:33 -07:00
|
|
|
}
|
|
|
|
|
2021-10-12 06:51:33 -07:00
|
|
|
#[derive(AbiExample, Debug)]
|
2021-06-01 07:16:17 -07:00
|
|
|
pub struct CostTracker {
|
2021-06-09 15:10:59 -07:00
|
|
|
account_cost_limit: u64,
|
|
|
|
block_cost_limit: u64,
|
2022-01-12 21:27:19 -08:00
|
|
|
vote_cost_limit: u64,
|
2021-06-09 15:10:59 -07:00
|
|
|
cost_by_writable_accounts: HashMap<Pubkey, u64>,
|
|
|
|
block_cost: u64,
|
2022-01-12 21:27:19 -08:00
|
|
|
vote_cost: u64,
|
2021-10-24 20:19:23 -07:00
|
|
|
transaction_count: u64,
|
2021-12-12 12:57:18 -08:00
|
|
|
account_data_size: u64,
|
2022-03-13 08:58:57 -07:00
|
|
|
|
|
|
|
/// The amount of total account data size remaining. If `Some`, then do not add transactions
|
|
|
|
/// that would cause `account_data_size` to exceed this limit.
|
|
|
|
account_data_size_limit: Option<u64>,
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
2021-10-12 06:51:33 -07:00
|
|
|
impl Default for CostTracker {
|
|
|
|
fn default() -> Self {
|
2022-02-22 08:44:23 -08:00
|
|
|
// Clippy doesn't like asserts in const contexts, so need to explicitly allow them. For
|
|
|
|
// more info, see this issue: https://github.com/rust-lang/rust-clippy/issues/8159
|
|
|
|
#![allow(clippy::assertions_on_constants)]
|
|
|
|
const _: () = assert!(MAX_WRITABLE_ACCOUNT_UNITS <= MAX_BLOCK_UNITS);
|
|
|
|
const _: () = assert!(MAX_VOTE_UNITS <= MAX_BLOCK_UNITS);
|
2021-10-12 06:51:33 -07:00
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
Self {
|
2022-02-22 08:44:23 -08:00
|
|
|
account_cost_limit: MAX_WRITABLE_ACCOUNT_UNITS,
|
|
|
|
block_cost_limit: MAX_BLOCK_UNITS,
|
|
|
|
vote_cost_limit: MAX_VOTE_UNITS,
|
2021-06-28 19:34:04 -07:00
|
|
|
cost_by_writable_accounts: HashMap::with_capacity(WRITABLE_ACCOUNTS_PER_BLOCK),
|
2021-06-01 07:16:17 -07:00
|
|
|
block_cost: 0,
|
2022-01-12 21:27:19 -08:00
|
|
|
vote_cost: 0,
|
2021-10-24 20:19:23 -07:00
|
|
|
transaction_count: 0,
|
2021-12-12 12:57:18 -08:00
|
|
|
account_data_size: 0,
|
2022-03-13 08:58:57 -07:00
|
|
|
account_data_size_limit: None,
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
}
|
2022-02-22 08:44:23 -08:00
|
|
|
}
|
2021-06-01 07:16:17 -07:00
|
|
|
|
2022-02-22 08:44:23 -08:00
|
|
|
impl CostTracker {
|
2022-03-13 08:58:57 -07:00
|
|
|
/// Construct and new CostTracker and set the account data size limit.
|
|
|
|
#[must_use]
|
|
|
|
pub fn new_with_account_data_size_limit(account_data_size_limit: Option<u64>) -> Self {
|
|
|
|
Self {
|
|
|
|
account_data_size_limit,
|
|
|
|
..Self::default()
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-10-19 12:37:33 -07:00
|
|
|
// bench tests needs to reset limits
|
2022-01-12 21:27:19 -08:00
|
|
|
pub fn set_limits(
|
|
|
|
&mut self,
|
|
|
|
account_cost_limit: u64,
|
|
|
|
block_cost_limit: u64,
|
|
|
|
vote_cost_limit: u64,
|
|
|
|
) {
|
2021-10-19 12:37:33 -07:00
|
|
|
self.account_cost_limit = account_cost_limit;
|
|
|
|
self.block_cost_limit = block_cost_limit;
|
2022-01-12 21:27:19 -08:00
|
|
|
self.vote_cost_limit = vote_cost_limit;
|
2021-10-19 12:37:33 -07:00
|
|
|
}
|
|
|
|
|
2021-07-15 20:51:27 -07:00
|
|
|
pub fn would_transaction_fit(
|
|
|
|
&self,
|
2022-01-12 21:27:19 -08:00
|
|
|
transaction: &SanitizedTransaction,
|
2021-10-19 12:37:33 -07:00
|
|
|
tx_cost: &TransactionCost,
|
|
|
|
) -> Result<(), CostTrackerError> {
|
2021-12-12 12:57:18 -08:00
|
|
|
self.would_fit(
|
|
|
|
&tx_cost.writable_accounts,
|
|
|
|
tx_cost.sum(),
|
|
|
|
tx_cost.account_data_size,
|
2022-01-12 21:27:19 -08:00
|
|
|
transaction,
|
2021-12-12 12:57:18 -08:00
|
|
|
)
|
2021-07-06 08:41:25 -07:00
|
|
|
}
|
|
|
|
|
2021-09-03 20:05:30 -07:00
|
|
|
pub fn add_transaction_cost(
|
|
|
|
&mut self,
|
2022-01-12 21:27:19 -08:00
|
|
|
transaction: &SanitizedTransaction,
|
2021-10-19 12:37:33 -07:00
|
|
|
tx_cost: &TransactionCost,
|
2021-09-03 20:05:30 -07:00
|
|
|
) {
|
2021-12-12 12:57:18 -08:00
|
|
|
self.add_transaction(
|
|
|
|
&tx_cost.writable_accounts,
|
|
|
|
tx_cost.sum(),
|
|
|
|
tx_cost.account_data_size,
|
2022-01-12 21:27:19 -08:00
|
|
|
transaction,
|
2021-12-12 12:57:18 -08:00
|
|
|
);
|
2021-07-06 08:41:25 -07:00
|
|
|
}
|
|
|
|
|
2021-10-05 06:57:39 -07:00
|
|
|
pub fn try_add(
|
|
|
|
&mut self,
|
2022-01-12 21:27:19 -08:00
|
|
|
transaction: &SanitizedTransaction,
|
2021-10-19 12:37:33 -07:00
|
|
|
tx_cost: &TransactionCost,
|
|
|
|
) -> Result<u64, CostTrackerError> {
|
2022-01-12 21:27:19 -08:00
|
|
|
let cost = tx_cost.sum();
|
|
|
|
self.would_fit(
|
|
|
|
&tx_cost.writable_accounts,
|
|
|
|
cost,
|
|
|
|
tx_cost.account_data_size,
|
|
|
|
transaction,
|
|
|
|
)?;
|
|
|
|
self.add_transaction(
|
|
|
|
&tx_cost.writable_accounts,
|
|
|
|
cost,
|
|
|
|
tx_cost.account_data_size,
|
|
|
|
transaction,
|
|
|
|
);
|
2021-06-01 07:16:17 -07:00
|
|
|
Ok(self.block_cost)
|
|
|
|
}
|
|
|
|
|
2021-10-24 20:19:23 -07:00
|
|
|
pub fn report_stats(&self, bank_slot: Slot) {
|
|
|
|
// skip reporting if block is empty
|
|
|
|
if self.transaction_count == 0 {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
let (costliest_account, costliest_account_cost) = self.find_costliest_account();
|
|
|
|
|
|
|
|
datapoint_info!(
|
|
|
|
"cost_tracker_stats",
|
|
|
|
("bank_slot", bank_slot as i64, i64),
|
|
|
|
("block_cost", self.block_cost as i64, i64),
|
2022-01-12 21:27:19 -08:00
|
|
|
("vote_cost", self.vote_cost as i64, i64),
|
2021-10-24 20:19:23 -07:00
|
|
|
("transaction_count", self.transaction_count as i64, i64),
|
|
|
|
(
|
|
|
|
"number_of_accounts",
|
|
|
|
self.cost_by_writable_accounts.len() as i64,
|
|
|
|
i64
|
|
|
|
),
|
|
|
|
("costliest_account", costliest_account.to_string(), String),
|
|
|
|
("costliest_account_cost", costliest_account_cost as i64, i64),
|
2021-12-12 12:57:18 -08:00
|
|
|
("account_data_size", self.account_data_size, i64),
|
2021-10-24 20:19:23 -07:00
|
|
|
);
|
|
|
|
}
|
2021-10-05 06:57:39 -07:00
|
|
|
|
2021-10-24 20:19:23 -07:00
|
|
|
fn find_costliest_account(&self) -> (Pubkey, u64) {
|
|
|
|
let mut costliest_account = Pubkey::default();
|
|
|
|
let mut costliest_account_cost = 0;
|
|
|
|
for (key, cost) in self.cost_by_writable_accounts.iter() {
|
|
|
|
if *cost > costliest_account_cost {
|
|
|
|
costliest_account = *key;
|
|
|
|
costliest_account_cost = *cost;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
(costliest_account, costliest_account_cost)
|
|
|
|
}
|
|
|
|
|
2021-12-12 12:57:18 -08:00
|
|
|
fn would_fit(
|
|
|
|
&self,
|
|
|
|
keys: &[Pubkey],
|
|
|
|
cost: u64,
|
|
|
|
account_data_len: u64,
|
2022-01-12 21:27:19 -08:00
|
|
|
transaction: &SanitizedTransaction,
|
2021-12-12 12:57:18 -08:00
|
|
|
) -> Result<(), CostTrackerError> {
|
2021-06-01 07:16:17 -07:00
|
|
|
// check against the total package cost
|
2022-01-12 21:27:19 -08:00
|
|
|
if self.block_cost.saturating_add(cost) > self.block_cost_limit {
|
2021-10-19 12:37:33 -07:00
|
|
|
return Err(CostTrackerError::WouldExceedBlockMaxLimit);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
2022-01-12 21:27:19 -08:00
|
|
|
// if vote transaction, check if it exceeds vote_transaction_limit
|
|
|
|
if transaction.is_simple_vote_transaction()
|
|
|
|
&& self.vote_cost.saturating_add(cost) > self.vote_cost_limit
|
|
|
|
{
|
|
|
|
return Err(CostTrackerError::WouldExceedVoteMaxLimit);
|
|
|
|
}
|
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
// check if the transaction itself is more costly than the account_cost_limit
|
2021-12-12 12:57:18 -08:00
|
|
|
if cost > self.account_cost_limit {
|
2021-10-19 12:37:33 -07:00
|
|
|
return Err(CostTrackerError::WouldExceedAccountMaxLimit);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
2022-03-13 08:58:57 -07:00
|
|
|
// NOTE: Check if the total accounts data size is exceeded *before* the block accounts data
|
|
|
|
// size. This way, transactions are not unnecessarily retried.
|
|
|
|
let account_data_size = self.account_data_size.saturating_add(account_data_len);
|
|
|
|
if let Some(account_data_size_limit) = self.account_data_size_limit {
|
|
|
|
if account_data_size > account_data_size_limit {
|
|
|
|
return Err(CostTrackerError::WouldExceedAccountDataTotalLimit);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if account_data_size > MAX_ACCOUNT_DATA_BLOCK_LEN {
|
|
|
|
return Err(CostTrackerError::WouldExceedAccountDataBlockLimit);
|
2021-12-12 12:57:18 -08:00
|
|
|
}
|
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
// check each account against account_cost_limit,
|
|
|
|
for account_key in keys.iter() {
|
2021-06-18 06:34:46 -07:00
|
|
|
match self.cost_by_writable_accounts.get(account_key) {
|
2021-06-01 07:16:17 -07:00
|
|
|
Some(chained_cost) => {
|
2022-01-12 21:27:19 -08:00
|
|
|
if chained_cost.saturating_add(cost) > self.account_cost_limit {
|
2021-10-19 12:37:33 -07:00
|
|
|
return Err(CostTrackerError::WouldExceedAccountMaxLimit);
|
2021-06-01 07:16:17 -07:00
|
|
|
} else {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
None => continue,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
Ok(())
|
|
|
|
}
|
|
|
|
|
2022-01-12 21:27:19 -08:00
|
|
|
fn add_transaction(
|
|
|
|
&mut self,
|
|
|
|
keys: &[Pubkey],
|
|
|
|
cost: u64,
|
|
|
|
account_data_size: u64,
|
|
|
|
transaction: &SanitizedTransaction,
|
|
|
|
) {
|
2021-06-01 07:16:17 -07:00
|
|
|
for account_key in keys.iter() {
|
2022-01-12 21:27:19 -08:00
|
|
|
let account_cost = self
|
2021-06-01 07:16:17 -07:00
|
|
|
.cost_by_writable_accounts
|
|
|
|
.entry(*account_key)
|
2022-01-12 21:27:19 -08:00
|
|
|
.or_insert(0);
|
|
|
|
*account_cost = account_cost.saturating_add(cost);
|
|
|
|
}
|
|
|
|
self.block_cost = self.block_cost.saturating_add(cost);
|
|
|
|
if transaction.is_simple_vote_transaction() {
|
|
|
|
self.vote_cost = self.vote_cost.saturating_add(cost);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
2021-12-12 12:57:18 -08:00
|
|
|
self.account_data_size = self.account_data_size.saturating_add(account_data_size);
|
2022-01-12 21:27:19 -08:00
|
|
|
self.transaction_count = self.transaction_count.saturating_add(1);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
#[cfg(test)]
|
|
|
|
mod tests {
|
2021-12-03 09:00:31 -08:00
|
|
|
use {
|
|
|
|
super::*,
|
|
|
|
crate::{
|
|
|
|
bank::Bank,
|
|
|
|
genesis_utils::{create_genesis_config, GenesisConfigInfo},
|
|
|
|
},
|
|
|
|
solana_sdk::{
|
|
|
|
hash::Hash,
|
|
|
|
signature::{Keypair, Signer},
|
|
|
|
system_transaction,
|
2022-02-03 03:00:12 -08:00
|
|
|
transaction::{DisabledAddressLoader, VersionedTransaction},
|
2021-12-03 09:00:31 -08:00
|
|
|
},
|
2022-01-12 21:27:19 -08:00
|
|
|
solana_vote_program::vote_transaction,
|
2021-12-03 09:00:31 -08:00
|
|
|
std::{cmp, sync::Arc},
|
2021-06-01 07:16:17 -07:00
|
|
|
};
|
|
|
|
|
2022-02-22 08:44:23 -08:00
|
|
|
impl CostTracker {
|
2022-03-13 08:58:57 -07:00
|
|
|
fn new(
|
|
|
|
account_cost_limit: u64,
|
|
|
|
block_cost_limit: u64,
|
|
|
|
vote_cost_limit: u64,
|
|
|
|
account_data_size_limit: Option<u64>,
|
|
|
|
) -> Self {
|
2022-02-22 08:44:23 -08:00
|
|
|
assert!(account_cost_limit <= block_cost_limit);
|
|
|
|
assert!(vote_cost_limit <= block_cost_limit);
|
|
|
|
Self {
|
|
|
|
account_cost_limit,
|
|
|
|
block_cost_limit,
|
|
|
|
vote_cost_limit,
|
2022-03-13 08:58:57 -07:00
|
|
|
account_data_size_limit,
|
2022-02-22 08:44:23 -08:00
|
|
|
..Self::default()
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
fn test_setup() -> (Keypair, Hash) {
|
|
|
|
solana_logger::setup();
|
|
|
|
let GenesisConfigInfo {
|
|
|
|
genesis_config,
|
|
|
|
mint_keypair,
|
|
|
|
..
|
|
|
|
} = create_genesis_config(10);
|
2021-08-05 12:50:25 -07:00
|
|
|
let bank = Arc::new(Bank::new_no_wallclock_throttle_for_tests(&genesis_config));
|
2021-06-01 07:16:17 -07:00
|
|
|
let start_hash = bank.last_blockhash();
|
|
|
|
(mint_keypair, start_hash)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn build_simple_transaction(
|
|
|
|
mint_keypair: &Keypair,
|
|
|
|
start_hash: &Hash,
|
2022-01-12 21:27:19 -08:00
|
|
|
) -> (SanitizedTransaction, Vec<Pubkey>, u64) {
|
2021-06-01 07:16:17 -07:00
|
|
|
let keypair = Keypair::new();
|
2022-01-12 21:27:19 -08:00
|
|
|
let simple_transaction = SanitizedTransaction::from_transaction_for_tests(
|
|
|
|
system_transaction::transfer(mint_keypair, &keypair.pubkey(), 2, *start_hash),
|
|
|
|
);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
(simple_transaction, vec![mint_keypair.pubkey()], 5)
|
|
|
|
}
|
|
|
|
|
2022-01-12 21:27:19 -08:00
|
|
|
fn build_simple_vote_transaction(
|
|
|
|
mint_keypair: &Keypair,
|
|
|
|
start_hash: &Hash,
|
|
|
|
) -> (SanitizedTransaction, Vec<Pubkey>, u64) {
|
|
|
|
let keypair = Keypair::new();
|
|
|
|
let transaction = vote_transaction::new_vote_transaction(
|
|
|
|
vec![42],
|
|
|
|
Hash::default(),
|
|
|
|
*start_hash,
|
|
|
|
mint_keypair,
|
|
|
|
&keypair,
|
|
|
|
&keypair,
|
|
|
|
None,
|
|
|
|
);
|
|
|
|
let message_hash = transaction.message.hash();
|
|
|
|
let vote_transaction = SanitizedTransaction::try_create(
|
|
|
|
VersionedTransaction::from(transaction),
|
|
|
|
message_hash,
|
|
|
|
Some(true),
|
2022-02-03 03:00:12 -08:00
|
|
|
&DisabledAddressLoader,
|
2022-01-12 21:27:19 -08:00
|
|
|
)
|
|
|
|
.unwrap();
|
|
|
|
(vote_transaction, vec![mint_keypair.pubkey()], 10)
|
|
|
|
}
|
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_initialization() {
|
2022-03-13 08:58:57 -07:00
|
|
|
let testee = CostTracker::new(10, 11, 8, None);
|
2021-06-01 07:16:17 -07:00
|
|
|
assert_eq!(10, testee.account_cost_limit);
|
|
|
|
assert_eq!(11, testee.block_cost_limit);
|
2022-01-12 21:27:19 -08:00
|
|
|
assert_eq!(8, testee.vote_cost_limit);
|
2021-06-01 07:16:17 -07:00
|
|
|
assert_eq!(0, testee.cost_by_writable_accounts.len());
|
|
|
|
assert_eq!(0, testee.block_cost);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_ok_add_one() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx, keys, cost) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
// build testee to have capacity for one simple transaction
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(cost, cost, cost, None);
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys, cost, 0, &tx).is_ok());
|
|
|
|
testee.add_transaction(&keys, cost, 0, &tx);
|
2021-06-01 07:16:17 -07:00
|
|
|
assert_eq!(cost, testee.block_cost);
|
2022-01-12 21:27:19 -08:00
|
|
|
assert_eq!(0, testee.vote_cost);
|
|
|
|
let (_costliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(cost, costliest_account_cost);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_ok_add_one_vote() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
let (tx, keys, cost) = build_simple_vote_transaction(&mint_keypair, &start_hash);
|
|
|
|
|
|
|
|
// build testee to have capacity for one simple transaction
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(cost, cost, cost, None);
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys, cost, 0, &tx).is_ok());
|
|
|
|
testee.add_transaction(&keys, cost, 0, &tx);
|
|
|
|
assert_eq!(cost, testee.block_cost);
|
|
|
|
assert_eq!(cost, testee.vote_cost);
|
|
|
|
let (_costliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(cost, costliest_account_cost);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
2021-12-12 12:57:18 -08:00
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_add_data() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx, keys, cost) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-12-12 12:57:18 -08:00
|
|
|
|
|
|
|
// build testee to have capacity for one simple transaction
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(cost, cost, cost, None);
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys, cost, 0, &tx).is_ok());
|
2021-12-12 12:57:18 -08:00
|
|
|
let old = testee.account_data_size;
|
2022-01-12 21:27:19 -08:00
|
|
|
testee.add_transaction(&keys, cost, 1, &tx);
|
2021-12-12 12:57:18 -08:00
|
|
|
assert_eq!(old + 1, testee.account_data_size);
|
|
|
|
}
|
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_ok_add_two_same_accounts() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two transactions with same signed account
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx1, keys1, cost1) = build_simple_transaction(&mint_keypair, &start_hash);
|
|
|
|
let (tx2, keys2, cost2) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
// build testee to have capacity for two simple transactions, with same accounts
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(cost1 + cost2, cost1 + cost2, cost1 + cost2, None);
|
2021-06-01 07:16:17 -07:00
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys1, cost1, 0, &tx1).is_ok());
|
|
|
|
testee.add_transaction(&keys1, cost1, 0, &tx1);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys2, cost2, 0, &tx2).is_ok());
|
|
|
|
testee.add_transaction(&keys2, cost2, 0, &tx2);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
assert_eq!(cost1 + cost2, testee.block_cost);
|
|
|
|
assert_eq!(1, testee.cost_by_writable_accounts.len());
|
2022-01-12 21:27:19 -08:00
|
|
|
let (_ccostliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(cost1 + cost2, costliest_account_cost);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_ok_add_two_diff_accounts() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two transactions with diff accounts
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx1, keys1, cost1) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
let second_account = Keypair::new();
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx2, keys2, cost2) = build_simple_transaction(&second_account, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
// build testee to have capacity for two simple transactions, with same accounts
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee =
|
|
|
|
CostTracker::new(cmp::max(cost1, cost2), cost1 + cost2, cost1 + cost2, None);
|
2021-06-01 07:16:17 -07:00
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys1, cost1, 0, &tx1).is_ok());
|
|
|
|
testee.add_transaction(&keys1, cost1, 0, &tx1);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys2, cost2, 0, &tx2).is_ok());
|
|
|
|
testee.add_transaction(&keys2, cost2, 0, &tx2);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
assert_eq!(cost1 + cost2, testee.block_cost);
|
|
|
|
assert_eq!(2, testee.cost_by_writable_accounts.len());
|
2022-01-12 21:27:19 -08:00
|
|
|
let (_ccostliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(std::cmp::max(cost1, cost2), costliest_account_cost);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_chain_reach_limit() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two transactions with same signed account
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx1, keys1, cost1) = build_simple_transaction(&mint_keypair, &start_hash);
|
|
|
|
let (tx2, keys2, cost2) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
// build testee to have capacity for two simple transactions, but not for same accounts
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee =
|
|
|
|
CostTracker::new(cmp::min(cost1, cost2), cost1 + cost2, cost1 + cost2, None);
|
2021-06-01 07:16:17 -07:00
|
|
|
// should have room for first transaction
|
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys1, cost1, 0, &tx1).is_ok());
|
|
|
|
testee.add_transaction(&keys1, cost1, 0, &tx1);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
// but no more sapce on the same chain (same signer account)
|
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys2, cost2, 0, &tx2).is_err());
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_reach_limit() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two transactions with diff accounts
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx1, keys1, cost1) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
let second_account = Keypair::new();
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx2, keys2, cost2) = build_simple_transaction(&second_account, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
// build testee to have capacity for each chain, but not enough room for both transactions
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(
|
|
|
|
cmp::max(cost1, cost2),
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
None,
|
|
|
|
);
|
2021-06-01 07:16:17 -07:00
|
|
|
// should have room for first transaction
|
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys1, cost1, 0, &tx1).is_ok());
|
|
|
|
testee.add_transaction(&keys1, cost1, 0, &tx1);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
// but no more room for package as whole
|
|
|
|
{
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.would_fit(&keys2, cost2, 0, &tx2).is_err());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_reach_vote_limit() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two mocking vote transactions with diff accounts
|
|
|
|
let (tx1, keys1, cost1) = build_simple_vote_transaction(&mint_keypair, &start_hash);
|
|
|
|
let second_account = Keypair::new();
|
|
|
|
let (tx2, keys2, cost2) = build_simple_vote_transaction(&second_account, &start_hash);
|
|
|
|
|
|
|
|
// build testee to have capacity for each chain, but not enough room for both votes
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(
|
|
|
|
cmp::max(cost1, cost2),
|
|
|
|
cost1 + cost2,
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
None,
|
|
|
|
);
|
2022-01-12 21:27:19 -08:00
|
|
|
// should have room for first vote
|
|
|
|
{
|
|
|
|
assert!(testee.would_fit(&keys1, cost1, 0, &tx1).is_ok());
|
|
|
|
testee.add_transaction(&keys1, cost1, 0, &tx1);
|
|
|
|
}
|
|
|
|
// but no more room for package as whole
|
|
|
|
{
|
|
|
|
assert!(testee.would_fit(&keys2, cost2, 0, &tx2).is_err());
|
|
|
|
}
|
|
|
|
// however there is room for none-vote tx3
|
|
|
|
{
|
|
|
|
let third_account = Keypair::new();
|
|
|
|
let (tx3, keys3, cost3) = build_simple_transaction(&third_account, &start_hash);
|
|
|
|
assert!(testee.would_fit(&keys3, cost3, 0, &tx3).is_ok());
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-12-12 12:57:18 -08:00
|
|
|
#[test]
|
2022-03-13 08:58:57 -07:00
|
|
|
fn test_cost_tracker_reach_data_block_limit() {
|
2021-12-12 12:57:18 -08:00
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two transactions with diff accounts
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx1, keys1, cost1) = build_simple_transaction(&mint_keypair, &start_hash);
|
2021-12-12 12:57:18 -08:00
|
|
|
let second_account = Keypair::new();
|
2022-01-12 21:27:19 -08:00
|
|
|
let (tx2, keys2, cost2) = build_simple_transaction(&second_account, &start_hash);
|
2021-12-12 12:57:18 -08:00
|
|
|
|
|
|
|
// build testee that passes
|
2022-03-13 08:58:57 -07:00
|
|
|
let testee = CostTracker::new(
|
|
|
|
cmp::max(cost1, cost2),
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
None,
|
|
|
|
);
|
2021-12-12 12:57:18 -08:00
|
|
|
assert!(testee
|
2022-03-13 08:58:57 -07:00
|
|
|
.would_fit(&keys1, cost1, MAX_ACCOUNT_DATA_BLOCK_LEN, &tx1)
|
2021-12-12 12:57:18 -08:00
|
|
|
.is_ok());
|
|
|
|
// data is too big
|
2022-03-13 08:58:57 -07:00
|
|
|
assert_eq!(
|
|
|
|
testee.would_fit(&keys2, cost2, MAX_ACCOUNT_DATA_BLOCK_LEN + 1, &tx2),
|
|
|
|
Err(CostTrackerError::WouldExceedAccountDataBlockLimit),
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_cost_tracker_reach_data_total_limit() {
|
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
|
|
|
// build two transactions with diff accounts
|
|
|
|
let (tx1, keys1, cost1) = build_simple_transaction(&mint_keypair, &start_hash);
|
|
|
|
let second_account = Keypair::new();
|
|
|
|
let (tx2, keys2, cost2) = build_simple_transaction(&second_account, &start_hash);
|
|
|
|
|
|
|
|
// build testee that passes
|
|
|
|
let remaining_account_data_size = 1234;
|
|
|
|
let testee = CostTracker::new(
|
|
|
|
cmp::max(cost1, cost2),
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
cost1 + cost2 - 1,
|
|
|
|
Some(remaining_account_data_size),
|
|
|
|
);
|
2021-12-12 12:57:18 -08:00
|
|
|
assert!(testee
|
2022-03-13 08:58:57 -07:00
|
|
|
.would_fit(&keys1, cost1, remaining_account_data_size, &tx1)
|
|
|
|
.is_ok());
|
|
|
|
// data is too big
|
|
|
|
assert_eq!(
|
|
|
|
testee.would_fit(&keys2, cost2, remaining_account_data_size + 1, &tx2),
|
|
|
|
Err(CostTrackerError::WouldExceedAccountDataTotalLimit),
|
|
|
|
);
|
2021-12-12 12:57:18 -08:00
|
|
|
}
|
|
|
|
|
2021-06-01 07:16:17 -07:00
|
|
|
#[test]
|
2021-10-19 12:37:33 -07:00
|
|
|
fn test_cost_tracker_try_add_is_atomic() {
|
2021-06-01 07:16:17 -07:00
|
|
|
let (mint_keypair, start_hash) = test_setup();
|
2022-01-12 21:27:19 -08:00
|
|
|
// build two mocking vote transactions with diff accounts
|
|
|
|
let (tx1, _keys1, _cost1) = build_simple_vote_transaction(&mint_keypair, &start_hash);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
let acct1 = Pubkey::new_unique();
|
|
|
|
let acct2 = Pubkey::new_unique();
|
|
|
|
let acct3 = Pubkey::new_unique();
|
|
|
|
let cost = 100;
|
|
|
|
let account_max = cost * 2;
|
|
|
|
let block_max = account_max * 3; // for three accts
|
|
|
|
|
2022-03-13 08:58:57 -07:00
|
|
|
let mut testee = CostTracker::new(account_max, block_max, block_max, None);
|
2021-06-01 07:16:17 -07:00
|
|
|
|
|
|
|
// case 1: a tx writes to 3 accounts, should success, we will have:
|
|
|
|
// | acct1 | $cost |
|
|
|
|
// | acct2 | $cost |
|
2021-10-24 20:19:23 -07:00
|
|
|
// | acct3 | $cost |
|
2021-06-01 07:16:17 -07:00
|
|
|
// and block_cost = $cost
|
|
|
|
{
|
|
|
|
let tx_cost = TransactionCost {
|
|
|
|
writable_accounts: vec![acct1, acct2, acct3],
|
|
|
|
execution_cost: cost,
|
2021-10-08 12:48:50 -07:00
|
|
|
..TransactionCost::default()
|
2021-06-01 07:16:17 -07:00
|
|
|
};
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.try_add(&tx1, &tx_cost).is_ok());
|
2021-10-24 20:19:23 -07:00
|
|
|
let (_costliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(cost, testee.block_cost);
|
|
|
|
assert_eq!(3, testee.cost_by_writable_accounts.len());
|
|
|
|
assert_eq!(cost, costliest_account_cost);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
// case 2: add tx writes to acct2 with $cost, should succeed, result to
|
|
|
|
// | acct1 | $cost |
|
|
|
|
// | acct2 | $cost * 2 |
|
2021-10-24 20:19:23 -07:00
|
|
|
// | acct3 | $cost |
|
2021-06-01 07:16:17 -07:00
|
|
|
// and block_cost = $cost * 2
|
|
|
|
{
|
|
|
|
let tx_cost = TransactionCost {
|
|
|
|
writable_accounts: vec![acct2],
|
|
|
|
execution_cost: cost,
|
2021-10-08 12:48:50 -07:00
|
|
|
..TransactionCost::default()
|
2021-06-01 07:16:17 -07:00
|
|
|
};
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.try_add(&tx1, &tx_cost).is_ok());
|
2021-10-24 20:19:23 -07:00
|
|
|
let (costliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(cost * 2, testee.block_cost);
|
|
|
|
assert_eq!(3, testee.cost_by_writable_accounts.len());
|
|
|
|
assert_eq!(cost * 2, costliest_account_cost);
|
|
|
|
assert_eq!(acct2, costliest_account);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
|
|
|
|
// case 3: add tx writes to [acct1, acct2], acct2 exceeds limit, should failed atomically,
|
|
|
|
// we shoudl still have:
|
|
|
|
// | acct1 | $cost |
|
2021-10-24 20:19:23 -07:00
|
|
|
// | acct2 | $cost * 2 |
|
|
|
|
// | acct3 | $cost |
|
|
|
|
// and block_cost = $cost * 2
|
2021-06-01 07:16:17 -07:00
|
|
|
{
|
|
|
|
let tx_cost = TransactionCost {
|
|
|
|
writable_accounts: vec![acct1, acct2],
|
|
|
|
execution_cost: cost,
|
2021-10-08 12:48:50 -07:00
|
|
|
..TransactionCost::default()
|
2021-06-01 07:16:17 -07:00
|
|
|
};
|
2022-01-12 21:27:19 -08:00
|
|
|
assert!(testee.try_add(&tx1, &tx_cost).is_err());
|
2021-10-24 20:19:23 -07:00
|
|
|
let (costliest_account, costliest_account_cost) = testee.find_costliest_account();
|
|
|
|
assert_eq!(cost * 2, testee.block_cost);
|
|
|
|
assert_eq!(3, testee.cost_by_writable_accounts.len());
|
|
|
|
assert_eq!(cost * 2, costliest_account_cost);
|
|
|
|
assert_eq!(acct2, costliest_account);
|
2021-06-01 07:16:17 -07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|