From: John Kacur <jkacur@redhat.com>
To: Tomas Glozar <tglozar@redhat.com>
Cc: linux-rt-users@vger.kernel.org
Subject: Re: [PATCH v2 2/3] rteval: Add run_on_isolcpus option to measurements
Date: Fri, 15 Sep 2023 15:01:39 -0400 (EDT) [thread overview]
Message-ID: <8e40bb39-a52b-9a58-23d3-0bfeb88abf@redhat.com> (raw)
In-Reply-To: <20230811095228.200772-3-tglozar@redhat.com>
On Fri, 11 Aug 2023, Tomas Glozar wrote:
> Add option --measurement-run-on-isolcpus for the user to specify to
> include isolcpus in the default cpulist (without --measurement-cpulist).
>
> A default value might also be specified in rteval.conf:
> [measurement]
> run-on-isolcpus: true
>
> Note that values different from true evaluate as false.
>
> Signed-off-by: Tomas Glozar <tglozar@redhat.com>
> ---
> rteval/modules/measurement/__init__.py | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/rteval/modules/measurement/__init__.py b/rteval/modules/measurement/__init__.py
> index 0e395be..5258dd7 100644
> --- a/rteval/modules/measurement/__init__.py
> +++ b/rteval/modules/measurement/__init__.py
> @@ -146,7 +146,15 @@ measurement profiles, based on their characteristics"""
>
> def SetupModuleOptions(self, parser):
> "Sets up all the measurement modules' parameters for the option parser"
> - self.__container.SetupModuleOptions(parser, self.__cfg)
> + grparser = self.__container.SetupModuleOptions(parser, self.__cfg)
> +
> + # Set up options specific for measurement module group
> + grparser.add_argument("--measurement-run-on-isolcpus",
> + dest="measurement___run_on_isolcpus",
> + action="store_true",
> + default=self.__cfg.GetSection("measurement").setdefault("run-on-isolcpus", "false").lower()
> + == "true",
> + help="Include isolated CPUs in default cpulist")
>
>
> def Setup(self, modparams):
> @@ -157,9 +165,10 @@ measurement profiles, based on their characteristics"""
>
> modcfg = self.__cfg.GetSection("measurement")
> cpulist = modcfg.cpulist
> + run_on_isolcpus = modcfg.run_on_isolcpus
>
> for (modname, modtype) in modcfg:
> - if modtype.lower() == 'module': # Only 'module' will be supported (ds)
> + if isinstance(modtype, str) and modtype.lower() == 'module': # Only 'module' will be supported (ds)
> # Extract the measurement modules info
> modinfo = self.__container.ModuleInfo(modname)
>
> @@ -179,6 +188,7 @@ measurement profiles, based on their characteristics"""
> # Setup this imported module inside the appropriate measurement profile
> self.__cfg.AppendConfig(modname, modparams)
> self.__cfg.AppendConfig(modname, {'cpulist':cpulist})
> + self.__cfg.AppendConfig(modname, {'run-on-isolcpus':run_on_isolcpus})
> mp.Setup(modname)
>
> del self.__container
> @@ -190,11 +200,12 @@ measurement profiles, based on their characteristics"""
> # Get the reports from all meaurement modules in all measurement profiles
> 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(cpulist).cpulist
> else:
> - cpulist = SysTop().default_cpus()
> + cpulist = SysTop().online_cpus() if run_on_isolcpus else SysTop().default_cpus()
> rep_n.newProp("measurecpus", collapse_cpulist(cpulist))
>
> for mp in self.__measureprofiles:
> --
> 2.41.0
>
>
Signed-off-by: John Kacur <jkacur@redhat.com>
next prev parent reply other threads:[~2023-09-15 19:03 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-11 9:52 [PATCH v2 0/3] rteval: Add cmdline option to run measurements on isolcpus by default Tomas Glozar
2023-08-11 9:52 ` [PATCH v2 1/3] rteval: Allow arguments specific to module group Tomas Glozar
2023-09-15 18:58 ` John Kacur
2023-08-11 9:52 ` [PATCH v2 2/3] rteval: Add run_on_isolcpus option to measurements Tomas Glozar
2023-09-15 19:01 ` John Kacur [this message]
2023-09-15 19:06 ` John Kacur
2023-08-11 9:52 ` [PATCH v2 3/3] rteval: Support run-on-isolcpus in cyclictest Tomas Glozar
2023-09-15 19:12 ` John Kacur
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=8e40bb39-a52b-9a58-23d3-0bfeb88abf@redhat.com \
--to=jkacur@redhat.com \
--cc=linux-rt-users@vger.kernel.org \
--cc=tglozar@redhat.com \
/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 NNTP newsgroup(s).