On Fri, 12 Oct 2012 09:57:56 +0100 Ian Campbell <ian.campb...@citrix.com> wrote:
> On Thu, 2012-10-11 at 22:58 +0100, Mukesh Rathor wrote: > > @@ -2177,8 +2210,19 @@ 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; > > + > > + /* 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; > > What do these end up being for the !xen_initial_domain case? I'd have > expected native_FOO. Yeah, right, we kept on changing the functions that they were set to, until it came down to just native_*. I just didn't think it didnt being set. Too much too fast... ok, time to slow down... :) :).. thanks Mukesh -- 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/