Messages in this thread | | | Date | Thu, 6 Aug 2009 12:15:53 +0200 | From | Andrea Arcangeli <> | Subject | Re: [RFC] respect the referenced bit of KVM guest pages? |
| |
On Wed, Aug 05, 2009 at 01:42:30PM -0400, Rik van Riel wrote: > Andrea Arcangeli wrote: > > On Wed, Aug 05, 2009 at 10:40:58AM +0800, Wu Fengguang wrote: > >> */ > >> - if ((vm_flags & VM_EXEC) && !PageAnon(page)) { > >> + if ((vm_flags & VM_EXEC) || PageAnon(page)) { > >> list_add(&page->lru, &l_active); > >> continue; > >> } > >> > > > > Please nuke the whole check and do an unconditional list_add; > > continue; there. > > <riel> aa: so you're saying we should _never_ add pages to the active > list at this point in the code > <aa> right > <riel> aa: and remove the list_add and continue completely > <aa> yes > <riel> aa: your email says the opposite :)
Posted more meaningful explanation in self-reply to the email that said the opposite, which tries to explains why I changed my mind (well my focus really was on VM_EXEC and I didn't change my mind about yet but then I'm flexible so I'm listening if somebody thinks it's good thing to keep it). The irc quote was greatly out of context and it missed all the previous conversation... I hope my mail explains my point in more detail than the above.
| |