Re: [PATCH v2 2/2] Documentation: More BananaPi documentation

On 2016-06-11 02:58, C.y wrote: > Jan, > > Please help me to review these updated patches. > And I think I forget to cc you in when I sent the patches. > > PS: The layout of mails here in Gmail seems to be a huge different from > google group. Thanks, I've queued both but will follow-up with a

[PATCH] Documentation: Rewrap Banana Pi README to 80 chars

For better readability in the absence of a markdown viewer. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- Documentation/setup-on-banana-pi-arm-board.md | 90 +-- 1 file changed, 57 insertions(+), 33 deletions(-) diff --git a/Documentation/setup-on-banana-

Re: Interrupt handling on ARM

On 2016-06-13 17:58, Antonios Motakis wrote: > > > On 13-Jun-16 17:41, Jan Kiszka wrote: >> On 2016-06-13 17:35, Antonios Motakis wrote: >>> Hello, >>> >>> On 13-Jun-16 08:16, Jan Kiszka wrote: >>>> Hi all, >>>> >>>> o

Re: Using DT overlays for adding virtual hardware

Hi Pantelis, coming back to this topic: On 2016-06-09 08:03, Jan Kiszka wrote: > OK, trial and error, and some interesting insights: I've played with DT > fragments and the overlay configfs patch of Pantelis [1] to have a > convenient start. Interestingly, I wasn't able to load a

Re: Using DT overlays for adding virtual hardware

On 2016-06-21 13:59, Jan Kiszka wrote: > On 2016-06-21 13:45, Pantelis Antoniou wrote: >> Hi Jan, >> >>> On Jun 21, 2016, at 14:43 , Jan Kiszka <jan.kis...@siemens.com> wrote: >>> >>> On 2016-06-21 13:35, Pantelis Antoniou wrote: >>>&g

Re: [PATCH] gitignore: add YCM and Vim config files

On 2016-06-20 06:37, Valentine Sinitsyn wrote: > Hi Jan, > > On 19.06.2016 23:53, Jan Kiszka wrote: >> On 2016-06-19 10:11, Valentine Sinitsyn wrote: >>> Signed-off-by: Valentine Sinitsyn <valentine.sinit...@gmail.com> >>> --- >>> .gitign

[PATCH] tools: Include EXTRA_CFLAGS into build

From: Jan Kiszka <jan.kis...@siemens.com> This allows to pass in additional flags or override existing ones, just like it is already possible for the parts under Kbuild control. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- tools/Makefile | 2 +- 1 file changed, 1 ins

Re: Interrupt limit

On 2016-06-23 13:47, Valentine Sinitsyn wrote: > On 23.06.2016 14:35, Jan Kiszka wrote: >> On 2016-06-23 13:28, Valentine Sinitsyn wrote: >>> Hi Jan, >>> >>> On 23.06.2016 13:53, Jan Kiszka wrote: >>>> On 2016-06-23 11:33, Valentine Sinitsyn wrote:

Re: [PATCH v3 00/13] preparatory patch series for Jailhouse AArch64 support

On 2016-06-23 20:37, Jan Kiszka wrote: > On 2016-06-23 10:27, Jan Kiszka wrote: >> I've merged this series with two modifications (build fix for patch 8, >> slight refactoring of patch 10) into next. I've also rebased my >> wip/arm64 mirror with the remaining patches on

Re: [PATCH v2 05/13] arm64: add work around for large SPIs on AMD Seattle

On 2016-06-17 21:13, antonios.mota...@huawei.com wrote: > From: Antonios Motakis > > The AMD Seattle board features SPI ids that are larger than 64, > which we do not support properly. This workaround allows us to > demonstrate working cells on this target, until we

[PATCH] arm: Remove unused struct pending_irq

From: Jan Kiszka <jan.kis...@siemens.com> Leftover of 8b121b498b. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/include/asm/irqchip.h | 8 1 file changed, 8 deletions(-) diff --git a/hypervisor/arch/arm/include/asm/irqchip.h b/hypervisor/arch

Re: [PATCH] x86: vmx: let bool functions return bool

[re-adding jailhouse-dev] On 2016-07-26 08:10, Nicholas Mc Guire wrote: > On Tue, Jul 26, 2016 at 07:58:44AM +0200, Jan Kiszka wrote: >> On 2016-07-25 21:52, Nicholas Mc Guire wrote: >>> This is really only a type cleanup - functions declared bool should >>> be re

Re: [PATCH] x86: vmx: let bool functions return bool

