summaryrefslogtreecommitdiff
path: root/Source/Core
diff options
context:
space:
mode:
authorJordan Woyak <jordan.woyak@gmail.com>2025-03-05 01:36:18 -0600
committerJordan Woyak <jordan.woyak@gmail.com>2025-03-15 14:40:00 -0500
commitb2ce3fbefc34fac9449e5f88f15c7a4da0825fac (patch)
tree79efaa3a1b090f897317add4ccabd85747cb0109 /Source/Core
parent225039f742d14eacd704490bb1a66dc119a91320 (diff)
PerformanceTracker: Pass chrono values instead of us s64.
Diffstat (limited to 'Source/Core')
-rw-r--r--Source/Core/VideoCommon/PerformanceMetrics.h2
-rw-r--r--Source/Core/VideoCommon/PerformanceTracker.cpp9
-rw-r--r--Source/Core/VideoCommon/PerformanceTracker.h5
3 files changed, 7 insertions, 9 deletions
diff --git a/Source/Core/VideoCommon/PerformanceMetrics.h b/Source/Core/VideoCommon/PerformanceMetrics.h
index d1f4211a46..672c07037d 100644
--- a/Source/Core/VideoCommon/PerformanceMetrics.h
+++ b/Source/Core/VideoCommon/PerformanceMetrics.h
@@ -47,7 +47,7 @@ public:
private:
PerformanceTracker m_fps_counter{"render_times.txt"};
PerformanceTracker m_vps_counter{"vblank_times.txt"};
- PerformanceTracker m_speed_counter{std::nullopt, 1000000};
+ PerformanceTracker m_speed_counter{std::nullopt, std::chrono::seconds{1}};
double m_graph_max_time = 0.0;
diff --git a/Source/Core/VideoCommon/PerformanceTracker.cpp b/Source/Core/VideoCommon/PerformanceTracker.cpp
index 93b33cff9a..64729a5cad 100644
--- a/Source/Core/VideoCommon/PerformanceTracker.cpp
+++ b/Source/Core/VideoCommon/PerformanceTracker.cpp
@@ -12,21 +12,20 @@
#include "Common/CommonTypes.h"
#include "Common/FileUtil.h"
-#include "Common/Timer.h"
#include "Core/Core.h"
#include "VideoCommon/VideoConfig.h"
static constexpr double SAMPLE_RC_RATIO = 0.25;
PerformanceTracker::PerformanceTracker(const std::optional<std::string> log_name,
- const std::optional<s64> sample_window_us)
+ const std::optional<DT> sample_window_duration)
: m_on_state_changed_handle{Core::AddOnStateChangedCallback([this](Core::State state) {
if (state == Core::State::Paused)
SetPaused(true);
else if (state == Core::State::Running)
SetPaused(false);
})},
- m_log_name{log_name}, m_sample_window_us{sample_window_us}
+ m_log_name{log_name}, m_sample_window_duration{sample_window_duration}
{
Reset();
}
@@ -92,8 +91,8 @@ void PerformanceTracker::Count()
DT PerformanceTracker::GetSampleWindow() const
{
// This reads a constant value and thus does not need a mutex
- return std::chrono::duration_cast<DT>(
- DT_us(m_sample_window_us.value_or(std::max(1, g_ActiveConfig.iPerfSampleUSec))));
+ return m_sample_window_duration.value_or(
+ duration_cast<DT>(DT_us{std::max(1, g_ActiveConfig.iPerfSampleUSec)}));
}
double PerformanceTracker::GetHzAvg() const
diff --git a/Source/Core/VideoCommon/PerformanceTracker.h b/Source/Core/VideoCommon/PerformanceTracker.h
index f24601c845..a41271abaa 100644
--- a/Source/Core/VideoCommon/PerformanceTracker.h
+++ b/Source/Core/VideoCommon/PerformanceTracker.h
@@ -4,7 +4,6 @@
#pragma once
#include <array>
-#include <chrono>
#include <fstream>
#include <optional>
#include <shared_mutex>
@@ -35,7 +34,7 @@ private:
public:
PerformanceTracker(const std::optional<std::string> log_name = std::nullopt,
- const std::optional<s64> sample_window_us = std::nullopt);
+ const std::optional<DT> sample_window_duration = std::nullopt);
~PerformanceTracker();
PerformanceTracker(const PerformanceTracker&) = delete;
@@ -84,7 +83,7 @@ private: // Functions for managing dt queue
TimePoint m_last_time;
// Amount of time to sample dt's over (defaults to config)
- const std::optional<s64> m_sample_window_us;
+ const std::optional<DT> m_sample_window_duration;
// Queue + Running Total used to calculate average dt
DT m_dt_total = DT::zero();