[lkml]   [2012]   [Jan]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRE: [PATCHv19 00/15] Contiguous Memory Allocator

On Thursday, January 26, 2012 4:32 PM Arnd Bergmann wrote:

> On Thursday 26 January 2012, Marek Szyprowski wrote:
> > Welcome everyone!
> >
> > Yes, that's true. This is yet another release of the Contiguous Memory
> > Allocator patches. This version mainly includes code cleanups requested
> > by Mel Gorman and a few minor bug fixes.
> Hi Marek,
> Thanks for keeping up this work! I really hope it works out for the
> next merge window.
> > TODO (optional):
> > - implement support for contiguous memory areas placed in HIGHMEM zone
> > - resolve issue with movable pages with pending io operations
> Can you clarify these? I believe the contiguous memory areas in highmem
> is something that should really be after the existing code is merged
> into the upstream kernel and should better not be listed as TODO here.

Ok, I will remove it from the TODO list. Core memory management is very
little dependence on HIGHMEM, it is more about DMA-mapping framework to
be aware that there might be no lowmem mappings for the allocated pages.
This can be easily added once the initial version got merged.

> I haven't followed the last two releases so closely. It seems that
> in v17 the movable pages with pending i/o was still a major problem
> but in v18 you added a solution. Is that right? What is still left
> to be done here then?

Since v18 the failed allocation is retried in a bit different place in
the contiguous memory area what heavily increased overall reliability.

This can be improved by making cma a bit more aware about pending io
operations, but I want to leave this after the initial merge.

I think that there are no major issues left to be resolved now.

Best regards
Marek Szyprowski
Samsung Poland R&D Center

 \ /
  Last update: 2012-01-26 16:51    [W:0.227 / U:23.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site