On 2016-07-25 21:52, Nicholas Mc Guire wrote: > This is really only a type cleanup - functions declared bool should > be returning bool. > > Fixes: c690fb976081 ("Jailhouse public release") > > Signed-off-by: Nicholas Mc Guire > --- > > Type inconsistency detected by

Re: [PATCH] match return type with function prototype

On 2016-07-22 16:00, Nicholas Mc Guire wrote: > start_bdf is at most 0x and thus can be promoted to int safely - no > need to make it unsigned int here which is not compliant with the return > type of pci_find_device(). > > Signed-off-by: Nicholas Mc Guire > --- > > The

Re: [PATCH 00/12] Assorted fixes and simplifications

On 2016-08-01 15:09, Claudio Fontana wrote: > just fix for a typo: > > On 31 July 2016 at 10:39, Jan Kiszka <jan.kis...@web.de> wrote: >> Besides config updates and some cleanups, this series comes with two >> major changes. >> >> The most user visible

Re: [PATCH 09/12] arm: Make arch_panic_stop more robust

On 2016-08-01 15:11, Claudio Fontana wrote: > On 31 July 2016 at 10:39, Jan Kiszka <jan.kis...@web.de> wrote: >> From: Jan Kiszka <jan.kis...@siemens.com> >> >> Do not rely on the PSCI CPU-off logic to stop a CPU in case of a fatal >> hypervisor failure -

Re: [PATCH] x86: vmx: let bool functions return bool

On 2016-07-26 08:20, Jan Kiszka wrote: > On 2016-07-26 08:10, Nicholas Mc Guire wrote: >> On Tue, Jul 26, 2016 at 07:58:44AM +0200, Jan Kiszka wrote: >>> >>> Can be done less verbose (I hope): >>> >>> return !!ok; >>> >> yup - thats

Re: [PATCH 12/12] x86: Require VT-d support also for QEMU

On 2016-07-31 10:39, Jan Kiszka wrote: > From: Jan Kiszka <jan.kis...@siemens.com> > > QEMU 2.7 will have all the features we need to run Jailhouse with VT-d > IOMMU. Remove the special cases from the code and document the new > command line. > > The requir

Re: Passing parameters into inmate through the cmdline

