[all-commits] [llvm/llvm-project] 654f5d: [scudo][standalone] Release secondary memory on purge

Kostya Kortchinsky via All-commits all-commits at lists.llvm.org
Thu Jan 30 13:23:14 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 654f5d684561d784fa8927d3de31b7dab3f28087
      https://github.com/llvm/llvm-project/commit/654f5d684561d784fa8927d3de31b7dab3f28087
  Author: Kostya Kortchinsky <kostyak at google.com>
  Date:   2020-01-30 (Thu, 30 Jan 2020)

  Changed paths:
    M compiler-rt/lib/scudo/standalone/combined.h
    M compiler-rt/lib/scudo/standalone/primary32.h
    M compiler-rt/lib/scudo/standalone/primary64.h
    M compiler-rt/lib/scudo/standalone/secondary.h

  Log Message:
  -----------
  [scudo][standalone] Release secondary memory on purge

Summary:
The Secondary's cache needs to be released when the Combined's
`releaseToOS` function is called (via `M_PURGE`) for example,
which this CL adds.

Additionally, if doing a forced release, we'll release the
transfer batch class as well since now we can do that.

There is a couple of other house keeping changes as well:
- read the page size only once in the Secondary Cache `store`
- remove the interval check for `CanRelease`: we are going to
  make that configurable via `mallopt` so this needs not be
  set in stone there.

Reviewers: cferris, hctim, pcc, eugenis

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D73730




More information about the All-commits mailing list