>>> On 17.09.14 at 16:07, <david.vra...@citrix.com> wrote: > On 17/09/14 05:12, Juergen Gross wrote: >> +static void __init xen_cleanmfnmap(unsigned long vaddr) >> +{ >> + unsigned long va = vaddr & PMD_MASK; >> + unsigned long pa; >> + pgd_t *pgd = pgd_offset_k(va); >> + pud_t *pud_page = pud_offset(pgd, 0); >> + pud_t *pud; >> + pmd_t *pmd; >> + pte_t *pte; >> + unsigned int i; >> + >> + set_pgd(pgd, __pgd(0)); >> + do { >> + pud = pud_page + pud_index(va); >> + if (pud_none(*pud)) { >> + va += PUD_SIZE; >> + } else if (pud_large(*pud)) { >> + pa = pud_val(*pud) & PHYSICAL_PAGE_MASK; >> + xen_free_ro_pages(pa, PUD_SIZE); >> + va += PUD_SIZE; > > Are you missing a ClearPagePinned(..) here?
No, this is a 1Gb data page, not a page table one. Jan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/