On Thu, Oct 3, 2013 at 8:51 AM, John Stultz <john.stultz@xxxxxxxxxx> wrote: > static inline int page_referenced(struct page *page, int is_locked, > struct mem_cgroup *memcg, > - unsigned long *vm_flags) > + unsigned long *vm_flags, > + int *is_vrange) > { > *vm_flags = 0; > + *is_vrange = 0; > return 0; > } I don't know if it is appropriate to add a parameter in such a core function for an optional functionality. Maybe the is_vrange flag should be squashed into the vm_flags ? I am not sure . -- Regards, Zhan Jianyu -- 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/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>