Re: [Linaro-mm-sig] [PATCH] dma-buf: heaps: Set allocation limit for system heap

On Thu, Jul 22, 2021 at 12:07 PM Hridya Valsaraju hridya@google.com wrote:

This patch limits the size of total memory that can be requested in a single allocation from the system heap. This would prevent a buggy/malicious client from depleting system memory by requesting for an extremely large allocation which might destabilize the system.

The limit is set to half the size of the device's total RAM which is the same as what was set by the deprecated ION system heap.

Signed-off-by: Hridya Valsaraju hridya@google.com

Seems sane to me, unless folks have better suggestions for allocation limits.

Reviewed-by: John Stultz john.stultz@linaro.org

thanks -john