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

Fix FreeBSD 11 #2802

Merged
merged 2 commits into from Aug 6, 2017
Merged

Fix FreeBSD 11 #2802

merged 2 commits into from Aug 6, 2017

Conversation

zao
Copy link
Contributor

@zao zao commented Aug 5, 2017

This pull request introduces a workaround for an unavailable environ in shared libraries by initializing a freebsd_environ in HPX startup for use in the runtime system.
We also remove MAP_NORESERVE from mmap code for FreeBSD, as the parameter previously was unimplemented and now is completely removed from FreeBSD 11.0.

zao added 2 commits August 5, 2017 21:42
FreeBSD libc.so.7 doesn't define `environ` when building a shared
library. Due to disallowing undefined symbols with `-z,defs` this
prevents `libhpx.so` from building.

We introduce a new variable `freebsd_environ` that gets initialized
during HPX startup which is used by any part of the system that
previously used the `environ` variable directly.
FreeBSD 11.0 removes the `MAP_NORESERVE` parameter to mmap. It has
previously not been implemented in the OS and as such, removing the flag
has no side effect.
Copy link
Member

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks a lot!

@hkaiser hkaiser merged commit d5639e0 into STEllAR-GROUP:master Aug 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants