Re: [PATCH mm] swapless page migration: fix swapops.h:97 BUG

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 24 May 2006, Christoph Lameter wrote:
> 
> Thats is very subtle. I hope there are no other areas where the child vma 
> has to be processed before the parent vma?

I've considered, and very much doubt it; and if there are, they'd
have to be peculiar to anonymous, since they'd already have been
wrong for the vma_prio_tree cases.

> An alternative would be to 
> make remove_anon_migration_ptes walk the list in reverse and add a big 
> comment.

No, even without this migration issue, I'd much rather have anon_vmas
inserted in the new list_add_tail way, matching the prio_tree direction.
It's asking for trouble to have opposite conventions there (and much
less surprising to insert child after parent than before).

Hugh
-
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]
  Powered by Linux