On Sat, 3 Feb 2007, Andrew Morton wrote:
> I wonder if it can be simpler. Make two changes:
Would be great if this could get simpler.
> a) If the scanner encounters an mlocked page on the LRU, take it off.
The current patch takes them off when mlock is set (which may not work
since the page may be off the LRU) and then has the scanner taking them
off. We could just remove the early one but what would this bring us?
> b) munlock() adds all affected pages to the LRU.
Hmmm... You mean without checking all the vmas of a page for VM_LOCKED? So they
are going to be removed again on the next pass? Ok. I see that makes it
simpler but it requires another reclaim scan.
> doesn't consume a page flag. Optional (and arguable) extension: scan the
> vmas during munmap, don't add page to LRU if it's still mlocked.
>
> Why _does_ your patch add a new page flag? That info is available via a
> vma scan.
The page flag allows a clean state transition of a page and accurate
keeping of statistics for MLOCKed pages. There were objections against the
fuzzy counting in the earlier incarnation and it was proposed that a page
flag be introduced. Without the flag we cannot know that the page is
already mapped by a VM_LOCKED vma without scanning over all vmas
referencing the page.
> > 1. We use the 21st page flag and we only have 20 on 32 bit NUMA platforms.
>
> Ow. How were you thinking of fixing that?
I thought someone else could come up with something. Maybe the one that
told me to use another page flag?
> > 2. Since mlocked pages are now off the LRU page migration will no longer
> > move them.
>
> Ow. That could be a right pain when we get around to using migration for
> memory-unplug?
We need to expand page migration anyways to allow the gerneral migration
of non-LRU pages.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
