Re: [PATCH v3] dma-buf/heaps: system_heap: avoid too much allocation

From: Jaewon Kim <jaewon31.kim@gmail.com>
To: "T.J. Mercier" <tjmercier@google.com>
Cc: Jaewon Kim <jaewon31.kim@samsung.com>,
	Michal Hocko <mhocko@suse.com>,
	jstultz@google.com,  Sumit Semwal <sumit.semwal@linaro.org>,
	daniel.vetter@ffwll.ch,
	 Andrew Morton <akpm@linux-foundation.org>,
	hannes@cmpxchg.org, linux-mm@kvack.org,
	 linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] dma-buf/heaps: system_heap: avoid too much allocation
Date: Thu, 13 Apr 2023 07:10:49 +0900	[thread overview]
Message-ID: <CAJrd-Uuj7b7JhGK+3HW5aEc9HiTs0JMKGiQJuJ_Z6Z8za0j4Cw@mail.gmail.com> (raw)
In-Reply-To: <CABdmKX2fA2nXaSb8k+LE1yeso=ZnboDtxhzmjzrS35GSKv73hQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 987 bytes --]

It was one of camera scenarios. I internally asked and heard that was not a
bug but normal. I think 2GB looks too big for one graphics buffer but it
could be for other purposes like camera. I think the system heap should
support that.

2023년 4월 13일 (목) 오전 1:49, T.J. Mercier <tjmercier@google.com>님이 작성:

> On Wed, Apr 12, 2023 at 4:38 AM Jaewon Kim <jaewon31.kim@samsung.com>
> wrote:
>
> > Yes I think you're right. As a allocator, dma-buf system heap looks to
> be loose
> > in memory allocation. Limiting dmabuf memory may be required. But I
> think there
> > is no nice and reasonable way so far. And the dma-buf system heap is
> being
> > widely used in Android mobile system. AFAIK the camera consumes huge
> memory
> > through this dma-buf system heap. I actually even looked a huge size
> request
> > over 2GB in one dma-buf request.
> >
> Hey can you point me to where you saw a request that big? That's a
> non-buggy request?!
>

[-- Attachment #2: Type: text/html, Size: 1402 bytes --]