Boost C++ Libraries: Ticket #5266: mapped_region::flush improvement

status changed; resolution set

Ion Gaztañaga — Fri, 01 Apr 2011 17:25:26 GMT

status new → closed
resolution → wontfix

Sorry, that wouldn't be portable for other platforms, so I'm afraid you'll need to call msync yourself with your custom params.

status, version changed; resolution deleted

dan.eloff@… — Mon, 09 Jan 2012 19:33:45 GMT

status closed → reopened
version Boost 1.46.0 → Boost Development Trunk
resolution wontfix

How about an overload:

mapped_region::flush(std::size_t mapping_offset, std::size_t numbytes, bool sync)

which uses msycn(MS_SYNC) on posix systems and FlushViewOfFile + FlushFileBuffers on windows

If there are platforms where it cannot be supported, flush with sync=true should throw an exception.

This removes the burden of writing a multiple platform flush(..., sync=true) from the user, which this library does a great job of for other memory mapped file operations. That should cover all the common use cases.

status changed; resolution set

Ion Gaztañaga — Thu, 10 Jan 2013 19:25:51 GMT

status reopened → closed
resolution → fixed

Implemented in Boost 1.51:

bool flush(std::size_t = 0, std::size_t = 0, bool = true);