Compare commits
No commits in common. "c8" and "imports/c8s/rteval-3.3-6.el8" have entirely different histories.
c8
...
imports/c8
2
.gitignore
vendored
2
.gitignore
vendored
@ -1 +1 @@
|
|||||||
SOURCES/rteval-3.7.tar.xz
|
SOURCES/rteval-3.3.tar.xz
|
||||||
|
1
.rteval.metadata
Normal file
1
.rteval.metadata
Normal file
@ -0,0 +1 @@
|
|||||||
|
557793c55592bfb3dc0e858274895af4347a2fe9 SOURCES/rteval-3.3.tar.xz
|
@ -1,68 +0,0 @@
|
|||||||
From 47fc74501aa5741fd5dcb2d04aacd857d3d87740 Mon Sep 17 00:00:00 2001
|
|
||||||
From: John Kacur <jkacur@redhat.com>
|
|
||||||
Date: Wed, 25 Oct 2023 10:59:28 -0400
|
|
||||||
Subject: [PATCH] Revert "rteval: Change the default kernel to compile to
|
|
||||||
linux-6.1.8"
|
|
||||||
|
|
||||||
This reverts commit 0f39c69610985b07ce2aa41142d2f0481da8e3a4.
|
|
||||||
|
|
||||||
For RHEL-8.x we want to continue using the same kernel for kcompile
|
|
||||||
|
|
||||||
Signed-off-by: John Kacur <jkacur@redhat.com>
|
|
||||||
---
|
|
||||||
Makefile | 2 +-
|
|
||||||
rteval/modules/loads/kcompile.py | 4 ++--
|
|
||||||
rteval/rteval.conf | 2 +-
|
|
||||||
3 files changed, 4 insertions(+), 4 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/Makefile b/Makefile
|
|
||||||
index b73e8c13f3e5..176435a8cfd5 100644
|
|
||||||
--- a/Makefile
|
|
||||||
+++ b/Makefile
|
|
||||||
@@ -18,7 +18,7 @@ PREFIX := /usr
|
|
||||||
DATADIR := $(DESTDIR)/$(PREFIX)/share
|
|
||||||
LOADDIR := loadsource
|
|
||||||
|
|
||||||
-KLOAD := $(LOADDIR)/linux-6.1.8.tar.xz
|
|
||||||
+KLOAD := $(LOADDIR)/linux-5.18.1.tar.xz
|
|
||||||
BLOAD := $(LOADDIR)/dbench-4.0.tar.gz
|
|
||||||
LOADS := $(KLOAD) $(BLOAD)
|
|
||||||
|
|
||||||
diff --git a/rteval/modules/loads/kcompile.py b/rteval/modules/loads/kcompile.py
|
|
||||||
index 8be79ce630d5..e531b60754e9 100644
|
|
||||||
--- a/rteval/modules/loads/kcompile.py
|
|
||||||
+++ b/rteval/modules/loads/kcompile.py
|
|
||||||
@@ -20,7 +20,7 @@ expand_cpulist = CpuList.expand_cpulist
|
|
||||||
compress_cpulist = CpuList.compress_cpulist
|
|
||||||
nonisolated_cpulist = CpuList.nonisolated_cpulist
|
|
||||||
|
|
||||||
-DEFAULT_KERNEL_PREFIX = "linux-6.1"
|
|
||||||
+DEFAULT_KERNEL_PREFIX = "linux-5.18"
|
|
||||||
|
|
||||||
class KBuildJob:
|
|
||||||
'''Class to manage a build job bound to a particular node'''
|
|
||||||
@@ -334,7 +334,7 @@ class Kcompile(CommandLineLoad):
|
|
||||||
|
|
||||||
def ModuleParameters():
|
|
||||||
return {"source": {"descr": "Source tar ball",
|
|
||||||
- "default": "linux-6.1.8.tar.xz",
|
|
||||||
+ "default": "linux-5.18.1.tar.xz",
|
|
||||||
"metavar": "TARBALL"},
|
|
||||||
"jobspercore": {"descr": "Number of working threads per core",
|
|
||||||
"default": 2,
|
|
||||||
diff --git a/rteval/rteval.conf b/rteval/rteval.conf
|
|
||||||
index 79e28032dc6b..1a8d0afd2642 100644
|
|
||||||
--- a/rteval/rteval.conf
|
|
||||||
+++ b/rteval/rteval.conf
|
|
||||||
@@ -18,7 +18,7 @@ dbench: external
|
|
||||||
stressng: module
|
|
||||||
|
|
||||||
[kcompile]
|
|
||||||
-source: linux-6.1.8.xz
|
|
||||||
+source: linux-5.18.1.xz
|
|
||||||
jobspercore: 2
|
|
||||||
|
|
||||||
[hackbench]
|
|
||||||
--
|
|
||||||
2.41.0
|
|
||||||
|
|
@ -1,278 +0,0 @@
|
|||||||
From 64ce7848dfabd2056d35c8a60f3354db45e36286 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Date: Thu, 18 Jan 2024 10:18:10 +0100
|
|
||||||
Subject: [PATCH 2/4] rteval: Add relative cpulists for measurements
|
|
||||||
|
|
||||||
Instead of specifying an absolute list of CPUs to run measurements on
|
|
||||||
in --measurement-cpulist, implement an option to specify a relative list
|
|
||||||
with respect to the current cpuset of rteval.
|
|
||||||
|
|
||||||
The relative cpulist can include CPUs both for addition and for removal,
|
|
||||||
e.g. +0,1,-7,8.
|
|
||||||
|
|
||||||
Also move the logic for processing cpulists specified by the user as
|
|
||||||
a string into cpulists usable by rteval to a single function.
|
|
||||||
|
|
||||||
Signed-off-by: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Signed-off-by: John Kacur <jkacur@redhat.com>
|
|
||||||
---
|
|
||||||
rteval-cmd | 26 +++++++-----
|
|
||||||
rteval/cpulist_utils.py | 33 ++++++++++++++++
|
|
||||||
rteval/modules/measurement/__init__.py | 9 +----
|
|
||||||
rteval/modules/measurement/cyclictest.py | 50 +++---------------------
|
|
||||||
rteval/systopology.py | 33 ++++++++++++++++
|
|
||||||
5 files changed, 90 insertions(+), 61 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/rteval-cmd b/rteval-cmd
|
|
||||||
index d224728..a5e8746 100755
|
|
||||||
--- a/rteval-cmd
|
|
||||||
+++ b/rteval-cmd
|
|
||||||
@@ -30,7 +30,7 @@ from rteval import RtEval, rtevalConfig
|
|
||||||
from rteval.modules.loads import LoadModules
|
|
||||||
from rteval.modules.measurement import MeasurementModules
|
|
||||||
from rteval.version import RTEVAL_VERSION
|
|
||||||
-from rteval.systopology import SysTopology
|
|
||||||
+from rteval.systopology import SysTopology, parse_cpulist_from_config
|
|
||||||
from rteval.modules.loads.kcompile import ModuleParameters
|
|
||||||
import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
@@ -339,26 +339,32 @@ if __name__ == '__main__':
|
|
||||||
|
|
||||||
ldcfg = config.GetSection('loads')
|
|
||||||
msrcfg = config.GetSection('measurement')
|
|
||||||
- if ldcfg.cpulist and msrcfg.cpulist:
|
|
||||||
+ msrcfg_cpulist_present = msrcfg.cpulist != ""
|
|
||||||
+ # Parse measurement cpulist using parse_cpulist_from_config to account for run-on-isolcpus
|
|
||||||
+ # and relative cpusets
|
|
||||||
+ cpulist = parse_cpulist_from_config(msrcfg.cpulist, msrcfg.run_on_isolcpus)
|
|
||||||
+ if msrcfg_cpulist_present and not cpulist_utils.is_relative(msrcfg.cpulist) and msrcfg.run_on_isolcpus:
|
|
||||||
+ logger.log(Log.WARN, "ignoring --measurement-run-on-isolcpus, since cpulist is specified")
|
|
||||||
+ msrcfg.cpulist = collapse_cpulist(cpulist)
|
|
||||||
+ if ldcfg.cpulist:
|
|
||||||
ldcfg.cpulist = remove_offline(ldcfg.cpulist)
|
|
||||||
- msrcfg.cpulist = remove_offline(msrcfg.cpulist)
|
|
||||||
# if we only specified one set of cpus (loads or measurement)
|
|
||||||
# default the other to the inverse of the specified list
|
|
||||||
- if not ldcfg.cpulist and msrcfg.cpulist:
|
|
||||||
+ if not ldcfg.cpulist and msrcfg_cpulist_present:
|
|
||||||
tmplist = expand_cpulist(msrcfg.cpulist)
|
|
||||||
tmplist = SysTopology().invert_cpulist(tmplist)
|
|
||||||
- ldcfg.cpulist = compress_cpulist(tmplist)
|
|
||||||
- msrcfg.cpulist = remove_offline(msrcfg.cpulist)
|
|
||||||
- if not msrcfg.cpulist and ldcfg.cpulist:
|
|
||||||
+ tmplist = cpulist_utils.online_cpulist(tmplist)
|
|
||||||
+ ldcfg.cpulist = collapse_cpulist(tmplist)
|
|
||||||
+ if not msrcfg_cpulist_present and ldcfg.cpulist:
|
|
||||||
tmplist = expand_cpulist(ldcfg.cpulist)
|
|
||||||
tmplist = SysTopology().invert_cpulist(tmplist)
|
|
||||||
- msrcfg.cpulist = compress_cpulist(tmplist)
|
|
||||||
- ldcfg.cpulist = remove_offline(ldcfg.cpulist)
|
|
||||||
+ tmplist = cpulist_utils.online_cpulist(tmplist)
|
|
||||||
+ msrcfg.cpulist = collapse_cpulist(tmplist)
|
|
||||||
|
|
||||||
if ldcfg.cpulist:
|
|
||||||
logger.log(Log.DEBUG, f"loads cpulist: {ldcfg.cpulist}")
|
|
||||||
# if --onlyload is specified msrcfg.cpulist is unused
|
|
||||||
- if msrcfg.cpulist and not rtevcfg.onlyload:
|
|
||||||
+ if msrcfg_cpulist_present and not rtevcfg.onlyload:
|
|
||||||
logger.log(Log.DEBUG, f"measurement cpulist: {msrcfg.cpulist}")
|
|
||||||
logger.log(Log.DEBUG, f"workdir: {rtevcfg.workdir}")
|
|
||||||
|
|
||||||
diff --git a/rteval/cpulist_utils.py b/rteval/cpulist_utils.py
|
|
||||||
index 402d579..7abc45a 100644
|
|
||||||
--- a/rteval/cpulist_utils.py
|
|
||||||
+++ b/rteval/cpulist_utils.py
|
|
||||||
@@ -126,3 +126,36 @@ def nonisolated_cpulist(cpulist):
|
|
||||||
isolated_cpulist = sysread(cpupath, "isolated")
|
|
||||||
isolated_cpulist = expand_cpulist(isolated_cpulist)
|
|
||||||
return list(set(cpulist).difference(set(isolated_cpulist)))
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def is_relative(cpulist):
|
|
||||||
+ return cpulist.startswith("+") or cpulist.startswith("-")
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def expand_relative_cpulist(cpulist):
|
|
||||||
+ """
|
|
||||||
+ Expand a relative cpulist into a tuple of lists.
|
|
||||||
+ :param cpulist: Relative cpulist of form +1,2,3,-4,5,6
|
|
||||||
+ :return: Tuple of two lists, one for added CPUs, one for removed CPUs
|
|
||||||
+ """
|
|
||||||
+ added_cpus = []
|
|
||||||
+ removed_cpus = []
|
|
||||||
+
|
|
||||||
+ if not cpulist:
|
|
||||||
+ return added_cpus, removed_cpus
|
|
||||||
+
|
|
||||||
+ cpus = None
|
|
||||||
+
|
|
||||||
+ for part in cpulist.split(','):
|
|
||||||
+ if part.startswith('+') or part.startswith('-'):
|
|
||||||
+ cpus = added_cpus if part[0] == '+' else removed_cpus
|
|
||||||
+ part = part[1:]
|
|
||||||
+ if '-' in part:
|
|
||||||
+ a, b = part.split('-')
|
|
||||||
+ a, b = int(a), int(b)
|
|
||||||
+ cpus.extend(list(range(a, b + 1)))
|
|
||||||
+ else:
|
|
||||||
+ a = int(part)
|
|
||||||
+ cpus.append(a)
|
|
||||||
+
|
|
||||||
+ return list(set(added_cpus)), list(set(removed_cpus))
|
|
||||||
diff --git a/rteval/modules/measurement/__init__.py b/rteval/modules/measurement/__init__.py
|
|
||||||
index 66dc9c5..11bd7b0 100644
|
|
||||||
--- a/rteval/modules/measurement/__init__.py
|
|
||||||
+++ b/rteval/modules/measurement/__init__.py
|
|
||||||
@@ -5,7 +5,7 @@
|
|
||||||
|
|
||||||
import libxml2
|
|
||||||
from rteval.modules import RtEvalModules, ModuleContainer
|
|
||||||
-from rteval.systopology import SysTopology as SysTop
|
|
||||||
+from rteval.systopology import parse_cpulist_from_config
|
|
||||||
import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
class MeasurementProfile(RtEvalModules):
|
|
||||||
@@ -183,12 +183,7 @@ measurement profiles, based on their characteristics"""
|
|
||||||
rep_n = libxml2.newNode("Measurements")
|
|
||||||
cpulist = self.__cfg.GetSection("measurement").cpulist
|
|
||||||
run_on_isolcpus = self.__cfg.GetSection("measurement").run_on_isolcpus
|
|
||||||
- if cpulist:
|
|
||||||
- # Convert str to list and remove offline cpus
|
|
||||||
- cpulist = cpulist_utils.expand_cpulist(cpulist)
|
|
||||||
- cpulist = cpulist_utils.online_cpulist(cpulist)
|
|
||||||
- else:
|
|
||||||
- cpulist = SysTop().online_cpus() if run_on_isolcpus else SysTop().default_cpus()
|
|
||||||
+ cpulist = parse_cpulist_from_config(cpulist, run_on_isolcpus)
|
|
||||||
rep_n.newProp("measurecpus", cpulist_utils.collapse_cpulist(cpulist))
|
|
||||||
|
|
||||||
for mp in self.__measureprofiles:
|
|
||||||
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
|
||||||
index fdca257..7224225 100644
|
|
||||||
--- a/rteval/modules/measurement/cyclictest.py
|
|
||||||
+++ b/rteval/modules/measurement/cyclictest.py
|
|
||||||
@@ -16,8 +16,7 @@ import math
|
|
||||||
import libxml2
|
|
||||||
from rteval.Log import Log
|
|
||||||
from rteval.modules import rtevalModulePrototype
|
|
||||||
-from rteval.systopology import cpuinfo
|
|
||||||
-from rteval.systopology import SysTopology
|
|
||||||
+from rteval.systopology import cpuinfo, parse_cpulist_from_config
|
|
||||||
import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
expand_cpulist = cpulist_utils.expand_cpulist
|
|
||||||
@@ -193,39 +192,9 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
self.__priority = int(self.__cfg.setdefault('priority', 95))
|
|
||||||
self.__buckets = int(self.__cfg.setdefault('buckets', 2000))
|
|
||||||
self.__numcores = 0
|
|
||||||
- self.__cpus = []
|
|
||||||
self.__cyclicdata = {}
|
|
||||||
- self.__sparse = False
|
|
||||||
- self.__run_on_isolcpus = bool(self.__cfg.setdefault('run-on-isolcpus', False))
|
|
||||||
-
|
|
||||||
- if self.__cfg.cpulist:
|
|
||||||
- self.__cpulist = self.__cfg.cpulist
|
|
||||||
- self.__cpus = expand_cpulist(self.__cpulist)
|
|
||||||
- # Only include online cpus
|
|
||||||
- self.__cpus = cpulist_utils.online_cpulist(self.__cpus)
|
|
||||||
- # Reset cpulist from the newly calculated self.__cpus
|
|
||||||
- self.__cpulist = cpulist_utils.collapse_cpulist(self.__cpus)
|
|
||||||
- self.__cpus = [str(c) for c in self.__cpus]
|
|
||||||
- self.__sparse = True
|
|
||||||
- if self.__run_on_isolcpus:
|
|
||||||
- self._log(Log.WARN, "ignoring --measurement-run-on-isolcpus, since cpulist is specified")
|
|
||||||
- else:
|
|
||||||
- self.__cpus = SysTopology().online_cpus_str()
|
|
||||||
- # Get the cpuset from the environment
|
|
||||||
- cpuset = os.sched_getaffinity(0)
|
|
||||||
- # Convert the elements to strings
|
|
||||||
- cpuset = [str(c) for c in cpuset]
|
|
||||||
- # Get isolated CPU list
|
|
||||||
- isolcpus = [str(c) for c in SysTopology().isolated_cpus()]
|
|
||||||
- # Only include cpus that are in the cpuset and isolated CPUs if run_on_isolcpus is enabled
|
|
||||||
- self.__cpus = [c for c in self.__cpus if c in cpuset or self.__run_on_isolcpus and c in isolcpus]
|
|
||||||
- if self.__run_on_isolcpus:
|
|
||||||
- self.__sparse = True
|
|
||||||
- self.__cpulist = cpulist_utils.collapse_cpulist([int(c) for c in self.__cpus])
|
|
||||||
-
|
|
||||||
- # Sort the list of cpus to align with the order reported by cyclictest
|
|
||||||
- self.__cpus.sort(key=int)
|
|
||||||
-
|
|
||||||
+ self.__cpulist = self.__cfg.cpulist
|
|
||||||
+ self.__cpus = [str(c) for c in expand_cpulist(self.__cpulist)]
|
|
||||||
self.__numcores = len(self.__cpus)
|
|
||||||
|
|
||||||
info = cpuinfo()
|
|
||||||
@@ -242,10 +211,7 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
logfnc=self._log)
|
|
||||||
self.__cyclicdata['system'].description = (f"({self.__numcores} cores) ") + info['0']['model name']
|
|
||||||
|
|
||||||
- if self.__sparse:
|
|
||||||
- self._log(Log.DEBUG, f"system using {self.__numcores} cpu cores")
|
|
||||||
- else:
|
|
||||||
- self._log(Log.DEBUG, f"system has {self.__numcores} cpu cores")
|
|
||||||
+ self._log(Log.DEBUG, f"system using {self.__numcores} cpu cores")
|
|
||||||
self.__started = False
|
|
||||||
self.__cyclicoutput = None
|
|
||||||
self.__breaktraceval = None
|
|
||||||
@@ -280,12 +246,8 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
f'-h {self.__buckets}',
|
|
||||||
f"-p{int(self.__priority)}",
|
|
||||||
]
|
|
||||||
- if self.__sparse:
|
|
||||||
- self.__cmd.append(f'-t{self.__numcores}')
|
|
||||||
- self.__cmd.append(f'-a{self.__cpulist}')
|
|
||||||
- else:
|
|
||||||
- self.__cmd.append('-t')
|
|
||||||
- self.__cmd.append('-a')
|
|
||||||
+ self.__cmd.append(f'-t{self.__numcores}')
|
|
||||||
+ self.__cmd.append(f'-a{self.__cpulist}')
|
|
||||||
|
|
||||||
if 'threads' in self.__cfg and self.__cfg.threads:
|
|
||||||
self.__cmd.append(f"-t{int(self.__cfg.threads)}")
|
|
||||||
diff --git a/rteval/systopology.py b/rteval/systopology.py
|
|
||||||
index 9e45762..6bcfc77 100644
|
|
||||||
--- a/rteval/systopology.py
|
|
||||||
+++ b/rteval/systopology.py
|
|
||||||
@@ -241,6 +241,39 @@ class SysTopology:
|
|
||||||
""" return a list of online cpus in cpulist """
|
|
||||||
return [c for c in self.online_cpus() if c in cpulist]
|
|
||||||
|
|
||||||
+
|
|
||||||
+def parse_cpulist_from_config(cpulist, run_on_isolcpus=False):
|
|
||||||
+ """
|
|
||||||
+ Generates a cpulist based on --*-cpulist argument given by user
|
|
||||||
+ :param cpulist: Value of --*-cpulist argument
|
|
||||||
+ :param run_on_isolcpus: Value of --*-run-on-isolcpus argument
|
|
||||||
+ :return: Sorted list of CPUs as integers
|
|
||||||
+ """
|
|
||||||
+ if cpulist and not cpulist_utils.is_relative(cpulist):
|
|
||||||
+ result = cpulist_utils.expand_cpulist(cpulist)
|
|
||||||
+ # Only include online cpus
|
|
||||||
+ result = cpulist_utils.online_cpulist(result)
|
|
||||||
+ else:
|
|
||||||
+ result = SysTopology().online_cpus()
|
|
||||||
+ # Get the cpuset from the environment
|
|
||||||
+ cpuset = os.sched_getaffinity(0)
|
|
||||||
+ # Get isolated CPU list
|
|
||||||
+ isolcpus = SysTopology().isolated_cpus()
|
|
||||||
+ if cpulist and cpulist_utils.is_relative(cpulist):
|
|
||||||
+ # Include cpus that are not removed in relative cpuset and are either in cpuset from affinity,
|
|
||||||
+ # isolcpus (with run_on_isolcpus enabled, or added by relative cpuset
|
|
||||||
+ added_cpus, removed_cpus = cpulist_utils.expand_relative_cpulist(cpulist)
|
|
||||||
+ result = [c for c in result
|
|
||||||
+ if (c in cpuset or
|
|
||||||
+ c in added_cpus or
|
|
||||||
+ run_on_isolcpus and c in isolcpus) and
|
|
||||||
+ c not in removed_cpus]
|
|
||||||
+ else:
|
|
||||||
+ # Only include cpus that are in the cpuset and isolated CPUs if run_on_isolcpus is enabled
|
|
||||||
+ result = [c for c in result if c in cpuset or run_on_isolcpus and c in isolcpus]
|
|
||||||
+ return result
|
|
||||||
+
|
|
||||||
+
|
|
||||||
if __name__ == "__main__":
|
|
||||||
|
|
||||||
def unit_test():
|
|
||||||
--
|
|
||||||
2.43.0
|
|
||||||
|
|
@ -1,616 +0,0 @@
|
|||||||
From a788ea3ec976e51309f0bfae1a41cce704a77f2d Mon Sep 17 00:00:00 2001
|
|
||||||
From: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Date: Thu, 18 Jan 2024 10:18:09 +0100
|
|
||||||
Subject: [PATCH 1/4] rteval: Convert CpuList class to a module
|
|
||||||
|
|
||||||
Move out code from CpuList class in rteval.systopology into a separate
|
|
||||||
module named rteval.cpulist_utils and avoid wrapping CPU lists in
|
|
||||||
a CpuList object.
|
|
||||||
|
|
||||||
Almost all uses of CpuList in the code either use the static methods of
|
|
||||||
the class or its constructor to filter online CPUs by running
|
|
||||||
CpuList(...).cpulist. The only exception to this are NumaNode and
|
|
||||||
SimNumaNode classes in systopology; these store a CpuList object,
|
|
||||||
however their .getcpulist() method extracts the list out. Thus,
|
|
||||||
the class is completely unnecessary.
|
|
||||||
|
|
||||||
Note: A better name for the module would be cpulist, consistent with the
|
|
||||||
original name of the class, but this name is already used for variables
|
|
||||||
throughout the code, hence cpulist_utils is used instead.
|
|
||||||
|
|
||||||
Signed-off-by: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Signed-off-by: John Kacur <jkacur@redhat.com>
|
|
||||||
---
|
|
||||||
rteval-cmd | 10 +-
|
|
||||||
rteval/cpulist_utils.py | 128 ++++++++++++++++++
|
|
||||||
rteval/modules/loads/__init__.py | 8 +-
|
|
||||||
rteval/modules/loads/hackbench.py | 9 +-
|
|
||||||
rteval/modules/loads/kcompile.py | 11 +-
|
|
||||||
rteval/modules/loads/stressng.py | 8 +-
|
|
||||||
rteval/modules/measurement/__init__.py | 8 +-
|
|
||||||
rteval/modules/measurement/cyclictest.py | 11 +-
|
|
||||||
rteval/systopology.py | 165 ++---------------------
|
|
||||||
9 files changed, 177 insertions(+), 181 deletions(-)
|
|
||||||
create mode 100644 rteval/cpulist_utils.py
|
|
||||||
|
|
||||||
diff --git a/rteval-cmd b/rteval-cmd
|
|
||||||
index 7c41429..d224728 100755
|
|
||||||
--- a/rteval-cmd
|
|
||||||
+++ b/rteval-cmd
|
|
||||||
@@ -30,11 +30,13 @@ from rteval import RtEval, rtevalConfig
|
|
||||||
from rteval.modules.loads import LoadModules
|
|
||||||
from rteval.modules.measurement import MeasurementModules
|
|
||||||
from rteval.version import RTEVAL_VERSION
|
|
||||||
-from rteval.systopology import CpuList, SysTopology
|
|
||||||
+from rteval.systopology import SysTopology
|
|
||||||
from rteval.modules.loads.kcompile import ModuleParameters
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
-compress_cpulist = CpuList.compress_cpulist
|
|
||||||
-expand_cpulist = CpuList.expand_cpulist
|
|
||||||
+compress_cpulist = cpulist_utils.compress_cpulist
|
|
||||||
+expand_cpulist = cpulist_utils.expand_cpulist
|
|
||||||
+collapse_cpulist = cpulist_utils.collapse_cpulist
|
|
||||||
|
|
||||||
def summarize(repfile, xslt):
|
|
||||||
""" Summarize an already existing XML report """
|
|
||||||
@@ -211,7 +213,7 @@ def remove_offline(cpulist):
|
|
||||||
""" return cpulist in collapsed compressed form with only online cpus """
|
|
||||||
tmplist = expand_cpulist(cpulist)
|
|
||||||
tmplist = SysTopology().online_cpulist(tmplist)
|
|
||||||
- return CpuList.collapse_cpulist(tmplist)
|
|
||||||
+ return collapse_cpulist(tmplist)
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == '__main__':
|
|
||||||
diff --git a/rteval/cpulist_utils.py b/rteval/cpulist_utils.py
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000..402d579
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/rteval/cpulist_utils.py
|
|
||||||
@@ -0,0 +1,128 @@
|
|
||||||
+# -*- coding: utf-8 -*-
|
|
||||||
+# SPDX-License-Identifier: GPL-2.0-or-later
|
|
||||||
+#
|
|
||||||
+# Copyright 2016 - Clark Williams <williams@redhat.com>
|
|
||||||
+# Copyright 2021 - John Kacur <jkacur@redhat.com>
|
|
||||||
+# Copyright 2023 - Tomas Glozar <tglozar@redhat.com>
|
|
||||||
+#
|
|
||||||
+"""Module providing utility functions for working with CPU lists"""
|
|
||||||
+
|
|
||||||
+import os
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+cpupath = "/sys/devices/system/cpu"
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def sysread(path, obj):
|
|
||||||
+ """ Helper function for reading system files """
|
|
||||||
+ with open(os.path.join(path, obj), "r") as fp:
|
|
||||||
+ return fp.readline().strip()
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def _online_file_exists():
|
|
||||||
+ """ Check whether machine / kernel is configured with online file """
|
|
||||||
+ # Note: some machines do not have cpu0/online so we check cpu1/online.
|
|
||||||
+ # In the case of machines with a single CPU, there is no cpu1, but
|
|
||||||
+ # that is not a problem, since a single CPU cannot be offline
|
|
||||||
+ return os.path.exists(os.path.join(cpupath, "cpu1/online"))
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def _isolated_file_exists():
|
|
||||||
+ """ Check whether machine / kernel is configured with isolated file """
|
|
||||||
+ return os.path.exists(os.path.join(cpupath, "isolated"))
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def collapse_cpulist(cpulist):
|
|
||||||
+ """
|
|
||||||
+ Collapse a list of cpu numbers into a string range
|
|
||||||
+ of cpus (e.g. 0-5, 7, 9)
|
|
||||||
+ """
|
|
||||||
+ cur_range = [None, None]
|
|
||||||
+ result = []
|
|
||||||
+ for cpu in cpulist + [None]:
|
|
||||||
+ if cur_range[0] is None:
|
|
||||||
+ cur_range[0] = cur_range[1] = cpu
|
|
||||||
+ continue
|
|
||||||
+ if cpu is not None and cpu == cur_range[1] + 1:
|
|
||||||
+ # Extend currently processed range
|
|
||||||
+ cur_range[1] += 1
|
|
||||||
+ else:
|
|
||||||
+ # Range processing finished, add range to string
|
|
||||||
+ result.append(f"{cur_range[0]}-{cur_range[1]}"
|
|
||||||
+ if cur_range[0] != cur_range[1]
|
|
||||||
+ else str(cur_range[0]))
|
|
||||||
+ # Reset
|
|
||||||
+ cur_range[0] = cur_range[1] = cpu
|
|
||||||
+ return ",".join(result)
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def compress_cpulist(cpulist):
|
|
||||||
+ """ return a string representation of cpulist """
|
|
||||||
+ if not cpulist:
|
|
||||||
+ return ""
|
|
||||||
+ if isinstance(cpulist[0], int):
|
|
||||||
+ return ",".join(str(e) for e in cpulist)
|
|
||||||
+ return ",".join(cpulist)
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def expand_cpulist(cpulist):
|
|
||||||
+ """ expand a range string into an array of cpu numbers
|
|
||||||
+ don't error check against online cpus
|
|
||||||
+ """
|
|
||||||
+ result = []
|
|
||||||
+
|
|
||||||
+ if not cpulist:
|
|
||||||
+ return result
|
|
||||||
+
|
|
||||||
+ for part in cpulist.split(','):
|
|
||||||
+ if '-' in part:
|
|
||||||
+ a, b = part.split('-')
|
|
||||||
+ a, b = int(a), int(b)
|
|
||||||
+ result.extend(list(range(a, b + 1)))
|
|
||||||
+ else:
|
|
||||||
+ a = int(part)
|
|
||||||
+ result.append(a)
|
|
||||||
+ return [int(i) for i in list(set(result))]
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def is_online(n):
|
|
||||||
+ """ check whether cpu n is online """
|
|
||||||
+ path = os.path.join(cpupath, f'cpu{n}')
|
|
||||||
+
|
|
||||||
+ # Some hardware doesn't allow cpu0 to be turned off
|
|
||||||
+ if not os.path.exists(path + '/online') and n == 0:
|
|
||||||
+ return True
|
|
||||||
+
|
|
||||||
+ return sysread(path, "online") == "1"
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def online_cpulist(cpulist):
|
|
||||||
+ """ Given a cpulist, return a cpulist of online cpus """
|
|
||||||
+ # This only works if the sys online files exist
|
|
||||||
+ if not _online_file_exists():
|
|
||||||
+ return cpulist
|
|
||||||
+ newlist = []
|
|
||||||
+ for cpu in cpulist:
|
|
||||||
+ if not _online_file_exists() and cpu == '0':
|
|
||||||
+ newlist.append(cpu)
|
|
||||||
+ elif is_online(int(cpu)):
|
|
||||||
+ newlist.append(cpu)
|
|
||||||
+ return newlist
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def isolated_cpulist(cpulist):
|
|
||||||
+ """Given a cpulist, return a cpulist of isolated CPUs"""
|
|
||||||
+ if not _isolated_file_exists():
|
|
||||||
+ return cpulist
|
|
||||||
+ isolated_cpulist = sysread(cpupath, "isolated")
|
|
||||||
+ isolated_cpulist = expand_cpulist(isolated_cpulist)
|
|
||||||
+ return list(set(isolated_cpulist) & set(cpulist))
|
|
||||||
+
|
|
||||||
+
|
|
||||||
+def nonisolated_cpulist(cpulist):
|
|
||||||
+ """Given a cpulist, return a cpulist of non-isolated CPUs"""
|
|
||||||
+ if not _isolated_file_exists():
|
|
||||||
+ return cpulist
|
|
||||||
+ isolated_cpulist = sysread(cpupath, "isolated")
|
|
||||||
+ isolated_cpulist = expand_cpulist(isolated_cpulist)
|
|
||||||
+ return list(set(cpulist).difference(set(isolated_cpulist)))
|
|
||||||
diff --git a/rteval/modules/loads/__init__.py b/rteval/modules/loads/__init__.py
|
|
||||||
index 13fba1e..0845742 100644
|
|
||||||
--- a/rteval/modules/loads/__init__.py
|
|
||||||
+++ b/rteval/modules/loads/__init__.py
|
|
||||||
@@ -11,7 +11,8 @@ import libxml2
|
|
||||||
from rteval.Log import Log
|
|
||||||
from rteval.rtevalConfig import rtevalCfgSection
|
|
||||||
from rteval.modules import RtEvalModules, rtevalModulePrototype
|
|
||||||
-from rteval.systopology import CpuList, SysTopology as SysTop
|
|
||||||
+from rteval.systopology import SysTopology as SysTop
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
class LoadThread(rtevalModulePrototype):
|
|
||||||
def __init__(self, name, config, logger=None):
|
|
||||||
@@ -117,10 +118,11 @@ class LoadModules(RtEvalModules):
|
|
||||||
cpulist = self._cfg.GetSection(self._module_config).cpulist
|
|
||||||
if cpulist:
|
|
||||||
# Convert str to list and remove offline cpus
|
|
||||||
- cpulist = CpuList(cpulist).cpulist
|
|
||||||
+ cpulist = cpulist_utils.expand_cpulist(cpulist)
|
|
||||||
+ cpulist = cpulist_utils.online_cpulist(cpulist)
|
|
||||||
else:
|
|
||||||
cpulist = SysTop().default_cpus()
|
|
||||||
- rep_n.newProp("loadcpus", CpuList.collapse_cpulist(cpulist))
|
|
||||||
+ rep_n.newProp("loadcpus", cpulist_utils.collapse_cpulist(cpulist))
|
|
||||||
|
|
||||||
return rep_n
|
|
||||||
|
|
||||||
diff --git a/rteval/modules/loads/hackbench.py b/rteval/modules/loads/hackbench.py
|
|
||||||
index cfc7063..a70fdb3 100644
|
|
||||||
--- a/rteval/modules/loads/hackbench.py
|
|
||||||
+++ b/rteval/modules/loads/hackbench.py
|
|
||||||
@@ -16,10 +16,11 @@ import errno
|
|
||||||
from signal import SIGKILL
|
|
||||||
from rteval.modules.loads import CommandLineLoad
|
|
||||||
from rteval.Log import Log
|
|
||||||
-from rteval.systopology import CpuList, SysTopology
|
|
||||||
+from rteval.systopology import SysTopology
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
-expand_cpulist = CpuList.expand_cpulist
|
|
||||||
-isolated_cpulist = CpuList.isolated_cpulist
|
|
||||||
+expand_cpulist = cpulist_utils.expand_cpulist
|
|
||||||
+isolated_cpulist = cpulist_utils.isolated_cpulist
|
|
||||||
|
|
||||||
class Hackbench(CommandLineLoad):
|
|
||||||
def __init__(self, config, logger):
|
|
||||||
@@ -61,7 +62,7 @@ class Hackbench(CommandLineLoad):
|
|
||||||
self.cpus[n] = [c for c in self.cpus[n] if c in expand_cpulist(self.cpulist)]
|
|
||||||
# if a cpulist was not specified, exclude isolated cpus
|
|
||||||
else:
|
|
||||||
- self.cpus[n] = CpuList.nonisolated_cpulist(self.cpus[n])
|
|
||||||
+ self.cpus[n] = cpulist_utils.nonisolated_cpulist(self.cpus[n])
|
|
||||||
|
|
||||||
# track largest number of cpus used on a node
|
|
||||||
node_biggest = len(self.cpus[n])
|
|
||||||
diff --git a/rteval/modules/loads/kcompile.py b/rteval/modules/loads/kcompile.py
|
|
||||||
index 0d02577..b606f7a 100644
|
|
||||||
--- a/rteval/modules/loads/kcompile.py
|
|
||||||
+++ b/rteval/modules/loads/kcompile.py
|
|
||||||
@@ -14,11 +14,12 @@ import subprocess
|
|
||||||
from rteval.modules import rtevalRuntimeError
|
|
||||||
from rteval.modules.loads import CommandLineLoad
|
|
||||||
from rteval.Log import Log
|
|
||||||
-from rteval.systopology import CpuList, SysTopology
|
|
||||||
+from rteval.systopology import SysTopology
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
-expand_cpulist = CpuList.expand_cpulist
|
|
||||||
-compress_cpulist = CpuList.compress_cpulist
|
|
||||||
-nonisolated_cpulist = CpuList.nonisolated_cpulist
|
|
||||||
+expand_cpulist = cpulist_utils.expand_cpulist
|
|
||||||
+compress_cpulist = cpulist_utils.compress_cpulist
|
|
||||||
+nonisolated_cpulist = cpulist_utils.nonisolated_cpulist
|
|
||||||
|
|
||||||
DEFAULT_KERNEL_PREFIX = "linux-5.18"
|
|
||||||
|
|
||||||
@@ -38,7 +39,7 @@ class KBuildJob:
|
|
||||||
os.mkdir(self.objdir)
|
|
||||||
|
|
||||||
# Exclude isolated CPUs if cpulist not set
|
|
||||||
- cpus_available = len(nonisolated_cpulist(self.node.cpus.cpulist))
|
|
||||||
+ cpus_available = len(nonisolated_cpulist(self.node.cpus))
|
|
||||||
|
|
||||||
if os.path.exists('/usr/bin/numactl') and not cpulist:
|
|
||||||
# Use numactl
|
|
||||||
diff --git a/rteval/modules/loads/stressng.py b/rteval/modules/loads/stressng.py
|
|
||||||
index 7c9e63f..cbcf6b7 100644
|
|
||||||
--- a/rteval/modules/loads/stressng.py
|
|
||||||
+++ b/rteval/modules/loads/stressng.py
|
|
||||||
@@ -7,10 +7,10 @@ import subprocess
|
|
||||||
import signal
|
|
||||||
from rteval.modules.loads import CommandLineLoad
|
|
||||||
from rteval.Log import Log
|
|
||||||
-from rteval.systopology import CpuList, SysTopology
|
|
||||||
+from rteval.systopology import SysTopology
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
-expand_cpulist = CpuList.expand_cpulist
|
|
||||||
-nonisolated_cpulist = CpuList.nonisolated_cpulist
|
|
||||||
+expand_cpulist = cpulist_utils.expand_cpulist
|
|
||||||
|
|
||||||
class Stressng(CommandLineLoad):
|
|
||||||
" This class creates a load module that runs stress-ng "
|
|
||||||
@@ -73,7 +73,7 @@ class Stressng(CommandLineLoad):
|
|
||||||
cpus[n] = [c for c in cpus[n] if c in expand_cpulist(self.cpulist)]
|
|
||||||
# if a cpulist was not specified, exclude isolated cpus
|
|
||||||
else:
|
|
||||||
- cpus[n] = CpuList.nonisolated_cpulist(cpus[n])
|
|
||||||
+ cpus[n] = cpulist_utils.nonisolated_cpulist(cpus[n])
|
|
||||||
|
|
||||||
|
|
||||||
# remove nodes with no cpus available for running
|
|
||||||
diff --git a/rteval/modules/measurement/__init__.py b/rteval/modules/measurement/__init__.py
|
|
||||||
index 41b8022..66dc9c5 100644
|
|
||||||
--- a/rteval/modules/measurement/__init__.py
|
|
||||||
+++ b/rteval/modules/measurement/__init__.py
|
|
||||||
@@ -5,7 +5,8 @@
|
|
||||||
|
|
||||||
import libxml2
|
|
||||||
from rteval.modules import RtEvalModules, ModuleContainer
|
|
||||||
-from rteval.systopology import CpuList, SysTopology as SysTop
|
|
||||||
+from rteval.systopology import SysTopology as SysTop
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
class MeasurementProfile(RtEvalModules):
|
|
||||||
"""Keeps and controls all the measurement modules with the same measurement profile"""
|
|
||||||
@@ -184,10 +185,11 @@ measurement profiles, based on their characteristics"""
|
|
||||||
run_on_isolcpus = self.__cfg.GetSection("measurement").run_on_isolcpus
|
|
||||||
if cpulist:
|
|
||||||
# Convert str to list and remove offline cpus
|
|
||||||
- cpulist = CpuList(cpulist).cpulist
|
|
||||||
+ cpulist = cpulist_utils.expand_cpulist(cpulist)
|
|
||||||
+ cpulist = cpulist_utils.online_cpulist(cpulist)
|
|
||||||
else:
|
|
||||||
cpulist = SysTop().online_cpus() if run_on_isolcpus else SysTop().default_cpus()
|
|
||||||
- rep_n.newProp("measurecpus", CpuList.collapse_cpulist(cpulist))
|
|
||||||
+ rep_n.newProp("measurecpus", cpulist_utils.collapse_cpulist(cpulist))
|
|
||||||
|
|
||||||
for mp in self.__measureprofiles:
|
|
||||||
mprep_n = mp.MakeReport()
|
|
||||||
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
|
||||||
index 1b14e7e..fdca257 100644
|
|
||||||
--- a/rteval/modules/measurement/cyclictest.py
|
|
||||||
+++ b/rteval/modules/measurement/cyclictest.py
|
|
||||||
@@ -17,9 +17,10 @@ import libxml2
|
|
||||||
from rteval.Log import Log
|
|
||||||
from rteval.modules import rtevalModulePrototype
|
|
||||||
from rteval.systopology import cpuinfo
|
|
||||||
-from rteval.systopology import CpuList, SysTopology
|
|
||||||
+from rteval.systopology import SysTopology
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
|
|
||||||
-expand_cpulist = CpuList.expand_cpulist
|
|
||||||
+expand_cpulist = cpulist_utils.expand_cpulist
|
|
||||||
|
|
||||||
class RunData:
|
|
||||||
'''class to keep instance data from a cyclictest run'''
|
|
||||||
@@ -201,9 +202,9 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
self.__cpulist = self.__cfg.cpulist
|
|
||||||
self.__cpus = expand_cpulist(self.__cpulist)
|
|
||||||
# Only include online cpus
|
|
||||||
- self.__cpus = CpuList(self.__cpus).cpulist
|
|
||||||
+ self.__cpus = cpulist_utils.online_cpulist(self.__cpus)
|
|
||||||
# Reset cpulist from the newly calculated self.__cpus
|
|
||||||
- self.__cpulist = CpuList.collapse_cpulist(self.__cpus)
|
|
||||||
+ self.__cpulist = cpulist_utils.collapse_cpulist(self.__cpus)
|
|
||||||
self.__cpus = [str(c) for c in self.__cpus]
|
|
||||||
self.__sparse = True
|
|
||||||
if self.__run_on_isolcpus:
|
|
||||||
@@ -220,7 +221,7 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
self.__cpus = [c for c in self.__cpus if c in cpuset or self.__run_on_isolcpus and c in isolcpus]
|
|
||||||
if self.__run_on_isolcpus:
|
|
||||||
self.__sparse = True
|
|
||||||
- self.__cpulist = CpuList.collapse_cpulist([int(c) for c in self.__cpus])
|
|
||||||
+ self.__cpulist = cpulist_utils.collapse_cpulist([int(c) for c in self.__cpus])
|
|
||||||
|
|
||||||
# Sort the list of cpus to align with the order reported by cyclictest
|
|
||||||
self.__cpus.sort(key=int)
|
|
||||||
diff --git a/rteval/systopology.py b/rteval/systopology.py
|
|
||||||
index bed5192..9e45762 100644
|
|
||||||
--- a/rteval/systopology.py
|
|
||||||
+++ b/rteval/systopology.py
|
|
||||||
@@ -9,12 +9,8 @@
|
|
||||||
import os
|
|
||||||
import os.path
|
|
||||||
import glob
|
|
||||||
-
|
|
||||||
-
|
|
||||||
-def sysread(path, obj):
|
|
||||||
- """ Helper function for reading system files """
|
|
||||||
- with open(os.path.join(path, obj), "r") as fp:
|
|
||||||
- return fp.readline().strip()
|
|
||||||
+import rteval.cpulist_utils as cpulist_utils
|
|
||||||
+from rteval.cpulist_utils import sysread
|
|
||||||
|
|
||||||
def cpuinfo():
|
|
||||||
''' return a dictionary of cpu keys with various cpu information '''
|
|
||||||
@@ -56,145 +52,6 @@ def cpuinfo():
|
|
||||||
return info
|
|
||||||
|
|
||||||
|
|
||||||
-#
|
|
||||||
-# class to provide access to a list of cpus
|
|
||||||
-#
|
|
||||||
-
|
|
||||||
-class CpuList:
|
|
||||||
- "Object that represents a group of system cpus"
|
|
||||||
-
|
|
||||||
- cpupath = '/sys/devices/system/cpu'
|
|
||||||
-
|
|
||||||
- def __init__(self, cpulist):
|
|
||||||
- if isinstance(cpulist, list):
|
|
||||||
- self.cpulist = cpulist
|
|
||||||
- elif isinstance(cpulist, str):
|
|
||||||
- self.cpulist = self.expand_cpulist(cpulist)
|
|
||||||
- self.cpulist = self.online_cpulist(self.cpulist)
|
|
||||||
- self.cpulist.sort()
|
|
||||||
-
|
|
||||||
- def __str__(self):
|
|
||||||
- return self.collapse_cpulist(self.cpulist)
|
|
||||||
-
|
|
||||||
- def __contains__(self, cpu):
|
|
||||||
- return cpu in self.cpulist
|
|
||||||
-
|
|
||||||
- def __len__(self):
|
|
||||||
- return len(self.cpulist)
|
|
||||||
-
|
|
||||||
- def getcpulist(self):
|
|
||||||
- """ return the list of cpus tracked """
|
|
||||||
- return self.cpulist
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def online_file_exists():
|
|
||||||
- """ Check whether machine / kernel is configured with online file """
|
|
||||||
- # Note: some machines do not have cpu0/online so we check cpu1/online.
|
|
||||||
- # In the case of machines with a single CPU, there is no cpu1, but
|
|
||||||
- # that is not a problem, since a single CPU cannot be offline
|
|
||||||
- return os.path.exists(os.path.join(CpuList.cpupath, "cpu1/online"))
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def isolated_file_exists():
|
|
||||||
- """ Check whether machine / kernel is configured with isolated file """
|
|
||||||
- return os.path.exists(os.path.join(CpuList.cpupath, "isolated"))
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def collapse_cpulist(cpulist):
|
|
||||||
- """
|
|
||||||
- Collapse a list of cpu numbers into a string range
|
|
||||||
- of cpus (e.g. 0-5, 7, 9)
|
|
||||||
- """
|
|
||||||
- cur_range = [None, None]
|
|
||||||
- result = []
|
|
||||||
- for cpu in cpulist + [None]:
|
|
||||||
- if cur_range[0] is None:
|
|
||||||
- cur_range[0] = cur_range[1] = cpu
|
|
||||||
- continue
|
|
||||||
- if cpu is not None and cpu == cur_range[1] + 1:
|
|
||||||
- # Extend currently processed range
|
|
||||||
- cur_range[1] += 1
|
|
||||||
- else:
|
|
||||||
- # Range processing finished, add range to string
|
|
||||||
- result.append(f"{cur_range[0]}-{cur_range[1]}"
|
|
||||||
- if cur_range[0] != cur_range[1]
|
|
||||||
- else str(cur_range[0]))
|
|
||||||
- # Reset
|
|
||||||
- cur_range[0] = cur_range[1] = cpu
|
|
||||||
- return ",".join(result)
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def compress_cpulist(cpulist):
|
|
||||||
- """ return a string representation of cpulist """
|
|
||||||
- if not cpulist:
|
|
||||||
- return ""
|
|
||||||
- if isinstance(cpulist[0], int):
|
|
||||||
- return ",".join(str(e) for e in cpulist)
|
|
||||||
- return ",".join(cpulist)
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def expand_cpulist(cpulist):
|
|
||||||
- """ expand a range string into an array of cpu numbers
|
|
||||||
- don't error check against online cpus
|
|
||||||
- """
|
|
||||||
- result = []
|
|
||||||
-
|
|
||||||
- if not cpulist:
|
|
||||||
- return result
|
|
||||||
-
|
|
||||||
- for part in cpulist.split(','):
|
|
||||||
- if '-' in part:
|
|
||||||
- a, b = part.split('-')
|
|
||||||
- a, b = int(a), int(b)
|
|
||||||
- result.extend(list(range(a, b + 1)))
|
|
||||||
- else:
|
|
||||||
- a = int(part)
|
|
||||||
- result.append(a)
|
|
||||||
- return [int(i) for i in list(set(result))]
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def is_online(n):
|
|
||||||
- """ check whether cpu n is online """
|
|
||||||
- path = os.path.join(CpuList.cpupath, f'cpu{n}')
|
|
||||||
-
|
|
||||||
- # Some hardware doesn't allow cpu0 to be turned off
|
|
||||||
- if not os.path.exists(path + '/online') and n == 0:
|
|
||||||
- return True
|
|
||||||
-
|
|
||||||
- return sysread(path, "online") == "1"
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def online_cpulist(cpulist):
|
|
||||||
- """ Given a cpulist, return a cpulist of online cpus """
|
|
||||||
- # This only works if the sys online files exist
|
|
||||||
- if not CpuList.online_file_exists():
|
|
||||||
- return cpulist
|
|
||||||
- newlist = []
|
|
||||||
- for cpu in cpulist:
|
|
||||||
- if not CpuList.online_file_exists() and cpu == '0':
|
|
||||||
- newlist.append(cpu)
|
|
||||||
- elif CpuList.is_online(int(cpu)):
|
|
||||||
- newlist.append(cpu)
|
|
||||||
- return newlist
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def isolated_cpulist(cpulist):
|
|
||||||
- """Given a cpulist, return a cpulist of isolated CPUs"""
|
|
||||||
- if not CpuList.isolated_file_exists():
|
|
||||||
- return cpulist
|
|
||||||
- isolated_cpulist = sysread(CpuList.cpupath, "isolated")
|
|
||||||
- isolated_cpulist = CpuList.expand_cpulist(isolated_cpulist)
|
|
||||||
- return list(set(isolated_cpulist) & set(cpulist))
|
|
||||||
-
|
|
||||||
- @staticmethod
|
|
||||||
- def nonisolated_cpulist(cpulist):
|
|
||||||
- """Given a cpulist, return a cpulist of non-isolated CPUs"""
|
|
||||||
- if not CpuList.isolated_file_exists():
|
|
||||||
- return cpulist
|
|
||||||
- isolated_cpulist = sysread(CpuList.cpupath, "isolated")
|
|
||||||
- isolated_cpulist = CpuList.expand_cpulist(isolated_cpulist)
|
|
||||||
- return list(set(cpulist).difference(set(isolated_cpulist)))
|
|
||||||
-
|
|
||||||
#
|
|
||||||
# class to abstract access to NUMA nodes in /sys filesystem
|
|
||||||
#
|
|
||||||
@@ -208,7 +65,8 @@ class NumaNode:
|
|
||||||
"""
|
|
||||||
self.path = path
|
|
||||||
self.nodeid = int(os.path.basename(path)[4:].strip())
|
|
||||||
- self.cpus = CpuList(sysread(self.path, "cpulist"))
|
|
||||||
+ self.cpus = cpulist_utils.expand_cpulist(sysread(self.path, "cpulist"))
|
|
||||||
+ self.cpus = cpulist_utils.online_cpulist(self.cpus)
|
|
||||||
self.getmeminfo()
|
|
||||||
|
|
||||||
def __contains__(self, cpu):
|
|
||||||
@@ -240,11 +98,11 @@ class NumaNode:
|
|
||||||
|
|
||||||
def getcpustr(self):
|
|
||||||
""" return list of cpus for this node as a string """
|
|
||||||
- return str(self.cpus)
|
|
||||||
+ return cpulist_utils.collapse_cpulist(self.cpus)
|
|
||||||
|
|
||||||
def getcpulist(self):
|
|
||||||
""" return list of cpus for this node """
|
|
||||||
- return self.cpus.getcpulist()
|
|
||||||
+ return self.cpus
|
|
||||||
|
|
||||||
class SimNumaNode(NumaNode):
|
|
||||||
"""class representing a simulated NUMA node.
|
|
||||||
@@ -257,7 +115,8 @@ class SimNumaNode(NumaNode):
|
|
||||||
|
|
||||||
def __init__(self):
|
|
||||||
self.nodeid = 0
|
|
||||||
- self.cpus = CpuList(sysread(SimNumaNode.cpupath, "possible"))
|
|
||||||
+ self.cpus = cpulist_utils.expand_cpulist(sysread(SimNumaNode.cpupath, "possible"))
|
|
||||||
+ self.cpus = cpulist_utils.online_cpulist(self.cpus)
|
|
||||||
self.getmeminfo()
|
|
||||||
|
|
||||||
def getmeminfo(self):
|
|
||||||
@@ -339,7 +198,7 @@ class SysTopology:
|
|
||||||
""" return a list of integers of all online cpus """
|
|
||||||
cpulist = []
|
|
||||||
for n in self.nodes:
|
|
||||||
- cpulist += self.getcpus(n)
|
|
||||||
+ cpulist += cpulist_utils.online_cpulist(self.getcpus(n))
|
|
||||||
cpulist.sort()
|
|
||||||
return cpulist
|
|
||||||
|
|
||||||
@@ -347,7 +206,7 @@ class SysTopology:
|
|
||||||
""" return a list of integers of all isolated cpus """
|
|
||||||
cpulist = []
|
|
||||||
for n in self.nodes:
|
|
||||||
- cpulist += CpuList.isolated_cpulist(self.getcpus(n))
|
|
||||||
+ cpulist += cpulist_utils.isolated_cpulist(self.getcpus(n))
|
|
||||||
cpulist.sort()
|
|
||||||
return cpulist
|
|
||||||
|
|
||||||
@@ -355,7 +214,7 @@ class SysTopology:
|
|
||||||
""" return a list of integers of all default schedulable cpus, i.e. online non-isolated cpus """
|
|
||||||
cpulist = []
|
|
||||||
for n in self.nodes:
|
|
||||||
- cpulist += CpuList.nonisolated_cpulist(self.getcpus(n))
|
|
||||||
+ cpulist += cpulist_utils.nonisolated_cpulist(self.getcpus(n))
|
|
||||||
cpulist.sort()
|
|
||||||
return cpulist
|
|
||||||
|
|
||||||
@@ -403,7 +262,7 @@ if __name__ == "__main__":
|
|
||||||
|
|
||||||
onlcpus = s.online_cpus()
|
|
||||||
print(f'onlcpus = {onlcpus}')
|
|
||||||
- onlcpus = CpuList.collapse_cpulist(onlcpus)
|
|
||||||
+ onlcpus = cpulist_utils.collapse_cpulist(onlcpus)
|
|
||||||
print(f'onlcpus = {onlcpus}')
|
|
||||||
|
|
||||||
onlcpus_str = s.online_cpus_str()
|
|
||||||
--
|
|
||||||
2.43.0
|
|
||||||
|
|
@ -0,0 +1,51 @@
|
|||||||
|
From 75667f4b92e3120bced7ae5464d97941edffd802 Mon Sep 17 00:00:00 2001
|
||||||
|
From: John Kacur <jkacur@redhat.com>
|
||||||
|
Date: Mon, 7 Feb 2022 16:09:13 -0500
|
||||||
|
Subject: [PATCH] rteval: Don't restrict measurement threads to inherited
|
||||||
|
cpumask
|
||||||
|
|
||||||
|
This patch reverses be811d28a471cfcaf7960289e00960e7f5b18f03
|
||||||
|
|
||||||
|
This patch syncs rteval to 795433f50f20ef7598db5cfe991b5386e4080d41 in
|
||||||
|
rt-tests
|
||||||
|
|
||||||
|
If the user does not specify a cpumask, rteval will still run on the
|
||||||
|
cpus according to the cpumask inherited from the runtime environment.
|
||||||
|
However, if the user specifies a cpumask, then it will overwrite the
|
||||||
|
runtime environment where allowed.
|
||||||
|
|
||||||
|
This will allow the user to run measurement threads on isolated cpus.
|
||||||
|
|
||||||
|
Signed-off-by: John Kacur <jkacur@redhat.com>
|
||||||
|
---
|
||||||
|
rteval/modules/measurement/cyclictest.py | 12 +-----------
|
||||||
|
1 file changed, 1 insertion(+), 11 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
||||||
|
index 5330d1466302..4ca310bf7490 100644
|
||||||
|
--- a/rteval/modules/measurement/cyclictest.py
|
||||||
|
+++ b/rteval/modules/measurement/cyclictest.py
|
||||||
|
@@ -217,19 +217,9 @@ class Cyclictest(rtevalModulePrototype):
|
||||||
|
else:
|
||||||
|
self.__cpus = online_cpus()
|
||||||
|
|
||||||
|
- # Sort the list of cpus to align with the order reported by
|
||||||
|
- # cyclictest
|
||||||
|
+ # Sort the list of cpus to align with the order reported by cyclictest
|
||||||
|
self.__cpus.sort(key=int)
|
||||||
|
|
||||||
|
- # Get the cpuset from the environment
|
||||||
|
- cpuset = os.sched_getaffinity(0)
|
||||||
|
-
|
||||||
|
- # Convert the elements to strings
|
||||||
|
- cpuset = [str(c) for c in cpuset]
|
||||||
|
-
|
||||||
|
- # Only include cpus that are in the cpuset
|
||||||
|
- self.__cpus = [c for c in self.__cpus if c in cpuset]
|
||||||
|
-
|
||||||
|
self.__numcores = len(self.__cpus)
|
||||||
|
|
||||||
|
info = cpuinfo()
|
||||||
|
--
|
||||||
|
2.34.1
|
||||||
|
|
@ -0,0 +1,53 @@
|
|||||||
|
From 19f9aa600ae331eb2fadf5f821f539bf24f0dea3 Mon Sep 17 00:00:00 2001
|
||||||
|
From: John Kacur <jkacur@redhat.com>
|
||||||
|
Date: Tue, 18 Jan 2022 15:03:36 -0500
|
||||||
|
Subject: [PATCH] rteval: Fix Popen for python3.6 where text=True is not
|
||||||
|
available
|
||||||
|
|
||||||
|
Fix using text=True in Popen for python-3.6 by using
|
||||||
|
encoding='utf-8' instead
|
||||||
|
|
||||||
|
Signed-off-by: John Kacur <jkacur@redhat.com>
|
||||||
|
---
|
||||||
|
rteval/sysinfo/services.py | 8 ++++----
|
||||||
|
1 file changed, 4 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/rteval/sysinfo/services.py b/rteval/sysinfo/services.py
|
||||||
|
index e5de22efc342..c85980e19165 100644
|
||||||
|
--- a/rteval/sysinfo/services.py
|
||||||
|
+++ b/rteval/sysinfo/services.py
|
||||||
|
@@ -62,11 +62,11 @@ class SystemServices:
|
||||||
|
if not [1 for p in reject if fnmatch.fnmatch(servicename, p)] \
|
||||||
|
and os.access(service, os.X_OK):
|
||||||
|
cmd = '%s -qs "\(^\|\W\)status)" %s' % (getcmdpath('grep'), service)
|
||||||
|
- c = subprocess.Popen(cmd, shell=True, text=True)
|
||||||
|
+ c = subprocess.Popen(cmd, shell=True, encoding='utf-8')
|
||||||
|
c.wait()
|
||||||
|
if c.returncode == 0:
|
||||||
|
cmd = ['env', '-i', 'LANG="%s"' % os.environ['LANG'], 'PATH="%s"' % os.environ['PATH'], 'TERM="%s"' % os.environ['TERM'], service, 'status']
|
||||||
|
- c = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
|
||||||
|
+ c = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, encoding='utf-8')
|
||||||
|
c.wait()
|
||||||
|
if c.returncode == 0 and (c.stdout.read() or c.stderr.read()):
|
||||||
|
ret_services[servicename] = 'running'
|
||||||
|
@@ -81,7 +81,7 @@ class SystemServices:
|
||||||
|
ret_services = {}
|
||||||
|
cmd = '%s list-unit-files -t service --no-legend' % getcmdpath('systemctl')
|
||||||
|
self.__log(Log.DEBUG, "cmd: %s" % cmd)
|
||||||
|
- c = subprocess.Popen(cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
|
||||||
|
+ c = subprocess.Popen(cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE, encoding='utf-8')
|
||||||
|
for p in c.stdout:
|
||||||
|
# p are lines like b'servicename.service status'
|
||||||
|
v = p.strip().split()
|
||||||
|
@@ -91,7 +91,7 @@ class SystemServices:
|
||||||
|
|
||||||
|
def services_get(self):
|
||||||
|
cmd = [getcmdpath('ps'), '-ocomm=', '1']
|
||||||
|
- c = subprocess.Popen(cmd, stdout=subprocess.PIPE, text=True)
|
||||||
|
+ c = subprocess.Popen(cmd, stdout=subprocess.PIPE, encoding='utf-8')
|
||||||
|
self.__init = c.stdout.read().strip()
|
||||||
|
if self.__init == 'systemd':
|
||||||
|
self.__log(Log.DEBUG, "Using systemd to get services status")
|
||||||
|
--
|
||||||
|
2.31.1
|
||||||
|
|
32
SOURCES/rteval-Fix-test-misses-threshold-assignment.patch
Normal file
32
SOURCES/rteval-Fix-test-misses-threshold-assignment.patch
Normal file
@ -0,0 +1,32 @@
|
|||||||
|
From aca562064a8d9f5927aa39bc108eac3bf4d0a56b Mon Sep 17 00:00:00 2001
|
||||||
|
From: John Kacur <jkacur@redhat.com>
|
||||||
|
Date: Fri, 14 Jan 2022 14:00:54 -0500
|
||||||
|
Subject: [PATCH] rteval: Fix test misses threshold assignment
|
||||||
|
|
||||||
|
Fix case where self.__cfg.threshold has a value but
|
||||||
|
instead of 'threshold' in self.__cfg,
|
||||||
|
'breaktrace' in self.__cfg.
|
||||||
|
by just checking whether self.__cfg.threshold has a value
|
||||||
|
if self._cfg.breaktrace does not have a value
|
||||||
|
|
||||||
|
Signed-off-by: John Kacur <jkacur@redhat.com>
|
||||||
|
---
|
||||||
|
rteval/modules/measurement/cyclictest.py | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
||||||
|
index cc74b467913d..af8adeeeb402 100644
|
||||||
|
--- a/rteval/modules/measurement/cyclictest.py
|
||||||
|
+++ b/rteval/modules/measurement/cyclictest.py
|
||||||
|
@@ -299,7 +299,7 @@ class Cyclictest(rtevalModulePrototype):
|
||||||
|
if 'breaktrace' in self.__cfg and self.__cfg.breaktrace:
|
||||||
|
self.__cmd.append("-b%d" % int(self.__cfg.breaktrace))
|
||||||
|
self.__cmd.append("--tracemark")
|
||||||
|
- elif 'threshold' in self.__cfg and self.__cfg.threshold:
|
||||||
|
+ elif self.__cfg.threshold:
|
||||||
|
self.__cmd.append("-b%d" % int(self.__cfg.threshold))
|
||||||
|
|
||||||
|
# Buffer for cyclictest data written to stdout
|
||||||
|
--
|
||||||
|
2.31.1
|
||||||
|
|
@ -0,0 +1,34 @@
|
|||||||
|
From 14b0cff41f13602579fd708748eb8a2ad93db85b Mon Sep 17 00:00:00 2001
|
||||||
|
From: John Kacur <jkacur@redhat.com>
|
||||||
|
Date: Mon, 14 Feb 2022 22:09:21 -0500
|
||||||
|
Subject: [PATCH] rteval: If the user doesn't specify a cpumask, use the
|
||||||
|
inherited one
|
||||||
|
|
||||||
|
If the user doesn't specify a cpumask, then we need to use the inherited
|
||||||
|
one to match what cyclictest currently does
|
||||||
|
|
||||||
|
Signed-off-by: John Kacur <jkacur@redhat.com>
|
||||||
|
---
|
||||||
|
rteval/modules/measurement/cyclictest.py | 6 ++++++
|
||||||
|
1 file changed, 6 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
||||||
|
index 4ca310bf7490..8fdd5341794a 100644
|
||||||
|
--- a/rteval/modules/measurement/cyclictest.py
|
||||||
|
+++ b/rteval/modules/measurement/cyclictest.py
|
||||||
|
@@ -216,6 +216,12 @@ class Cyclictest(rtevalModulePrototype):
|
||||||
|
self.__sparse = True
|
||||||
|
else:
|
||||||
|
self.__cpus = online_cpus()
|
||||||
|
+ # Get the cpuset from the environment
|
||||||
|
+ cpuset = os.sched_getaffinity(0)
|
||||||
|
+ # Convert the elements to strings
|
||||||
|
+ cpuset = [str(c) for c in cpuset]
|
||||||
|
+ # Only include cpus that are in the cpuset
|
||||||
|
+ self.__cpus = [c for c in self.__cpus if c in cpuset]
|
||||||
|
|
||||||
|
# Sort the list of cpus to align with the order reported by cyclictest
|
||||||
|
self.__cpus.sort(key=int)
|
||||||
|
--
|
||||||
|
2.34.1
|
||||||
|
|
@ -0,0 +1,60 @@
|
|||||||
|
From 9dff629f186313beebb96594d236dd9268bef1b1 Mon Sep 17 00:00:00 2001
|
||||||
|
From: John Kacur <jkacur@redhat.com>
|
||||||
|
Date: Wed, 26 Jan 2022 10:06:33 -0500
|
||||||
|
Subject: [PATCH] rteval: Increase default buckets from 2000 to 3500
|
||||||
|
|
||||||
|
Increase the default buckets from 2000 to 3500
|
||||||
|
|
||||||
|
With commit 0292c8963611 we skip statistics reporting if we run out of
|
||||||
|
buckets. Increase the default number of buckets to make this less likely
|
||||||
|
to occur.
|
||||||
|
|
||||||
|
Signed-off-by: John Kacur <jkacur@redhat.com>
|
||||||
|
---
|
||||||
|
doc/rteval.8 | 2 +-
|
||||||
|
rteval/modules/measurement/cyclictest.py | 2 +-
|
||||||
|
rteval/rteval.conf | 2 +-
|
||||||
|
3 files changed, 3 insertions(+), 3 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/doc/rteval.8 b/doc/rteval.8
|
||||||
|
index fa24509ce7e5..25dcfcc298e7 100644
|
||||||
|
--- a/doc/rteval.8
|
||||||
|
+++ b/doc/rteval.8
|
||||||
|
@@ -122,7 +122,7 @@ Measurement thread interval in microseconds (default: 100)
|
||||||
|
Interval increment in microseconds (default: 0)
|
||||||
|
.TP
|
||||||
|
.B \-\-cyclictest-buckets=NBUCKETS
|
||||||
|
-Number of 1 microsecond histogram buckets (default: 2000)
|
||||||
|
+Number of 1 microsecond histogram buckets (default: 3500)
|
||||||
|
.TP
|
||||||
|
.B \-\-hackbench-jobspercore=N
|
||||||
|
Number of jobs per online-core for hackbench load
|
||||||
|
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
||||||
|
index af8adeeeb402..5330d1466302 100644
|
||||||
|
--- a/rteval/modules/measurement/cyclictest.py
|
||||||
|
+++ b/rteval/modules/measurement/cyclictest.py
|
||||||
|
@@ -450,7 +450,7 @@ def ModuleParameters():
|
||||||
|
"default": 100,
|
||||||
|
"metavar": "INTV_US"},
|
||||||
|
"buckets": {"descr": "Histogram width",
|
||||||
|
- "default": 2000,
|
||||||
|
+ "default": 3500,
|
||||||
|
"metavar": "NUM"},
|
||||||
|
"priority": {"descr": "Run cyclictest with the given priority",
|
||||||
|
"default": 95,
|
||||||
|
diff --git a/rteval/rteval.conf b/rteval/rteval.conf
|
||||||
|
index 7ce6ef0b5acc..aa0a53bcfc63 100644
|
||||||
|
--- a/rteval/rteval.conf
|
||||||
|
+++ b/rteval/rteval.conf
|
||||||
|
@@ -6,7 +6,7 @@ duration: 60.0
|
||||||
|
report_interval: 600
|
||||||
|
|
||||||
|
[cyclictest]
|
||||||
|
-buckets: 2000
|
||||||
|
+buckets: 3500
|
||||||
|
interval: 100
|
||||||
|
distance: 0
|
||||||
|
priority: 95
|
||||||
|
--
|
||||||
|
2.34.1
|
||||||
|
|
@ -1,85 +0,0 @@
|
|||||||
From 768daa63ec8e2299da53afe081e7304b37fc91cf Mon Sep 17 00:00:00 2001
|
|
||||||
From: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Date: Wed, 29 Nov 2023 10:34:55 +0100
|
|
||||||
Subject: [PATCH 2/9] rteval: Minor improvements to CpuList class
|
|
||||||
|
|
||||||
- Remove unnecessary if-else from online_file_exists
|
|
||||||
- Use cpupath in online_file_exists
|
|
||||||
- In is_online, remove check for n in cpuset and make it static
|
|
||||||
- Mark also the remaining methods static since they do not rely on
|
|
||||||
any fields of the class
|
|
||||||
|
|
||||||
Signed-off-by: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
- Removed incorrect line from commit message
|
|
||||||
Signed-off-by: John Kacur <jkacur@redhat.com>
|
|
||||||
---
|
|
||||||
rteval/systopology.py | 29 +++++++++++++++--------------
|
|
||||||
1 file changed, 15 insertions(+), 14 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/rteval/systopology.py b/rteval/systopology.py
|
|
||||||
index ea8e242..60ac8e8 100644
|
|
||||||
--- a/rteval/systopology.py
|
|
||||||
+++ b/rteval/systopology.py
|
|
||||||
@@ -82,12 +82,17 @@ class CpuList:
|
|
||||||
def __len__(self):
|
|
||||||
return len(self.cpulist)
|
|
||||||
|
|
||||||
+ def getcpulist(self):
|
|
||||||
+ """ return the list of cpus tracked """
|
|
||||||
+ return self.cpulist
|
|
||||||
+
|
|
||||||
@staticmethod
|
|
||||||
def online_file_exists():
|
|
||||||
""" Check whether machine / kernel is configured with online file """
|
|
||||||
- if os.path.exists('/sys/devices/system/cpu/cpu1/online'):
|
|
||||||
- return True
|
|
||||||
- return False
|
|
||||||
+ # Note: some machines do not have cpu0/online so we check cpu1/online.
|
|
||||||
+ # In the case of machines with a single CPU, there is no cpu1, but
|
|
||||||
+ # that is not a problem, since a single CPU cannot be offline
|
|
||||||
+ return os.path.exists(os.path.join(CpuList.cpupath, "cpu1/online"))
|
|
||||||
|
|
||||||
@staticmethod
|
|
||||||
def isolated_file_exists():
|
|
||||||
@@ -147,14 +152,9 @@ class CpuList:
|
|
||||||
result.append(a)
|
|
||||||
return [int(i) for i in list(set(result))]
|
|
||||||
|
|
||||||
- def getcpulist(self):
|
|
||||||
- """ return the list of cpus tracked """
|
|
||||||
- return self.cpulist
|
|
||||||
-
|
|
||||||
- def is_online(self, n):
|
|
||||||
+ @staticmethod
|
|
||||||
+ def is_online(n):
|
|
||||||
""" check whether cpu n is online """
|
|
||||||
- if n not in self.cpulist:
|
|
||||||
- raise RuntimeError(f"invalid cpu number {n}")
|
|
||||||
path = os.path.join(CpuList.cpupath, f'cpu{n}')
|
|
||||||
|
|
||||||
# Some hardware doesn't allow cpu0 to be turned off
|
|
||||||
@@ -163,16 +163,17 @@ class CpuList:
|
|
||||||
|
|
||||||
return sysread(path, "online") == "1"
|
|
||||||
|
|
||||||
- def online_cpulist(self, cpulist):
|
|
||||||
+ @staticmethod
|
|
||||||
+ def online_cpulist(cpulist):
|
|
||||||
""" Given a cpulist, return a cpulist of online cpus """
|
|
||||||
# This only works if the sys online files exist
|
|
||||||
- if not self.online_file_exists():
|
|
||||||
+ if not CpuList.online_file_exists():
|
|
||||||
return cpulist
|
|
||||||
newlist = []
|
|
||||||
for cpu in cpulist:
|
|
||||||
- if not self.online_file_exists() and cpu == '0':
|
|
||||||
+ if not CpuList.online_file_exists() and cpu == '0':
|
|
||||||
newlist.append(cpu)
|
|
||||||
- elif self.is_online(int(cpu)):
|
|
||||||
+ elif CpuList.is_online(int(cpu)):
|
|
||||||
newlist.append(cpu)
|
|
||||||
return newlist
|
|
||||||
|
|
||||||
--
|
|
||||||
2.43.0
|
|
||||||
|
|
@ -1,225 +0,0 @@
|
|||||||
From 4b4f50900e38a931ed9585c6cf200b74c0120a20 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Date: Wed, 29 Nov 2023 10:34:54 +0100
|
|
||||||
Subject: [PATCH 1/9] rteval: Refactor collapse_cpulist in systopology
|
|
||||||
|
|
||||||
Instead of having duplicate code in two functions, one top-level and
|
|
||||||
one member function of CpuList, have only one static function in
|
|
||||||
CpuList.
|
|
||||||
|
|
||||||
Additionally re-write the implementation to use a more straight forward
|
|
||||||
one-pass algorithm.
|
|
||||||
|
|
||||||
Signed-off-by: Tomas Glozar <tglozar@redhat.com>
|
|
||||||
Signed-off-by: John Kacur <jkacur@redhat.com>
|
|
||||||
---
|
|
||||||
rteval-cmd | 4 +-
|
|
||||||
rteval/modules/loads/__init__.py | 4 +-
|
|
||||||
rteval/modules/measurement/__init__.py | 4 +-
|
|
||||||
rteval/modules/measurement/cyclictest.py | 6 +--
|
|
||||||
rteval/systopology.py | 68 ++++++++----------------
|
|
||||||
5 files changed, 30 insertions(+), 56 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/rteval-cmd b/rteval-cmd
|
|
||||||
index 6f613a3..7c41429 100755
|
|
||||||
--- a/rteval-cmd
|
|
||||||
+++ b/rteval-cmd
|
|
||||||
@@ -30,7 +30,7 @@ from rteval import RtEval, rtevalConfig
|
|
||||||
from rteval.modules.loads import LoadModules
|
|
||||||
from rteval.modules.measurement import MeasurementModules
|
|
||||||
from rteval.version import RTEVAL_VERSION
|
|
||||||
-from rteval.systopology import CpuList, SysTopology, collapse_cpulist
|
|
||||||
+from rteval.systopology import CpuList, SysTopology
|
|
||||||
from rteval.modules.loads.kcompile import ModuleParameters
|
|
||||||
|
|
||||||
compress_cpulist = CpuList.compress_cpulist
|
|
||||||
@@ -211,7 +211,7 @@ def remove_offline(cpulist):
|
|
||||||
""" return cpulist in collapsed compressed form with only online cpus """
|
|
||||||
tmplist = expand_cpulist(cpulist)
|
|
||||||
tmplist = SysTopology().online_cpulist(tmplist)
|
|
||||||
- return collapse_cpulist(tmplist)
|
|
||||||
+ return CpuList.collapse_cpulist(tmplist)
|
|
||||||
|
|
||||||
|
|
||||||
if __name__ == '__main__':
|
|
||||||
diff --git a/rteval/modules/loads/__init__.py b/rteval/modules/loads/__init__.py
|
|
||||||
index aca0c9f..13fba1e 100644
|
|
||||||
--- a/rteval/modules/loads/__init__.py
|
|
||||||
+++ b/rteval/modules/loads/__init__.py
|
|
||||||
@@ -11,7 +11,7 @@ import libxml2
|
|
||||||
from rteval.Log import Log
|
|
||||||
from rteval.rtevalConfig import rtevalCfgSection
|
|
||||||
from rteval.modules import RtEvalModules, rtevalModulePrototype
|
|
||||||
-from rteval.systopology import collapse_cpulist, CpuList, SysTopology as SysTop
|
|
||||||
+from rteval.systopology import CpuList, SysTopology as SysTop
|
|
||||||
|
|
||||||
class LoadThread(rtevalModulePrototype):
|
|
||||||
def __init__(self, name, config, logger=None):
|
|
||||||
@@ -120,7 +120,7 @@ class LoadModules(RtEvalModules):
|
|
||||||
cpulist = CpuList(cpulist).cpulist
|
|
||||||
else:
|
|
||||||
cpulist = SysTop().default_cpus()
|
|
||||||
- rep_n.newProp("loadcpus", collapse_cpulist(cpulist))
|
|
||||||
+ rep_n.newProp("loadcpus", CpuList.collapse_cpulist(cpulist))
|
|
||||||
|
|
||||||
return rep_n
|
|
||||||
|
|
||||||
diff --git a/rteval/modules/measurement/__init__.py b/rteval/modules/measurement/__init__.py
|
|
||||||
index 2a0556b..41b8022 100644
|
|
||||||
--- a/rteval/modules/measurement/__init__.py
|
|
||||||
+++ b/rteval/modules/measurement/__init__.py
|
|
||||||
@@ -5,7 +5,7 @@
|
|
||||||
|
|
||||||
import libxml2
|
|
||||||
from rteval.modules import RtEvalModules, ModuleContainer
|
|
||||||
-from rteval.systopology import collapse_cpulist, CpuList, SysTopology as SysTop
|
|
||||||
+from rteval.systopology import CpuList, SysTopology as SysTop
|
|
||||||
|
|
||||||
class MeasurementProfile(RtEvalModules):
|
|
||||||
"""Keeps and controls all the measurement modules with the same measurement profile"""
|
|
||||||
@@ -187,7 +187,7 @@ measurement profiles, based on their characteristics"""
|
|
||||||
cpulist = CpuList(cpulist).cpulist
|
|
||||||
else:
|
|
||||||
cpulist = SysTop().online_cpus() if run_on_isolcpus else SysTop().default_cpus()
|
|
||||||
- rep_n.newProp("measurecpus", collapse_cpulist(cpulist))
|
|
||||||
+ rep_n.newProp("measurecpus", CpuList.collapse_cpulist(cpulist))
|
|
||||||
|
|
||||||
for mp in self.__measureprofiles:
|
|
||||||
mprep_n = mp.MakeReport()
|
|
||||||
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py
|
|
||||||
index 0af1d31..1b14e7e 100644
|
|
||||||
--- a/rteval/modules/measurement/cyclictest.py
|
|
||||||
+++ b/rteval/modules/measurement/cyclictest.py
|
|
||||||
@@ -17,7 +17,7 @@ import libxml2
|
|
||||||
from rteval.Log import Log
|
|
||||||
from rteval.modules import rtevalModulePrototype
|
|
||||||
from rteval.systopology import cpuinfo
|
|
||||||
-from rteval.systopology import CpuList, SysTopology, collapse_cpulist
|
|
||||||
+from rteval.systopology import CpuList, SysTopology
|
|
||||||
|
|
||||||
expand_cpulist = CpuList.expand_cpulist
|
|
||||||
|
|
||||||
@@ -203,7 +203,7 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
# Only include online cpus
|
|
||||||
self.__cpus = CpuList(self.__cpus).cpulist
|
|
||||||
# Reset cpulist from the newly calculated self.__cpus
|
|
||||||
- self.__cpulist = collapse_cpulist(self.__cpus)
|
|
||||||
+ self.__cpulist = CpuList.collapse_cpulist(self.__cpus)
|
|
||||||
self.__cpus = [str(c) for c in self.__cpus]
|
|
||||||
self.__sparse = True
|
|
||||||
if self.__run_on_isolcpus:
|
|
||||||
@@ -220,7 +220,7 @@ class Cyclictest(rtevalModulePrototype):
|
|
||||||
self.__cpus = [c for c in self.__cpus if c in cpuset or self.__run_on_isolcpus and c in isolcpus]
|
|
||||||
if self.__run_on_isolcpus:
|
|
||||||
self.__sparse = True
|
|
||||||
- self.__cpulist = collapse_cpulist(self.__cpus)
|
|
||||||
+ self.__cpulist = CpuList.collapse_cpulist([int(c) for c in self.__cpus])
|
|
||||||
|
|
||||||
# Sort the list of cpus to align with the order reported by cyclictest
|
|
||||||
self.__cpus.sort(key=int)
|
|
||||||
diff --git a/rteval/systopology.py b/rteval/systopology.py
|
|
||||||
index 62ad355..ea8e242 100644
|
|
||||||
--- a/rteval/systopology.py
|
|
||||||
+++ b/rteval/systopology.py
|
|
||||||
@@ -10,25 +10,6 @@ import os
|
|
||||||
import os.path
|
|
||||||
import glob
|
|
||||||
|
|
||||||
-# Utility version of collapse_cpulist that doesn't require a CpuList object
|
|
||||||
-def collapse_cpulist(cpulist):
|
|
||||||
- """ Collapse a list of cpu numbers into a string range
|
|
||||||
- of cpus (e.g. 0-5, 7, 9) """
|
|
||||||
- if len(cpulist) == 0:
|
|
||||||
- return ""
|
|
||||||
- idx = CpuList.longest_sequence(cpulist)
|
|
||||||
- if idx == 0:
|
|
||||||
- seq = str(cpulist[0])
|
|
||||||
- else:
|
|
||||||
- if idx == 1:
|
|
||||||
- seq = f"{cpulist[0]},{cpulist[idx]}"
|
|
||||||
- else:
|
|
||||||
- seq = f"{cpulist[0]}-{cpulist[idx]}"
|
|
||||||
-
|
|
||||||
- rest = collapse_cpulist(cpulist[idx+1:])
|
|
||||||
- if rest == "":
|
|
||||||
- return seq
|
|
||||||
- return ",".join((seq, rest))
|
|
||||||
|
|
||||||
def sysread(path, obj):
|
|
||||||
""" Helper function for reading system files """
|
|
||||||
@@ -93,7 +74,7 @@ class CpuList:
|
|
||||||
self.cpulist.sort()
|
|
||||||
|
|
||||||
def __str__(self):
|
|
||||||
- return self.__collapse_cpulist(self.cpulist)
|
|
||||||
+ return self.collapse_cpulist(self.cpulist)
|
|
||||||
|
|
||||||
def __contains__(self, cpu):
|
|
||||||
return cpu in self.cpulist
|
|
||||||
@@ -114,35 +95,28 @@ class CpuList:
|
|
||||||
return os.path.exists(os.path.join(CpuList.cpupath, "isolated"))
|
|
||||||
|
|
||||||
@staticmethod
|
|
||||||
- def longest_sequence(cpulist):
|
|
||||||
- """ return index of last element of a sequence that steps by one """
|
|
||||||
- lim = len(cpulist)
|
|
||||||
- for idx, _ in enumerate(cpulist):
|
|
||||||
- if idx+1 == lim:
|
|
||||||
- break
|
|
||||||
- if int(cpulist[idx+1]) != (int(cpulist[idx])+1):
|
|
||||||
- return idx
|
|
||||||
- return lim - 1
|
|
||||||
-
|
|
||||||
- def __collapse_cpulist(self, cpulist):
|
|
||||||
- """ Collapse a list of cpu numbers into a string range
|
|
||||||
+ def collapse_cpulist(cpulist):
|
|
||||||
+ """
|
|
||||||
+ Collapse a list of cpu numbers into a string range
|
|
||||||
of cpus (e.g. 0-5, 7, 9)
|
|
||||||
"""
|
|
||||||
- if len(cpulist) == 0:
|
|
||||||
- return ""
|
|
||||||
- idx = self.longest_sequence(cpulist)
|
|
||||||
- if idx == 0:
|
|
||||||
- seq = str(cpulist[0])
|
|
||||||
- else:
|
|
||||||
- if idx == 1:
|
|
||||||
- seq = f"{cpulist[0]},{cpulist[idx]}"
|
|
||||||
+ cur_range = [None, None]
|
|
||||||
+ result = []
|
|
||||||
+ for cpu in cpulist + [None]:
|
|
||||||
+ if cur_range[0] is None:
|
|
||||||
+ cur_range[0] = cur_range[1] = cpu
|
|
||||||
+ continue
|
|
||||||
+ if cpu is not None and cpu == cur_range[1] + 1:
|
|
||||||
+ # Extend currently processed range
|
|
||||||
+ cur_range[1] += 1
|
|
||||||
else:
|
|
||||||
- seq = f"{cpulist[0]}-{cpulist[idx]}"
|
|
||||||
-
|
|
||||||
- rest = self.__collapse_cpulist(cpulist[idx+1:])
|
|
||||||
- if rest == "":
|
|
||||||
- return seq
|
|
||||||
- return ",".join((seq, rest))
|
|
||||||
+ # Range processing finished, add range to string
|
|
||||||
+ result.append(f"{cur_range[0]}-{cur_range[1]}"
|
|
||||||
+ if cur_range[0] != cur_range[1]
|
|
||||||
+ else str(cur_range[0]))
|
|
||||||
+ # Reset
|
|
||||||
+ cur_range[0] = cur_range[1] = cpu
|
|
||||||
+ return ",".join(result)
|
|
||||||
|
|
||||||
@staticmethod
|
|
||||||
def compress_cpulist(cpulist):
|
|
||||||
@@ -428,7 +402,7 @@ if __name__ == "__main__":
|
|
||||||
|
|
||||||
onlcpus = s.online_cpus()
|
|
||||||
print(f'onlcpus = {onlcpus}')
|
|
||||||
- onlcpus = collapse_cpulist(onlcpus)
|
|
||||||
+ onlcpus = CpuList.collapse_cpulist(onlcpus)
|
|
||||||
print(f'onlcpus = {onlcpus}')
|
|
||||||
|
|
||||||
onlcpus_str = s.online_cpus_str()
|
|
||||||
--
|
|
||||||
2.43.0
|
|
||||||
|
|
@ -1,566 +0,0 @@
|
|||||||
From de8e25ff3a30dbdbba6fb1b68ea0921dff55cd91 Mon Sep 17 00:00:00 2001
|
|
||||||
From: John Kacur <jkacur@redhat.com>
|
|
||||||
Date: Mon, 13 Nov 2023 14:32:19 -0500
|
|
||||||
Subject: [PATCH] rteval: Remove upstream spec file
|
|
||||||
|
|
||||||
Specfiles should be maintained by distributions and not in the upstream
|
|
||||||
code. In practice they are not maintained upstream except for version
|
|
||||||
numbers, so just remove the specfile.
|
|
||||||
|
|
||||||
This also moves a lot of functionality around rpms in the Makefile, but
|
|
||||||
this functionality exists in tools such as rpmbuild and friends anyway.
|
|
||||||
|
|
||||||
Signed-off-by: John Kacur <jkacur@redhat.com>
|
|
||||||
---
|
|
||||||
Makefile | 37 ----
|
|
||||||
rteval.spec | 484 ----------------------------------------------------
|
|
||||||
2 files changed, 521 deletions(-)
|
|
||||||
delete mode 100644 rteval.spec
|
|
||||||
|
|
||||||
diff --git a/Makefile b/Makefile
|
|
||||||
index 14f74e087eff..ee4cca555b95 100644
|
|
||||||
--- a/Makefile
|
|
||||||
+++ b/Makefile
|
|
||||||
@@ -79,49 +79,12 @@ rpm_prep:
|
|
||||||
|
|
||||||
rpms rpm: rpm_prep rtevalrpm loadrpm
|
|
||||||
|
|
||||||
-rtevalrpm: rteval-$(VERSION).tar.bz2
|
|
||||||
- cp $^ rpm/SOURCES
|
|
||||||
- cp rteval.spec rpm/SPECS
|
|
||||||
- rpmbuild -ba --define "_topdir $(HERE)/rpm" rpm/SPECS/rteval.spec
|
|
||||||
-
|
|
||||||
-rtevalsrpm: rteval-$(VERSION).tar.bz2
|
|
||||||
- cp $^ rpm/SOURCES
|
|
||||||
- cp rteval.spec rpm/SPECS
|
|
||||||
- rpmbuild -bs --define "_topdir $(HERE)/rpm" rpm/SPECS/rteval.spec
|
|
||||||
-
|
|
||||||
-
|
|
||||||
-xmlrpcrpm: rteval-xmlrpc-$(XMLRPCVER).tar.gz
|
|
||||||
- cp rteval-xmlrpc-$(XMLRPCVER).tar.gz rpm/SOURCES/
|
|
||||||
- cp server/rteval-parser.spec rpm/SPECS/
|
|
||||||
- rpmbuild -ba --define "_topdir $(HERE)/rpm" rpm/SPECS/rteval-parser.spec
|
|
||||||
-
|
|
||||||
-xmlsrpm: rteval-xmlrpc-$(XMLRPCVER).tar.gz
|
|
||||||
- cp rteval-xmlrpc-$(XMLRPCVER).tar.gz rpm/SOURCES/
|
|
||||||
- cp server/rteval-parser.spec rpm/SPECS/
|
|
||||||
- rpmbuild -bs --define "_topdir $(HERE)/rpm" rpm/SPECS/rteval-parser.spec
|
|
||||||
-
|
|
||||||
-loadrpm:
|
|
||||||
- rm -rf rpm-loads
|
|
||||||
- mkdir -p rpm-loads/{BUILD,RPMS,SRPMS,SOURCES,SPECS}
|
|
||||||
- cp rteval-loads.spec rpm-loads/SPECS
|
|
||||||
- cp $(LOADS) rpm-loads/SOURCES
|
|
||||||
- rpmbuild -ba --define "_topdir $(HERE)/rpm-loads" rpm-loads/SPECS/rteval-loads.spec
|
|
||||||
-
|
|
||||||
-rpmlint: rpms
|
|
||||||
- @echo "==============="
|
|
||||||
- @echo "running rpmlint"
|
|
||||||
- rpmlint -v $(shell find ./rpm -type f -name "*.rpm") \
|
|
||||||
- $(shell find ./rpm-loads -type f -name "*.rpm") \
|
|
||||||
- $(shell find ./rpm/SPECS -type f -name "rteval*.spec") \
|
|
||||||
- $(shell find ./rpm-loads/SPECS -type f -name "rteval*.spec" )
|
|
||||||
|
|
||||||
help:
|
|
||||||
@echo ""
|
|
||||||
@echo "rteval Makefile targets:"
|
|
||||||
@echo ""
|
|
||||||
@echo " runit: do a short testrun locally [default]"
|
|
||||||
- @echo " rpm: run rpmbuild for all rpms"
|
|
||||||
- @echo " rpmlint: run rpmlint against all rpms/srpms/specfiles"
|
|
||||||
@echo " tarfile: create the source tarball"
|
|
||||||
@echo " install: install rteval locally"
|
|
||||||
@echo " clean: cleanup generated files"
|
|
||||||
diff --git a/rteval.spec b/rteval.spec
|
|
||||||
deleted file mode 100644
|
|
||||||
index b5842f0d8206..000000000000
|
|
||||||
--- a/rteval.spec
|
|
||||||
+++ /dev/null
|
|
||||||
@@ -1,484 +0,0 @@
|
|
||||||
-%{!?python_sitelib: %define python_sitelib %(%{__python} -c "from distutils.sysconfig import get_python_lib; print get_python_lib()")}
|
|
||||||
-%{!?python_ver: %define python_ver %(%{__python} -c "import sys ; print sys.version[:3]")}
|
|
||||||
-
|
|
||||||
-Name: rteval
|
|
||||||
-Version: 3.7
|
|
||||||
-Release: 1%{?dist}
|
|
||||||
-Summary: Utility to evaluate system suitability for RT Linux
|
|
||||||
-
|
|
||||||
-Group: Development/Tools
|
|
||||||
-License: GPLv2
|
|
||||||
-URL: http://git.kernel.org/?p=linux/kernel/git/clrkwllms/rteval.git
|
|
||||||
-Source0: rteval-%{version}.tar.bz2
|
|
||||||
-BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
|
|
||||||
-
|
|
||||||
-BuildRequires: python3-devel
|
|
||||||
-Requires: platform-python
|
|
||||||
-Requires: python3-ethtool python3-lxml
|
|
||||||
-Requires: python3-dmidecode >= 3.10
|
|
||||||
-Requires: rt-tests >= 0.97
|
|
||||||
-Requires: rteval-loads >= 1.4
|
|
||||||
-Requires: rteval-common => %{version}-%{release}
|
|
||||||
-Requires: sysstat
|
|
||||||
-Requires: bzip2
|
|
||||||
-Requires: kernel-headers
|
|
||||||
-Requires: sos
|
|
||||||
-BuildArch: noarch
|
|
||||||
-Obsoletes: rteval <= 1.7
|
|
||||||
-Requires: numactl
|
|
||||||
-
|
|
||||||
-%description
|
|
||||||
-The rteval script is a utility for measuring various aspects of
|
|
||||||
-realtime behavior on a system under load. The script unpacks the
|
|
||||||
-kernel source, and then goes into a loop, running hackbench and
|
|
||||||
-compiling a kernel tree. During that loop the cyclictest program
|
|
||||||
-is run to measure event response time. After the run time completes,
|
|
||||||
-a statistical analysis of the event response times is done and printed
|
|
||||||
-to the screen.
|
|
||||||
-
|
|
||||||
-
|
|
||||||
-%package common
|
|
||||||
-Summary: Common rteval files
|
|
||||||
-BuildArch: noarch
|
|
||||||
-
|
|
||||||
-%description common
|
|
||||||
-Common files used by rteval, rteval-xmlrpc and rteval-parser
|
|
||||||
-
|
|
||||||
-%prep
|
|
||||||
-%setup -q
|
|
||||||
-
|
|
||||||
-# version sanity check (make sure specfile and rteval.py match)
|
|
||||||
-cp rteval/version.py rtevalversion.py
|
|
||||||
-srcver=$(%{__python} -c "from rtevalversion import RTEVAL_VERSION; print RTEVAL_VERSION")
|
|
||||||
-rm -rf rtevalversion.py
|
|
||||||
-if [ $srcver != %{version} ]; then
|
|
||||||
- printf "\n***\n*** rteval spec file version do not match the rteval/rteval.py version\n***\n\n"
|
|
||||||
- exit -1
|
|
||||||
-fi
|
|
||||||
-
|
|
||||||
-%build
|
|
||||||
-%{__python} setup.py build
|
|
||||||
-
|
|
||||||
-%install
|
|
||||||
-%{__python} setup.py install --root=$RPM_BUILD_ROOT
|
|
||||||
-
|
|
||||||
-%clean
|
|
||||||
-rm -rf $RPM_BUILD_ROOT
|
|
||||||
-
|
|
||||||
-%files common
|
|
||||||
-%doc COPYING
|
|
||||||
-%dir %{_datadir}/%{name}
|
|
||||||
-%{python_sitelib}/rteval/rtevalclient.py*
|
|
||||||
-%{python_sitelib}/rteval/rtevalConfig.py*
|
|
||||||
-%{python_sitelib}/rteval/rtevalXMLRPC.py*
|
|
||||||
-%{python_sitelib}/rteval/version.py*
|
|
||||||
-%{python_sitelib}/rteval/Log.py*
|
|
||||||
-%{python_sitelib}/rteval/misc.py*
|
|
||||||
-%{python_sitelib}/rteval/systopology.py*
|
|
||||||
-
|
|
||||||
-%files
|
|
||||||
-%defattr(-,root,root,-)
|
|
||||||
-%if "%{python_ver}" >= "2.5"
|
|
||||||
-%{python_sitelib}/*.egg-info
|
|
||||||
-%endif
|
|
||||||
-
|
|
||||||
-%doc COPYING README doc/rteval.txt
|
|
||||||
-%{_mandir}/man8/rteval.8.gz
|
|
||||||
-%config(noreplace) %{_sysconfdir}/rteval.conf
|
|
||||||
-%{_datadir}/%{name}/rteval_*.xsl
|
|
||||||
-%{python_sitelib}/rteval/__init__.py*
|
|
||||||
-%{python_sitelib}/rteval/rtevalMailer.py*
|
|
||||||
-%{python_sitelib}/rteval/rtevalReport.py*
|
|
||||||
-%{python_sitelib}/rteval/xmlout.py*
|
|
||||||
-%{python_sitelib}/rteval/modules
|
|
||||||
-%{python_sitelib}/rteval/sysinfo
|
|
||||||
-/usr/bin/rteval
|
|
||||||
-
|
|
||||||
-%changelog
|
|
||||||
-* Thu Mar 16 2017 Clark Williams <williams@redhat.com> - 2.14-1
|
|
||||||
-- removed leftover import of systopology from sysinfo
|
|
||||||
-
|
|
||||||
-* Wed Mar 15 2017 Clark Williams <williams@redhat.com> - 2.13-2
|
|
||||||
-- Updated specfile to correct version and bz [1382155]
|
|
||||||
-
|
|
||||||
-* Tue Sep 20 2016 Clark Williams <williams@rehdat.com> - 2.12-1
|
|
||||||
-- handle empty environment variables SUDO_USER and USER [1312057]
|
|
||||||
-
|
|
||||||
-* Tue Aug 30 2016 Clark Williams <williams@rehdat.com> - 2.11-1
|
|
||||||
-- make sure we return non-zero for early exit from tests
|
|
||||||
-
|
|
||||||
-* Wed Aug 3 2016 Clark Williams <williams@rehdat.com> - 2.10-1
|
|
||||||
-- bumped version for RHEL 7.3 release
|
|
||||||
-
|
|
||||||
-* Mon May 9 2016 Clark Williams <williams@redhat.com> - 2.9.1
|
|
||||||
-- default cpulist for modules if only one specified [1333831]
|
|
||||||
-
|
|
||||||
-* Tue Apr 26 2016 Clark Williams <williams@redhat.com> - 2.8.1
|
|
||||||
-- add the --version option to print the rteval version
|
|
||||||
-- made the --cyclictest-breaktrace option work properly [1209986]
|
|
||||||
-
|
|
||||||
-* Fri Apr 1 2016 Clark Williams <williams@redhat.com> - 2.7.1
|
|
||||||
-- treat SIGINT and SIGTERM as valid end-of-run events [1278757]
|
|
||||||
-- added cpulist options to man page
|
|
||||||
-
|
|
||||||
-* Thu Feb 11 2016 Clark Williams <williams@redhat.com> - 2.6.1
|
|
||||||
-- update to make --loads-cpulist and --measurement-cpulist work [1306437]
|
|
||||||
-
|
|
||||||
-* Thu Dec 10 2015 Clark Williams <williams@refhat.com> - 2.5-1
|
|
||||||
-- stop using old numactl --cpubind argument
|
|
||||||
-
|
|
||||||
-* Wed Dec 9 2015 Clark Williams <williams@refhat.com> - 2.4.2
|
|
||||||
-- added Require of package numactl
|
|
||||||
-
|
|
||||||
-* Tue Nov 17 2015 Clark Williams <williams@refhat.com> - 2.4.1
|
|
||||||
-- rework hackbench load to not generate cross-node traffic [1282826]
|
|
||||||
-
|
|
||||||
-* Wed Aug 12 2015 Clark Williams <williams@redhat.com> - 2.3-1
|
|
||||||
-- comment out HWLatDetect module from default config [1245699]
|
|
||||||
-
|
|
||||||
-* Wed Jun 10 2015 Clark Williams <williams@redhat.com> - 2.2-1
|
|
||||||
-- add --loads-cpulist and --measurement-cpulist to allow cpu placement [1230401]
|
|
||||||
-
|
|
||||||
-* Thu Apr 23 2015 Luis Claudio R. Goncalves <lgoncalv@redhat.com> - 2.1-8
|
|
||||||
-- load default configs when no config file is specified (Jiri kastner) [1212452]
|
|
||||||
-
|
|
||||||
-* Wed Jan 14 2015 Clark Williams <williams@redhat.com> - 2.1-7
|
|
||||||
-- added requires of bzip2 to specfile [1151567]
|
|
||||||
-
|
|
||||||
-* Thu Jan 8 2015 Clark Williams <williams@redhat.com> - 2.1-6
|
|
||||||
-- cleaned up product documentation [1173315]
|
|
||||||
-
|
|
||||||
-* Mon Nov 10 2014 Luis Claudio R. Goncalves <lgoncalv@redhat.com> - 2.1-5
|
|
||||||
-- rebuild for RHEL-7.1 (1151567)
|
|
||||||
-
|
|
||||||
-* Thu Mar 27 2014 Clark Williams <williams@redhat.com> - 2.1-4
|
|
||||||
-- cherry-picked old commit to deal with installdir problem
|
|
||||||
-
|
|
||||||
-* Wed Mar 26 2014 Clark Williams <williams@redhat.com> - 2.1-3
|
|
||||||
-- added sysstat requires to specfile
|
|
||||||
-
|
|
||||||
-* Tue Mar 12 2013 David Sommerseth <davids@redhat.com> - 2.1-2
|
|
||||||
-- Migrated from libxslt-python to python-lxml
|
|
||||||
-
|
|
||||||
-* Fri Jan 18 2013 David Sommerseth <davids@redhat.com> - 2.1-1
|
|
||||||
-- Made some log lines clearer
|
|
||||||
-- cyclictest: Added --cyclictest-breaktrace feature
|
|
||||||
-- cyclictest: Removed --cyclictest-distance option
|
|
||||||
-- cyclictest: Use a tempfile buffer for cyclictest's stdout data
|
|
||||||
-- cyclictest: Report if breaktrace was triggered
|
|
||||||
-- cyclictest: Make the unit test work again
|
|
||||||
-- cyclictest: Only log and show statistic data when samples are collected
|
|
||||||
-- Copyright updates
|
|
||||||
-
|
|
||||||
-* Thu Jan 17 2013 David Sommerseth <davids@redhat.com> - 2.0.1-1
|
|
||||||
-- Fix up type casting in the core module code
|
|
||||||
-- hwlatdetect: Add some more useful debug info
|
|
||||||
-- Reworked the run logic for modules - allow them to flag they won't run
|
|
||||||
-- Fixed a few log messages in load modules
|
|
||||||
-- Add a 30 seconds sleep before unleashing the measurement threads
|
|
||||||
-
|
|
||||||
-* Thu Jan 10 2013 David Sommerseth <davids@redhat.com> - 2.0-3
|
|
||||||
-- Separate out RTEVAL_VERSION into rteval.version, to avoid
|
|
||||||
- massive BuildRequirements
|
|
||||||
-
|
|
||||||
-* Fri Dec 21 2012 David Sommerseth <davids@redhat.com> - 2.0-2
|
|
||||||
-- Split out common files into rteval-common
|
|
||||||
-
|
|
||||||
-* Fri Dec 21 2012 David Sommerseth <davids@redhat.com> - 2.0-1
|
|
||||||
-- Updated to rteval v2.0 and reworked spec file to use setup.py directly
|
|
||||||
-
|
|
||||||
-* Tue Oct 23 2012 Clark Williams <williams@redhat.com> - 1.36-1
|
|
||||||
-- deal with system not having dmidecode python module
|
|
||||||
-- make sure to cast priority parameter to int
|
|
||||||
-- from Raphaël Beamonte <raphael.beamonte@gmail.com>:
|
|
||||||
- - Rewrite of the get_kthreads method to make it cross-distribution
|
|
||||||
- - Adds getcmdpath method to use which to locate the used commands
|
|
||||||
- - Rewrite of the get_services method to make it cross-distribution
|
|
||||||
-
|
|
||||||
-* Mon Apr 2 2012 Clark Williams <williams@redhat.com> - 1.35-1
|
|
||||||
-- fix thinko where SIGINT and SIGTERM handlers were commented out
|
|
||||||
-
|
|
||||||
-* Thu Jan 12 2012 Clark Williams <williams@redhat.com> - 1.34-1
|
|
||||||
-- fix missing config merge in rteval.py to pass parameters
|
|
||||||
- down to cyclictest
|
|
||||||
-- modify hackbench to use helper function to start process
|
|
||||||
-
|
|
||||||
-* Sat May 14 2011 Clark Williams <williams@redhat.com> - 1.33-1
|
|
||||||
-- modify hackbench cutoff to be 0.75GB/core
|
|
||||||
-
|
|
||||||
-* Mon Aug 23 2010 Clark Williams <williams@redhat.com> - 1.32-1
|
|
||||||
-- update docs
|
|
||||||
-- refactor some RTEval methods to utility functions
|
|
||||||
-- modify hackbench.py not to run under low memory conditions
|
|
||||||
-- clean up XML generation to deal with new hackbench code
|
|
||||||
-- clean up XSL code to deal with new XML 'run' attribute
|
|
||||||
-- from David Sommerseth <davids@redhat.com>:
|
|
||||||
- - improve CPU socket counting logic
|
|
||||||
- - delay log directory creation until actually needed
|
|
||||||
-- from Gowrishankar <gowrishankar.m@in.ibm.com>:
|
|
||||||
- - check if the core id really exists (multithreading fix)
|
|
||||||
-
|
|
||||||
-* Mon Jul 26 2010 Clark Williams <williams@redhat.com> - 1.31-1
|
|
||||||
-- from David Sommerseth <davids@redhat.com>:
|
|
||||||
- - Updated hackbench implementation to avoid overusing resources
|
|
||||||
- - Don't show NUMA node information if it's missing in the summary.xml
|
|
||||||
- - Show CPU cores properly
|
|
||||||
-
|
|
||||||
-* Wed Jul 21 2010 Clark Williams <williams@redhat.com> - 1.30-1
|
|
||||||
-- added code to hackbench to try to detect and ease memory pressure
|
|
||||||
-
|
|
||||||
-* Fri Jul 16 2010 Clark Williams <williams@redhat.com> - 1.29-1
|
|
||||||
-- fixed incorrect type value in kcompile.py
|
|
||||||
-
|
|
||||||
-* Fri Jul 16 2010 Clark Williams <williams@redhat.com> - 1.28-1
|
|
||||||
-- added logic to loads to adjust number of jobs based on ratio
|
|
||||||
- of memory per core
|
|
||||||
-
|
|
||||||
-* Wed Jul 14 2010 Clark Williams <williams@redhat.com> - 1.27-1
|
|
||||||
-- modified hackbench to go back to using threads rather than
|
|
||||||
- processes for units of work
|
|
||||||
-- added memory size, number of numa nodes and run duration to the
|
|
||||||
- parameter dictionary passed to all loads and cyclictest
|
|
||||||
-
|
|
||||||
-* Tue Jul 13 2010 Clark Williams <williams@redhat.com> - 1.26-1
|
|
||||||
-- modified hackbench parameters to reduce memory consumption
|
|
||||||
-
|
|
||||||
-* Mon Jul 12 2010 Clark Williams <williams@redhat.com> - 1.25-1
|
|
||||||
-- fixed cyclictest bug that caused everything to be uniprocessor
|
|
||||||
-- updated source copyrights to 2010
|
|
||||||
-
|
|
||||||
-* Fri Jul 9 2010 Clark Williams <williams@redhat.com> - 1.24-1
|
|
||||||
-- modified hackbench arguments and added new parameters for
|
|
||||||
- hackbench in rteval.conf
|
|
||||||
-
|
|
||||||
-* Thu Jul 8 2010 Clark Williams <williams@redhat.com> - 1.23-1
|
|
||||||
-- version bump to deal with out-of-sync cvs issue
|
|
||||||
-
|
|
||||||
-* Thu Jul 8 2010 Clark Williams <williams@redhat.com> - 1.22-1
|
|
||||||
-- merged David Sommerseth <davids@redhat.com> changes to use
|
|
||||||
- hackbench from rt-tests packages rather than carry local copy
|
|
||||||
-- converted all loads and cyclictest to pass __init__ parameters
|
|
||||||
- in a dictionary rather than as discrete parameters
|
|
||||||
-- added logging for load output
|
|
||||||
-
|
|
||||||
-* Tue Apr 13 2010 Clark Williams <williams@redhat.com> - 1.21-1
|
|
||||||
-- from Luis Claudio Goncalves <lgoncalv@redhat.com>:
|
|
||||||
- - remove unecessary wait() call in cyclictest.py
|
|
||||||
- - close /dev/null after using it
|
|
||||||
- - call subprocess.wait() when needed
|
|
||||||
- - remove delayloop code in hackbench.py
|
|
||||||
-- from David Sommerseth <davids@redhat.com>:
|
|
||||||
- - add SIGINT handler
|
|
||||||
- - handle non-root user case
|
|
||||||
- - process DMI warnings before command line arguments
|
|
||||||
- - added --annotate feature to rteval
|
|
||||||
- - updates to xmlrpc code
|
|
||||||
-
|
|
||||||
-* Tue Apr 6 2010 Clark Williams <williams@redhat.com> - 1.20-1
|
|
||||||
-- code fixes from Luis Claudio Goncalves <lgoncalv@redhat.com>
|
|
||||||
-- from David Sommerseth <davids@redhat.com>:
|
|
||||||
- - xmlrpc server updates
|
|
||||||
- - cputopology.py for recording topology in xml
|
|
||||||
- - added NUMA node recording for run data
|
|
||||||
- - rpmlint fixes
|
|
||||||
-- added start of rteval whitepaper in docs dir
|
|
||||||
-
|
|
||||||
-* Tue Mar 16 2010 Clark Williams <williams@redhat.com> - 1.19-1
|
|
||||||
-- add ability for --summarize to read tarfiles
|
|
||||||
-- from David Sommerseth <davids@redhat.com>
|
|
||||||
- - gather info about loaded kernel modules for XML file
|
|
||||||
- - added child tracking to hackbench to prevent zombies
|
|
||||||
-
|
|
||||||
-* Tue Feb 16 2010 Clark Williams <williams@redhat.com> - 1.18-1
|
|
||||||
-- fix usage of python 2.6 features on RHEL5 (python 2.4)
|
|
||||||
-
|
|
||||||
-* Tue Feb 16 2010 Clark Williams <williams@redhat.com> - 1.17-1
|
|
||||||
-- added logic to filter non-printables from service status output
|
|
||||||
- so that we have legal XML output
|
|
||||||
-- added logic to hackbench.py to cleanup properly at the end
|
|
||||||
- of the test
|
|
||||||
-
|
|
||||||
-* Thu Feb 11 2010 Clark Williams <williams@redhat.com> - 1.16-1
|
|
||||||
-- fix errors in show_remaining_time() introduced because
|
|
||||||
- time values are floats rather than ints
|
|
||||||
-
|
|
||||||
-* Thu Feb 11 2010 Clark Williams <williams@redhat.com> - 1.15-1
|
|
||||||
-- added logic to use --numa and --smp options of new cyclictest
|
|
||||||
-- added countdown report for time remaining in a run
|
|
||||||
-
|
|
||||||
-* Tue Feb 9 2010 Clark Williams <williams@redhat.com> - 1.14-1
|
|
||||||
-- David Sommerseth <davids@redhat.com>:
|
|
||||||
- merged XMLReport() changes for hwcert suite
|
|
||||||
-
|
|
||||||
-* Tue Dec 22 2009 Clark Williams <williams@redhat.com> - 1.13-1
|
|
||||||
-- added cyclictest default initializers
|
|
||||||
-- added sanity checks to statistics reduction code
|
|
||||||
-- updated release checklist to include origin push
|
|
||||||
-- updated Makefile clean and help targets
|
|
||||||
-- davids updates (mainly for v7 integration):
|
|
||||||
- - Add explicit sys.path directory to the python sitelib+
|
|
||||||
- '/rteval'
|
|
||||||
- - Send program arguments via RtEval() constructor
|
|
||||||
- - Added more DMI data into the summary.xml report
|
|
||||||
- - Fixed issue with not including all devices in the
|
|
||||||
- OnBoardDeviceInfo tag
|
|
||||||
-
|
|
||||||
-* Thu Dec 3 2009 David Sommerseth <davids@redhat.com> - 1.12-2
|
|
||||||
-- fixed Makefile and specfile to include and install the
|
|
||||||
- rteval/rteval_histogram_raw.py source file for gaining
|
|
||||||
- raw access to histogram data
|
|
||||||
-- Removed xmlrpc package during merge against master_ipv4 branch
|
|
||||||
-
|
|
||||||
-* Wed Nov 25 2009 Clark Williams <williams@redhat.com> - 1.12-1
|
|
||||||
-- fix incorrect reporting of measurement thread priorities
|
|
||||||
-
|
|
||||||
-* Mon Nov 16 2009 Clark Williams <williams@redhat.com> - 1.11-5
|
|
||||||
-- ensure that no double-slashes ("//") appear in the symlink
|
|
||||||
- path for /usr/bin/rteval (problem with rpmdiff)
|
|
||||||
-
|
|
||||||
-* Tue Nov 10 2009 Clark Williams <williams@redhat.com> - 1.11-4
|
|
||||||
-- changed symlink back to install and tracked by %%files
|
|
||||||
-
|
|
||||||
-* Mon Nov 9 2009 Clark Williams <williams@redhat.com> - 1.11-3
|
|
||||||
-- changed symlink generation from %%post to %%posttrans
|
|
||||||
-
|
|
||||||
-* Mon Nov 9 2009 Clark Williams <williams@redhat.com> - 1.11-2
|
|
||||||
-- fixed incorrect dependency for libxslt
|
|
||||||
-
|
|
||||||
-* Fri Nov 6 2009 Clark Williams <williams@redhat.com> - 1.11-1
|
|
||||||
-- added base OS info to XML file and XSL report
|
|
||||||
-- created new package rteval-loads for the load source code
|
|
||||||
-
|
|
||||||
-* Wed Nov 4 2009 Clark Williams <williams@redhat.com> - 1.10-1
|
|
||||||
-- added config file section for cyclictest and two settable
|
|
||||||
- parameters, buckets and interval
|
|
||||||
-
|
|
||||||
-* Thu Oct 29 2009 Clark Williams <williams@redhat.com> - 1.9-1
|
|
||||||
-- merged davids updates:
|
|
||||||
- -H option (raw histogram data)
|
|
||||||
- cleaned up xsl files
|
|
||||||
- fixed cpu sorting
|
|
||||||
-
|
|
||||||
-* Mon Oct 26 2009 David Sommerseth <davids@redhat.com> - 1.8-3
|
|
||||||
-- Fixed rpmlint complaints
|
|
||||||
-
|
|
||||||
-* Mon Oct 26 2009 David Sommerseth <davids@redhat.com> - 1.8-2
|
|
||||||
-- Added xmlrpc package, containing the XML-RPC mod_python modules
|
|
||||||
-
|
|
||||||
-* Tue Oct 20 2009 Clark Williams <williams@redhat.com> - 1.8-1
|
|
||||||
-- split kcompile and hackbench into sub-packages
|
|
||||||
-- reworked Makefile (and specfile) install/uninstall logic
|
|
||||||
-- fixed sysreport incorrect plugin option
|
|
||||||
-- catch failure when running on root-squashed NFS
|
|
||||||
-
|
|
||||||
-* Tue Oct 13 2009 Clark Williams <williams@redhat.com> - 1.7-1
|
|
||||||
-- added kthread status to xml file
|
|
||||||
-- merged davids changes for option processing and additions
|
|
||||||
- to xml summary
|
|
||||||
-
|
|
||||||
-* Tue Oct 13 2009 Clark Williams <williams@redhat.com> - 1.6-1
|
|
||||||
-- changed stat calculation to loop less
|
|
||||||
-- added methods to grab service and kthread status
|
|
||||||
-
|
|
||||||
-* Mon Oct 12 2009 Clark Williams <williams@redhat.com> - 1.5-1
|
|
||||||
-- changed cyclictest to use less memory when doing statisics
|
|
||||||
- calculations
|
|
||||||
-- updated debug output to use module name prefixes
|
|
||||||
-- changed option processing to only process config file once
|
|
||||||
-
|
|
||||||
-* Fri Oct 9 2009 Clark Williams <williams@redhat.com> - 1.4-1
|
|
||||||
-- changed cyclictest to use histogram rather than sample array
|
|
||||||
-- calcuated statistics directly from histogram
|
|
||||||
-- changed sample interval to 100us
|
|
||||||
-- added -a (affinity) argument to force cpu affinity for
|
|
||||||
- measurement threads
|
|
||||||
-
|
|
||||||
-* Thu Sep 24 2009 David Sommerseth <davids@redhat.com> - 1.3-3
|
|
||||||
-- Cleaned up the spec file and made rpmlint happy
|
|
||||||
-
|
|
||||||
-* Wed Sep 23 2009 David Sommerseth <davids@redhat.com> - 1.3-2
|
|
||||||
-- Removed version number from /usr/share/rteval path
|
|
||||||
-
|
|
||||||
-* Tue Sep 22 2009 Clark Williams <williams@redhat.com> - 1.3-1
|
|
||||||
-- changes from davids:
|
|
||||||
- * changed report code to sort by processor id
|
|
||||||
- * added report submission retry logic
|
|
||||||
- * added emailer class
|
|
||||||
-
|
|
||||||
-* Fri Sep 18 2009 Clark Williams <williams@redhat.com> - 1.2-1
|
|
||||||
-- added config file handling for modifying load behavior and
|
|
||||||
- setting defaults
|
|
||||||
-- added units in report per IBM request
|
|
||||||
-
|
|
||||||
-* Wed Aug 26 2009 Clark Williams <williams@redhat.com> - 1.1-2
|
|
||||||
-- missed a version change in rteval/rteval.py
|
|
||||||
-
|
|
||||||
-* Wed Aug 26 2009 Clark Williams <williams@redhat.com> - 1.1-1
|
|
||||||
-- modified cyclictest.py to start cyclictest threads with a
|
|
||||||
- 'distance' of zero, meaning they all have the same measurement
|
|
||||||
- interval
|
|
||||||
-
|
|
||||||
-* Tue Aug 25 2009 Clark Williams <williams@redhat.com> - 1.0-1
|
|
||||||
-- merged davids XMLRPC fixes
|
|
||||||
-- fixed --workdir option
|
|
||||||
-- verion bump to 1.0
|
|
||||||
-
|
|
||||||
-* Thu Aug 13 2009 Clark Williams <williams@redhat.com> - 0.9-2
|
|
||||||
-- fixed problem with incorrect version in rteval.py
|
|
||||||
-
|
|
||||||
-* Tue Aug 4 2009 Clark Williams <williams@redhat.com> - 0.9-1
|
|
||||||
-- merged dsommers XMLRPC and database changes
|
|
||||||
-- Specify minimum python-dmidecode version, which got native XML support
|
|
||||||
-- Added rteval_dmi.xsl
|
|
||||||
-- Fixed permission issues in /usr/share/rteval-x.xx
|
|
||||||
-
|
|
||||||
-* Wed Jul 22 2009 Clark Williams <williams@redhat.com> - 0.8-1
|
|
||||||
-- added code to capture clocksource info
|
|
||||||
-- added code to copy dmesg info to report directory
|
|
||||||
-- added code to display clocksource info in report
|
|
||||||
-- added --summarize option to display summary of existing report
|
|
||||||
-- added helpfile target to Makefile
|
|
||||||
-
|
|
||||||
-* Thu Mar 26 2009 Clark Williams <williams@torg> - 0.7-1
|
|
||||||
-- added require for python-schedutils to specfile
|
|
||||||
-- added default for cyclictest output file
|
|
||||||
-- added help parameter to option parser data
|
|
||||||
-- renamed xml output file to summary.xml
|
|
||||||
-- added routine to create tarfile of result files
|
|
||||||
-
|
|
||||||
-* Wed Mar 18 2009 Clark Williams <williams@torg> - 0.6-6
|
|
||||||
-- added code to handle binary data coming from DMI tables
|
|
||||||
-
|
|
||||||
-* Wed Mar 18 2009 Clark Williams <williams@torg> - 0.6-5
|
|
||||||
-- fixed logic for locating XSL template (williams)
|
|
||||||
-- fixed another stupid typo in specfile (williams)
|
|
||||||
-
|
|
||||||
-* Wed Mar 18 2009 Clark Williams <williams@torg> - 0.6-4
|
|
||||||
-- fixed specfile to install rteval_text.xsl in /usr/share directory
|
|
||||||
-
|
|
||||||
-* Wed Mar 18 2009 Clark Williams <williams@torg> - 0.6-3
|
|
||||||
-- added Requires for libxslt-python (williams)
|
|
||||||
-- fixed race condition in xmlout constructor/destructor (williams)
|
|
||||||
-
|
|
||||||
-* Wed Mar 18 2009 Clark Williams <williams@torg> - 0.6-2
|
|
||||||
-- added Requires for libxslt (williams)
|
|
||||||
-- fixed stupid typo in rteval/rteval.py (williams)
|
|
||||||
-
|
|
||||||
-* Wed Mar 18 2009 Clark Williams <williams@torg> - 0.6-1
|
|
||||||
-- added xml output logic (williams, dsommers)
|
|
||||||
-- added xlst template for report generator (dsommers)
|
|
||||||
-- added dmi/smbios output to report (williams)
|
|
||||||
-- added __del__ method to hackbench to cleanup after run (williams)
|
|
||||||
-- modified to always keep run data (williams)
|
|
||||||
-
|
|
||||||
-* Fri Feb 20 2009 Clark Williams <williams@torg> - 0.5-1
|
|
||||||
-- fixed tab/space mix problem
|
|
||||||
-- added report path line to report
|
|
||||||
-
|
|
||||||
-* Fri Feb 20 2009 Clark Williams <williams@torg> - 0.4-1
|
|
||||||
-- reworked report output
|
|
||||||
-- handle keyboard interrupt better
|
|
||||||
-- removed duration mismatch between rteval and cyclictest
|
|
||||||
-
|
|
||||||
-* Mon Feb 2 2009 Clark Williams <williams@torg> - 0.3-1
|
|
||||||
-- initial checkin
|
|
||||||
--
|
|
||||||
2.41.0
|
|
||||||
|
|
@ -1,6 +1,6 @@
|
|||||||
Name: rteval
|
Name: rteval
|
||||||
Version: 3.7
|
Version: 3.3
|
||||||
Release: 4%{?dist}
|
Release: 6%{?dist}
|
||||||
Summary: Utility to evaluate system suitability for RT Linux
|
Summary: Utility to evaluate system suitability for RT Linux
|
||||||
|
|
||||||
Group: Development/Tools
|
Group: Development/Tools
|
||||||
@ -11,12 +11,11 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
|
|||||||
|
|
||||||
BuildRequires: python3-devel
|
BuildRequires: python3-devel
|
||||||
Requires: platform-python
|
Requires: platform-python
|
||||||
|
Requires: python3-ethtool
|
||||||
Requires: python3-lxml
|
Requires: python3-lxml
|
||||||
Requires: python3-libxml2
|
|
||||||
Requires: python3-dmidecode >= 3.10
|
Requires: python3-dmidecode >= 3.10
|
||||||
Requires: python3-requests
|
|
||||||
Requires: rt-tests >= 1.5-11
|
Requires: rt-tests >= 1.5-11
|
||||||
Requires: rteval-loads >= 1.6-2
|
Requires: rteval-loads >= 1.5-1
|
||||||
Requires: sysstat
|
Requires: sysstat
|
||||||
Requires: xz bzip2 tar gzip m4 make gawk
|
Requires: xz bzip2 tar gzip m4 make gawk
|
||||||
Requires: kernel-headers
|
Requires: kernel-headers
|
||||||
@ -33,13 +32,11 @@ Requires: libmpc libmpc-devel
|
|||||||
Obsoletes: rteval-common <= 3.1
|
Obsoletes: rteval-common <= 3.1
|
||||||
|
|
||||||
#Patches
|
#Patches
|
||||||
Patch1: Revert-rteval-Change-the-default-kernel.patch
|
Patch1: rteval-Fix-test-misses-threshold-assignment.patch
|
||||||
Patch2: rteval-Remove-upstream-spec-file.patch
|
Patch2: rteval-Fix-Popen-for-python3.6-where-text-True-is-no.patch
|
||||||
Patch3: rteval-Refactor-collapse_cpulist-in-systopology.patch
|
Patch3: rteval-Increase-default-buckets-from-2000-to-3500.patch
|
||||||
Patch4: rteval-Minor-improvements-to-CpuList-class.patch
|
Patch4: rteval-Don-t-restrict-measurement-threads-to-inherit.patch
|
||||||
Patch5: rteval-Convert-CpuList-class-to-a-module.patch
|
Patch5: rteval-If-the-user-doesn-t-specify-a-cpumask-use-the.patch
|
||||||
Patch6: rteval-Add-relative-cpulists-for-measurements.patch
|
|
||||||
|
|
||||||
|
|
||||||
%description
|
%description
|
||||||
The rteval script is a utility for measuring various aspects of
|
The rteval script is a utility for measuring various aspects of
|
||||||
@ -57,8 +54,6 @@ to the screen.
|
|||||||
%patch3 -p1
|
%patch3 -p1
|
||||||
%patch4 -p1
|
%patch4 -p1
|
||||||
%patch5 -p1
|
%patch5 -p1
|
||||||
%patch6 -p1
|
|
||||||
|
|
||||||
|
|
||||||
%build
|
%build
|
||||||
%{__python3} setup.py build
|
%{__python3} setup.py build
|
||||||
@ -66,6 +61,9 @@ to the screen.
|
|||||||
%install
|
%install
|
||||||
%{__python3} setup.py install --root=$RPM_BUILD_ROOT
|
%{__python3} setup.py install --root=$RPM_BUILD_ROOT
|
||||||
|
|
||||||
|
%clean
|
||||||
|
rm -rf $RPM_BUILD_ROOT
|
||||||
|
|
||||||
%files
|
%files
|
||||||
%defattr(-,root,root,-)
|
%defattr(-,root,root,-)
|
||||||
%{python3_sitelib}/*.egg-info
|
%{python3_sitelib}/*.egg-info
|
||||||
@ -77,8 +75,8 @@ to the screen.
|
|||||||
%{python3_sitelib}/rteval/rtevalXMLRPC.py*
|
%{python3_sitelib}/rteval/rtevalXMLRPC.py*
|
||||||
%{python3_sitelib}/rteval/version.py*
|
%{python3_sitelib}/rteval/version.py*
|
||||||
%{python3_sitelib}/rteval/Log.py*
|
%{python3_sitelib}/rteval/Log.py*
|
||||||
|
%{python3_sitelib}/rteval/misc.py*
|
||||||
%{python3_sitelib}/rteval/systopology.py*
|
%{python3_sitelib}/rteval/systopology.py*
|
||||||
%{python3_sitelib}/rteval/cpulist_utils.py*
|
|
||||||
%{_mandir}/man8/rteval.8.gz
|
%{_mandir}/man8/rteval.8.gz
|
||||||
%config(noreplace) %{_sysconfdir}/rteval.conf
|
%config(noreplace) %{_sysconfdir}/rteval.conf
|
||||||
%{_datadir}/%{name}/rteval_*.xsl
|
%{_datadir}/%{name}/rteval_*.xsl
|
||||||
@ -93,100 +91,6 @@ to the screen.
|
|||||||
%{python3_sitelib}/rteval/__pycache__/*
|
%{python3_sitelib}/rteval/__pycache__/*
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
* Wed Jan 31 2024 Tomas Glozar <tglozar@redhat.com> - 3.7-4
|
|
||||||
- Added patchset for relative cpuset functionality from upstream
|
|
||||||
Resolves: RHEL-21926
|
|
||||||
|
|
||||||
* Wed Nov 15 2023 John Kacur <jkacur@redhat.com> - 3.7-3
|
|
||||||
- Add an rpminspect file to pass gating which is failing due to
|
|
||||||
python egg-info directory permissions
|
|
||||||
Resolves: RHEL-16401
|
|
||||||
|
|
||||||
* Mon Nov 13 2023 John Kacur <jkacur@redhat.com> - 3.7-2
|
|
||||||
- Remove upstream spec files
|
|
||||||
Resolves: RHEL-9189
|
|
||||||
|
|
||||||
* Wed Oct 25 2023 John Kacur <jkacur@redhat.com> - 3.7-1
|
|
||||||
- Rebase to upstream rteval-3.7
|
|
||||||
- Revert the change to use a newer default kernel
|
|
||||||
Resolves: RHEL-8967
|
|
||||||
|
|
||||||
* Thu Oct 05 2023 Tomas Glozar <tglozar@redhat.com> - 3.5-9
|
|
||||||
- Added patch set that enables rteval to do load calculations and reporting
|
|
||||||
correctly on systems with isolated CPUs
|
|
||||||
jiraProject== RHEL-8680
|
|
||||||
|
|
||||||
* Wed Oct 04 2023 John Kacur <jkacur@redhat.com> - 3.5-8
|
|
||||||
- Added patches to use argparse instead of deprecated optparse
|
|
||||||
jiraProject == RHEL-9029
|
|
||||||
|
|
||||||
* Fri Aug 11 2023 John Kacur <jkacur@redhat.com> - 3.5-7
|
|
||||||
- Added patches to use f-strings where possible, no functional change
|
|
||||||
jiraProject == RHEL-797
|
|
||||||
|
|
||||||
* Thu Jun 08 2023 John Kacur <jkacur@redhat.com> - 3.5-6
|
|
||||||
- Add tests/tests.yml and tests/scripts/run_tests.sh for gating
|
|
||||||
Resolves: rhbz#2213609
|
|
||||||
jiraProject == RHELPLAN-159326
|
|
||||||
|
|
||||||
* Wed Jun 07 2023 John Kacur <jkacur@redhat.com> - 3.5-5
|
|
||||||
- Added code to check if the /proc/net/if_net6 file exists.
|
|
||||||
Resolves: rhbz#2210106
|
|
||||||
jiraProject == RHELPLAN-158239
|
|
||||||
|
|
||||||
* Wed Feb 08 2023 John Kacur <jkacur@redhat.com> - 3.5-4
|
|
||||||
- Add check to catch python-dmidecode if it fails
|
|
||||||
Resolves: rhbz#2168373
|
|
||||||
jiraProject == RHELPLAN-148048
|
|
||||||
|
|
||||||
* Thu Oct 27 2022 John Kacur <jkacur@redhat.com> - 3.5-3
|
|
||||||
- Fix "DMI WARNING" when not running as root
|
|
||||||
- Don't attempt to get DMI info if dmidecode returns warnings
|
|
||||||
Resolves: rhbz#2136926
|
|
||||||
|
|
||||||
* Mon Oct 17 2022 John Kacur <jkacur@redhat.com> - 3.5-2
|
|
||||||
- Remove dependency on python-ethtool by using inline code
|
|
||||||
- Add Requires of python-libxml2
|
|
||||||
Resolves: rhbz#2131377
|
|
||||||
|
|
||||||
* Fri Sep 23 2022 John Kacur <jkacur@redhat.com> - 3.5-1
|
|
||||||
- Rebase to rteval-3.5 upstream
|
|
||||||
Resolves: rhbz#2119172
|
|
||||||
|
|
||||||
* Thu Sep 22 2022 Leah Leshchinsky <lleshchi@redhat.com> - 3.4-5
|
|
||||||
- Add measurement and load location to run report
|
|
||||||
Resolves: rhbz#2082260
|
|
||||||
|
|
||||||
* Tue Sep 13 2022 John Kacur <jkacur@redhat.com> - 3.4-4
|
|
||||||
- Make use of systopology instead of misc everywhere
|
|
||||||
- Allow user to enter compressed form of cpulist
|
|
||||||
Resolves: rhbz#2121535
|
|
||||||
|
|
||||||
* Mon Sep 12 2022 John Kacur <jkacur@redhat.com> - 3.4-3
|
|
||||||
- Add option for downloading the kernel to compile as a load
|
|
||||||
- Add a manpage entry for the kernel download option
|
|
||||||
Resolves: rhbz#2107710
|
|
||||||
|
|
||||||
* Tue Jun 28 2022 John Kacur <jkacur@redhat.com> - 3.4-2
|
|
||||||
- Add back __pycache__ to the rhel-8.7 spec file
|
|
||||||
Resolves: rhbz#2069354
|
|
||||||
|
|
||||||
* Tue Jun 28 2022 John Kacur <jkacur@redhat.com> - 3.4-1
|
|
||||||
- Rebase to rteval-3.4 upstream
|
|
||||||
Resolves: rhbz#2069354
|
|
||||||
|
|
||||||
* Wed Jun 22 2022 John Kacur <jkacur@redhat.com> - 3.3-9
|
|
||||||
- Add upstream kcompile patches
|
|
||||||
Resolves: rhbz#2093478
|
|
||||||
|
|
||||||
* Tue Jun 07 2022 John Kacur <jkacur@redhat.com> - 3.3-8
|
|
||||||
- Updates the Requires for rteval-loads with the correct kernel version
|
|
||||||
Resolves: rhbz#2093478
|
|
||||||
|
|
||||||
* Tue Jun 07 2022 John Kacur <jkacur@redhat.com> - 3.3-7
|
|
||||||
- Change the default kernel to compile to linux-5.18.1
|
|
||||||
Resolves: rhbz#2093478
|
|
||||||
|
|
||||||
* Tue Feb 15 2022 John Kacur <jkacur@redhat.com> - 3.3-6
|
* Tue Feb 15 2022 John Kacur <jkacur@redhat.com> - 3.3-6
|
||||||
- Use inherited cpumask if user doesn't specify a cpumask
|
- Use inherited cpumask if user doesn't specify a cpumask
|
||||||
Resolves: rhbz#2012284
|
Resolves: rhbz#2012284
|
||||||
|
Loading…
Reference in New Issue
Block a user