Opened 14 years ago
Closed 11 years ago
#2696 closed Bugs (fixed)
max_size for boost pool
Reported by: | Nikolay Mladenov | Owned by: | Chris Newbold |
---|---|---|---|
Milestone: | Boost 1.38.0 | Component: | pool |
Version: | Boost 1.37.0 | Severity: | Problem |
Keywords: | Cc: | boostpool@… |
Description
The malloc_needs_resize, ordered_malloc_needs_resize and ordered_malloc members of the pool class double the size of the chunk requested from the operating system every time they cannot satisfy request.
Consider a scenario in which the client allocates about 900Mb before releases the initial size of the pool is 128Mb. on the 1st request from the client the pool will request 128 MB from the OS when the client uses 128M -> 256MB will be requested from the os when the client uses 384M -> 512MB will be requested from the os when the client uses 896M -> 1GB will be requested from the os (which is already likely to fail)
when the client reaches 900M the pool will have close to 2Gb memory allocated from the OS.
But the client will be perfectly happy if the pool just allocates 128M from the OS when it is exhausted, provided 128Mb is large enough to satisfy any single request from the client.
The supplied patch allow for a cap on the chunks the pool is allocating from the OS. This is done by adding a default last parameter max_size which when left at the default of 0 leads to the uncapped behavior.
Attachments (2)
Change History (10)
by , 14 years ago
Attachment: | pool.patch added |
---|
comment:1 by , 14 years ago
Component: | None → pool |
---|---|
Owner: | set to |
by , 12 years ago
Attachment: | boost-pool-maxsize-patch-issue2696.diff added |
---|
comment:2 by , 12 years ago
Cc: | added |
---|
Note to all:
Marshall Clow plans to apply this to Trunk (and then release) to make sure it gets into 1.46. We still need a test case for this though...
comment:3 by , 12 years ago
Cc: | added; removed |
---|
comment:5 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:6 by , 12 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
We should update the docs and tests for this issue - reopened for now so we don't forget :)
Update to original patch to apply to r66834 -- tested locally without failures, GCC 4.4 Linux.