DR> And do what if they're not? What behavior are you trying to fix DR> from the pagewalk code with respect to page-aligned addresses? Any DR> specific examples? Sorry, I thought I detailed this in the patch header. In walk_pte_entry(), the exit condition is when the end address is equal to the start address + n*PAGE_SIZE. If they're not both page aligned, then we'll never exit the loop and we'll start handing bad pte entries to the handler function. As was pointed out earlier in the thread, we could "solve" this by making the exit condition be > instead of ==. However, that changes the entirety of walk_page_range() from requiring page-aligned attributes to silently tolerating them. IMHO, it's better to just declare/check/enforce that they are. I hit this recently because I was working with a prototype syscall that took an address range from userspace and walked the pages. I ended up passing non-page-aligned addresses, not knowing that walk_page_range() needed it, and it took me a few days to figure out why my pte_entry handler got a few good entries and then garbage until I crashed. I turned on DEBUG_VM and got zero additional help. With the proposed patch, I would have received a helpful smack in the head. Does that make sense? -- Dan Smith IBM Linux Technology Center email: danms@xxxxxxxxxx -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. 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>