Commit 67d17618 authored by Dave Martin's avatar Dave Martin
Browse files

arm64: Select SVE vector lengths via the command line



In order to support use cases such as migration, it may be
important in some situations to restrict the set of SVE vector
lengths available to the guest.  It can also be useful to observe
the behaviour of guest OSes with different vector lengths.

To enable testing and experimentation for such configurations, this
patch adds a command-line option to allow setting of the set of
vector lengths to be made available to the guest.

For now, the setting is global: no means is offered to configure
individual guest vcpus independently of each other.

By default all vector lengths that the host can support are given
to the guest, as before.
Signed-off-by: default avatarDave Martin <Dave.Martin@arm.com>
parent c83e481f
#ifndef KVM__KVM_CONFIG_ARCH_H
#define KVM__KVM_CONFIG_ARCH_H
int sve_vls_parser(const struct option *opt, const char *arg, int unset);
#define ARM_OPT_ARCH_RUN(cfg) \
OPT_BOOLEAN('\0', "aarch32", &(cfg)->aarch32_guest, \
"Run AArch32 guest"), \
......@@ -16,7 +18,11 @@
OPT_BOOLEAN('\0', "enable-sve", &(cfg)->enable_sve, \
"Enable SVE for the guest"), \
OPT_BOOLEAN('\0', "disable-sve", &(cfg)->disable_sve, \
"Disable SVE for the guest"),
"Disable SVE for the guest"), \
OPT_CALLBACK('\0', "sve-vls", &(cfg)->sve_vqs, \
"comma-separated list of vector lengths, in 128-bit units", \
"Set of vector lengths to enable for the guest", \
sve_vls_parser, NULL),
#include "arm-common/kvm-config-arch.h"
......
#include <errno.h>
#include <stdio.h>
#include <string.h>
#include "kvm/kvm-cpu.h"
#include "kvm/kvm.h"
#include "kvm/virtio.h"
#include <asm/ptrace.h>
#include <asm/sigcontext.h>
#define COMPAT_PSR_F_BIT 0x00000040
#define COMPAT_PSR_I_BIT 0x00000080
......@@ -12,6 +17,65 @@
#define SCTLR_EL1_E0E_MASK (1 << 24)
#define SCTLR_EL1_EE_MASK (1 << 25)
/*
* Work around old kernel headers that lack these definitions in
* <asm/sigcontext.h>:
*/
#ifndef SVE_VQ_MIN
#define SVE_VQ_MIN 1
#endif
#ifndef SVE_VQ_MAX
#define SVE_VQ_MAX 512
#endif
int sve_vls_parser(const struct option *opt, const char *arg, int unset)
{
size_t offset = 0;
int vq, n, t;
u64 (*vqs)[(SVE_VQ_MAX + 1 - SVE_VQ_MIN + 63) / 64];
u64 **cfg_vqs = opt->value;
if (*cfg_vqs) {
pr_err("sve-vls: SVE vector lengths set may only be specified once");
return -1;
}
vqs = calloc(1, sizeof *vqs);
if (!vqs)
die("%s", strerror(ENOMEM));
offset = 0;
while (arg[offset]) {
n = -1;
t = sscanf(arg + offset,
offset == 0 ? "%i%n" : ",%i%n",
&vq, &n);
if (t == EOF || t < 1 || n <= 0) {
pr_err("sve-vls: Comma-separated list of vector lengths required");
goto error;
}
if (vq < SVE_VQ_MIN || vq > SVE_VQ_MAX) {
pr_err("sve-vls: Invalid vector length %d", vq);
goto error;
}
vq -= SVE_VQ_MIN;
(*vqs)[vq / 64] |= (u64)1 << (vq % 64);
offset += n;
}
*cfg_vqs = *vqs;
return 0;
error:
free(vqs);
return -1;
}
static __u64 __core_reg_id(__u64 offset)
{
__u64 id = KVM_REG_ARM64 | KVM_REG_ARM_CORE | offset;
......@@ -180,6 +244,16 @@ void kvm_cpu__select_features(struct kvm *kvm, struct kvm_vcpu_init *init)
static int configure_sve(struct kvm_cpu *vcpu)
{
int feature = KVM_ARM_VCPU_SVE;
struct kvm_one_reg r = {
.id = KVM_REG_ARM64_SVE_VLS,
.addr = (u64)vcpu->kvm->cfg.arch.sve_vqs,
};
if (vcpu->kvm->cfg.arch.sve_vqs)
if (ioctl(vcpu->vcpu_fd, KVM_SET_ONE_REG, &r)) {
pr_err("Cannot set requested SVE vector lengths");
return -1;
}
if (ioctl(vcpu->vcpu_fd, KVM_ARM_VCPU_FINALIZE, &feature)) {
pr_err("KVM_ARM_VCPU_FINALIZE: %s", strerror(errno));
......@@ -191,9 +265,13 @@ static int configure_sve(struct kvm_cpu *vcpu)
int kvm_cpu__configure_features(struct kvm_cpu *vcpu)
{
if (vcpu->kvm->cfg.arch.enable_sve)
if (vcpu->kvm->cfg.arch.enable_sve) {
if (configure_sve(vcpu))
return -1;
} else {
if (vcpu->kvm->cfg.arch.sve_vqs)
pr_warning("SVE vector lengths ignored");
}
return 0;
}
......
......@@ -12,6 +12,7 @@ struct kvm_config_arch {
u64 kaslr_seed;
bool enable_sve;
bool disable_sve;
u64 *sve_vqs;
bool enable_ptrauth;
bool disable_ptrauth;
enum irqchip_type irqchip;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment