github/reth
1
1
Fork 0
mirror of https://github.com/paradigmxyz/reth.git synced 2026-04-30 03:01:58 -04:00
Code Issues Packages Projects Releases Wiki Activity

nits

Browse Source
This commit is contained in:
Alexey Shekhirin
2025-12-23 17:32:04 +00:00
committed by Alexey Shekhirin
parent 68163c45df
commit cd164dc902
1 changed files with 6 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
crates/engine/tree/src/tree/cached_state.rs
View File

@@ -425,21 +425,14 @@ impl<S: HashedPostStateProvider> HashedPostStateProvider for CachedStateProvider
///
/// ## Storage Invalidation
///
/// When an account is destroyed (SELFDESTRUCT), all its storage must be invalidated.
/// This is handled using timestamps:
/// - Each storage entry stores the timestamp when it was inserted
/// - Each account tracks when it was last wiped (destroyed)
/// - On lookup, if the entry's timestamp <= wipe timestamp, the entry is stale
///
/// Since EIP-6780, SELFDESTRUCT only works within the same transaction where the
/// contract was created, so we only need to track wiped accounts for the current block.
/// The wipe map is cleared after each block execution via [`Self::clear_wiped_accounts`].
/// contract was created, so we don't need to handle clearing the storage.
#[derive(Debug, Clone)]
pub(crate) struct ExecutionCache {
/// Cache for contract bytecode, keyed by code hash.
code_cache: Arc<FixedCache<B256, Option<Bytecode>, FbBuildHasher<32>>>,
/// Flat storage cache: maps `(Address, StorageKey)` to timestamped storage value.
/// Flat storage cache: maps `(Address, StorageKey)` to storage value.
storage_cache: Arc<FixedCache<(Address, StorageKey), StorageValue>>,
/// Cache for basic account information (nonce, balance, code hash).
@@ -502,7 +495,7 @@ impl ExecutionCache {
Ok(if miss { CachedStatus::NotCached(result) } else { CachedStatus::Cached(result) })
}
/// Insert storage value into cache with current timestamp.
/// Insert storage value into cache.
pub(crate) fn insert_storage(
&self,
address: Address,
@@ -512,16 +505,6 @@ impl ExecutionCache {
self.storage_cache.insert((address, key), value.unwrap_or_default());
}
/// Insert multiple storage values into cache for a single account.
pub(crate) fn insert_storage_bulk<I>(&self, address: Address, storage_entries: I)
where
I: IntoIterator<Item = (StorageKey, Option<StorageValue>)>,
{
for (key, value) in storage_entries {
self.insert_storage(address, key, value)
}
}
/// Inserts the post-execution state changes into the cache.
///
/// This method is called after transaction execution to update the cache with
@@ -582,11 +565,9 @@ impl ExecutionCache {
};
// Now we iterate over all storage and make updates to the cached storage values
let storage_entries = account
.storage
.iter()
.map(|(storage_key, slot)| ((*storage_key).into(), Some(slot.present_value)));
self.insert_storage_bulk(*addr, storage_entries);
for (key, slot) in &account.storage {
self.insert_storage(*addr, (*key).into(), Some(slot.present_value));
}
// Insert will update if present, so we just use the new account info as the new value
// for the account cache