Skip to content

New dynamic - #6

Open
danielobiri wants to merge 2523 commits into
becsedi:developfrom
danielobiri:new_dynamic
Open

New dynamic#6
danielobiri wants to merge 2523 commits into
becsedi:developfrom
danielobiri:new_dynamic

Conversation

@danielobiri

Copy link
Copy Markdown

No description provided.

Open Source Botand others added 30 commits August 24, 2022 17:08
Summary:
GitHub commits:
facebook@e2e2e00facebook/fb303@bf47b75facebook/fbthrift@3b79333facebook/watchman@27791ccfacebookexperimental/rust-shed@c78af95
Reviewed By: zpao
fbshipit-source-id: 9926940a6375d6cdd7770b19673c9f80c86bd560
Summary:
GitHub commits:
facebook@8118866facebook/fb303@63e5773facebook/fbthrift@c127159facebook/watchman@dce5c01facebookexperimental/rust-shed@88af8c8
Reviewed By: zpao
fbshipit-source-id: 7ca490b118c72dc4642edcd294ed384ecb862e13
Summary:
GitHub commits:
facebook/fbthrift@e795837
Reviewed By: zpao
fbshipit-source-id: 5596f3b4d27c514c15c615f83d471a66a80844de
Summary:
GitHub commits:
facebook/fbthrift@1e2c611facebookarchive/profilo@befd6ae
Reviewed By: zpao
fbshipit-source-id: 1e6d60daa672ac5213510fe0d99f7ed071c85001
Summary:
GitHub commits:
facebook/fbthrift@a3fe0f1
Reviewed By: zpao
fbshipit-source-id: 207d5e36e99a8b686a444a38a88449327f717036
Summary:
GitHub commits:
facebook/fbthrift@35478e7facebook/litho@20d1c9e
Reviewed By: zpao
fbshipit-source-id: 4046d0887f0522916b588b2d0c4274a7710654dc
Summary:
GitHub commits:
facebook/folly@05b1dc5
Reviewed By: zpao
fbshipit-source-id: 5085b6b37522dac4142059011f3f4254653932fd
Summary:
AdRanker ASAN canary flagged a possible UBSan violation.
## Error
Failed Run: https://fburl.com/servicelab/apytosry
```
#0 0x562e3adb59bc in facebook::cachelib::objcache2::ObjectCacheSizeController<facebook::cachelib::CacheAllocator<facebook::cachelib::LruCacheTrait> >::work() buck-out/v2/gen/fbcode/47d914adeee3d982/cachelib/experimental/objcache2/__object-cache__/headers/cachelib/experimental/objcache2/ObjectCacheSizeController-inl.h
#1 0x562de7610f78 in facebook::cachelib::PeriodicWorker::loop() fbcode/cachelib/common/PeriodicWorker.cpp:55
becsedi#2 0x7f7632c524e4 in execute_native_thread_routine /home/engshare/third-party2/libgcc/11.x/src/gcc-11.x/x86_64-facebook-linux/libstdc++-v3/src/c++11/../../../.././libstdc++-v3/src/c++11/thread.cc:82:18
becsedi#3 0x7f7632f6ec0e in start_thread /home/engshare/third-party2/glibc/2.34/src/glibc-2.34/nptl/pthread_create.c:434:8
becsedi#4 0x7f76330011db in clone3 /home/engshare/third-party2/glibc/2.34/src/glibc-2.34/misc/../sysdeps/unix/sysv/linux/x86_64/clone3.S:81
UndefinedBehaviorSanitizer: integer-divide-by-zero buck-out/v2/gen/fbcode/47d914adeee3d982/cachelib/experimental/objcache2/__object-cache__/headers/cachelib/experimental/objcache2/ObjectCacheSizeController-inl.h:33:40 in
```
Reviewed By: jiayuebao
Differential Revision: D39024188
fbshipit-source-id: 64ad644c360565e638fa3ca74616a315038382ab
Sergei Vinogradovand others added 30 commits September 13, 2022 19:54
The issue was caused by incorrect behaviour of the
CacheAllocator<CacheTrait>::tryEvictToNextMemoryTier method in case the
evicted item is expired. We cannot simply return a handle to it, but we need
to remove it from the access container and MM container.
Fix ReaperSkippingSlabTraversalWhileSlabReleasing test
Fix moveRegularItemWithSync and add tests
Background data movement using periodic workers. Attempts to evict/promote items per given thresholds for each class. These reduce p99 latency since there is a higher chance that an allocation slot is free in the tier we are allocating in.
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.

9 participants

@danielobiri@jiayuebao@therealgymmy@jaesoo-fb@igchor@guptask@victoria-mcgrath@mcengija@byrnedj