From 2ca6fdf538935a03e0de6cd0fdaffc21799fad1a Mon Sep 17 00:00:00 2001 From: Alwin Berger Date: Thu, 10 Oct 2024 16:33:19 +0200 Subject: [PATCH] simplify stored execution times per testcase --- fuzzers/FRET/src/time/clock.rs | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/fuzzers/FRET/src/time/clock.rs b/fuzzers/FRET/src/time/clock.rs index 14370bb030..eb299111e7 100644 --- a/fuzzers/FRET/src/time/clock.rs +++ b/fuzzers/FRET/src/time/clock.rs @@ -215,6 +215,7 @@ pub struct ClockTimeFeedback { impl Feedback for ClockTimeFeedback where S: State + UsesInput + MaybeHasClientPerfMonitor + HasMetadata, + ::Input: Default { #[allow(clippy::wrong_self_convention)] fn is_interesting( @@ -231,20 +232,15 @@ where { #[cfg(feature="trace_job_response_times")] { - if let Some(t) = &self.select_task { + if self.select_task.is_some() { let observer = observers.match_name::>("systemstate").unwrap(); - if let Some(time) = observer.worst_job_instances.get(t) { - self.exec_time = Some(Duration::from_nanos(*time)); - return Ok(false); - } else { - self.exec_time = Some(Duration::from_nanos(0)); - return Ok(false); - } + self.exec_time = Some(Duration::from_nanos(observer.last_runtime())); + return Ok(false) } } // TODO Replace with match_name_type when stable let observer = observers.match_name::(self.name()).unwrap(); - self.exec_time = Some(Duration::from_nanos(observer.last_runtime() << QEMU_ICOUNT_SHIFT)); // Assume a somewhat realistic multiplier of clock, it does not matter + self.exec_time = Some(Duration::from_nanos(observer.last_runtime())); Ok(false) }