Skip to content

Commit

Permalink
Correct calculation of nr_nodes and re-enable move_pages test
Browse files Browse the repository at this point in the history
This was pointed out by @bjsprakash in #8.

After the bug is corrected, we can re-enable the test in `make check`,
since most machines these days will have at least two nodes by default.

Travis-CI still fails with this test (one node only available), so keep
skipping it there.
  • Loading branch information
filbranden committed Jul 3, 2018
1 parent 09d294e commit d1bc165
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions Makefile.am
Expand Up @@ -134,6 +134,7 @@ TESTS = \
test/checkaffinity \
test/checktopology \
test/distance \
test/move_pages \
test/nodemap \
test/numademo \
test/regress \
Expand Down
2 changes: 1 addition & 1 deletion test/move_pages.c
Expand Up @@ -28,7 +28,7 @@ int main(int argc, char **argv)

pagesize = getpagesize();

nr_nodes = numa_max_node();
nr_nodes = numa_max_node() + 1;

if (nr_nodes < 2) {
printf("A minimum of 2 nodes is required for this test.\n");
Expand Down

0 comments on commit d1bc165

Please sign in to comment.