From ae8149077c187676029693075961936c0197adf8 Mon Sep 17 00:00:00 2001 From: Douglas Raillard Date: Mon, 3 Feb 2025 15:49:50 +0000 Subject: [PATCH] collector/ftrace: Remove cpu_frequency_devlib event in FtraceCollector.stop() Emiting the current frequency of all CPUs in the stop() hook is not useful as the current frequency should already be known from the trace. Either the event is emitted every time the frequency changes and the up-to-date information is available, or the frequency never changes (e.g. userspace governor) and the frequencies will be known from emitting cpu_frequency_devlib in start(). --- devlib/collector/ftrace.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/devlib/collector/ftrace.py b/devlib/collector/ftrace.py index 9a887ef..ddbc748 100644 --- a/devlib/collector/ftrace.py +++ b/devlib/collector/ftrace.py @@ -328,9 +328,6 @@ class FtraceCollector(CollectorBase): if self.functions and self.tracer is None: self.target.execute('echo 0 > {}'.format(self.function_profile_file), as_root=True) - if 'cpufreq' in self.target.modules: - self.logger.debug('Trace CPUFreq frequencies') - self.target.cpufreq.trace_frequencies() self.stop_time = time.time() if self.automark: self.mark_stop()