Skip to content

[Upstreaming] Added file-based SHM segment option in SHM module - #74

Closed
guptask wants to merge 2254 commits into
pmem:mainfrom
guptask:shm_segments
Closed

[Upstreaming] Added file-based SHM segment option in SHM module#74
guptask wants to merge 2254 commits into
pmem:mainfrom
guptask:shm_segments

Conversation

@guptask

@guptaskguptask commented Jun 2, 2022

Copy link
Copy Markdown
Collaborator

Added file-based SHM segment option in SHM module and CacheAllocator. This is first half of the changes needed to enable multi-tiering in CacheLib. The multi-tiering config APIs are in a separate PR (facebook#138). These config APIs are needed to enable multi-tiering. Once that PR is merged, the second half of the multi-tier changes can be upstreamed via a separate PR.


This change is Reviewable

@guptaskguptask self-assigned this Jun 2, 2022
@guptask
guptaskforce-pushed the shm_segments branch 10 times, most recently from 0f84af5 to 2eb9eacCompareJune 9, 2022 00:17
@guptask
guptaskforce-pushed the shm_segments branch 11 times, most recently from e3c00af to 4275bafCompareJune 13, 2022 09:20
svcscm svcscmand others added 8 commits June 13, 2022 03:11
Summary:
GitHub commits:
facebook/fbthrift@80469bdfacebook/litho@09210b4facebook/watchman@6c29e8afacebookincubator/velox@fb7b62b
Reviewed By: jurajh-fb
fbshipit-source-id: 234fd86e822cef60781048bce907b9dae25187a0
Summary: Clean up `ItemHandle` in CacheAllocator
Reviewed By: therealgymmy
Differential Revision: D34334392
fbshipit-source-id: 0ba983dc0e4f2614b8074ad70585f91d1af3e5e9
Summary:
GitHub commits:
facebook@c8dc407facebook/fbthrift@8900b1e
Reviewed By: jurajh-fb
fbshipit-source-id: f470e19dcba367551a58d23a5588a3bfa0e3ebfb
svcscm svcscmand others added 26 commits June 13, 2022 03:11
Summary:
GitHub commits:
facebook/folly@847a391
Reviewed By: wittgenst
fbshipit-source-id: 6fb2934c88a00aa3a346543952b21660cf019959
Summary:
GitHub commits:
facebook/fbthrift@9a8e286
Reviewed By: wittgenst
fbshipit-source-id: 548ede7f73eef04044608c8c4ff08cde6ac0b1c6
Summary:
GitHub commits:
facebook/folly@1d196d6
Reviewed By: wittgenst
fbshipit-source-id: 85b66175cbcac8d719708d4cc92533a96408ce84
Summary:
GitHub commits:
facebook/fbthrift@85148bc
Reviewed By: wittgenst
fbshipit-source-id: d5e0a075a19357ddcc2f8cee208e257c9c5f4b4e
It's implementation is mostly based on PosixShmSegment.
Also, extend ShmManager and ShmSegmentOpts to support this new
segment type.
After introducing file segment type, nameToKey_ does not provide
enough information to recover/remove segments on restart.
This commit fixes that by replacing nameToKey_ with nameToOpts_.
Previously, the Key from nameToKey_ map was only used in a single
DCHECK().
It wrongly assumed that the only possible segment type is
PosixSysV segment.
@guptaskguptask closed this Jun 13, 2022
@guptask
guptask deleted the shm_segments branch June 13, 2022 10:29
guptask added a commit to guptask/CacheLib that referenced this pull request Mar 30, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@guptask@jiayuebao@luciang@therealgymmy@hslee2008@igchor