On 2016-07-30 16:38, Maxim Baryshnikov wrote: > Hello everyone, > > I am trying to use the cmdline feature in Jailhouse, but it seems > that it does not work :( For example, when I load apic-demo like this: > > #jailhouse cell load apic-demo -a 0xf -s > "pollute_cache" > > Then,

[PATCH 11/12] configs: Adjust QEMU config to upcoming release 2.7

From: Jan Kiszka <jan.kis...@siemens.com> While 2.7 is still in -rc, we will significantly benefit from this release: it comes with VT-d support that now includes interrupt remapping. Therefore move forward to this QEMU version. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --

[PATCH 01/12] configs: Expand ivshmem regions to 1 MB

From: Jan Kiszka <jan.kis...@siemens.com> More space helps with throughput when using the shared memory for inter-cell networking. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- configs/ivshmem-demo.c | 6 +++--- configs/linux-x86-demo.c | 8 configs/qemu-vm.

[PATCH 04/12] core: Deny shutdown with still existing non-root cells

From: Jan Kiszka <jan.kis...@siemens.com> Simplify the shutdown logic by requiring the root cell driver to explicitly destroy all non-root cells first. In particular, it helps to resolve the tricky handover of non-root cell CPUs in unplugged state to Linux. At this chance, properly do

[PATCH 07/12] driver: Gracefully handle zero-sized load requests

From: Jan Kiszka <jan.kis...@siemens.com> We currently fail if a load request has a size of 0, because mapping an empty region fails. While such requests are often caused by corrupted files there may also be valid use cases. So better handle this gracefully. Signed-off-by: Jan Kiszka &l

[PATCH 00/12] Assorted fixes and simplifications

disabling the hypervisor. This shifts efforts out of the hypervisor and allows to drop some of its code. Jan CC: Jan Kiszka <jan.kis...@siemens.com> Jan Kiszka (12): configs: Expand ivshmem regions to 1 MB driver: Make a bunch of cell functions static driver: Destroy non-root cells

[PATCH 12/12] x86: Require VT-d support also for QEMU

From: Jan Kiszka <jan.kis...@siemens.com> QEMU 2.7 will have all the features we need to run Jailhouse with VT-d IOMMU. Remove the special cases from the code and document the new command line. The requirement on the kernel version is not exact but a safe guess based on tests. Support f

[PATCH 06/12] core: Fix printing of invalid MMIO accesses

From: Jan Kiszka <jan.kis...@siemens.com> mem->phys_start is u64, but we only print up to unsigned long. Adjust the input type. Resolves broken size reports on 32-bit ARM. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/mmio.c | 3 ++- 1 file changed, 2 i

[PATCH 08/12] arm: Remove return code from irqchip_cpu_reset

From: Jan Kiszka <jan.kis...@siemens.com> Only the GICv3 actually returns a non-zero value, and that only if the reset was called before the irqchip was fully initialized, i.e. as part of an error rollback triggered via irqchip_cpu_shutdown. Therefore, remove the error return code and only

Re: [PATCH 03/12] driver: Destroy non-root cells before disabling the hypervisor

Cheers, > Tony > > On 31-Jul-16 10:39, Jan Kiszka wrote: >> From: Jan Kiszka <jan.kis...@siemens.com> >> >> The hypervisor's disable function will lose support for destroying >> non-root cells along with the shutdown. Prepare for this by performing >> an expl

Re: [PATCH] inmate: x86: fixup apic_freq to be unsigned long

On 2016-08-01 19:24, Nicholas Mc Guire wrote: > apic_timer_init is expected to return unsigned long but was returning > unsigned long long. As the math should be safe with apic_freq being > unsigned long, the type is simply adjusted to match the function > declaration. > > Fixes: 29b8b410c3e7

Re: [PATCH V2] x86: vmx: make bool functions return bool

On 2016-08-01 19:17, Nicholas Mc Guire wrote: > This is really only a type cleanup - functions declared bool should > be returning bool. > > Fixes: c690fb976081 ("Jailhouse public release") > > Signed-off-by: Nicholas Mc Guire > --- > V2: a hopefully more reasonable solution

Re: Question about the sysconfig.c

On 2016-08-04 06:38, Meng Xu wrote: > Hi Jan, > > This is a follow-up question as to the closed issue at > https://github.com/siemens/jailhouse/issues/15 > > As you suggested, I'm reposting the follow-up question in the mailing > list. Hopefully, I'm doing it in a right way. > > I didn't

[PATCH 17/17] Documentation: Add patch requirement to Banana Pi setup description

As of f7ee922b0bca, we need a tiny patch of the root Linux kernel on ARM. Update the Banana Pi documentation accordingly. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- Documentation/setup-on-banana-pi-arm-board.md | 26 +- 1 file changed, 25 insertions

[PATCH 13/17] arm: Enforce disabled MMU during setup_mmu_el2

Instead of silently returning if the MMU is already on in EL2, make sure that it is off and rather risk a crash if that wasn't true. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/mmu_hyp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

[PATCH 14/17] arm: Remove infinite loop from end of setup_mmu_el2

This is never reached, the assembly code before this line ensures it However, don't place a __builtin_unreachable here, or the compiler will start to drop essential code of the caller. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/mmu_hyp.c | 3 --- 1 file chan

[PATCH 08/17] arm: Move access_banked_reg as access_banked_r13_r14 into trap.c

By making it specific for access_cell_reg, it becomes obvious that there are not other cases than r13 and r14. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/include/asm/traps.h | 14 -- hypervisor/arch/arm/traps.c | 20 ++---

[PATCH 16/17] TODO: Update ARM entries

Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- TODO.md | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/TODO.md b/TODO.md index ce9a564..92950af 100644 --- a/TODO.md +++ b/TODO.md @@ -11,13 +11,16 @@ x86 support ARM support - v7 (

[PATCH 15/17] arm: Fix coding style in arch_cpu_init

No functional changes. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hypervisor/arch/arm/setup.c b/hypervisor/arch/arm/setup.c index 91e6ff7..8e9f54f 100644 --- a/hypervisor/ar

[PATCH 01/17] arm: Re-indent psci.h

No functional changes, just better alignment with surrounding code. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/include/asm/psci.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/hypervisor/arch/arm/include/asm/psc

[PATCH 04/17] arm: Move arm_read/write_banked_reg definition to processor.h

Access to those registers is not only trap-related. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/include/asm/processor.h | 6 ++ hypervisor/arch/arm/include/asm/traps.h | 6 -- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/hype

[PATCH 03/17] arm: Reorder includes according to Jailhouse standard

Generic includes should always come first, and alphabetical sorting is desired. No functional changes. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/mmu_hyp.c | 4 ++-- hypervisor/arch/arm/psci.c| 2 +- hypervisor/arch/arm/setup.c | 8 hyperviso

[PATCH 11/17] arm: Combine multiple function __attribute__

Cosmetic cleanup, no functional changes. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/mmu_hyp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hypervisor/arch/arm/mmu_hyp.c b/hypervisor/arch/arm/mmu_hyp.c index f9b3d8d..93758bc

[PATCH 05/17] arm: Move _access_banked as arm_rw_banked_reg to processor.h

Also a generic service that should be defined near arm_read/write_banked_reg. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/include/asm/processor.h | 8 hypervisor/arch/arm/include/asm/traps.h | 22 +++--- 2 files changed, 15 inse

[PATCH 00/17] ARM: Additional cleanups

Most of these are of cosmetic nature, finding while reworking to code for the previously sent series. Jan Jan Kiszka (17): arm: Re-indent psci.h arm: Adjust includes of percpu.h arm: Reorder includes according to Jailhouse standard arm: Move arm_read/write_banked_reg definition

Re: [PATCH v3 0/8] Replace jailhouse-cell-list by native C implementation

On 2016-08-13 00:00, Ralf Ramsauer wrote: > Hi, > > On 08/08/2016 11:23 PM, Ralf Ramsauer wrote: >> Hi, >> >> just realised that sysfs entries are created before valid cell ID are >> assigned. > a short general question on the creation of cells: > > For any type of cell, the corresponding sysfs

[PATCH 22/27] arm: Introduce infrastructure for reworked CPU management

of those flags takes place in check_events. The flags are generally protected now by a spinlock, resolving many of the subtle wake-up races of the old code. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/control.c | 174 ++ hype

[PATCH 23/27] arm: Reorder function to match final layout

Non-functional change to match the arch_*_cpu function ordering of x86 also on ARM. Helps when comparing code and will reduce the churn of the following commits a bit. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/control.

[PATCH 25/27] arm: Switch to new CPU management services

without proper PSCI support: it is no longer required to add stubs for PSCI-based platforms. Consequently, those for TK1 and Allwinner A20 are removed. Now obsolete PSCI fragments will be purged in a separate patch. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/Ma

[PATCH 04/27] arm: Rename arch_mmu API to arm_paging

This is more consistent with existing paging-related code. It also implies we should have related prototypes in asm/paging.h. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/control.c | 8 hypervisor/arch/arm/include/asm/control.

[PATCH 17/27] arm: Refactor arch_cpu_tlb_flush to arm_paging_vcpu_flush_tlbs

Move resetting of flush_vcpu_caches to the callers the actually require it, then remove the misleading cpu_data argument (service only works for the calling CPU) and rename it so that it is consistent with other cache flushing services. Signed-off-by: Jan Kiszka <jan.kis...@siemens.

[PATCH 20/27] arm: Let arm_paging_vcpu_init take paging structure as argument

Prepare for setting a VCPU to different paging structures than those of its owner cell. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/control.c| 2 +- hypervisor/arch/arm/include/asm/paging.h | 4 ++-- hypervisor/arch/arm/mmu_cell.c

[PATCH 00/27] ARM: Cache maintenance & CPU management fixes

gier <marc.zyng...@arm.com> CC: Mark Rutland <mark.rutl...@arm.com> Jan Kiszka (27): driver: Evaluate and maintain Linux' __boot_cpu_mode on ARM driver: Flush D-cache after cell loading on ARMv7 core: Add MIN macro arm: Rename arch_mmu API to arm_paging arm: A

[PATCH 05/27] arm: Align description arch_paging_flush_cpu_caches with actual logic

and comments to what the code already did. CC: Marc Zyngier <marc.zyng...@arm.com> CC: Mark Rutland <mark.rutl...@arm.com> Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hypervisor/arch/arm/include/asm/paging.h | 4 ++-- hypervisor/arch/arm/include/asm/sysregs.h | 2 +-

[PATCH 01/27] driver: Evaluate and maintain Linux' __boot_cpu_mode on ARM

of the hypervisor stub Linux should have installed instead of simply crashing the system when it is missing. Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- driver/main.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/driver/main.c b/driver/main.c index 0a76e18..7

Re: [PATCH 01/27] driver: Evaluate and maintain Linux' __boot_cpu_mode on ARM

On 2016-08-10 12:31, Jan Kiszka wrote: > On 2016-08-10 11:38, Ralf Ramsauer wrote: >> Hi Jan, >> >> I'm curious, which upstream patch patch are you talking about? I tried >> to export is_hyp_mode_available a couple of months ago but wasn't >> accepted upstream.

Re: ioctl function not completing and hypervisor output

On 2016-07-19 04:37, Dube, Swaraj wrote: > Hi Jan, > > I had a discussion with my team regarding the board specs but before that we > believe there are some things that are missing or incorrect in myboard.c > (generated C file using jailhouse-config-create) especially after reading the >

Re: ARM: Managing stage-2 page table updates

On 2016-07-20 13:17, Mark Rutland wrote: > On Wed, Jul 20, 2016 at 12:19:20PM +0200, Jan Kiszka wrote: >> Again my question: What are the requirements regarding cache maintenance >> when allowing a guest to run with caches off? > > a) There must be no cacheable alias for t

Re: ARM: Managing stage-2 page table updates

On 2016-07-20 12:07, Marc Zyngier wrote: > On 20/07/16 10:47, Mark Rutland wrote: >> Hi Jan, >> >> On Wed, Jul 20, 2016 at 10:05:01AM +0100, Marc Zyngier wrote: >>> On 20/07/16 07:09, Jan Kiszka wrote: >>>> I've two questions for a while now: >>&g

Re: ioctl function not completing and hypervisor output

On 2016-07-20 10:48, Dube, Swaraj wrote: > Hi Jan, > > I have discussed that matter with my team and we need to get clearance from > the management about passing out the addresses and stuff but my target > platform is the Intel Apollo lake SoC, you may have information about this > however I

ARM: Managing stage-2 page table updates

Hi all, I'd like to have some word by an expert on what we have to do in Jailhouse when changing the stage-2 page tables. This is what currently happens for 32-bit ARM (maybe Tony can add a word on potential deviations for ARM64): - all affected CPUs, except the one executing the changes, are

Re: ioctl function not completing and hypervisor output

-Original Message- > From: jailhouse-dev@googlegroups.com [mailto:jailhouse-dev@googlegroups.com] > On Behalf Of Valentine Sinitsyn > Sent: Thursday, July 14, 2016 5:23 PM > To: Dube, Swaraj <swaraj.d...@intel.com>; Jan Kiszka > <jan.kis...@siemens.com>; Jailhouse

Re: Failed to boot jailhouse

On 2016-06-27 10:38, Raz wrote: > Thanks Jan. > You were right. > As a user I have experience with RT-systems solution. So I am a bit > puzzled because I would expect to see a cpu dropped from both the host > and the guest. First of all, the QEMU setup is for testing purposes - do not confuse

Re: [PATCH] x86: vmx: Enable INVPCID if reported via CPUID

On 2016-07-08 19:33, Valentine Sinitsyn wrote: > Hi Jan, > > On 08.07.2016 22:25, Jan Kiszka wrote: >> Linux kernel 4.6 and later make use of this instruction and crash if we >> do not allow it. It flushes TLB mappings, but only on the caller's >> logical CPU. >

Re: Caching mode = 0 Missing

On 2016-07-05 19:46, Jan Kiszka wrote: > On 2016-07-05 09:36, swaraj.d...@intel.com wrote: >> Hi, >> >> I just ran a hardware check for Jailhouse on my intel board using the >> following command: >> >> tools/jailhouse-hardware-check configs/myboard.cell &

[PATCH] x86: Officially support VT-d Caching Mode != 0

We already flush all domain-related caches after reconfigurations, irrespective of the address ranges changed and how they were changed (made valid or invalid). So the feature check during unit setup is actually superfluous - remove it. Signed-off-by: Jan Kiszka <jan.kis...@siemens.

Re: Anyone using MSI-X?

On 2016-07-05 20:49, Valentine Sinitsyn wrote: > On 05.07.2016 22:51, Jan Kiszka wrote: >> On 2016-07-05 13:21, Valentine Sinitsyn wrote: >>> Hi all, >>> >>> I faced a strange issue regarding MSI-X on my board. >>> >>> A USB host (xchi_hcd)

Re: [PATCH 0/7] Replace jailhouse-cell-list by native C implementation

On 2016-08-04 21:28, Ralf Ramsauer wrote: > Hi, > > this is a short series that does not introduce any functional changes. > > I'd like to replace jailhouse-cell-list. It's a rudimentary function and > requires python. So I reimplemented cell-list natively in C and did some > cleanup work on

Re: [PATCH 0/7] Replace jailhouse-cell-list by native C implementation

On 2016-08-05 11:44, Ralf Ramsauer wrote: > Hi Jan, > > On 08/05/2016 07:57 AM, Jan Kiszka wrote: >> On 2016-08-04 21:28, Ralf Ramsauer wrote: >>> Hi, >>> >>> this is a short series that does not introduce any functional changes. >>> >>>

Re: [PATCH v3 8/8] tools: jailhouse: reimplement jailhouse-cell-list in C

> index d5a535b..000 > --- a/tools/jailhouse-cell-list > +++ /dev/null > @@ -1,40 +0,0 @@ > -#!/usr/bin/env python > - > -# Jailhouse, a Linux-based partitioning hypervisor > -# > -# Copyright (c) Siemens AG, 2014 > -# > -# Authors: > -# Jan Kiszka <jan.kis.

Re: [PATCH 4/7] tools: jailhouse: prepare read_file() to read files from sysfs

On 2016-08-04 21:28, Ralf Ramsauer wrote: > read_file() aborts if the number of read bytes does not equal to the > corresponding size in struct stat. st_size and the actual file size may > differ if sysfs is involved. > > If read() returns a number smaller than st.st_size does not necessarily >

Re: [PATCH 1/7] tools: jailhouse: reduce redundant usage of "/dev/jailhouse"

On 2016-08-04 21:28, Ralf Ramsauer wrote: > The device file "/dev/jailhouse" appears in several locations. Reducing > the occurence of this string makes code more flexible. > > Signed-off-by: Ralf Ramsauer > --- > tools/jailhouse.c | 5 +++-- > 1 file changed, 3

Re: [PATCH 7/7] tools: jailhouse: introduce STR_(N)EQ macros

On 2016-08-04 21:28, Ralf Ramsauer wrote: > Jailhouse command line tool makes heavy use of string comparisons for > parsing command line arguments. Simplify string comparisons by using > macros. > > Signed-off-by: Ralf Ramsauer > --- > tools/jailhouse.c | 39

Re: [PATCH 6/7] tools: jailhouse: replace jailhouse-cell-list by native implementation

On 2016-08-06 13:05, Ralf Ramsauer wrote: > On 08/06/2016 12:06 PM, Jan Kiszka wrote: >> If we want to go down this way, sysfs should first of all be improved >> to support this better: >> >> diff --git a/Documentation/sysfs-entries.txt >> b/Documentation/s

Re: config-create: unusable PCI bar configuration

Hi Bene, On 2016-08-09 17:39, Benedikt Spranger wrote: > Hello everyone, > > "jailhouse config create" creates a unusable PCI bar configuration: > > PCI device: > 06:00.0 Ethernet controller: Realtek Semiconductor Co., Ltd. > RTL8111/8168/8411 PCI Express Gigabit Ethernet Controller (rev 06) >

Re: [PATCH v2 8/8] tools: jailhouse: reimplement jailhouse-cell-list in C

On 2016-08-07 22:08, Ralf Ramsauer wrote: > On 08/07/2016 09:53 PM, Jan Kiszka wrote: >> On 2016-08-07 19:41, Ralf Ramsauer wrote: >>> And keep the output format of the former python script. >>> >> >> Not yet using scandir with alphasort- why? >&g

Re: [PATCH v2 7/8] tools: jailhouse: do not use struct stat to predict buffer size

On 2016-08-08 11:00, Valentine Sinitsyn wrote: > On 08.08.2016 13:51, Jan Kiszka wrote: >> On 2016-08-08 10:26, Ralf Ramsauer wrote: >>> On 08/08/2016 07:25 AM, Jan Kiszka wrote: >>>> On 2016-08-07 22:08, Ralf Ramsauer wrote: >>>>> On 08/07/2016 09:52

Re: [PATCH v2 7/8] tools: jailhouse: do not use struct stat to predict buffer size

On 2016-08-08 11:22, Valentine Sinitsyn wrote: > On 08.08.2016 14:16, Jan Kiszka wrote: >> On 2016-08-08 11:13, Valentine Sinitsyn wrote: >>> On 08.08.2016 14:03, Jan Kiszka wrote: >>>> On 2016-08-08 11:00, Valentine Sinitsyn wrote: >>>>> On 08.08.2016

Re: ARM: Managing stage-2 page table updates

On 2016-08-01 19:23, Antonios Motakis wrote: > > > On 20-Jul-16 13:31, Jan Kiszka wrote: >> On 2016-07-20 13:17, Mark Rutland wrote: >>> On Wed, Jul 20, 2016 at 12:19:20PM +0200, Jan Kiszka wrote: >>>> Again my question: What are the requirements regarding

Re: [PATCH V2] x86: vmx: make bool functions return bool

On 2016-08-02 10:49, Nicholas Mc Guire wrote: > On Mon, Aug 01, 2016 at 10:23:21PM +0200, Jan Kiszka wrote: >> On 2016-08-01 19:17, Nicholas Mc Guire wrote: >>> This is really only a type cleanup - functions declared bool should >>> be returning bool. >>> >&g

Re: Jailhouse and legacy interrupts

On 2016-06-30 12:49, Gabriele Sonnu wrote: > Hi, > > I'm trying to run a custom minimal RT OS inside Jailhouse. > I successfully booted it by making a custom loader, but it freezes > trying to use the timer - it just waits forever for the interrupt to be > served. > The OS is single core, and

Re: Failed to boot jailhouse

rrno 22] Invalid argument > >The linux-x86-demo.cell is only cell I found ? > > > > On Mon, Jun 27, 2016 at 8:43 PM, Jan Kiszka <jan.kis...@web.de > <mailto:jan.kis...@web.de>> wrote: > > On 2016-06-27 17:12, Raz wrote: > > >

