On (15/09/07 14:14), Goswin von Brederlow didst pronounce: > Andrew Morton <[EMAIL PROTECTED]> writes: > > > On Tue, 11 Sep 2007 14:12:26 +0200 Jörn Engel <[EMAIL PROTECTED]> wrote: > > > >> While I agree with your concern, those numbers are quite silly. The > >> chances of 99.8% of pages being free and the remaining 0.2% being > >> perfectly spread across all 2MB large_pages are lower than those of SHA1 > >> creating a collision. > > > > Actually it'd be pretty easy to craft an application which allocates seven > > pages for pagecache, then one for <something>, then seven for pagecache, > > then > > one for <something>, etc. > > > > I've had test apps which do that sort of thing accidentally. The result > > wasn't pretty. > > Except that the applications 7 pages are movable and the <something> > would have to be unmovable. And then they should not share the same > memory region. At least they should never be allowed to interleave in > such a pattern on a larger scale. >
It is actually really easy to force regions to never share. At the moment, there is a fallback list that determines a preference for what block to mix. The reason why this isn't enforced is the cost of moving. On x86 and x86_64, a block of interest is usually 2MB or 4MB. Clearing out one of those pages to prevent any mixing would be bad enough. On PowerPC, it's potentially 16MB. On IA64, it's 1GB. As this was fragmentation avoidance, not guarantees, the decision was made to not strictly enforce the types of pages within a block as the cost cannot be made back unless the system was making agressive use of large pages. This is not the case with Linux. > The only way a fragmentation catastroph can be (proovable) avoided is > by having so few unmovable objects that size + max waste << ram > size. The smaller the better. Allowing movable and unmovable objects > to mix means that max waste goes way up. In your example waste would > be 7*size. With 2MB uper order limit it would be 511*size. > > I keep coming back to the fact that movable objects should be moved > out of the way for unmovable ones. Anything else just allows > fragmentation to build up. > This is easily achieved, just really really expensive because of the amount of copying that would have to take place. It would also compel that min_free_kbytes be at least one free PAGEBLOCK_NR_PAGES and likely MIGRATE_TYPES * PAGEBLOCK_NR_PAGES to reduce excessive copying. That is a lot of free memory to keep around which is why fragmentation avoidance doesn't do it. -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html