linux-rt-users.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Wagner <dwagner@suse.de>
To: Clark Williams <williams@redhat.com>, John Kacur <jkacur@redhat.com>
Cc: linux-rt-users@vger.kernel.org, Daniel Wagner <dwagner@suse.de>
Subject: [rt-tests v3 08/16] rt-numa: Use mask size for iterator limit
Date: Tue, 26 Jan 2021 10:14:52 +0100	[thread overview]
Message-ID: <20210126091500.31735-9-dwagner@suse.de> (raw)
In-Reply-To: <20210126091500.31735-1-dwagner@suse.de>

The bitmask structure knows its size use, thus use it as upper limit
in the loop.

Signed-off-by: Daniel Wagner <dwagner@suse.de>
---
 src/lib/rt-numa.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/rt-numa.c b/src/lib/rt-numa.c
index 7e99eab60681..b1bbb8c8052c 100644
--- a/src/lib/rt-numa.c
+++ b/src/lib/rt-numa.c
@@ -33,7 +33,7 @@ static int cpu_for_thread_sp(int thread_num, int max_cpus, struct bitmask *cpuma
 	m = thread_num % num_cpus;
 
 	/* there are num_cpus bits set, we want position of m'th one */
-	for (i = 0, cpu = 0; i < max_cpus; i++) {
+	for (i = 0, cpu = 0; i < cpumask->size; i++) {
 		if (numa_bitmask_isbitset(cpumask, i)) {
 			if (cpu == m)
 				return i;
@@ -97,7 +97,7 @@ static void use_current_cpuset(int max_cpus, struct bitmask *cpumask)
 	 * Clear bits that are not set in both the cpuset from the
 	 * environment, and in the user specified affinity.
 	 */
-	for (i = 0; i < max_cpus; i++) {
+	for (i = 0; i < cpumask->size; i++) {
 		if ((!numa_bitmask_isbitset(cpumask, i)) ||
 		    (!numa_bitmask_isbitset(curmask, i)))
 			numa_bitmask_clearbit(cpumask, i);
-- 
2.30.0


  parent reply	other threads:[~2021-01-26 18:42 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-26  9:14 [rt-tests v3 00/16] rt-numa.h cleanups Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 01/16] cyclictest: Use numa API directly Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 02/16] cyclictest: Mimik --smp behavior with --affinity Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 03/16] cyclictest: Simplify --smp vs --affinity vs --threads argument logic Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 04/16] signaltest: Use affinity_mask for steering thread placement Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 05/16] signaltest: Simplify --smp vs --affinity vs --threads argument logic Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 06/16] rt-numa: Remove unused definitions and numa_initialize() Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 07/16] rt-numa: Add generic cpu_for_thread() helper Daniel Wagner
2021-01-26  9:14 ` Daniel Wagner [this message]
2021-01-26  9:14 ` [rt-tests v3 09/16] rt-numa: Remove max_cpus argument from parse_cpusmask Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 10/16] signaltest: Remove unused max_cpus argument from process_options Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 11/16] cyclictest: " Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 12/16] rt-numa: Use CPU_SETSIZE as upper loop limit Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 13/16] rt-numa: Remove used max_cpus argument from cpu_for_thread() Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 14/16] cyclictest: Remove max cpus used verbose information Daniel Wagner
2021-01-26  9:14 ` [rt-tests v3 15/16] cyclictest: Remove unecessary local variable Daniel Wagner
2021-01-26  9:15 ` [rt-tests v3 16/16] rt-tests: Rename error.h to rt-error.h Daniel Wagner

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=20210126091500.31735-9-dwagner@suse.de \
    --to=dwagner@suse.de \
    --cc=jkacur@redhat.com \
    --cc=linux-rt-users@vger.kernel.org \
    --cc=williams@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).