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

Reimplement support for flags to mmap (nosync nocore) #38

Open
bovine opened this issue Jul 9, 2013 · 3 comments
Open

Reimplement support for flags to mmap (nosync nocore) #38

bovine opened this issue Jul 9, 2013 · 3 comments
Assignees

Comments

@bovine
Copy link
Member

bovine commented Jul 9, 2013

During the Boost reimplementation in the cpp branch, support for the "flags" argument to mmap() had to be removed because Boost did not support overriding it at the time.

However, Boost Version 1.54.0 now supports platform-specific flags to mapped_region so we should re-add support for it.

https://svn.boost.org/trac/boost/ticket/8030
http://boost.2283326.n4.nabble.com/interprocess-Platform-specific-flags-for-mapped-region-td4642667.html
http://www.boost.org/users/history/version_1_54_0.html

@ghost ghost assigned bovine Jul 9, 2013
@bovine
Copy link
Member Author

bovine commented Jul 16, 2013

currently waiting for FreeBSD ports to be updated to boost 1.54 first. I've contacted the ports maintainer and he said: "I'm exp-running to make sure it doesn't break a port depending one it. Might need a couple of more weeks to finish it."

@gahr
Copy link
Collaborator

gahr commented Jul 9, 2014

Just in case this was forgotten, we now ship boost 1.55.0 :)

@bovine
Copy link
Member Author

bovine commented Aug 4, 2014

To specify the "options" argument of mapped_region to managed_mapped_file, we need to do this:

Hi Jeff,

You need to pass those options from several classes and the interface will become very ugly (too many arguments) say:

basic_managed_mapped_file's constructor -> managed_open_or_create_impl's constructor ->
managed_open_or_create_impl::priv_open_or_create

and in this function, pass it to the newly created mapped_region in the map_options_t argument.

Adding this and other options would require a modification to the interface of managed classes and I haven't had time to try to design a cleaner interface to pass multiple options.

Hope this helps,

Ion

@bovine bovine removed this from the cpp milestone Jul 8, 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

No branches or pull requests

2 participants