Hi Laura, On Wed, Jun 12, 2013 at 06:23:29PM +0100, Laura Abbott wrote: > Other architectures define various set_memory functions to allow > attributes to be changed (e.g. set_memory_x, set_memory_rw, etc.) > Currently, these functions are missing on ARM. Define these in an > appropriate manner for ARM. [...] > +int set_memory_ro(unsigned long addr, int numpages) > +{ > + unsigned long start = addr; > + unsigned long size = PAGE_SIZE*numpages; > + unsigned end = start + size; > + > + apply_to_page_range(&init_mm, start, size, pte_set_ro, NULL); > + dsb(); > + flush_tlb_kernel_range(start, end); The TLB_WB flag gives you the dsb, so you don't need to code it explicitly here (same comment for the other occurrences in this patch). Will -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html