add excutor creation trace timings
This commit is contained in:
committed by
Trent Nelson
parent
7cb147fdcd
commit
a4ca18a54d
1
Cargo.lock
generated
1
Cargo.lock
generated
@ -4517,6 +4517,7 @@ dependencies = [
|
|||||||
"log 0.4.14",
|
"log 0.4.14",
|
||||||
"rand 0.7.3",
|
"rand 0.7.3",
|
||||||
"solana-measure",
|
"solana-measure",
|
||||||
|
"solana-metrics",
|
||||||
"solana-program-runtime",
|
"solana-program-runtime",
|
||||||
"solana-runtime",
|
"solana-runtime",
|
||||||
"solana-sdk",
|
"solana-sdk",
|
||||||
|
1
programs/bpf/Cargo.lock
generated
1
programs/bpf/Cargo.lock
generated
@ -2543,6 +2543,7 @@ dependencies = [
|
|||||||
"libsecp256k1 0.6.0",
|
"libsecp256k1 0.6.0",
|
||||||
"log",
|
"log",
|
||||||
"solana-measure",
|
"solana-measure",
|
||||||
|
"solana-metrics",
|
||||||
"solana-program-runtime",
|
"solana-program-runtime",
|
||||||
"solana-sdk",
|
"solana-sdk",
|
||||||
"solana_rbpf",
|
"solana_rbpf",
|
||||||
|
@ -15,6 +15,7 @@ byteorder = "1.4.3"
|
|||||||
log = "0.4.14"
|
log = "0.4.14"
|
||||||
libsecp256k1 = "0.6.0"
|
libsecp256k1 = "0.6.0"
|
||||||
solana-measure = { path = "../../measure", version = "=1.9.4" }
|
solana-measure = { path = "../../measure", version = "=1.9.4" }
|
||||||
|
solana-metrics = { path = "../../metrics", version = "=1.9.4" }
|
||||||
solana-program-runtime = { path = "../../program-runtime", version = "=1.9.4" }
|
solana-program-runtime = { path = "../../program-runtime", version = "=1.9.4" }
|
||||||
solana-sdk = { path = "../../sdk", version = "=1.9.4" }
|
solana-sdk = { path = "../../sdk", version = "=1.9.4" }
|
||||||
solana_rbpf = "=0.2.21"
|
solana_rbpf = "=0.2.21"
|
||||||
|
@ -8,6 +8,9 @@ pub mod upgradeable;
|
|||||||
pub mod upgradeable_with_jit;
|
pub mod upgradeable_with_jit;
|
||||||
pub mod with_jit;
|
pub mod with_jit;
|
||||||
|
|
||||||
|
#[macro_use]
|
||||||
|
extern crate solana_metrics;
|
||||||
|
|
||||||
use {
|
use {
|
||||||
crate::{
|
crate::{
|
||||||
serialization::{deserialize_parameters, serialize_parameters},
|
serialization::{deserialize_parameters, serialize_parameters},
|
||||||
@ -121,9 +124,14 @@ pub fn create_executor(
|
|||||||
.is_active(&reject_all_elf_rw::id()),
|
.is_active(&reject_all_elf_rw::id()),
|
||||||
..Config::default()
|
..Config::default()
|
||||||
};
|
};
|
||||||
|
let program_id;
|
||||||
|
let load_elf_us: u64;
|
||||||
|
let verify_elf_us: u64;
|
||||||
|
let mut jit_compile_us = 0u64;
|
||||||
let mut executable = {
|
let mut executable = {
|
||||||
let keyed_accounts = invoke_context.get_keyed_accounts()?;
|
let keyed_accounts = invoke_context.get_keyed_accounts()?;
|
||||||
let programdata = keyed_account_at_index(keyed_accounts, programdata_account_index)?;
|
let programdata = keyed_account_at_index(keyed_accounts, programdata_account_index)?;
|
||||||
|
program_id = *programdata.unsigned_key();
|
||||||
let mut load_elf_time = Measure::start("load_elf_time");
|
let mut load_elf_time = Measure::start("load_elf_time");
|
||||||
let executable = Executable::<BpfError, ThisInstructionMeter>::from_elf(
|
let executable = Executable::<BpfError, ThisInstructionMeter>::from_elf(
|
||||||
&programdata.try_account_ref()?.data()[programdata_offset..],
|
&programdata.try_account_ref()?.data()[programdata_offset..],
|
||||||
@ -132,10 +140,11 @@ pub fn create_executor(
|
|||||||
syscall_registry,
|
syscall_registry,
|
||||||
);
|
);
|
||||||
load_elf_time.stop();
|
load_elf_time.stop();
|
||||||
|
load_elf_us = load_elf_time.as_us();
|
||||||
invoke_context.timings.create_executor_load_elf_us = invoke_context
|
invoke_context.timings.create_executor_load_elf_us = invoke_context
|
||||||
.timings
|
.timings
|
||||||
.create_executor_load_elf_us
|
.create_executor_load_elf_us
|
||||||
.saturating_add(load_elf_time.as_us());
|
.saturating_add(load_elf_us);
|
||||||
executable
|
executable
|
||||||
}
|
}
|
||||||
.map_err(|e| map_ebpf_error(invoke_context, e))?;
|
.map_err(|e| map_ebpf_error(invoke_context, e))?;
|
||||||
@ -144,24 +153,33 @@ pub fn create_executor(
|
|||||||
verifier::check(text_bytes, &config)
|
verifier::check(text_bytes, &config)
|
||||||
.map_err(|e| map_ebpf_error(invoke_context, EbpfError::UserError(e.into())))?;
|
.map_err(|e| map_ebpf_error(invoke_context, EbpfError::UserError(e.into())))?;
|
||||||
verify_code_time.stop();
|
verify_code_time.stop();
|
||||||
|
verify_elf_us = verify_code_time.as_us();
|
||||||
invoke_context.timings.create_executor_verify_code_us = invoke_context
|
invoke_context.timings.create_executor_verify_code_us = invoke_context
|
||||||
.timings
|
.timings
|
||||||
.create_executor_verify_code_us
|
.create_executor_verify_code_us
|
||||||
.saturating_add(verify_code_time.as_us());
|
.saturating_add(verify_elf_us);
|
||||||
if use_jit {
|
if use_jit {
|
||||||
let mut jit_compile_time = Measure::start("jit_compile_time");
|
let mut jit_compile_time = Measure::start("jit_compile_time");
|
||||||
let jit_compile_result =
|
let jit_compile_result =
|
||||||
Executable::<BpfError, ThisInstructionMeter>::jit_compile(&mut executable);
|
Executable::<BpfError, ThisInstructionMeter>::jit_compile(&mut executable);
|
||||||
jit_compile_time.stop();
|
jit_compile_time.stop();
|
||||||
|
jit_compile_us = jit_compile_time.as_us();
|
||||||
invoke_context.timings.create_executor_jit_compile_us = invoke_context
|
invoke_context.timings.create_executor_jit_compile_us = invoke_context
|
||||||
.timings
|
.timings
|
||||||
.create_executor_jit_compile_us
|
.create_executor_jit_compile_us
|
||||||
.saturating_add(jit_compile_time.as_us());
|
.saturating_add(jit_compile_us);
|
||||||
if let Err(err) = jit_compile_result {
|
if let Err(err) = jit_compile_result {
|
||||||
ic_msg!(invoke_context, "Failed to compile program {:?}", err);
|
ic_msg!(invoke_context, "Failed to compile program {:?}", err);
|
||||||
return Err(InstructionError::ProgramFailedToCompile);
|
return Err(InstructionError::ProgramFailedToCompile);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
datapoint_trace!(
|
||||||
|
"create_executor_trace",
|
||||||
|
("program_id", program_id.to_string(), String),
|
||||||
|
("load_elf_us", load_elf_us, i64),
|
||||||
|
("verify_elf_us", verify_elf_us, i64),
|
||||||
|
("jit_compile_us", jit_compile_us, i64),
|
||||||
|
);
|
||||||
Ok(Arc::new(BpfExecutor { executable }))
|
Ok(Arc::new(BpfExecutor { executable }))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user