On Wed, Feb 29, 2012 at 11:28:30AM +0000, bill4carson wrote:

On 2012?02?29? 18:32, Catalin Marinas wrote:

quoted

On Mon, Feb 13, 2012 at 09:44:22AM +0000, Bill Carson wrote:

quoted

+static inline void huge_ptep_set_wrprotect(struct mm_struct *mm,
+					   unsigned long addr, pte_t *ptep)
+{
+	pte_t old_pte = *ptep;
+	set_huge_pte_at(mm, addr, ptep, pte_wrprotect(old_pte));
+}

You could use the generic ptep_set_wrprotect()

I'm a bit of confused about this.

generic ptep_set_wrprotect() can not set huge pte, that's why
set_huge_pte_at is used instead here.

Ah, the generic one can only work with with LPAE where set_huge_pte_at()
is just a set_pte_at(). So this part looks good.

-- 
Catalin