Messages in this thread | | | Subject | Re: [Xen-devel] [PATCH V3 3/6]: PVH: mmu related changes. | From | Ian Campbell <> | Date | Thu, 18 Oct 2012 11:47:10 +0100 |
| |
On Thu, 2012-10-18 at 01:31 +0100, Mukesh Rathor wrote: > * the initial domain. For guests using the toolstack, they are in: > @@ -2177,8 +2210,20 @@ static const struct pv_mmu_ops xen_mmu_ops __initconst = { > > void __init xen_init_mmu_ops(void) > { > - x86_init.mapping.pagetable_reserve = xen_mapping_pagetable_reserve; > x86_init.paging.pagetable_init = xen_pagetable_init; > + > + if (xen_feature(XENFEAT_auto_translated_physmap)) { > + pv_mmu_ops.flush_tlb_others = xen_flush_tlb_others; > +#if 0
I don't recall seeing this #if 0 before -- is it new and/or deliberate?
If it is a marker for future work can we leave it out for now?
> + /* For PCI devices to map iomem. */ > + if (xen_initial_domain()) { > + pv_mmu_ops.set_pte = native_set_pte; > + pv_mmu_ops.set_pte_at = native_set_pte_at; > + } > +#endif > + return; > + } > + x86_init.mapping.pagetable_reserve = xen_mapping_pagetable_reserve; > pv_mmu_ops = xen_mmu_ops; >
| |