[compiler-rt] [scudo] Update secondary cache released pages bound. (PR #106466)

via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 16:24:50 PDT 2024


================
@@ -389,17 +388,14 @@ class MapAllocatorCache {
         const uptr HeaderPos = AllocPos - HeadersSize;
         const uptr MaxAllowedFragmentedBytes =
             MaxAllowedFragmentedPages * PageSize;
-        if (HeaderPos > CommitBase + CommitSize)
+        if (HeaderPos > CommitBase + CommitSize || HeaderPos < CommitBase)
           continue;
----------------
ChiaHungDuan wrote:

Let's only do the change of `MaxReleasedCachePages` in this pull request. The change of `AllocPos` to `roundDown(HeaderPos, PageSize)` still introduce 1 page difference. It's better to do it separately

https://github.com/llvm/llvm-project/pull/106466


More information about the llvm-commits mailing list