Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: remove reallocations in compute hashes #22

Closed
wants to merge 1 commit into from

Conversation

0xdeafbeef
Copy link
Member

184cdf0 - realloc
7af2b0 - alloc::raw_vec::RawVec<T,A>::grow_one
7c03b0 - everscale_types::cell::cell_context::CellParts::compute_hashes
7bc2f0 - <everscale_types::cell::cell_impl::sync::EmptyCellContext as everscale_types::cell::cell_context::CellContext>::finalize_cell
7c5bc0 - everscale_types::cell::builder::CellBuilder::build_ext
7d2960 - everscale_types::merkle::proof::BuilderImpl<S>::build
7d62b0 - everscale_types::merkle::update::BuilderImpl::build
13af190 - tycho_collator::collator::build_block::create_merkle_update
13a2790 - tycho_collator::collator::build_block::<impl tycho_collator::collator::CollatorStdImpl>::finalize_block
137a370 - <rayon_core::job::HeapJob<BODY> as rayon_core::job::Job>::execute
aa2540 - rayon_core::registry::WorkerThread::wait_until_cold
a9fa50 - rayon_core::registry::ThreadBuilder::run
a9d330 - std::sys::backtrace::__rust_begin_short_backtrace
a9d550 - core::ops::function::FnOnce::call_once{{vtable.shim}}
Size      | Count     | Percentage | Distribution
----------+-----------+------------+--------------------------------------------------
128 B      |     65145 |    100.00% | ##################################################
Total allocations: 8.4 MiB in 65145 allocations

Snapshot of 10m node work with 1000 sampling ratio.

@Rexagon
Copy link
Member

Rexagon commented Oct 12, 2024

Fixed by 8a2410b

@Rexagon Rexagon closed this Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants