Re: [Xen-devel] [PATCH 00/22] xen/arm: P2M clean-up and fixes

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Wed, 20 Jul 2016, Julien Grall wrote:
> Hello all,
> 
> This patch series contains a bunch of clean-up and fixes for the P2M code on
> ARM. The major changes are:
>     - Restrict usage of get_page_from_gva to the current vCPU
>     - Deduce the memory attributes from the p2m type
>     - Switch to read-write lock to improve performance
>     - Simplify the TLB flush for a given domain
> 
> I have provided a branch will all the patches applied on my repo:
> git://xenbits/xenbits.xen.org/people/julieng/xen-unstable.git p2m-cleanup-v1

I have committed the first 4 patches to staging. I agree that patch #1
and #3 should be backported to the stable trees. I'll do so when they
pass the gate.


> Yours sincerely,
> 
> Julien Grall (22):
>   xen/arm: system: Use the correct parameter name in local_irq_restore
>   xen/arm: p2m: Pass the vCPU in parameter to get_page_from_gva
>   xen/arm: p2m: Restrict usage of get_page_from_gva to the current vCPU
>   xen/arm: p2m: Fix multi-lines coding style comments
>   xen/arm: p2m: Clean-up mfn_to_p2m_entry
>   xen/arm: p2m: Use the typesafe MFN in mfn_to_p2m_entry
>   xen/arm: p2m: Use p2m_is_foreign in get_page_from_gfn to avoid open
>     coding
>   xen/arm: p2m: Simplify p2m type check by using bitmask
>   xen/arm: p2m: Use a whitelist rather than blacklist in
>     get_page_from_gfn
>   xen/arm: p2m: Differentiate cacheable vs non-cacheable MMIO
>   xen/arm: p2m: Find the memory attributes based on the p2m type
>   xen/arm: p2m: Remove unnecessary locking
>   xen/arm: p2m: Introduce p2m_{read,write}_{,un}lock helpers
>   xen/arm: p2m: Switch the p2m lock from spinlock to rwlock
>   xen/arm: Don't call p2m_alloc_table from arch_domain_create
>   xen/arm: p2m: Move the vttbr field from arch_domain to p2m_domain
>   xen/arm: p2m: Don't need to restore the state for an idle vCPU.
>   xen/arm: p2m: Rework the context switch to another VTTBR in
>     flush_tlb_domain
>   xen/arm: p2m: Inline p2m_load_VTTBR into p2m_restore_state
>   xen/arm: Don't export flush_tlb_domain
>   xen/arm: p2m: Replace flush_tlb_domain by p2m_flush_tlb
>   xen/arm: p2m: Pass the p2m in parameter rather the domain when it is
>     possible
> 
>  xen/arch/arm/domain.c              |   3 -
>  xen/arch/arm/guestcopy.c           |   6 +-
>  xen/arch/arm/p2m.c                 | 255 
> +++++++++++++++++++------------------
>  xen/arch/arm/traps.c               |   4 +-
>  xen/include/asm-arm/arm32/system.h |   2 +-
>  xen/include/asm-arm/arm64/system.h |   2 +-
>  xen/include/asm-arm/domain.h       |   1 -
>  xen/include/asm-arm/flushtlb.h     |   3 -
>  xen/include/asm-arm/mm.h           |   2 +-
>  xen/include/asm-arm/p2m.h          |  85 +++++++++----
>  10 files changed, 194 insertions(+), 169 deletions(-)
> 
> -- 
> 1.9.1
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel