[compiler-rt] [scudo] Enable "Delayed release to OS" feature for Android (PR #65942)

Christopher Ferris via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 26 11:17:15 PDT 2023


cferris1000 wrote:

Before you go through with this, what is the benefit to this change? If there is nothing on the system that would benefit from this, I don't think this is a good idea. The problem is that it's very difficult to tell what the RSS increase will be. We can measure some of it, but because this is memory coming from the secondary, it can be a large increase in RSS from a single process. For example, it could be as bad a > 10MB for a single process.

I do have a proposal specific to Android that sets all adb shell spawned processes to set the decay timer to the max. I also have a way to make sure that any command-line tools spawned from an app also get the same decay timer setting.  I think this would be a better choice for this solution. I've already got the code that implements this and have tested it and it appears to work. I should have CLs up soon once I convince myself this is not going to be a problem.

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


More information about the llvm-commits mailing list