Received: by 10.192.165.148 with SMTP id m20csp1081675imm; Fri, 27 Apr 2018 12:19:54 -0700 (PDT) X-Google-Smtp-Source: AB8JxZqMwJ8a7/mVQn32EdpEWsVtsb3MJFMQ90B2xQYz0arg8+xCSyggU8kt0WMWTY5WzMfDcKYP X-Received: by 2002:a17:902:4464:: with SMTP id k91-v6mr3279025pld.219.1524856793984; Fri, 27 Apr 2018 12:19:53 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1524856793; cv=none; d=google.com; s=arc-20160816; b=AZaCOCEoAfJ2763Pw62vXue8jA7XQPnRK/2eYNj0HkaQkUKgcEd7nKibJzvqWA059d ZPFC7ioHi9J7vIdKCihb24oXIp3EJqnzPw0V55zj4CfLzD1eYbGY1gxwg/dVYQjZwf03 F/l6532sKFMmUe6ZkkpQODLywofSnVvic/a7wh2njT9tourYuybOBNy4Y6rKjvFBU7p/ 6yOjhiFuT/C4KN627fpD3oCG3qR4O4k0lVI8OMPXWgrdTwhY/x0E8844oQFSnkmd1z8W l9A5T+azi/BPQ/gKHECDvQ4xAsuZ/Hvj1WY8hl9mRqA1vCzA9pu5H/XS8bw25XIAuYX3 /n3w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=CyT4x0djIGvjl5TiI0GUdnzpypAoQmYHoziM8yqY5MA=; b=IlVvJD6llfxcNT2htftd6lJfNK0/7PXgRutALUGNfvrdLVn+5CfEp4bd3BQD7t7jpc Bk375raNczEnNsQ8CN0VpntDgnDlO1R1SJTZvP1rw6INI83bRQT0Y/k8MDl+fhWlU6v9 yYYZAD9DzAvpBJqW/sj0a3oqrGGm/MNyjVCYPF/pTfY9DY3UYs7v82TwyDEplsdx8HHd vo0Czhyy/Iu8gj1Pg1wlANXNLrL9UuxNmELcRiAF4bS4+CWmcwT2k4djY8qkTFUrkWdS vMW0GyN8oGGiufl+/8DPh0GDF58/QU+v2TfpsuOvQ49/s/xFm0HBgjLY5Dzn7QiNE7ah a73A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f7-v6si1709843pgn.476.2018.04.27.12.19.39; Fri, 27 Apr 2018 12:19:53 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932868AbeD0TSD (ORCPT + 99 others); Fri, 27 Apr 2018 15:18:03 -0400 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:45476 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932793AbeD0TSB (ORCPT ); Fri, 27 Apr 2018 15:18:01 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A561B1529; Fri, 27 Apr 2018 12:18:01 -0700 (PDT) Received: from localhost (unknown [10.37.9.163]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D1FC83F487; Fri, 27 Apr 2018 12:18:00 -0700 (PDT) Date: Fri, 27 Apr 2018 21:17:58 +0200 From: Christoffer Dall To: Eric Auger Cc: eric.auger.pro@gmail.com, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, marc.zyngier@arm.com, peter.maydell@linaro.org, andre.przywara@arm.com Subject: Re: [PATCH v4 11/12] KVM: arm/arm64: Implement KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION Message-ID: <20180427191758.GQ13249@C02W217FHV2R.local> References: <1524838505-3823-1-git-send-email-eric.auger@redhat.com> <1524838505-3823-12-git-send-email-eric.auger@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1524838505-3823-12-git-send-email-eric.auger@redhat.com> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Eric, On Fri, Apr 27, 2018 at 04:15:04PM +0200, Eric Auger wrote: > Now all the internals are ready to handle multiple redistributor > regions, let's allow the userspace to register them. > > Signed-off-by: Eric Auger > > --- > v3 -> v4: > - vgic_v3_rdist_region_from_index is introduced in this patch > > v2 -> v3: > - early exit if vgic_v3_rdist_region_from_index() fails > --- > virt/kvm/arm/vgic/vgic-kvm-device.c | 42 +++++++++++++++++++++++++++++++++++-- > virt/kvm/arm/vgic/vgic-mmio-v3.c | 4 ++-- > virt/kvm/arm/vgic/vgic-v3.c | 14 +++++++++++++ > virt/kvm/arm/vgic/vgic.h | 13 +++++++++++- > 4 files changed, 68 insertions(+), 5 deletions(-) > > diff --git a/virt/kvm/arm/vgic/vgic-kvm-device.c b/virt/kvm/arm/vgic/vgic-kvm-device.c > index e7b5a86..00e03d3 100644 > --- a/virt/kvm/arm/vgic/vgic-kvm-device.c > +++ b/virt/kvm/arm/vgic/vgic-kvm-device.c > @@ -65,7 +65,8 @@ int kvm_vgic_addr(struct kvm *kvm, unsigned long type, u64 *addr, bool write) > { > int r = 0; > struct vgic_dist *vgic = &kvm->arch.vgic; > - phys_addr_t *addr_ptr, alignment; > + phys_addr_t *addr_ptr = NULL; > + phys_addr_t alignment; > uint64_t undef_value = VGIC_ADDR_UNDEF; > > mutex_lock(&kvm->lock); > @@ -92,7 +93,7 @@ int kvm_vgic_addr(struct kvm *kvm, unsigned long type, u64 *addr, bool write) > if (r) > break; > if (write) { > - r = vgic_v3_set_redist_base(kvm, *addr); > + r = vgic_v3_set_redist_base(kvm, 0, *addr, 0); > goto out; > } > rdreg = list_first_entry(&vgic->rd_regions, > @@ -103,6 +104,42 @@ int kvm_vgic_addr(struct kvm *kvm, unsigned long type, u64 *addr, bool write) > addr_ptr = &rdreg->base; > break; > } > + case KVM_VGIC_V3_ADDR_TYPE_REDIST_REGION: > + { > + struct vgic_redist_region *rdreg; > + uint8_t index; > + > + r = vgic_check_type(kvm, KVM_DEV_TYPE_ARM_VGIC_V3); > + if (r) > + break; > + > + index = *addr & KVM_VGIC_V3_RDIST_INDEX_MASK; > + > + if (write) { > + gpa_t base = *addr & KVM_VGIC_V3_RDIST_BASE_MASK; > + uint32_t count = (*addr & KVM_VGIC_V3_RDIST_COUNT_MASK) > + >> KVM_VGIC_V3_RDIST_COUNT_SHIFT; > + uint8_t flags = (*addr & KVM_VGIC_V3_RDIST_FLAGS_MASK) > + >> KVM_VGIC_V3_RDIST_FLAGS_SHIFT; > + > + if (!count || flags) > + r = -EINVAL; > + else > + r = vgic_v3_set_redist_base(kvm, index, > + base, count); > + goto out; > + } > + > + rdreg = vgic_v3_rdist_region_from_index(kvm, index); > + if (!rdreg) { > + r = -ENODEV; > + goto out; > + } > + > + *addr_ptr = rdreg->base & index & > + (uint64_t)rdreg->count << KVM_VGIC_V3_RDIST_COUNT_SHIFT; This still looks wrong, please see my reply to v3. Thanks, -Christoffer