Skip to content

Conversation

@shamisp
Copy link
Contributor

@shamisp shamisp commented May 24, 2016

The OPAL_CMA_NEED_SYSCALL_DEFS is always defined/set to 0 or 1. Therefore
instead of checking if the macro is defined, we have to look at the value
itself.

Signed-off-by: Pavel Shamis (Pasha) pasharesearch@gmail.com

The OPAL_CMA_NEED_SYSCALL_DEFS is always defined/set to 0 or 1.  Therefore
instead of checking if the macro is defined, we have to look at the value
itself.

Signed-off-by: Pavel Shamis (Pasha) <pasharesearch@gmail.com>
@shamisp
Copy link
Contributor Author

shamisp commented May 24, 2016

FYI @hjelmn

@hjelmn
Copy link
Member

hjelmn commented May 24, 2016

@shamisp Good catch. btl/sm is on its way out but I will merge this once jenkins finishes.

@jsquyres
Copy link
Member

@shamisp Has ARM updated its schedule A form to include your name?

@shamisp
Copy link
Contributor Author

shamisp commented May 24, 2016

@jsquyres - I don't think so ? Is it a blocker ? (I can start the process)

@jsquyres
Copy link
Member

We do try to hold off on unauthorized commits until the paperwork has been emailed to IU.

Updating the schedule A is trivial; it doesn't require a lawyer's signature.

@shamisp
Copy link
Contributor Author

shamisp commented May 31, 2016

@jsquyres - the CLA schedule A was updated. You should have the copy. 10x

@jsquyres
Copy link
Member

Excellent; thank you!

@shamisp
Copy link
Contributor Author

shamisp commented May 31, 2016

@hjelmn - are we good to go ?

@hjelmn hjelmn merged commit 5d4bcce into open-mpi:master Jun 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants