From: tglozar@redhat.com
To: linux-rt-users@vger.kernel.org
Cc: jkacur@redhat.com, Tomas Glozar <tglozar@redhat.com>
Subject: [PATCH v2 0/3] rteval: Refactor CPU list logic
Date: Thu, 23 Nov 2023 08:24:33 +0100 [thread overview]
Message-ID: <20231123072436.270267-1-tglozar@redhat.com> (raw)
From: Tomas Glozar <tglozar@redhat.com>
The main change of this patchset is the removal of the CpuList class, moving
the related logic to a new module rteval.cpulist_utils. Motivation for this
is further explained in the corresponding commit description.
In addition to that, there are a few smaller fixes in the first two commits.
These can be merged separately from the removal of the CpuList class
in the third commit.
v2: First patch ("rteval: Refactor collapse_cpulist in systopology")
broke --measurement-run-on-isolcpus because the new implementation of
collapse_cpulist does not allow for string cpus in input. Changed to
always convert cpus to string when passing to collapse_cpulist.
Third patch ("rteval: Convert CpuList class to a module") was missing
replacing the old calls to CpuList functions with the new cpulist_util ones
at a few places, causing rteval to fail to run with --*-cpulist option. Fixed.
Tomas Glozar (3):
rteval: Refactor collapse_cpulist in systopology
rteval: Minor improvements to CpuList class
rteval: Convert CpuList class to a module
rteval-cmd | 17 +-
rteval/cpulist_utils.py | 125 +++++++++++++++
rteval/modules/loads/__init__.py | 8 +-
rteval/modules/loads/hackbench.py | 9 +-
rteval/modules/loads/kcompile.py | 14 +-
rteval/modules/loads/stressng.py | 9 +-
rteval/modules/measurement/__init__.py | 8 +-
rteval/modules/measurement/cyclictest.py | 12 +-
rteval/systopology.py | 190 ++---------------------
9 files changed, 175 insertions(+), 217 deletions(-)
create mode 100644 rteval/cpulist_utils.py
--
2.41.0
next reply other threads:[~2023-11-23 7:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-23 7:24 tglozar [this message]
2023-11-23 7:24 ` [PATCH v2 1/3] rteval: Refactor collapse_cpulist in systopology tglozar
2023-11-23 7:24 ` [PATCH v2 2/3] rteval: Minor improvements to CpuList class tglozar
2023-11-23 16:20 ` John Kacur
2023-11-24 7:05 ` Tomas Glozar
2023-11-24 15:27 ` John Kacur
2023-11-24 16:51 ` Tomas Glozar
2023-11-24 20:06 ` John Kacur
2023-11-27 7:52 ` Tomas Glozar
2023-11-24 21:38 ` gene heskett
2023-11-23 7:24 ` [PATCH v2 3/3] rteval: Convert CpuList class to a module tglozar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20231123072436.270267-1-tglozar@redhat.com \
--to=tglozar@redhat.com \
--cc=jkacur@redhat.com \
--cc=linux-rt-users@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).