On Wed, Apr 13, 2011 at 05:00:49PM +1000, Benjamin Herrenschmidt wrote: > On Wed, 2011-04-13 at 09:44 +0300, Alexey Dobriyan wrote: > > > Yes, I take Hugh's version because vm_flags_t is ugly to me. And > > arch > > > dependent variable size is problematic. > > > > Who said it should have arch-dependent size? > > Right, it shouldn't. My original patch did that to avoid thinking about > archs that manipulated it from asm such as ARM but that wasn't the right > thing to do. But that doesn't invalidate having a type. No, we don't manipulate it. We only test for VM_EXEC in it in asm. -- 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>