On 2017-12-12 21:40, Ralf Ramsauer wrote: > Hohoho, > > this series decouples inmates from hypervisor (besides printk-core.c). > > To achieve that, I introduced a new global include directory in the root > path of the project. > > Rationale is as follows: > Inmates could accidentally include headers from > 'hypervisor/include/jailhouse', such as utils.h, mmio.h or others. > Furthermore, we intermixed headers of different licenses in this > directory which complicates clearness. All includes that are > dual-licensed are needed by both, hypervisor and inmates. Let's > outsource those headers to their own directory. > > I also moved the project-global configuration header config.h to > 'include/jailhouse/config.h' > > This series would also facilitate efforts to entirely decouple inmates. > > Compile-time tested on x86 (think that suffices), run-time tested on arm > and arm64. > > Jan, you might want to pick those patches from here: > https://github.com/lfd/jailhouse/tree/inmate-decouple > > Cheers > Ralf > > changes since v3: > - collected all signed-off's > - rebased to latest next (no conflicts) > - compile test for all archs > > changes since v2: > - Include Signed-Offs of Jean-Philippe and Lokesh > - rebased to next (and respect that GICv3 is unconditionally compiled for > all > ARM architectures) > > changes since v1: > - squashed some patches > - improved commit messages > - Cc Jean-Phillippe and Antonios for sysregs.h patches > > > Ralf Ramsauer (6): > core: move hypercall interface to global include directory > core: move cell-config.h to global include directory > core: move hypervisor configuration to include/jailhouse/config.h > inmates, arm: provide own sysregs.h > inmates, arm64: provide own sysregs.h > inmates: remove hypervisor-local include directory > > .gitignore | 2 +- > Documentation/hypervisor-configuration.md | 2 +- > Documentation/setup-on-banana-pi-arm-board.md | 2 +- > Kbuild | 2 +- > configs/Makefile | 7 +- > driver/Makefile | 5 +- > hypervisor/Makefile | 4 +- > .../arch/arm}/asm/jailhouse_hypercall.h | 0 > .../arch/arm64}/asm/jailhouse_hypercall.h | 26 ++++++++ > .../arch/x86}/asm/jailhouse_hypercall.h | 0 > .../include => include}/jailhouse/cell-config.h | 0 > .../include => include}/jailhouse/hypercall.h | 0 > inmates/Makefile | 4 +- > inmates/lib/arm/include/asm/sysregs.h | 76 > ++++++++++++++++++++++ > inmates/lib/arm64/include/asm/sysregs.h | 55 ++++++++++++++++ > 15 files changed, 174 insertions(+), 11 deletions(-) > rename {hypervisor/arch/arm/include => > include/arch/arm}/asm/jailhouse_hypercall.h (100%) > rename {hypervisor/arch/arm64/include => > include/arch/arm64}/asm/jailhouse_hypercall.h (63%) > rename {hypervisor/arch/x86/include => > include/arch/x86}/asm/jailhouse_hypercall.h (100%) > rename {hypervisor/include => include}/jailhouse/cell-config.h (100%) > rename {hypervisor/include => include}/jailhouse/hypercall.h (100%) > create mode 100644 inmates/lib/arm/include/asm/sysregs.h > create mode 100644 inmates/lib/arm64/include/asm/sysregs.h >
Thanks, merged now with fixups. One follow-up path will come. Jan -- Siemens AG, Corporate Technology, CT RDA ITP SES-DE Corporate Competence Center Embedded Linux -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
