From: Andre Przywara Date: Wed, 7 Sep 2016 00:47:49 +0000 (+0100) Subject: ARM: vITS: handle MAPC command X-Git-Tag: archive/raspbian/4.11.1-1+rpi1~1^2~66^2~2012 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=9d133712b4f0e7e870e9f5d4e6422a549dca4454;p=xen.git ARM: vITS: handle MAPC command The MAPC command associates a given collection ID with a given redistributor, thus mapping collections to VCPUs. We just store the vcpu_id in the collection table for that. Signed-off-by: Andre Przywara Acked-by: Julien Grall Acked-by: Stefano Stabellini --- diff --git a/xen/arch/arm/vgic-v3-its.c b/xen/arch/arm/vgic-v3-its.c index 8a2a0d288a..14cb1f04ce 100644 --- a/xen/arch/arm/vgic-v3-its.c +++ b/xen/arch/arm/vgic-v3-its.c @@ -114,6 +114,25 @@ static paddr_t get_baser_phys_addr(uint64_t reg) return reg & GENMASK(47, 12); } +/* Must be called with the ITS lock held. */ +static int its_set_collection(struct virt_its *its, uint16_t collid, + coll_table_entry_t vcpu_id) +{ + paddr_t addr = get_baser_phys_addr(its->baser_coll); + + /* The collection table entry must be able to store a VCPU ID. */ + BUILD_BUG_ON(BIT(sizeof(coll_table_entry_t) * 8) < MAX_VIRT_CPUS); + + ASSERT(spin_is_locked(&its->its_lock)); + + if ( collid >= its->max_collections ) + return -ENOENT; + + return vgic_access_guest_memory(its->d, + addr + collid * sizeof(coll_table_entry_t), + &vcpu_id, sizeof(vcpu_id), true); +} + /* Must be called with the ITS lock held. */ static struct vcpu *get_vcpu_from_collection(struct virt_its *its, uint16_t collid) @@ -281,6 +300,29 @@ static int its_handle_int(struct virt_its *its, uint64_t *cmdptr) return 0; } +static int its_handle_mapc(struct virt_its *its, uint64_t *cmdptr) +{ + uint32_t collid = its_cmd_get_collection(cmdptr); + uint64_t rdbase = its_cmd_mask_field(cmdptr, 2, 16, 44); + + if ( collid >= its->max_collections ) + return -1; + + if ( rdbase >= its->d->max_vcpus ) + return -1; + + spin_lock(&its->its_lock); + + if ( its_cmd_get_validbit(cmdptr) ) + its_set_collection(its, collid, rdbase); + else + its_set_collection(its, collid, UNMAPPED_COLLECTION); + + spin_unlock(&its->its_lock); + + return 0; +} + #define ITS_CMD_BUFFER_SIZE(baser) ((((baser) & 0xff) + 1) << 12) #define ITS_CMD_OFFSET(reg) ((reg) & GENMASK(19, 5)) @@ -320,6 +362,9 @@ static int vgic_its_handle_cmds(struct domain *d, struct virt_its *its) case GITS_CMD_INT: ret = its_handle_int(its, command); break; + case GITS_CMD_MAPC: + ret = its_handle_mapc(its, command); + break; case GITS_CMD_SYNC: /* We handle ITS commands synchronously, so we ignore SYNC. */ break;