diff options
author | John Kacur <jkacur@redhat.com> | 2022-07-26 20:58:46 -0400 |
---|---|---|
committer | John Kacur <jkacur@redhat.com> | 2022-08-03 11:42:22 -0400 |
commit | 7399c8f9fdf8e1c152dde1ca86a757559dfc72ab (patch) | |
tree | 262287730dbf26e12e90bc706de5133da461737d | |
parent | d7e4b9e9c350b3572ebcf2b3dff20f21be0c21ca (diff) | |
download | rteval-7399c8f9fdf8e1c152dde1ca86a757559dfc72ab.tar.gz |
rteval: cyclictest: Reset cpulist from newly calculated cpus
After we recalculate self.__cpus to only include online cpus, we also
need to reset the self.__cpulist which is the collapsed form of the list
that we pass as parameters to cyclictest, otherwise we can potentially
pass an offline cpu as a parameter to cyclictest which will then fail
Signed-off-by: John Kacur <jkacur@redhat.com>
-rw-r--r-- | rteval/modules/measurement/cyclictest.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/rteval/modules/measurement/cyclictest.py b/rteval/modules/measurement/cyclictest.py index 9e7f4ba..e235b83 100644 --- a/rteval/modules/measurement/cyclictest.py +++ b/rteval/modules/measurement/cyclictest.py @@ -36,7 +36,7 @@ import libxml2 from rteval.Log import Log from rteval.modules import rtevalModulePrototype from rteval.misc import cpuinfo -from rteval.systopology import CpuList, SysTopology +from rteval.systopology import CpuList, SysTopology, collapse_cpulist expand_cpulist = CpuList.expand_cpulist @@ -220,6 +220,8 @@ class Cyclictest(rtevalModulePrototype): self.__cpus = expand_cpulist(self.__cpulist) # 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.__cpus = [str(c) for c in self.__cpus] self.__sparse = True else: |