Re: [PATCH 2 of 2] make: do not support GCOV in out of kernel components

On 2017-02-07 13:32, Henning Schild wrote: > If the root-cell kernel is built with GCOV (CONFIG_GCOV_PROFILE_ALL) > kbuild will build everything with GCOV support. For the driver it is > ok to inherit that, but for out of kernel parts like hypervisor, > tools, inmates it is not. > >

Re: [PATCH 1 of 2] gitignore: ignore GCOV output

On 2017-02-07 16:52, Henning Schild wrote: > On Wed, 8 Feb 2017 00:02:39 +0900 > Valentine Sinitsyn wrote: > >> Hi Henning, >> >> I recall that there was a policy against adding tools-generated files >> to .gitignore, as .config/git/ignore seems more appropriate >>

Re: [PATCH 2 of 2 v2] make: do not support GCOV in out of kernel components

On 2017-02-08 16:20, Henning Schild wrote: > If the root-cell kernel is built with GCOV (CONFIG_GCOV_PROFILE_ALL) > kbuild will build everything with GCOV support. For the driver it is > ok to inherit that, but for out of kernel parts like hypervisor, > tools, inmates it is not. > >

Re: Linux: ioremap_page_range is no longer exported

On 2017-02-03 10:11, Henning Schild wrote: > On Thu, 2 Feb 2017 18:36:59 +0100 > Jan Kiszka <jan.kis...@siemens.com> wrote: > >> On 2017-02-02 18:17, Ralf Ramsauer wrote: >>> Hi, >>> >>> upstream Linux commit 3277953 (mm: do not export ioremap_page

Re: how to enable I-cache for an inmate

On 2017-02-03 16:34, Vitaly Andrianov wrote: > Hi Jan, > > On 02/03/2017 09:12 AM, Jan Kiszka wrote: >> Hi Vitaly, >> >> On 2017-02-03 14:57, Vitaly Andrianov wrote: >>> Hello, >>> >>> I'm working on Jailhouse support for TI AM572x

Re: how to enable I-cache for an inmate

nger apply). Jan 8< >From 7d01e57ffc40a5343d6f8de5e4c51084f8fdd775 Mon Sep 17 00:00:00 2001 From: Jan Kiszka <jan.kis...@siemens.com> Date: Mon, 25 Jul 2016 08:03:23 +0200 Subject: [PATCH] inmates: arm: Enable caches during startup --- inmates/lib/arm/header.S | 28 +++

