From 1ed9f604b45cad7edc6205e5706ac446297b2dfb Mon Sep 17 00:00:00 2001 From: John Kacur Date: Fri, 7 Feb 2025 15:00:11 -0500 Subject: [PATCH] Make cyclictest the default measurement module for rteval - Make cyclictest the default measurement module for rteval - Make the default interval for timerlat in rteval 1100us Resolves: RHEL-78394 Signed-off-by: John Kacur --- ...ult-measurement-module-to-cyclictest.patch | 45 +++++++++++++++++++ ...ault-interval-for-timerlat-to-1100us.patch | 31 +++++++++++++ rteval.spec | 9 +++- 3 files changed, 84 insertions(+), 1 deletion(-) create mode 100644 rteval-Change-default-measurement-module-to-cyclictest.patch create mode 100644 rteval-Change-the-default-interval-for-timerlat-to-1100us.patch diff --git a/rteval-Change-default-measurement-module-to-cyclictest.patch b/rteval-Change-default-measurement-module-to-cyclictest.patch new file mode 100644 index 0000000..23bf6b8 --- /dev/null +++ b/rteval-Change-default-measurement-module-to-cyclictest.patch @@ -0,0 +1,45 @@ +From 950b6dd5d7812d101bf30ac0b4c94bedcc2a9d23 Mon Sep 17 00:00:00 2001 +From: John Kacur +Date: Fri, 7 Feb 2025 14:36:17 -0500 +Subject: [PATCH 2/2] rteval: Change default measurement module to cyclictest + +Change the default measurement module in rteval to cyclictest +until we can fix a hang in rtla timerlat hist + +Signed-off-by: John Kacur +--- + rteval-cmd | 2 +- + rteval.conf | 4 ++-- + 2 files changed, 3 insertions(+), 3 deletions(-) + +diff --git a/rteval-cmd b/rteval-cmd +index 9875903c0107..4e13d312a24a 100755 +--- a/rteval-cmd ++++ b/rteval-cmd +@@ -250,7 +250,7 @@ if __name__ == '__main__': + + if not config.HasSection('measurement'): + config.AppendConfig('measurement', { +- 'timerlat' : 'module', ++ 'cyclictest' : 'module', + 'sysstat' : 'module'}) + + # Prepare log levels before loading modules, not to have unwanted log messages +diff --git a/rteval.conf b/rteval.conf +index 46a9b1743716..601410b51c28 100644 +--- a/rteval.conf ++++ b/rteval.conf +@@ -6,8 +6,8 @@ duration: 60.0 + report_interval: 600 + + [measurement] +-# cyclictest: module +-timerlat: module ++cyclictest: module ++# timerlat: module + + [loads] + kcompile: module +-- +2.48.1 + diff --git a/rteval-Change-the-default-interval-for-timerlat-to-1100us.patch b/rteval-Change-the-default-interval-for-timerlat-to-1100us.patch new file mode 100644 index 0000000..8aa0797 --- /dev/null +++ b/rteval-Change-the-default-interval-for-timerlat-to-1100us.patch @@ -0,0 +1,31 @@ +From 686fefa1e63da70b0c3e4da9ea816e334fef1782 Mon Sep 17 00:00:00 2001 +From: John Kacur +Date: Fri, 7 Feb 2025 14:31:14 -0500 +Subject: [PATCH 1/2] rteval: Change the default interval for timerlat to + 1100us + +Change the default interval for timerlat in rteval to 1100us to make it less +likely to encounter a hang in rtla timerlat hist, until we have a better +solution. + +Signed-off-by: John Kacur +--- + rteval/modules/measurement/timerlat.py | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/rteval/modules/measurement/timerlat.py b/rteval/modules/measurement/timerlat.py +index 5bfc495217ea..fe29f34bccbd 100644 +--- a/rteval/modules/measurement/timerlat.py ++++ b/rteval/modules/measurement/timerlat.py +@@ -509,7 +509,7 @@ class Timerlat(rtevalModulePrototype): + def ModuleParameters(): + """ default parameters """ + return {"interval": {"descr": "Base interval or period of threads in microseconds", +- "default": 100, ++ "default": 1100, + "metavar": "INTV_US"}, + "priority": {"descr": "Run rtla timerlat with this priority", + "default": 95, +-- +2.48.1 + diff --git a/rteval.spec b/rteval.spec index eb1a5fd..4858c0a 100644 --- a/rteval.spec +++ b/rteval.spec @@ -1,6 +1,6 @@ Name: rteval Version: 3.9 -Release: 2%{?dist} +Release: 3%{?dist} Summary: Utility to evaluate system suitability for RT Linux Group: Development/Tools @@ -34,6 +34,8 @@ BuildArch: noarch #Patches Patch1: rteval-Fix-missing-statistics-for-system-with-timerl.patch +Patch2: rteval-Change-the-default-interval-for-timerlat-to-1100us.patch +Patch3: rteval-Change-default-measurement-module-to-cyclictest.patch %description The rteval script is a utility for measuring various aspects of @@ -67,6 +69,11 @@ to the screen. %{_bindir}/rteval %changelog +* Fri Feb 07 2025 John Kacur - 3.9-3 +- Make the default interval for timerlat in rteval 1100us +- Make cyclictest the default measurement module for rteval +Resolves: RHEL-78394 + * Fri Dec 13 2024 John Kacur - 3.9-2 - Fix missing system statistics Resolves: RHEL-70166