From: Andi Kleen <andi@firstfloor.org>
To: cpw@sgi.com, linux-numa@vger.kernel.org
Subject: [PATCH] Fix test/prefered
Date: Sat, 10 Jan 2009 06:54:08 +0100 [thread overview]
Message-ID: <20090110055408.GC26290@one.firstfloor.org> (raw)
Fix test/prefered
Couple of bugs here:
- First the kernel errors out of the nodemask is < MAXNUMNODES,
so have to discover that at runtime. There's no function
in libnuma for this (perhaps there should be one?), so i
wrote one here.
- The loop set the node in the wrong mask, breaking the test
Signed-off-by: Andi Kleen <ak@linux.intel.com>
diff -urp numactl-2.0.2/test/prefered.c numactl-2.0.2-hack//test/prefered.c
--- numactl-2.0.2/test/prefered.c 2008-08-05 16:36:58.000000000 +0200
+++ numactl-2.0.2-hack//test/prefered.c 2009-01-10 06:30:11.000000000 +0100
@@ -6,20 +6,38 @@
#include <assert.h>
#include <unistd.h>
#include <stdlib.h>
+#include <errno.h>
#define err(x) perror(x),exit(1)
+/* Discover MAX_NUMNODE of the kernel */
+int max_numnode(void)
+{
+ int v;
+ unsigned size = 128;
+ struct bitmask *mask = numa_bitmask_alloc(size);
+ while (get_mempolicy(NULL, mask->maskp, mask->size, &v, MPOL_F_ADDR) < 0 &&
+ errno == EINVAL) {
+ numa_bitmask_free(mask);
+ size <<= 1;
+ }
+ numa_bitmask_free(mask);
+ printf("size %d\n", size);
+ return size;
+}
+
int main(void)
{
int max = numa_max_node();
+ int maxmask = max_numnode();
struct bitmask *nodes, *mask;
int pagesize = getpagesize();
int i;
int pol;
int node;
int err = 0;
- nodes = numa_bitmask_alloc(max+1);
- mask = numa_bitmask_alloc(max+1);
+ nodes = numa_bitmask_alloc(maxmask);
+ mask = numa_bitmask_alloc(maxmask);
for (i = max; i >= 0; --i) {
char *mem = mmap(NULL, pagesize*(max+1), PROT_READ|PROT_WRITE,
@@ -33,9 +51,9 @@ int main(void)
numa_bitmask_clearall(nodes);
numa_bitmask_clearall(mask);
- numa_bitmask_setbit(mask, i);
+ numa_bitmask_setbit(nodes, i);
if (mbind(adr, pagesize, MPOL_PREFERRED, nodes->maskp,
nodes->size, 0) < 0)
err("mbind");
--
ak@linux.intel.com
next reply other threads:[~2009-01-10 5:54 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-10 5:54 Andi Kleen [this message]
2009-01-10 6:18 ` [PATCH] Fix test/prefered v2 Andi Kleen
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=20090110055408.GC26290@one.firstfloor.org \
--to=andi@firstfloor.org \
--cc=cpw@sgi.com \
--cc=linux-numa@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 NNTP newsgroup(s).