Re: Linux: ioremap_page_range is no longer exported

On 2017-02-02 18:17, Ralf Ramsauer wrote: > Hi, > > upstream Linux commit 3277953 (mm: do not export ioremap_page_range > symbol for external module) unexports the ioremap_page_range symbol, > which we need for jailhouse_ioremap(). > > The unexport was introduced in 4.10-rc6. To be able to load

Re: [PATCH v4 09/12] driver: implement continous console reading on /dev/jailhouse

On 2017-01-19 21:11, Ralf Ramsauer wrote: > Signed-off-by: Ralf Ramsauer > --- > Documentation/debug-output.md | 8 +++- > driver/main.c | 97 > +++ > 2 files changed, 104 insertions(+), 1 deletion(-) > >

Re: [Qemu-devel] Towards an ivshmem 2.0?

On 2017-01-23 15:19, Markus Armbruster wrote: > Jan Kiszka <jan.kis...@siemens.com> writes: > >> Hi, >> >> some of you may know that we are using a shared memory device similar to >> ivshmem in the partitioning hypervisor Jailhouse [1]. >> >> We sta

Re: [PATCH v4 09/12] driver: implement continous console reading on /dev/jailhouse

On 2017-01-24 17:54, Ralf Ramsauer wrote: > On 01/24/2017 05:04 PM, Jan Kiszka wrote: >> On 2017-01-19 21:11, Ralf Ramsauer wrote: >>> Signed-off-by: Ralf Ramsauer <ralf.ramsa...@oth-regensburg.de> >>> --- >>> Documentation/debug-output.md | 8 ++

