Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

numTasks potential bug #329

Closed
wadudmiah opened this issue Feb 8, 2021 · 5 comments
Closed

numTasks potential bug #329

wadudmiah opened this issue Feb 8, 2021 · 5 comments
Labels

Comments

@wadudmiah
Copy link

In the IOR documentation, it says:

numTasks - number of tasks that should participate in the test [0] NOTE: **0 denotes all tasks**

But when I set numTasks to 0, I get the following runtime error:

ior ERROR: too few tasks for testing, errno 12, Cannot allocate memory (ior.c:1573)

The source has the following line that produces this error:

if (test->numTasks <= 0)
                ERR("too few tasks for testing");

If I'm not mistaken, isn't this a bug?

@JulianKunkel
Copy link
Collaborator

It seems this is a bug: Updating documentation isn't a strongsuit, yet but work in progress.

@wadudmiah
Copy link
Author

So I presume numTasks should be equal to or less than the number of MPI processes (as in mpirun -n <procs>)?

@JulianKunkel
Copy link
Collaborator

At the moment, yes. It seems sensible to set 0 to be as you state (this is the size in the MPI communicator)

@wadudmiah
Copy link
Author

Hi Julian, I didn't realise you were based in Reading. I'm just up the road in Didcot!

@JulianKunkel
Copy link
Collaborator

It appears that if you set it to -1, then it uses numTasks. I added PR #330 to fix it. Cool. Let me know if you have any other questions.

JulianKunkel added a commit that referenced this issue Feb 8, 2021
Bugfix user docu for ranks #329. Bugfix missmatching barriers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants