mbox series

[Xen-devel,v2,0/7] xen/arm: Clean-up traps.c

Message ID 20170912103622.18562-1-julien.grall@arm.com
Headers show
Series xen/arm: Clean-up traps.c | expand

Message

Julien Grall Sept. 12, 2017, 10:36 a.m. UTC
Hi all,

xen/arch/arm/traps.c is beginning to get very big. This series is moving out
the co-processor and sysreg emulation in separate files. This will avoid to
grow traps.c when adding more registers emulation (coming soon).

A branch with this series has been pushed:

https://xenbits.xen.org/git-http/people/julieng/xen-unstable.git
branch cleanup-traps-v2

Cheers,

Cc: volodymyr_babchuk@epam.com

Julien Grall (7):
  xen/arm: traps: Re-order the includes alphabetically
  xen/arm: Move arch/arm/vtimer.h to include/asm-arm/vtimer.h
  xen/arm: traps: Export a bunch of helpers to handle emulation
  xen/arm: Move sysreg emulation outside of traps.c
  xen/arm: Move co-processor emulation outside of traps.c
  xen/arm: Move sysregs.h in arm64 sub-directory
  xen/arm: Limit the scope of cpregs.h

 xen/arch/arm/Makefile                      |   1 +
 xen/arch/arm/arm64/Makefile                |   1 +
 xen/arch/arm/arm64/vsysreg.c               | 229 ++++++++++
 xen/arch/arm/domain.c                      |   2 +-
 xen/arch/arm/smp.c                         |   1 -
 xen/arch/arm/traps.c                       | 702 ++---------------------------
 xen/arch/arm/vcpreg.c                      | 452 +++++++++++++++++++
 xen/arch/arm/vgic-v3.c                     |   1 +
 xen/arch/arm/vtimer.c                      |   2 +
 xen/include/asm-arm/arm32/processor.h      |   2 +
 xen/include/asm-arm/arm32/traps.h          |  13 +
 xen/include/asm-arm/arm64/processor.h      |   2 +
 xen/include/asm-arm/{ => arm64}/sysregs.h  |  10 +-
 xen/include/asm-arm/arm64/traps.h          |  18 +
 xen/include/asm-arm/percpu.h               |   1 -
 xen/include/asm-arm/processor.h            |   2 -
 xen/include/asm-arm/traps.h                |  44 ++
 xen/{arch/arm => include/asm-arm}/vtimer.h |   0
 18 files changed, 811 insertions(+), 672 deletions(-)
 create mode 100644 xen/arch/arm/arm64/vsysreg.c
 create mode 100644 xen/arch/arm/vcpreg.c
 create mode 100644 xen/include/asm-arm/arm32/traps.h
 rename xen/include/asm-arm/{ => arm64}/sysregs.h (98%)
 create mode 100644 xen/include/asm-arm/arm64/traps.h
 create mode 100644 xen/include/asm-arm/traps.h
 rename xen/{arch/arm => include/asm-arm}/vtimer.h (100%)

Comments

Stefano Stabellini Sept. 12, 2017, 9:57 p.m. UTC | #1
On Tue, 12 Sep 2017, Julien Grall wrote:
> Hi all,
> 
> xen/arch/arm/traps.c is beginning to get very big. This series is moving out
> the co-processor and sysreg emulation in separate files. This will avoid to
> grow traps.c when adding more registers emulation (coming soon).
> 
> A branch with this series has been pushed:
> 
> https://xenbits.xen.org/git-http/people/julieng/xen-unstable.git
> branch cleanup-traps-v2

I committed the first two patches. Only a couple of very minor comments
on the rest.


> Cheers,
> 
> Cc: volodymyr_babchuk@epam.com
> 
> Julien Grall (7):
>   xen/arm: traps: Re-order the includes alphabetically
>   xen/arm: Move arch/arm/vtimer.h to include/asm-arm/vtimer.h
>   xen/arm: traps: Export a bunch of helpers to handle emulation
>   xen/arm: Move sysreg emulation outside of traps.c
>   xen/arm: Move co-processor emulation outside of traps.c
>   xen/arm: Move sysregs.h in arm64 sub-directory
>   xen/arm: Limit the scope of cpregs.h
> 
>  xen/arch/arm/Makefile                      |   1 +
>  xen/arch/arm/arm64/Makefile                |   1 +
>  xen/arch/arm/arm64/vsysreg.c               | 229 ++++++++++
>  xen/arch/arm/domain.c                      |   2 +-
>  xen/arch/arm/smp.c                         |   1 -
>  xen/arch/arm/traps.c                       | 702 ++---------------------------
>  xen/arch/arm/vcpreg.c                      | 452 +++++++++++++++++++
>  xen/arch/arm/vgic-v3.c                     |   1 +
>  xen/arch/arm/vtimer.c                      |   2 +
>  xen/include/asm-arm/arm32/processor.h      |   2 +
>  xen/include/asm-arm/arm32/traps.h          |  13 +
>  xen/include/asm-arm/arm64/processor.h      |   2 +
>  xen/include/asm-arm/{ => arm64}/sysregs.h  |  10 +-
>  xen/include/asm-arm/arm64/traps.h          |  18 +
>  xen/include/asm-arm/percpu.h               |   1 -
>  xen/include/asm-arm/processor.h            |   2 -
>  xen/include/asm-arm/traps.h                |  44 ++
>  xen/{arch/arm => include/asm-arm}/vtimer.h |   0
>  18 files changed, 811 insertions(+), 672 deletions(-)
>  create mode 100644 xen/arch/arm/arm64/vsysreg.c
>  create mode 100644 xen/arch/arm/vcpreg.c
>  create mode 100644 xen/include/asm-arm/arm32/traps.h
>  rename xen/include/asm-arm/{ => arm64}/sysregs.h (98%)
>  create mode 100644 xen/include/asm-arm/arm64/traps.h
>  create mode 100644 xen/include/asm-arm/traps.h
>  rename xen/{arch/arm => include/asm-arm}/vtimer.h (100%)
> 
> -- 
> 2.11.0
>