Re: Better shared data structure management and resizable shared data structures
Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>
On Wed, Apr 8, 2026 at 1:39 AM Andres Freund <andres@anarazel.de> wrote: > > Hi, > > On 2026-04-07 22:48:17 +0300, Heikki Linnakangas wrote: > > > +/* > > > + * ShmemResizeStruct() --- resize a resizable shared memory structure. > > > + * > > > + * The new size must be within [minimum_size, maximum_size]. If the structure > > > + * is being shrunk, the memory pages that are no longer needed are freed. If > > > + * the structure is being expanded, the memory pages that are needed for the > > > + * new size are allocated. See EstimateAllocatedSize() for explanation of which > > > + * pages are allocated for a resizable structure. > > > + */ > > > +void > > > +ShmemResizeStruct(const char *name, Size new_size) > > > > This interface only allows shrinking and growing the allocated region at the > > end, but the underlying mechanism is madvise(MADV_REMOVE) and > > madvise(MADV_WRITE_POPULATE), which supports also "punching holes", i.e. > > freeing memory in the middle of a region. Do we gain anything by restricting > > ourselves to changing the size at the end? It seems to me that it could be > > handy to punch holes for some use cases. > > Agreed. The hard part may be the "communication" with the user about how > granular the punches can be. Because that will depend on things like > huge_pages, huge_page_size and may depend on what alignment you happened to > get. > We can extend it that way if there is a valid usecase. For now I kept it simple for two reasons: 1. Buffer manager structures shrink and expand only at the end right now. Longer note on buffer lookup table later. This effort started with buffer resizing and didn't want to expand scope more than what's needed. 2. Not all the approaches we tried to implement resizable shared memory have the facility to free memory in the middle. Usually they have a facility to shrink or expand at the end. If we offer ability to free memory in the middle based on facilities on one platform, we will face big hurdles when supporting other platforms. I think it's better to avoid it when it's not needed. Buffer lookup table is fixed. It may benefit from punching holes in the middle if we can somehow get pages worth of free entries together somewhere in the middle. First it's not easy to perform such compaction. But even if implement compaction, we can collect those entries at the end instead of in the middle; the current API will still be useful. Is there any other usecase you are envisioning? I also think that it will be better to introduce a new ShmemFreeStructPart()/ShmemAllocStructPart() instead of the current ShmemResizeStruct(). > > > What's the portability story? I understand that this is Linux-only at the > > moment, but what platforms can we support in the future, and what's the > > effort? I think BSD's have similar capabilities with plain mmap() and > > MADV_FREE if I read the man pages right. > > At least linux' MADV_FREE is only for private mappings. It's not clear in at > least freebsd's man page, but the described use case makes me suspect it may > be similar there. > looks so. FreeBSD also has fallocate with PUNCH_HOLES. We could use it with fd created using memfd_create() on .and it will need memfd_create(). I haven't checked whether that works. > > > What about macOS and Windows? This doesn't necessarily need to be fully > > portable, if some OS's don't have the capabilities we need, but would be > > nice to know what's possible. > > Looks like windows has OfferVirtualMemory > https://learn.microsoft.com/en-us/windows/win32/api/memoryapi/nf-memoryapi-offervirtualmemory > but it's not clear to me if it actually does what we need when multiple > processes are attached. > Those APIs look similar to madvise+ MADV_REMOVE/MADV_WRITE_POPULATE, with specific and cleaner interface. At least worth a try. > I suspect it's going to be a lot easier once we're threaded... The reason I > am ok with doing resizing this way before threading is because it's > architecturally pretty similar to what you'd want to do once threaded, so it's > not a huge dead end. But I'm doubtful we'll find facilities that allow this > across processes in all operating systems... check -- Best Wishes, Ashutosh Bapat
Commits
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Tidy up #ifdef USE_INJECTION_POINTS guards
- 9480c585df6c 19 (unreleased) landed
-
Convert all remaining subsystems to use the new shmem allocation API
- 9b5acad3f40f 19 (unreleased) landed
-
Convert buffer manager to use the new shmem allocation functions
- a4b6139dcceb 19 (unreleased) landed
-
Add alignment option to ShmemRequestStruct()
- dacfe81a0de5 19 (unreleased) landed
-
Convert AIO to use the new shmem allocation functions
- 58a1573385ed 19 (unreleased) landed
-
Convert SLRUs to use the new shmem allocation functions
- 2e0943a8597e 19 (unreleased) landed
-
Refactor shmem initialization code in predicate.c
- 4c9eca5afea0 19 (unreleased) landed
-
Use the new shmem allocation functions in a few core subsystems
- c6d55714ba4c 19 (unreleased) landed
-
Convert lwlock.c to use the new shmem allocation functions
- a006bc7b1699 19 (unreleased) landed
-
Introduce a registry of built-in shmem subsystems
- 1fc2e9fbc0a3 19 (unreleased) landed
-
Convert pg_stat_statements to use the new shmem allocation functions
- d4885af3d653 19 (unreleased) landed
-
Add a test module to test after-startup shmem allocations
- 6409994c7dd8 19 (unreleased) landed
-
Introduce a new mechanism for registering shared memory areas
- 283e823f9dcb 19 (unreleased) landed
-
Move some code from shmem.c and shmem.h
- 6ef9bee29310 19 (unreleased) landed
-
Improve test_lwlock_tranches
- 92a685e4070d 19 (unreleased) landed
-
Test pg_stat_statements across crash restart
- 148fe2b05df5 19 (unreleased) landed
-
Refactor PredicateLockShmemInit to not reuse var for different things
- 3fd057772827 19 (unreleased) landed
-
Refactor ShmemIndex initialization
- 6b8238cb6aa7 19 (unreleased) landed
-
Add a new shmem_request_hook hook.
- 4f2400cb3f10 15.0 cited