[all-commits] [llvm/llvm-project] 990acd: [scudo][standalone] Reduce Android region sizes to...

Kostya Kortchinsky via All-commits all-commits at lists.llvm.org
Wed Jan 22 09:42:44 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 990acd450cf198cc6a82a28e011fa602e44512ad
      https://github.com/llvm/llvm-project/commit/990acd450cf198cc6a82a28e011fa602e44512ad
  Author: Kostya Kortchinsky <kostyak at google.com>
  Date:   2020-01-22 (Wed, 22 Jan 2020)

  Changed paths:
    M compiler-rt/lib/scudo/standalone/allocator_config.h

  Log Message:
  -----------
  [scudo][standalone] Reduce Android region sizes to 128MB

Summary:
Unity is making irresponsible assumptions as to how clumped up memory
should be. With larger regions, we break those, resulting in errors
like:

"Using memoryadresses from more that 16GB of memory"

This is unfortunately one of those situations where we have to bend to
existing code because we doubt it's going to change any time soon.

128MB should be enough, but we could be flirting with OOMs in the
higher class sizes.

Reviewers: cferris, eugenis, hctim, morehouse, pcc

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

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




More information about the All-commits mailing list