On Wed, May 4, 2011 at 8:37 PM, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: > I seriously considered making that "skip stack guard page" and the > "mlock lookup" be two separate bits, because conceptually they are > really pretty independent, but right now the only _users_ seem to be > tied together, so I kept it as one single bit (FOLL_MLOCK). Yes, this seems best for now. > But as far as I can tell, the attached patch is 100% equivalent to > what we do now, except for that "skip stack guard pages only for > mlock" change. > > Comments? I like this patch because it seems to make the logic more > straightforward. > > But somebody else should double-check my logic. It makes perfect sense. Reviewed-by: Michel Lespinasse <walken@xxxxxxxxxx> (I would argue for it to go to stable trees as well) -- Michel "Walken" Lespinasse A program is never fully debugged until the last user dies. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>