Message ID | 1529072910-16156-5-git-send-email-eric.auger@redhat.com |
---|---|
State | New |
Headers | show |
Series | KVM/ARM: virt-3.0: Multiple redistributor regions and 256MB ECAM region | expand |
On Fri, Jun 15, 2018 at 04:28:23PM +0200, Eric Auger wrote: > Let's check if KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION is supported. > If not, we check the number of redist region is equal to 1 and use the > legacy KVM_VGIC_V3_ADDR_TYPE_REDIST attribute. Otherwise we use > the new attribute and allow to register multiple regions to the > KVM device. > > Signed-off-by: Eric Auger <eric.auger@redhat.com> > Reviewed-by: Peter Maydell <peter.maydell@linaro.org> > > --- > > v1 (virt3.0)-> v2 > - added a comment to explain why the regions are registered in > reverse order > > v2 -> v3: > - In kvm_arm_gicv3_realize rename val into add_ormask local variable and > add a comment > - start the redist region registration from s->nb_redist_regions - 1 > downwards > --- > hw/intc/arm_gicv3_kvm.c | 37 ++++++++++++++++++++++++++++++++++--- > 1 file changed, 34 insertions(+), 3 deletions(-) > > diff --git a/hw/intc/arm_gicv3_kvm.c b/hw/intc/arm_gicv3_kvm.c > index ed7b719..5cb94fc 100644 > --- a/hw/intc/arm_gicv3_kvm.c > +++ b/hw/intc/arm_gicv3_kvm.c > @@ -753,6 +753,7 @@ static void kvm_arm_gicv3_realize(DeviceState *dev, Error **errp) > { > GICv3State *s = KVM_ARM_GICV3(dev); > KVMARMGICv3Class *kgc = KVM_ARM_GICV3_GET_CLASS(s); > + bool multiple_redist_region_allowed; > Error *local_err = NULL; > int i; > > @@ -789,6 +790,18 @@ static void kvm_arm_gicv3_realize(DeviceState *dev, Error **errp) > return; > } > > + multiple_redist_region_allowed = > + kvm_device_check_attr(s->dev_fd, KVM_DEV_ARM_VGIC_GRP_ADDR, > + KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION); > + > + if (!multiple_redist_region_allowed && s->nb_redist_regions > 1) { > + error_setg(errp, "Multiple VGICv3 redistributor regions are not " > + "supported by this host kernel"); > + error_append_hint(errp, "A maximum of %d VCPUs can be used", > + s->redist_region_count[0]); > + return; > + } > + > kvm_device_access(s->dev_fd, KVM_DEV_ARM_VGIC_GRP_NR_IRQS, > 0, &s->num_irq, true, &error_abort); > > @@ -798,9 +811,27 @@ static void kvm_arm_gicv3_realize(DeviceState *dev, Error **errp) > > kvm_arm_register_device(&s->iomem_dist, -1, KVM_DEV_ARM_VGIC_GRP_ADDR, > KVM_VGIC_V3_ADDR_TYPE_DIST, s->dev_fd, 0); > - kvm_arm_register_device(&s->iomem_redist[0], -1, > - KVM_DEV_ARM_VGIC_GRP_ADDR, > - KVM_VGIC_V3_ADDR_TYPE_REDIST, s->dev_fd, 0); > + > + if (!multiple_redist_region_allowed) { > + kvm_arm_register_device(&s->iomem_redist[0], -1, > + KVM_DEV_ARM_VGIC_GRP_ADDR, > + KVM_VGIC_V3_ADDR_TYPE_REDIST, s->dev_fd, 0); > + } else { > + /* we register regions in reverse order as "devices" are inserted at Capital w on we would be nicer, but probably not worth respinning for > + * the head of a QSLIST and the list is then popped from the head > + * onwards by kvm_arm_machine_init_done() > + */ > + for (i = s->nb_redist_regions - 1; i >= 0; i--) { > + /* Address mask made of the rdist region index and count */ > + uint64_t addr_ormask = > + i | ((uint64_t)s->redist_region_count[i] << 52); > + > + kvm_arm_register_device(&s->iomem_redist[i], -1, > + KVM_DEV_ARM_VGIC_GRP_ADDR, > + KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION, > + s->dev_fd, addr_ormask); > + } > + } > > if (kvm_has_gsi_routing()) { > /* set up irq routing */ > -- > 2.5.5 > Reviewed-by: Andrew Jones <drjones@redhat.com>
diff --git a/hw/intc/arm_gicv3_kvm.c b/hw/intc/arm_gicv3_kvm.c index ed7b719..5cb94fc 100644 --- a/hw/intc/arm_gicv3_kvm.c +++ b/hw/intc/arm_gicv3_kvm.c @@ -753,6 +753,7 @@ static void kvm_arm_gicv3_realize(DeviceState *dev, Error **errp) { GICv3State *s = KVM_ARM_GICV3(dev); KVMARMGICv3Class *kgc = KVM_ARM_GICV3_GET_CLASS(s); + bool multiple_redist_region_allowed; Error *local_err = NULL; int i; @@ -789,6 +790,18 @@ static void kvm_arm_gicv3_realize(DeviceState *dev, Error **errp) return; } + multiple_redist_region_allowed = + kvm_device_check_attr(s->dev_fd, KVM_DEV_ARM_VGIC_GRP_ADDR, + KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION); + + if (!multiple_redist_region_allowed && s->nb_redist_regions > 1) { + error_setg(errp, "Multiple VGICv3 redistributor regions are not " + "supported by this host kernel"); + error_append_hint(errp, "A maximum of %d VCPUs can be used", + s->redist_region_count[0]); + return; + } + kvm_device_access(s->dev_fd, KVM_DEV_ARM_VGIC_GRP_NR_IRQS, 0, &s->num_irq, true, &error_abort); @@ -798,9 +811,27 @@ static void kvm_arm_gicv3_realize(DeviceState *dev, Error **errp) kvm_arm_register_device(&s->iomem_dist, -1, KVM_DEV_ARM_VGIC_GRP_ADDR, KVM_VGIC_V3_ADDR_TYPE_DIST, s->dev_fd, 0); - kvm_arm_register_device(&s->iomem_redist[0], -1, - KVM_DEV_ARM_VGIC_GRP_ADDR, - KVM_VGIC_V3_ADDR_TYPE_REDIST, s->dev_fd, 0); + + if (!multiple_redist_region_allowed) { + kvm_arm_register_device(&s->iomem_redist[0], -1, + KVM_DEV_ARM_VGIC_GRP_ADDR, + KVM_VGIC_V3_ADDR_TYPE_REDIST, s->dev_fd, 0); + } else { + /* we register regions in reverse order as "devices" are inserted at + * the head of a QSLIST and the list is then popped from the head + * onwards by kvm_arm_machine_init_done() + */ + for (i = s->nb_redist_regions - 1; i >= 0; i--) { + /* Address mask made of the rdist region index and count */ + uint64_t addr_ormask = + i | ((uint64_t)s->redist_region_count[i] << 52); + + kvm_arm_register_device(&s->iomem_redist[i], -1, + KVM_DEV_ARM_VGIC_GRP_ADDR, + KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION, + s->dev_fd, addr_ormask); + } + } if (kvm_has_gsi_routing()) { /* set up irq routing */