Re: non-root linux jetson tk1

On 2017-01-26 11:10, Dan Zach wrote: > On Thursday, 26 January 2017 11:04:37 UTC+2, Ralf Ramsauer wrote: >> Hi Dan, >> >> On 01/26/2017 09:29 AM, Dan Zach wrote: >>> I am in process of starting the non-root linux on the inmate. >>> The inmate RAM starts at 0xe800 (phys and virt). >>> The

Re: [Qemu-devel] Towards an ivshmem 2.0?

On 2017-01-30 13:19, Markus Armbruster wrote: >>> Can you explain why not letting the guest map the shared memory into its >>> address space on its own just like any other piece of device memory is a >>> requirement? >> >> It requires reconfiguration of the sensitive 2nd level page tables >>

[PATCH] tools: Add completion for 'jailhouse console'

From: Jan Kiszka <jan.kis...@siemens.com> Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- tools/jailhouse-completion.bash | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/tools/jailhouse-completion.bash b/tools/jailhouse-completion.bash index 3c59

[PATCH] Documentation: Update non-root Linux description

From: Jan Kiszka <jan.kis...@siemens.com> We now have "jailhouse cell linux" support for all architectures, thus the original ARM64 description is no longer needed. Remove it and generalize the x86 documentation to cover all architectures. Signed-off-by: Jan Kiszka <jan

