[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20100820121124Z.fujita.tomonori@lab.ntt.co.jp>
Date: Fri, 20 Aug 2010 12:12:50 +0900
From: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
To: m.nazarewicz@...sung.com
Cc: kyungmin.park@...sung.com, fujita.tomonori@....ntt.co.jp,
linux-mm@...ck.org, dwalker@...eaurora.org, linux@....linux.org.uk,
corbet@....net, p.osciak@...sung.com,
broonie@...nsource.wolfsonmicro.com, linux-kernel@...r.kernel.org,
hvaibhav@...com, hverkuil@...all.nl, kgene.kim@...sung.com,
zpfeffer@...eaurora.org, jaeryul.oh@...sung.com,
linux-media@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
m.szyprowski@...sung.com
Subject: Re: [PATCH/RFCv3 0/6] The Contiguous Memory Allocator framework
> >> We hope this method included at mainline kernel if possible.
> >> It's really needed feature for our multimedia frameworks.
> >
> > You got any comments from mm people?
> >
> > Virtually, this adds a new memory allocator implementation that steals
> > some memory from memory allocator during boot process. Its API looks
> > completely different from the API for memory allocator. That doesn't
> > sound appealing to me much. This stuff couldn't be integrated well
> > into memory allocator?
>
> What kind of integration do you mean? I see three levels:
>
> 1. Integration on API level meaning that some kind of existing API is used
> instead of new cma_*() calls. CMA adds notion of devices and memory
> types which is new to all the other APIs (coherent has notion of devices
> but that's not enough). This basically means that no existing API can be
> used for CMA. On the other hand, removing notion of devices and memory
> types would defeat the whole purpose of CMA thus destroying the solution
> that CMA provides.
You can create something similar to the existing API for memory
allocator.
For example, blk_kmalloc/blk_alloc_pages was proposed as memory
allocator API with notion of an address range for allocated memory. It
wasn't merged for other reasons though.
I don't mean that this is necessary for the inclusion (I'm not the
person to ack or nack this). I just expect the similarity of memory
allocator API.
> 2. Reuse of memory pools meaning that memory reserved by CMA can then be
> used by other allocation mechanisms. This is of course possible. For
> instance coherent could easily be implemented as a wrapper to CMA.
> This is doable and can be done in the future after CMA gets more
> recognition.
>
> 3. Reuse of algorithms meaning that allocation algorithms used by other
> allocators will be used with CMA regions. This is doable as well and
> can be done in the future.
Well, why can't we do the above before the inclusion?
Anyway, I think that comments from mm people would be helpful to merge
this.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists