mbox series

[Xen-devel,00/15] xen/arm: Bunch of clean-up/improvement

Message ID 20180716172712.20294-1-julien.grall@arm.com
Headers show
Series xen/arm: Bunch of clean-up/improvement | expand

Message

Julien Grall July 16, 2018, 5:26 p.m. UTC
Hi all,

This is patch series is a bunch of clean-up/improvement I collected while
working on the P2M and trap subsystems.

Cheers,

Julien Grall (15):
  xen/arm: cpregs: Allow HSR_CPREG* to receive more than 1 parameter
  xen/arm: cpregs: Fix typo in the documentation of TTBCR
  xen/arm: Introduce helpers to clear/flags flags in HCR_EL2
  xen/arm: p2m: Reduce the locking section in get_page_from_gva
  xen/arm: p2m: Limit call to mem access code use in get_page_from_gva
  xen/arm: Rework lpae_mapping
  xen/arm: Rework lpae_table
  xen/arm: Rename lpae_valid to lpae_is_valid
  xen/arm: guest_walk: Use lpae_is_mapping to simplify the code
  xen/arm: Introduce helpers to get/set an MFN from/to an LPAE entry
  xen/arm: Allow lpae_is_{table, mapping} helpers to work on invalid
    entry
  xen/arm: p2m: Rename ret to mfn in p2m_lookup
  xen/arm: p2m: Introduce a new variable removing_mapping in
    __p2m_set_entry
  xen/arm: guest_walk_tables: Switch the return to bool
  xen/arm: traps: Move the implementation of GUEST_BUG_ON in traps.h

 xen/arch/arm/guest_walk.c        |  54 +++++++++----------
 xen/arch/arm/mem_access.c        |   2 +-
 xen/arch/arm/mm.c                |  18 +++----
 xen/arch/arm/p2m.c               | 110 ++++++++++++++++++++++++---------------
 xen/arch/arm/traps.c             |  27 +---------
 xen/include/asm-arm/cpregs.h     |   6 +--
 xen/include/asm-arm/guest_walk.h |   8 +--
 xen/include/asm-arm/lpae.h       |  27 +++++-----
 xen/include/asm-arm/processor.h  |  18 +++++++
 xen/include/asm-arm/traps.h      |  24 +++++++++
 10 files changed, 168 insertions(+), 126 deletions(-)

Comments

Julien Grall July 23, 2018, 5:12 p.m. UTC | #1
Ping?

Cheers,

On 16/07/18 18:26, Julien Grall wrote:
> Hi all,
> 
> This is patch series is a bunch of clean-up/improvement I collected while
> working on the P2M and trap subsystems.
> 
> Cheers,
> 
> Julien Grall (15):
>    xen/arm: cpregs: Allow HSR_CPREG* to receive more than 1 parameter
>    xen/arm: cpregs: Fix typo in the documentation of TTBCR
>    xen/arm: Introduce helpers to clear/flags flags in HCR_EL2
>    xen/arm: p2m: Reduce the locking section in get_page_from_gva
>    xen/arm: p2m: Limit call to mem access code use in get_page_from_gva
>    xen/arm: Rework lpae_mapping
>    xen/arm: Rework lpae_table
>    xen/arm: Rename lpae_valid to lpae_is_valid
>    xen/arm: guest_walk: Use lpae_is_mapping to simplify the code
>    xen/arm: Introduce helpers to get/set an MFN from/to an LPAE entry
>    xen/arm: Allow lpae_is_{table, mapping} helpers to work on invalid
>      entry
>    xen/arm: p2m: Rename ret to mfn in p2m_lookup
>    xen/arm: p2m: Introduce a new variable removing_mapping in
>      __p2m_set_entry
>    xen/arm: guest_walk_tables: Switch the return to bool
>    xen/arm: traps: Move the implementation of GUEST_BUG_ON in traps.h
> 
>   xen/arch/arm/guest_walk.c        |  54 +++++++++----------
>   xen/arch/arm/mem_access.c        |   2 +-
>   xen/arch/arm/mm.c                |  18 +++----
>   xen/arch/arm/p2m.c               | 110 ++++++++++++++++++++++++---------------
>   xen/arch/arm/traps.c             |  27 +---------
>   xen/include/asm-arm/cpregs.h     |   6 +--
>   xen/include/asm-arm/guest_walk.h |   8 +--
>   xen/include/asm-arm/lpae.h       |  27 +++++-----
>   xen/include/asm-arm/processor.h  |  18 +++++++
>   xen/include/asm-arm/traps.h      |  24 +++++++++
>   10 files changed, 168 insertions(+), 126 deletions(-)
>
Julien Grall Aug. 22, 2018, 3:45 p.m. UTC | #2
Hi,

On 16/07/18 18:26, Julien Grall wrote:
> Julien Grall (15):
>    xen/arm: cpregs: Allow HSR_CPREG* to receive more than 1 parameter
>    xen/arm: cpregs: Fix typo in the documentation of TTBCR
>    xen/arm: p2m: Reduce the locking section in get_page_from_gva
>    xen/arm: p2m: Limit call to mem access code use in get_page_from_gva
>    xen/arm: Rework lpae_mapping
>    xen/arm: Rework lpae_table
>    xen/arm: Rename lpae_valid to lpae_is_valid
>    xen/arm: guest_walk: Use lpae_is_mapping to simplify the code
>    xen/arm: p2m: Rename ret to mfn in p2m_lookup
>    xen/arm: p2m: Introduce a new variable removing_mapping in
>      __p2m_set_entry
I have committed those 10 patches. The rest requires some rework and 
therefore a respin.

Cheers,