Re: [PATCH] NOMMU: Pages allocated to a ramfs inode's pagecachemay get wrongly discarded

From: Peter Zijlstra
Date: Fri Mar 13 2009 - 03:58:02 EST


On Fri, 2009-03-13 at 11:03 +0900, KOSAKI Motohiro wrote:
> Hi
>
> > > Which as Peter points out:
> > >
> > > The ramfs stuff is rather icky in that it adds the pages to the aging
> > > list, marks them dirty, but does not provide a writeout method.
> > >
> > > This will make the paging code scan over them (continuously) trying to
> > > clean them, failing that (lack of writeout method) and putting them back
> > > on the list.
> > >
> > > Not requiring the pages to be added to the LRU would be a really good idea.
> > > They are not discardable, be it in MMU or NOMMU mode, except when the inode
> > > itself is discarded.
> >
> > Yep, these pages shouldn't be on the LRU at all. I guess that will
> > require some tweaks to core filemap.c code.
>
> IMHO, UNEVICTABLE_LRU already does lru isolation.
> only rest prblem is, getting rid of "depends on MMU" line in mm/Kconfig.
>
> Am I missing anything?

Yes, the need to take something off that shouldn't be there to begin
with.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/