Re: [Qemu-devel] Towards an ivshmem 2.0?

On 2017-01-30 09:02, Markus Armbruster wrote: > Jan Kiszka <jan.kis...@web.de> writes: > >> On 2017-01-29 15:00, Marc-André Lureau wrote: >>> Hi >>> >>> On Sun, Jan 29, 2017 at 12:44 PM Jan Kiszka <jan.kis...@web.de >>> <mailto:jan.

Re: [Qemu-devel] Towards an ivshmem 2.0?

On 2017-01-30 09:00, Markus Armbruster wrote: > Jan Kiszka <jan.kis...@web.de> writes: > >> On 2017-01-27 20:36, Markus Armbruster wrote: >>> Jan Kiszka <jan.kis...@web.de> writes: >>> >>>> On 2017-01-23 15:19, Markus Armbruster w

Re: Jetson TK1 not shutting down using jailhouse

On 2017-01-20 17:42, Andreas Kölbl wrote: > Hi, > > my Jetson TK1 does not power off after shutting it down, if I created a > cell before which uses the first CPU. > > I've created a cell config with 0x1 as cpu mask. I enable the > jetson-demo.cell, create my cell, disable jailhouse and try to

Re: [PATCH 0/2] Add 16-bit console divider support

On 2017-01-20 10:55, andreas.koe...@st.oth-regensburg.de wrote: > From: Andreas Koelbl > > At the moment we only support dividers being less or equal than an 8-bit > value. > This affects both the x86 and the arm architecture. > > Andreas Kölbl (2): >

Re: [PATCH v4 00/12] Implement virtual hypervisor console

On 2017-01-19 21:11, Ralf Ramsauer wrote: > Hi, > > I implemented a sysfs interface for the hypervisor console. This is useful for > debugging devices without UARTs. At least as long as the root-cell doesn't > crash... > > Additionally, I implemented continuous reading of the hypervisor console

Re: [Qemu-devel] Towards an ivshmem 2.0?

On 2017-01-29 15:00, Marc-André Lureau wrote: > Hi > > On Sun, Jan 29, 2017 at 12:44 PM Jan Kiszka <jan.kis...@web.de > <mailto:jan.kis...@web.de>> wrote: > > >> Of course, I'm careful with investing much time into expanding the > >> exist

  1   2   3   4   5   6   7   8   9   10   >