| Message ID | 1463018504-26432-1-git-send-email-van.freenix@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Peng, On 12/05/16 03:01, Peng Fan wrote: > CPU0 is using the boot pages table before relocating xen and > xen_second is not part of them. So, no need to flush the TLB > when filling xen_second. > > Signed-off-by: Peng Fan <van.freenix@gmail.com> > Cc: Stefano Stabellini <sstabellini@kernel.org> > Cc: Julien Grall <julien.grall@arm.com> This will be Xen 4.8 material: Reviewed-by: Julien Grall <julien.grall@arm.com> Regards,
diff --git a/xen/arch/arm/mm.c b/xen/arch/arm/mm.c index 94ea054..addd699 100644 --- a/xen/arch/arm/mm.c +++ b/xen/arch/arm/mm.c @@ -447,7 +447,6 @@ void __init setup_pagetables(unsigned long boot_phys_offset, paddr_t xen_paddr) dest_va = BOOT_RELOC_VIRT_START; pte = mfn_to_xen_entry(xen_paddr >> PAGE_SHIFT, WRITEALLOC); write_pte(xen_second + second_table_offset(dest_va), pte); - flush_xen_data_tlb_range_va_local(dest_va, SECOND_SIZE); /* Calculate virt-to-phys offset for the new location */ phys_offset = xen_paddr - (unsigned long) _start;
CPU0 is using the boot pages table before relocating xen and xen_second is not part of them. So, no need to flush the TLB when filling xen_second. Signed-off-by: Peng Fan <van.freenix@gmail.com> Cc: Stefano Stabellini <sstabellini@kernel.org> Cc: Julien Grall <julien.grall@arm.com> --- V2: Following Julien's comments: split the V1 patch into two patches. This patch only remove tlb flush. refine commit log xen/arch/arm/mm.c | 1 - 1 file changed, 1 deletion(-)