Messages in this thread | | | From | Hridya Valsaraju <> | Date | Tue, 3 Aug 2021 14:39:19 -0700 | Subject | Re: [PATCH] dma-buf: heaps: Set allocation limit for system heap |
| |
On Mon, Aug 2, 2021 at 7:18 PM John Stultz <john.stultz@linaro.org> wrote: > > 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>
Thank you for taking a look John!
Regards, Hridya
> > thanks > -john
| |