[Xen-devel,v2,00/21] xen/arm: Bunch of clean-ups

Message ID 20181031181313.8028-1-julien.grall@arm.com
Headers show
Series
  • xen/arm: Bunch of clean-ups
Related show

Message

Julien Grall Oct. 31, 2018, 6:12 p.m.
Hi all,

This patch series is a bunch of clean-ups I have collected the past few weeks.

The main goal is to reduce the scope of some includes and reduce the size of
processor.h by moving some parts in new/current headers.

All the patches but #1 and #3 have been reviewed by Andrii.

Cheers,

Julien Grall (21):
  xen/arm: traps: Constify show_*, do_unexpected_trap and do_bug_frame
    parameters
  xen/arm: regs: Convert guest_mode to a static inline helper
  xen/arm: Remove __init from prototype
  xen/arm: bugs: Move do_bug_frame to traps.h
  xen/arm: Consolidate CPU identification in cpufeature.{c,h}
  xen/arm: Move VABORT_GEN_BY_GUEST to traps.h and turned into inline
  xen/arm: gic-3: Remove unused includes
  xen/arm: gic-v3: Re-order includes in alphabetical order
  xen/arm: Move HSR defines in a new header hsr.h
  xen/arm: Move SYSREG accessors in sysregs.h
  xen/arm: Move out of processor.h traps related variable/function
  xen/arm: Only include stringify.h when necessary
  xen/arm: Only include vreg.h when necessary
  xen/arm: Remove unnecessary includes in asm/vgic.h
  xen/arm: Remove unnecessary includes in asm/mmio.h
  xen/arm: Remove unnecessary includes in traps.c
  xen/arm: Remove unnecessary includes in asm/p2m.h
  xen/arm: Remove unnecessary includes in asm-arm/acpi.h
  xen/arm: Remove unnecessary includes in asm/current.h
  xen/arm: platform: Don't include p2m.h in exynos5 and omap5
  xen/arm: Move vgic_* helpers from gic.h to vgic.h

 xen/arch/arm/Makefile                 |   1 -
 xen/arch/arm/acpi/lib.c               |   1 +
 xen/arch/arm/arm32/entry.S            |   1 +
 xen/arch/arm/arm32/proc-v7.S          |   1 +
 xen/arch/arm/arm32/traps.c            |   1 +
 xen/arch/arm/arm64/traps.c            |   1 +
 xen/arch/arm/bootfdt.c                |   2 +-
 xen/arch/arm/cpu.c                    |  68 -------
 xen/arch/arm/cpuerrata.c              |   1 +
 xen/arch/arm/cpufeature.c             |  42 ++++
 xen/arch/arm/device.c                 |   1 +
 xen/arch/arm/gic-v3-lpi.c             |   1 +
 xen/arch/arm/gic-v3.c                 |  28 ++-
 xen/arch/arm/platforms/exynos5.c      |   1 -
 xen/arch/arm/platforms/omap5.c        |   1 -
 xen/arch/arm/psci.c                   |   1 +
 xen/arch/arm/traps.c                  |  37 ++--
 xen/arch/arm/vcpreg.c                 |   1 +
 xen/arch/arm/vgic-v2.c                |   1 +
 xen/arch/arm/vgic-v3-its.c            |   1 +
 xen/arch/arm/vpl011.c                 |   1 +
 xen/include/asm-arm/acpi.h            |   8 +-
 xen/include/asm-arm/alternative.h     |   3 +-
 xen/include/asm-arm/arm32/processor.h |  62 ------
 xen/include/asm-arm/arm32/sysregs.h   |  75 +++++++
 xen/include/asm-arm/arm64/hsr.h       | 122 ++++++++++++
 xen/include/asm-arm/arm64/processor.h |  27 ---
 xen/include/asm-arm/arm64/sysregs.h   | 132 +++----------
 xen/include/asm-arm/bug.h             |   4 -
 xen/include/asm-arm/cpregs.h          |   2 -
 xen/include/asm-arm/cpufeature.h      | 134 +++++++++++++
 xen/include/asm-arm/current.h         |   2 -
 xen/include/asm-arm/device.h          |  10 +-
 xen/include/asm-arm/event.h           |   1 -
 xen/include/asm-arm/gic.h             |   3 -
 xen/include/asm-arm/hsr.h             | 217 +++++++++++++++++++++
 xen/include/asm-arm/iommu.h           |   4 +-
 xen/include/asm-arm/mmio.h            |   4 +-
 xen/include/asm-arm/p2m.h             |   3 -
 xen/include/asm-arm/page.h            |   1 +
 xen/include/asm-arm/percpu.h          |   8 +-
 xen/include/asm-arm/platform.h        |   7 +-
 xen/include/asm-arm/processor.h       | 354 +---------------------------------
 xen/include/asm-arm/regs.h            |  22 ++-
 xen/include/asm-arm/setup.h           |   9 +-
 xen/include/asm-arm/sysregs.h         |  22 +++
 xen/include/asm-arm/time.h            |   2 +
 xen/include/asm-arm/traps.h           |  16 ++
 xen/include/asm-arm/vgic.h            |   9 +-
 xen/include/asm-arm/vpl011.h          |   1 -
 xen/include/xen/device_tree.h         |   5 +-
 51 files changed, 742 insertions(+), 720 deletions(-)
 delete mode 100644 xen/arch/arm/cpu.c
 create mode 100644 xen/include/asm-arm/arm32/sysregs.h
 create mode 100644 xen/include/asm-arm/arm64/hsr.h
 create mode 100644 xen/include/asm-arm/hsr.h
 create mode 100644 xen/include/asm-arm/sysregs.h

Comments

Stefano Stabellini Nov. 9, 2018, 6:15 p.m. | #1
I committed the whole series

On Wed, 31 Oct 2018, Julien Grall wrote:
> Hi all,
> 
> This patch series is a bunch of clean-ups I have collected the past few weeks.
> 
> The main goal is to reduce the scope of some includes and reduce the size of
> processor.h by moving some parts in new/current headers.
> 
> All the patches but #1 and #3 have been reviewed by Andrii.
> 
> Cheers,
> 
> Julien Grall (21):
>   xen/arm: traps: Constify show_*, do_unexpected_trap and do_bug_frame
>     parameters
>   xen/arm: regs: Convert guest_mode to a static inline helper
>   xen/arm: Remove __init from prototype
>   xen/arm: bugs: Move do_bug_frame to traps.h
>   xen/arm: Consolidate CPU identification in cpufeature.{c,h}
>   xen/arm: Move VABORT_GEN_BY_GUEST to traps.h and turned into inline
>   xen/arm: gic-3: Remove unused includes
>   xen/arm: gic-v3: Re-order includes in alphabetical order
>   xen/arm: Move HSR defines in a new header hsr.h
>   xen/arm: Move SYSREG accessors in sysregs.h
>   xen/arm: Move out of processor.h traps related variable/function
>   xen/arm: Only include stringify.h when necessary
>   xen/arm: Only include vreg.h when necessary
>   xen/arm: Remove unnecessary includes in asm/vgic.h
>   xen/arm: Remove unnecessary includes in asm/mmio.h
>   xen/arm: Remove unnecessary includes in traps.c
>   xen/arm: Remove unnecessary includes in asm/p2m.h
>   xen/arm: Remove unnecessary includes in asm-arm/acpi.h
>   xen/arm: Remove unnecessary includes in asm/current.h
>   xen/arm: platform: Don't include p2m.h in exynos5 and omap5
>   xen/arm: Move vgic_* helpers from gic.h to vgic.h
> 
>  xen/arch/arm/Makefile                 |   1 -
>  xen/arch/arm/acpi/lib.c               |   1 +
>  xen/arch/arm/arm32/entry.S            |   1 +
>  xen/arch/arm/arm32/proc-v7.S          |   1 +
>  xen/arch/arm/arm32/traps.c            |   1 +
>  xen/arch/arm/arm64/traps.c            |   1 +
>  xen/arch/arm/bootfdt.c                |   2 +-
>  xen/arch/arm/cpu.c                    |  68 -------
>  xen/arch/arm/cpuerrata.c              |   1 +
>  xen/arch/arm/cpufeature.c             |  42 ++++
>  xen/arch/arm/device.c                 |   1 +
>  xen/arch/arm/gic-v3-lpi.c             |   1 +
>  xen/arch/arm/gic-v3.c                 |  28 ++-
>  xen/arch/arm/platforms/exynos5.c      |   1 -
>  xen/arch/arm/platforms/omap5.c        |   1 -
>  xen/arch/arm/psci.c                   |   1 +
>  xen/arch/arm/traps.c                  |  37 ++--
>  xen/arch/arm/vcpreg.c                 |   1 +
>  xen/arch/arm/vgic-v2.c                |   1 +
>  xen/arch/arm/vgic-v3-its.c            |   1 +
>  xen/arch/arm/vpl011.c                 |   1 +
>  xen/include/asm-arm/acpi.h            |   8 +-
>  xen/include/asm-arm/alternative.h     |   3 +-
>  xen/include/asm-arm/arm32/processor.h |  62 ------
>  xen/include/asm-arm/arm32/sysregs.h   |  75 +++++++
>  xen/include/asm-arm/arm64/hsr.h       | 122 ++++++++++++
>  xen/include/asm-arm/arm64/processor.h |  27 ---
>  xen/include/asm-arm/arm64/sysregs.h   | 132 +++----------
>  xen/include/asm-arm/bug.h             |   4 -
>  xen/include/asm-arm/cpregs.h          |   2 -
>  xen/include/asm-arm/cpufeature.h      | 134 +++++++++++++
>  xen/include/asm-arm/current.h         |   2 -
>  xen/include/asm-arm/device.h          |  10 +-
>  xen/include/asm-arm/event.h           |   1 -
>  xen/include/asm-arm/gic.h             |   3 -
>  xen/include/asm-arm/hsr.h             | 217 +++++++++++++++++++++
>  xen/include/asm-arm/iommu.h           |   4 +-
>  xen/include/asm-arm/mmio.h            |   4 +-
>  xen/include/asm-arm/p2m.h             |   3 -
>  xen/include/asm-arm/page.h            |   1 +
>  xen/include/asm-arm/percpu.h          |   8 +-
>  xen/include/asm-arm/platform.h        |   7 +-
>  xen/include/asm-arm/processor.h       | 354 +---------------------------------
>  xen/include/asm-arm/regs.h            |  22 ++-
>  xen/include/asm-arm/setup.h           |   9 +-
>  xen/include/asm-arm/sysregs.h         |  22 +++
>  xen/include/asm-arm/time.h            |   2 +
>  xen/include/asm-arm/traps.h           |  16 ++
>  xen/include/asm-arm/vgic.h            |   9 +-
>  xen/include/asm-arm/vpl011.h          |   1 -
>  xen/include/xen/device_tree.h         |   5 +-
>  51 files changed, 742 insertions(+), 720 deletions(-)
>  delete mode 100644 xen/arch/arm/cpu.c
>  create mode 100644 xen/include/asm-arm/arm32/sysregs.h
>  create mode 100644 xen/include/asm-arm/arm64/hsr.h
>  create mode 100644 xen/include/asm-arm/hsr.h
>  create mode 100644 xen/include/asm-arm/sysregs.h
> 
> -- 
> 2.11.0
>