On Fri, Jul 02, 2010 at 02:47:25PM +0900, Naoya Horiguchi wrote:
Why is this a break and not a continue? Couldn't you have small and large
pages in the same list?
There's more code that handles LRU in this file. Do they all handle huge pages
I also noticed we do not always lock all sub pages in the huge page. Now if
IO happens it will lock on subpages, not the head page. But this code
handles all subpages as a unit. Could this cause locking problems?
Perhaps it would be safer to lock all sub pages always? Or would
need to audit other page users to make sure they always lock on the head
and do the same here.
Hmm page reference counts may have the same issue?
I think I would prefer a proper if else over a goto here.
The lookup should probably just call a helper to make this function more readable
(like lookup_address(), unfortunately that's x86 specific right now)
Better reuse copy_huge_page() instead of open coding.
firstname.lastname@example.org -- Speaking for myself only.