Re: [PATCH v5 00/22] vITS save/restore

Verified on Cavium ThunderX platform Continousily migrated Guest more than 500 times b/w different hosts. Please feel free to add Tested-by: Prakash, Brahmajyosyula ___ kvmarm mailing list

Re: [PATCH v5 22/22] KVM: arm64: vgic-v3: KVM_DEV_ARM_VGIC_SAVE_PENDING_TABLES

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > This patch adds a new attribute to GICV3 KVM device > KVM_DEV_ARM_VGIC_GRP_CTRL group. This Allows the userspace to > flush all GICR pending tables into guest RAM. At the moment > we do not offer any restore control as

Re: [PATCH v5 21/22] KVM: arm64: vgic-its: Fix pending table sync

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > In its_sync_lpi_pending_table() we currently ignore the > target_vcpu of the LPIs. We sync the pending bit found in > the vcpu pending table even if the LPI is not targeting it. > > Also in vgic_its_cmd_handle_invall() we

Re: [PATCH v5 20/22] KVM: arm64: vgic-its: Device table save/restore

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > This patch saves the device table entries into guest RAM. > Both flat table and 2 stage tables are supported. DeviceId > indexing is used. > > For each device listed in the device table, we also save > the translation

Re: [PATCH v5 19/22] KVM: arm64: vgic-its: ITT save and restore

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > Introduce routines to save and restore device ITT and their > interrupt table entries (ITE). > > The routines will be called on device table save and > restore. They will become static in subsequent patches. > >

Re: [PATCH v5 17/22] KVM: arm64: vgic-its: Collection table save/restore

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > The save path copies the collection entries into guest RAM > at the GPA specified in the BASER register. This obviously > requires the BASER to be set. The last written element is a > dummy collection table entry. > > We

Re: [PATCH v5 18/22] KVM: arm64: vgic-its: vgic_its_check_id returns the entry's GPA

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > As vgic_its_check_id() computes the device/collection entry's > GPA, let's return it so that new callers can retrieve it easily. > > Signed-off-by: Eric Auger Tested-by: Prakash, Brahmajyosyula

Re: [PATCH v5 16/22] KVM: arm64: vgic-its: Add infrastructure for table lookup

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > Add a generic lookup_table() helper whose role consists in > scanning a contiguous table located in guest RAM and applying > a callback on each entry. Entries can be handled as linked lists > since the callback may return

Re: [PATCH v5 15/22] KVM: arm64: vgic-its: vgic_its_alloc_ite/device

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > Add two new helpers to allocate an its ite and an its device. > This will avoid duplication on restore path. > > Signed-off-by: Eric Auger > Acked-by: Marc Zyngier Tested-by:

Re: [PATCH v5 14/22] KVM: arm64: vgic-its: KVM_DEV_ARM_ITS_SAVE/RESTORE_TABLES

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > Introduce new attributes in KVM_DEV_ARM_VGIC_GRP_CTRL group: > - KVM_DEV_ARM_ITS_SAVE_TABLES: saves the ITS tables into guest RAM > - KVM_DEV_ARM_ITS_RESTORE_TABLES: restores them into VGIC internal > structures. > > We

Re: [PATCH v5 13/22] KVM: arm64: vgic-its: Check the device id matches TYPER DEVBITS range

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > On MAPD we currently check the device id can be stored in the device table. > Let's first check it can be encoded within the range defined by TYPER > DEVBITS. > > Signed-off-by: Eric Auger

Re: [PATCH v5 12/22] KVM: arm64: vgic-its: Interpret MAPD ITT_addr field

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > Up to now the MAPD ITT_addr had been ignored. We will need it > for save/restore. Let's record it in the its_device struct. > > Signed-off-by: Eric Auger Tested-by: Prakash, Brahmajyosyula

Re: [PATCH v5 11/22] KVM: arm64: vgic-its: Interpret MAPD Size field and check related errors

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > Up to now the MAPD's ITT size field has been ignored. It encodes > the number of eventid bit minus 1. It should be used to check > the eventid when a MAPTI command is issued on a device. Let's > store the number of

Re: [PATCH v5 10/22] KVM: arm64: vgic-its: Implement vgic_mmio_uaccess_write_its_iidr

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > The GITS_IIDR revision field is used to encode the migration ABI > revision. So we need to restore it to check the table layout is > readable by the destination. > > By writing the IIDR, userspace thus force the ABI

Re: [PATCH v5 09/22] KVM: arm64: vgic-its: Introduce migration ABI infrastructure

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > We plan to support different migration ABIs, ie. characterizing > the ITS table layout format in guest RAM. Typically a new ABI will > be needed if vLPIs get supported for nested use case. > > So let's introduce an array

Re: [PATCH v5 08/22] KVM: arm64: vgic-its: Implement vgic_mmio_uaccess_write_its_creadr

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > GITS_CREADR needs to be restored so let's implement the associated > uaccess_write_its callback. The write only is allowed if the its > is disabled. > > Signed-off-by: Eric Auger Tested-by:

Re: [PATCH v5 07/22] KVM: arm64: vgic-its: Implement vgic_its_has_attr_regs and attr_regs_access

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > This patch implements vgic_its_has_attr_regs and vgic_its_attr_regs_access > upon the MMIO framework. VGIC ITS KVM device KVM_DEV_ARM_VGIC_GRP_ITS_REGS > group becomes functional. > > At least GITS_CREADR and GITS_IIDR

Re: [PATCH v5 06/22] KVM: arm/arm64: vgic: expose (un)lock_all_vcpus

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > We need to use those helpers in vgic-its.c so let's > expose them in the private vgic header. > > Signed-off-by: Eric Auger > Acked-by: Marc Zyngier Tested-by: Prakash,

Re: [PATCH v5 05/22] KVM: arm64: vgic-its: KVM_DEV_ARM_VGIC_GRP_ITS_REGS group

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > The ITS KVM device exposes a new KVM_DEV_ARM_VGIC_GRP_ITS_REGS > group which allows the userspace to save/restore ITS registers. > > At this stage the get/set/has operations are not yet implemented. > > Signed-off-by:

Re: [PATCH v5 04/22] arm/arm64: vgic: turn vgic_find_mmio_region into public

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > We plan to use vgic_find_mmio_region in vgic-its.c so let's > turn it into a public function. > > Also let's take the opportunity to rename the region parameter > into regions to emphasize this latter is an array of

Re: [PATCH v5 03/22] KVM: arm/arm64: vgic-its: rename itte into ite

On Fri, Apr 14, 2017 at 3:45 PM, Eric Auger wrote: > The actual abbreviation for the interrupt translation table entry > is ITE. Let's rename all itte instances by ite. > > Signed-off-by: Eric Auger > Acked-by: Marc Zyngier

Aarch64: Qemu master or 2.9.0-rc1 breaks compatibility with 4.10 kernel.

/lib/aarch64-linux-gnu/libglib-2.0.so.0 Backtrace stopped: previous frame identical to this frame (corrupt stack?) Verified guest booting on host 4.10 kernel by reverting "GICv3: support state save/restore" patches and guest booted without issues. Regards, Prakash B __

Re: [RFC PATCH 09/33] irqchip/gic-v3-its: Split out property table allocation

PROPBASE_SZ); gic_flush_dcache_to_poc(page_address(prop_page), LPI_PROPBASE_SZ); > > + return prop_page; > +} Regards, Prakash B ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

Re: [PATCH] KVM: arm64: Increase number of memslots to 512

a.prak...@cavium.com> Thanks, prakash B ___ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm