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

From: John Stultz
Date: Mon Aug 02 2021 - 22:18:26 EST


On Thu, Jul 22, 2021 at 12:07 PM Hridya Valsaraju <hridya@xxxxxxxxxx> 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@xxxxxxxxxx>

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

Reviewed-by: John Stultz <john.stultz@xxxxxxxxxx>

thanks
-john