From: cdall@linaro.org (Christoffer Dall)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/5] Clarify GICC_PMR export format and remove struct vmcr
Date: Tue, 21 Mar 2017 12:05:25 +0100 [thread overview]
Message-ID: <20170321110530.15857-1-cdall@linaro.org> (raw)
This patch series addresses two issues with the way we export the GIC
CPU interface registers to userspace.
First, we clarify that the ABI is slightly broken and misleading, in
that we export the GICC_PMR using an invented format where we put the
GICH_VMCR.VMPriMask field in the lower five bits of the GICC_PMR
exported to userspace.
Second, since we don't have any non-version-specific users of the struct
vmcr representation, we don't have any clear benefit of using this
indirection inside the kernel, and this has already led to confusion.
Instead, slightly rework some of the user access functions and get rid
of the VMCR structure entirely, which results in a rather nice diffstat
considering I've added quite a few lines of comments.
Based on v4.11-rc1.
Thanks,
-Christoffer
Christoffer Dall (5):
KVM: arm/arm64: Clarify GICC_PMR export format
KVM: arm64: vgic: Factor out access_gic_ctlr into separate r/w
functions
KVM: arm64: vgic: Rename vgic_v3_cpu to vgic_cpu
KVM: arm64: vgic: Get rid of struct vmcr for GICv3
KVM: arm/arm64: vgic: Get rid of struct vmcr for GICv2
Documentation/virtual/kvm/devices/arm-vgic.txt | 6 +
arch/arm64/kvm/vgic-sys-reg-v3.c | 239 ++++++++++++-------------
virt/kvm/arm/vgic/vgic-mmio-v2.c | 51 ++++--
virt/kvm/arm/vgic/vgic-mmio.c | 16 --
virt/kvm/arm/vgic/vgic-v2.c | 29 ---
virt/kvm/arm/vgic/vgic-v3.c | 38 ----
virt/kvm/arm/vgic/vgic.h | 16 --
7 files changed, 155 insertions(+), 240 deletions(-)
--
2.9.0
next reply other threads:[~2017-03-21 11:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-21 11:05 Christoffer Dall [this message]
2017-03-21 11:05 ` [PATCH 1/5] KVM: arm/arm64: Clarify GICC_PMR export format Christoffer Dall
2017-03-21 11:05 ` [PATCH 2/5] KVM: arm64: vgic: Factor out access_gic_ctlr into separate r/w functions Christoffer Dall
2017-03-21 11:05 ` [PATCH 3/5] KVM: arm64: vgic: Rename vgic_v3_cpu to vgic_cpu Christoffer Dall
2017-03-21 11:05 ` [PATCH 4/5] KVM: arm64: vgic: Get rid of struct vmcr for GICv3 Christoffer Dall
2017-03-21 14:17 ` Marc Zyngier
2017-03-21 11:05 ` [PATCH 5/5] KVM: arm/arm64: vgic: Get rid of struct vmcr for GICv2 Christoffer Dall
2017-03-21 14:36 ` Marc Zyngier
2017-03-21 16:01 ` Christoffer Dall
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170321110530.15857-1-cdall@linaro.org \
--to=cdall@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).