Re: [PATCH][8/8] mm: lru interface change

Stone Wang wrote:

I may have missed something very trivial, but... why are they on a
list at all if they don't get scanned

Get the locked pages on a list is necessary for page management,
scatter the locked pages around isnt a good idea.

This doesn't make sense. Whether or not they're on a list does not
change the fact that they may be "scattered".

Also, we could add some kinds of scan to the locked pages,
if we find that it's necessary.

This is a valid reason. But at present you don't scan them, do you?
So you should add them to a list in patch 9 where you add the
machanism to scan them as well, right?

--
SUSE Labs, Novell Inc.

Send instant messages to your online friends http://au.messenger.yahoo.com -

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/

---

• References:
  • [PATCH][8/8] mm: lru interface change
    • From: "Stone Wang" <[email protected]>
  • Re: [PATCH][8/8] mm: lru interface change
    • From: Nick Piggin <[email protected]>
  • Re: [PATCH][8/8] mm: lru interface change
    • From: "Stone Wang" <[email protected]>

• Prev by Date: Re: [PATCH] hpet header sanitization
• Next by Date: Re: slab: Add transfer_objects() function
• Previous by thread: Re: [PATCH][8/8] mm: lru interface change
• Next by thread: [PATCH] kernel crash in powernow-k8 after lost ticks detected
• Index(es):
  • Date
  • Thread

[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]