2022-01-12 11:13:18 +03:00
|
|
|
/*
|
|
|
|
* RISC-V implementation of KVM hooks
|
|
|
|
*
|
|
|
|
* Copyright (c) 2020 Huawei Technologies Co., Ltd
|
|
|
|
*
|
|
|
|
* This program is free software; you can redistribute it and/or modify it
|
|
|
|
* under the terms and conditions of the GNU General Public License,
|
|
|
|
* version 2 or later, as published by the Free Software Foundation.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope it will be useful, but WITHOUT
|
|
|
|
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
|
|
|
|
* more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License along with
|
|
|
|
* this program. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include "qemu/osdep.h"
|
|
|
|
#include <sys/ioctl.h>
|
|
|
|
|
|
|
|
#include <linux/kvm.h>
|
|
|
|
|
|
|
|
#include "qemu/timer.h"
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
#include "qapi/error.h"
|
2022-01-12 11:13:18 +03:00
|
|
|
#include "qemu/error-report.h"
|
|
|
|
#include "qemu/main-loop.h"
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
#include "qapi/visitor.h"
|
2022-01-12 11:13:18 +03:00
|
|
|
#include "sysemu/sysemu.h"
|
|
|
|
#include "sysemu/kvm.h"
|
|
|
|
#include "sysemu/kvm_int.h"
|
|
|
|
#include "cpu.h"
|
|
|
|
#include "trace.h"
|
2023-09-25 20:57:01 +03:00
|
|
|
#include "hw/core/accel-cpu.h"
|
2022-01-12 11:13:18 +03:00
|
|
|
#include "hw/pci/pci.h"
|
|
|
|
#include "exec/memattrs.h"
|
|
|
|
#include "exec/address-spaces.h"
|
|
|
|
#include "hw/boards.h"
|
|
|
|
#include "hw/irq.h"
|
2023-07-27 13:24:35 +03:00
|
|
|
#include "hw/intc/riscv_imsic.h"
|
2022-01-12 11:13:18 +03:00
|
|
|
#include "qemu/log.h"
|
|
|
|
#include "hw/loader.h"
|
2022-01-12 11:13:22 +03:00
|
|
|
#include "kvm_riscv.h"
|
2022-01-12 11:13:24 +03:00
|
|
|
#include "sbi_ecall_interface.h"
|
|
|
|
#include "chardev/char-fe.h"
|
2022-01-12 11:13:26 +03:00
|
|
|
#include "migration/migration.h"
|
2022-01-12 11:13:27 +03:00
|
|
|
#include "sysemu/runstate.h"
|
2023-07-27 13:24:35 +03:00
|
|
|
#include "hw/riscv/numa.h"
|
2022-01-12 11:13:18 +03:00
|
|
|
|
2023-08-30 16:35:03 +03:00
|
|
|
void riscv_kvm_aplic_request(void *opaque, int irq, int level)
|
|
|
|
{
|
|
|
|
kvm_set_irq(kvm_state, irq, !!level);
|
|
|
|
}
|
|
|
|
|
2023-09-13 12:13:21 +03:00
|
|
|
static bool cap_has_mp_state;
|
|
|
|
|
2023-12-08 21:38:35 +03:00
|
|
|
static uint64_t kvm_riscv_reg_id_ulong(CPURISCVState *env, uint64_t type,
|
2022-01-12 11:13:19 +03:00
|
|
|
uint64_t idx)
|
|
|
|
{
|
|
|
|
uint64_t id = KVM_REG_RISCV | type | idx;
|
|
|
|
|
|
|
|
switch (riscv_cpu_mxl(env)) {
|
|
|
|
case MXL_RV32:
|
|
|
|
id |= KVM_REG_SIZE_U32;
|
|
|
|
break;
|
|
|
|
case MXL_RV64:
|
|
|
|
id |= KVM_REG_SIZE_U64;
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
g_assert_not_reached();
|
|
|
|
}
|
|
|
|
return id;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:31 +03:00
|
|
|
static uint64_t kvm_riscv_reg_id_u32(uint64_t type, uint64_t idx)
|
|
|
|
{
|
|
|
|
return KVM_REG_RISCV | KVM_REG_SIZE_U32 | type | idx;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:32 +03:00
|
|
|
static uint64_t kvm_riscv_reg_id_u64(uint64_t type, uint64_t idx)
|
|
|
|
{
|
|
|
|
return KVM_REG_RISCV | KVM_REG_SIZE_U64 | type | idx;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:35 +03:00
|
|
|
#define RISCV_CORE_REG(env, name) \
|
|
|
|
kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_CORE, \
|
|
|
|
KVM_REG_RISCV_CORE_REG(name))
|
2022-01-12 11:13:20 +03:00
|
|
|
|
2023-12-08 21:38:35 +03:00
|
|
|
#define RISCV_CSR_REG(env, name) \
|
|
|
|
kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_CSR, \
|
|
|
|
KVM_REG_RISCV_CSR_REG(name))
|
2022-01-12 11:13:20 +03:00
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
#define RISCV_CONFIG_REG(env, name) \
|
2023-12-08 21:38:35 +03:00
|
|
|
kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_CONFIG, \
|
|
|
|
KVM_REG_RISCV_CONFIG_REG(name))
|
2023-12-08 21:38:34 +03:00
|
|
|
|
2023-12-08 21:38:33 +03:00
|
|
|
#define RISCV_TIMER_REG(name) kvm_riscv_reg_id_u64(KVM_REG_RISCV_TIMER, \
|
2022-01-12 11:13:26 +03:00
|
|
|
KVM_REG_RISCV_TIMER_REG(name))
|
|
|
|
|
2023-12-08 21:38:31 +03:00
|
|
|
#define RISCV_FP_F_REG(idx) kvm_riscv_reg_id_u32(KVM_REG_RISCV_FP_F, idx)
|
2022-01-12 11:13:20 +03:00
|
|
|
|
2023-12-08 21:38:32 +03:00
|
|
|
#define RISCV_FP_D_REG(idx) kvm_riscv_reg_id_u64(KVM_REG_RISCV_FP_D, idx)
|
2022-01-12 11:13:20 +03:00
|
|
|
|
|
|
|
#define KVM_RISCV_GET_CSR(cs, env, csr, reg) \
|
|
|
|
do { \
|
target/riscv/kvm: fix shadowing in kvm_riscv_(get|put)_regs_csr
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() use an 'int ret' variable
that is used to do an early 'return' if ret > 0. Both are being called
in functions that are also declaring a 'ret' integer, initialized with
'0', and this integer is used as return of the function.
The result is that the compiler is less than pleased and is pointing
shadowing errors:
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_get_regs_csr':
../target/riscv/kvm/kvm-cpu.c:90:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
90 | int ret = kvm_get_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:539:5: note: in expansion of macro 'KVM_RISCV_GET_CSR'
539 | KVM_RISCV_GET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:536:9: note: shadowed declaration is here
536 | int ret = 0;
| ^~~
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_put_regs_csr':
../target/riscv/kvm/kvm-cpu.c:98:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
98 | int ret = kvm_set_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:556:5: note: in expansion of macro 'KVM_RISCV_SET_CSR'
556 | KVM_RISCV_SET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:553:9: note: shadowed declaration is here
553 | int ret = 0;
| ^~~
The macros are doing early returns for non-zero returns and the local
'ret' variable for both functions is used just to do 'return 0', so
remove them from kvm_riscv_get_regs_csr() and kvm_riscv_put_regs_csr()
and do a straight 'return 0' in the end.
For good measure let's also rename the 'ret' variables in
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() to '_ret' to make them more
resilient to these kind of errors.
Fixes: 937f0b4512 ("target/riscv: Implement kvm_arch_get_registers")
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-ID: <20231123101338.1040134-1-dbarboza@ventanamicro.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
2023-11-23 13:13:38 +03:00
|
|
|
int _ret = kvm_get_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
|
|
|
|
if (_ret) { \
|
|
|
|
return _ret; \
|
2022-01-12 11:13:20 +03:00
|
|
|
} \
|
|
|
|
} while (0)
|
|
|
|
|
2022-01-12 11:13:21 +03:00
|
|
|
#define KVM_RISCV_SET_CSR(cs, env, csr, reg) \
|
|
|
|
do { \
|
target/riscv/kvm: fix shadowing in kvm_riscv_(get|put)_regs_csr
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() use an 'int ret' variable
that is used to do an early 'return' if ret > 0. Both are being called
in functions that are also declaring a 'ret' integer, initialized with
'0', and this integer is used as return of the function.
The result is that the compiler is less than pleased and is pointing
shadowing errors:
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_get_regs_csr':
../target/riscv/kvm/kvm-cpu.c:90:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
90 | int ret = kvm_get_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:539:5: note: in expansion of macro 'KVM_RISCV_GET_CSR'
539 | KVM_RISCV_GET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:536:9: note: shadowed declaration is here
536 | int ret = 0;
| ^~~
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_put_regs_csr':
../target/riscv/kvm/kvm-cpu.c:98:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
98 | int ret = kvm_set_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:556:5: note: in expansion of macro 'KVM_RISCV_SET_CSR'
556 | KVM_RISCV_SET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:553:9: note: shadowed declaration is here
553 | int ret = 0;
| ^~~
The macros are doing early returns for non-zero returns and the local
'ret' variable for both functions is used just to do 'return 0', so
remove them from kvm_riscv_get_regs_csr() and kvm_riscv_put_regs_csr()
and do a straight 'return 0' in the end.
For good measure let's also rename the 'ret' variables in
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() to '_ret' to make them more
resilient to these kind of errors.
Fixes: 937f0b4512 ("target/riscv: Implement kvm_arch_get_registers")
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-ID: <20231123101338.1040134-1-dbarboza@ventanamicro.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
2023-11-23 13:13:38 +03:00
|
|
|
int _ret = kvm_set_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
|
|
|
|
if (_ret) { \
|
|
|
|
return _ret; \
|
2022-01-12 11:13:21 +03:00
|
|
|
} \
|
|
|
|
} while (0)
|
|
|
|
|
2023-12-08 21:38:33 +03:00
|
|
|
#define KVM_RISCV_GET_TIMER(cs, name, reg) \
|
2022-01-12 11:13:26 +03:00
|
|
|
do { \
|
2023-12-08 21:38:33 +03:00
|
|
|
int ret = kvm_get_one_reg(cs, RISCV_TIMER_REG(name), ®); \
|
2022-01-12 11:13:26 +03:00
|
|
|
if (ret) { \
|
|
|
|
abort(); \
|
|
|
|
} \
|
|
|
|
} while (0)
|
|
|
|
|
2023-12-08 21:38:33 +03:00
|
|
|
#define KVM_RISCV_SET_TIMER(cs, name, reg) \
|
2022-01-12 11:13:26 +03:00
|
|
|
do { \
|
2023-12-08 21:38:33 +03:00
|
|
|
int ret = kvm_set_one_reg(cs, RISCV_TIMER_REG(name), ®); \
|
2022-01-12 11:13:26 +03:00
|
|
|
if (ret) { \
|
|
|
|
abort(); \
|
|
|
|
} \
|
|
|
|
} while (0)
|
|
|
|
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
typedef struct KVMCPUConfig {
|
|
|
|
const char *name;
|
|
|
|
const char *description;
|
|
|
|
target_ulong offset;
|
|
|
|
int kvm_reg_id;
|
|
|
|
bool user_set;
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
bool supported;
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
} KVMCPUConfig;
|
|
|
|
|
|
|
|
#define KVM_MISA_CFG(_bit, _reg_id) \
|
|
|
|
{.offset = _bit, .kvm_reg_id = _reg_id}
|
|
|
|
|
|
|
|
/* KVM ISA extensions */
|
|
|
|
static KVMCPUConfig kvm_misa_ext_cfgs[] = {
|
|
|
|
KVM_MISA_CFG(RVA, KVM_RISCV_ISA_EXT_A),
|
|
|
|
KVM_MISA_CFG(RVC, KVM_RISCV_ISA_EXT_C),
|
|
|
|
KVM_MISA_CFG(RVD, KVM_RISCV_ISA_EXT_D),
|
|
|
|
KVM_MISA_CFG(RVF, KVM_RISCV_ISA_EXT_F),
|
|
|
|
KVM_MISA_CFG(RVH, KVM_RISCV_ISA_EXT_H),
|
|
|
|
KVM_MISA_CFG(RVI, KVM_RISCV_ISA_EXT_I),
|
|
|
|
KVM_MISA_CFG(RVM, KVM_RISCV_ISA_EXT_M),
|
|
|
|
};
|
|
|
|
|
2023-10-18 22:56:33 +03:00
|
|
|
static void kvm_cpu_get_misa_ext_cfg(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
KVMCPUConfig *misa_ext_cfg = opaque;
|
|
|
|
target_ulong misa_bit = misa_ext_cfg->offset;
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(obj);
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
bool value = env->misa_ext_mask & misa_bit;
|
|
|
|
|
|
|
|
visit_type_bool(v, name, &value, errp);
|
|
|
|
}
|
|
|
|
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
static void kvm_cpu_set_misa_ext_cfg(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
KVMCPUConfig *misa_ext_cfg = opaque;
|
|
|
|
target_ulong misa_bit = misa_ext_cfg->offset;
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(obj);
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
bool value, host_bit;
|
|
|
|
|
|
|
|
if (!visit_type_bool(v, name, &value, errp)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
host_bit = env->misa_ext_mask & misa_bit;
|
|
|
|
|
|
|
|
if (value == host_bit) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!value) {
|
|
|
|
misa_ext_cfg->user_set = true;
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Forbid users to enable extensions that aren't
|
|
|
|
* available in the hart.
|
|
|
|
*/
|
|
|
|
error_setg(errp, "Enabling MISA bit '%s' is not allowed: it's not "
|
|
|
|
"enabled in the host", misa_ext_cfg->name);
|
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:31 +03:00
|
|
|
static void kvm_riscv_update_cpu_misa_ext(RISCVCPU *cpu, CPUState *cs)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
uint64_t id, reg;
|
|
|
|
int i, ret;
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(kvm_misa_ext_cfgs); i++) {
|
|
|
|
KVMCPUConfig *misa_cfg = &kvm_misa_ext_cfgs[i];
|
|
|
|
target_ulong misa_bit = misa_cfg->offset;
|
|
|
|
|
|
|
|
if (!misa_cfg->user_set) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* If we're here we're going to disable the MISA bit */
|
|
|
|
reg = 0;
|
2023-12-08 21:38:35 +03:00
|
|
|
id = kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_ISA_EXT,
|
|
|
|
misa_cfg->kvm_reg_id);
|
2023-07-06 13:17:31 +03:00
|
|
|
ret = kvm_set_one_reg(cs, id, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
/*
|
|
|
|
* We're not checking for -EINVAL because if the bit is about
|
|
|
|
* to be disabled, it means that it was already enabled by
|
|
|
|
* KVM. We determined that by fetching the 'isa' register
|
|
|
|
* during init() time. Any error at this point is worth
|
|
|
|
* aborting.
|
|
|
|
*/
|
|
|
|
error_report("Unable to set KVM reg %s, error %d",
|
|
|
|
misa_cfg->name, ret);
|
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
env->misa_ext &= ~misa_bit;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
#define KVM_EXT_CFG(_name, _prop, _reg_id) \
|
2023-09-12 16:24:17 +03:00
|
|
|
{.name = _name, .offset = CPU_CFG_OFFSET(_prop), \
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
.kvm_reg_id = _reg_id}
|
|
|
|
|
|
|
|
static KVMCPUConfig kvm_multi_ext_cfgs[] = {
|
2023-10-12 19:46:03 +03:00
|
|
|
KVM_EXT_CFG("zicbom", ext_zicbom, KVM_RISCV_ISA_EXT_ZICBOM),
|
2023-10-12 19:46:04 +03:00
|
|
|
KVM_EXT_CFG("zicboz", ext_zicboz, KVM_RISCV_ISA_EXT_ZICBOZ),
|
2023-10-23 18:39:25 +03:00
|
|
|
KVM_EXT_CFG("zicntr", ext_zicntr, KVM_RISCV_ISA_EXT_ZICNTR),
|
2023-10-31 23:51:50 +03:00
|
|
|
KVM_EXT_CFG("zicsr", ext_zicsr, KVM_RISCV_ISA_EXT_ZICSR),
|
|
|
|
KVM_EXT_CFG("zifencei", ext_zifencei, KVM_RISCV_ISA_EXT_ZIFENCEI),
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
KVM_EXT_CFG("zihintpause", ext_zihintpause, KVM_RISCV_ISA_EXT_ZIHINTPAUSE),
|
2023-10-23 18:39:27 +03:00
|
|
|
KVM_EXT_CFG("zihpm", ext_zihpm, KVM_RISCV_ISA_EXT_ZIHPM),
|
2023-10-31 23:51:50 +03:00
|
|
|
KVM_EXT_CFG("zba", ext_zba, KVM_RISCV_ISA_EXT_ZBA),
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
KVM_EXT_CFG("zbb", ext_zbb, KVM_RISCV_ISA_EXT_ZBB),
|
2023-10-31 23:51:50 +03:00
|
|
|
KVM_EXT_CFG("zbs", ext_zbs, KVM_RISCV_ISA_EXT_ZBS),
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
KVM_EXT_CFG("ssaia", ext_ssaia, KVM_RISCV_ISA_EXT_SSAIA),
|
|
|
|
KVM_EXT_CFG("sstc", ext_sstc, KVM_RISCV_ISA_EXT_SSTC),
|
|
|
|
KVM_EXT_CFG("svinval", ext_svinval, KVM_RISCV_ISA_EXT_SVINVAL),
|
2023-10-31 23:51:50 +03:00
|
|
|
KVM_EXT_CFG("svnapot", ext_svnapot, KVM_RISCV_ISA_EXT_SVNAPOT),
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
KVM_EXT_CFG("svpbmt", ext_svpbmt, KVM_RISCV_ISA_EXT_SVPBMT),
|
|
|
|
};
|
|
|
|
|
2023-07-06 13:17:37 +03:00
|
|
|
static void *kvmconfig_get_cfg_addr(RISCVCPU *cpu, KVMCPUConfig *kvmcfg)
|
|
|
|
{
|
|
|
|
return (void *)&cpu->cfg + kvmcfg->offset;
|
|
|
|
}
|
|
|
|
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
static void kvm_cpu_cfg_set(RISCVCPU *cpu, KVMCPUConfig *multi_ext,
|
|
|
|
uint32_t val)
|
|
|
|
{
|
2023-07-06 13:17:37 +03:00
|
|
|
bool *ext_enabled = kvmconfig_get_cfg_addr(cpu, multi_ext);
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
|
|
|
|
*ext_enabled = val;
|
|
|
|
}
|
|
|
|
|
|
|
|
static uint32_t kvm_cpu_cfg_get(RISCVCPU *cpu,
|
|
|
|
KVMCPUConfig *multi_ext)
|
|
|
|
{
|
2023-07-06 13:17:37 +03:00
|
|
|
bool *ext_enabled = kvmconfig_get_cfg_addr(cpu, multi_ext);
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
|
|
|
|
return *ext_enabled;
|
|
|
|
}
|
|
|
|
|
2023-10-18 22:56:33 +03:00
|
|
|
static void kvm_cpu_get_multi_ext_cfg(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
KVMCPUConfig *multi_ext_cfg = opaque;
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(obj);
|
|
|
|
bool value = kvm_cpu_cfg_get(cpu, multi_ext_cfg);
|
|
|
|
|
|
|
|
visit_type_bool(v, name, &value, errp);
|
|
|
|
}
|
|
|
|
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
static void kvm_cpu_set_multi_ext_cfg(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
KVMCPUConfig *multi_ext_cfg = opaque;
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(obj);
|
|
|
|
bool value, host_val;
|
|
|
|
|
|
|
|
if (!visit_type_bool(v, name, &value, errp)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
host_val = kvm_cpu_cfg_get(cpu, multi_ext_cfg);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Ignore if the user is setting the same value
|
|
|
|
* as the host.
|
|
|
|
*/
|
|
|
|
if (value == host_val) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!multi_ext_cfg->supported) {
|
|
|
|
/*
|
|
|
|
* Error out if the user is trying to enable an
|
|
|
|
* extension that KVM doesn't support. Ignore
|
|
|
|
* option otherwise.
|
|
|
|
*/
|
|
|
|
if (value) {
|
|
|
|
error_setg(errp, "KVM does not support disabling extension %s",
|
|
|
|
multi_ext_cfg->name);
|
|
|
|
}
|
|
|
|
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
multi_ext_cfg->user_set = true;
|
|
|
|
kvm_cpu_cfg_set(cpu, multi_ext_cfg, value);
|
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:38 +03:00
|
|
|
static KVMCPUConfig kvm_cbom_blocksize = {
|
|
|
|
.name = "cbom_blocksize",
|
2023-09-12 16:24:17 +03:00
|
|
|
.offset = CPU_CFG_OFFSET(cbom_blocksize),
|
2023-07-06 13:17:38 +03:00
|
|
|
.kvm_reg_id = KVM_REG_RISCV_CONFIG_REG(zicbom_block_size)
|
|
|
|
};
|
|
|
|
|
|
|
|
static KVMCPUConfig kvm_cboz_blocksize = {
|
|
|
|
.name = "cboz_blocksize",
|
2023-09-12 16:24:17 +03:00
|
|
|
.offset = CPU_CFG_OFFSET(cboz_blocksize),
|
2023-07-06 13:17:38 +03:00
|
|
|
.kvm_reg_id = KVM_REG_RISCV_CONFIG_REG(zicboz_block_size)
|
|
|
|
};
|
|
|
|
|
|
|
|
static void kvm_cpu_set_cbomz_blksize(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
KVMCPUConfig *cbomz_cfg = opaque;
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(obj);
|
|
|
|
uint16_t value, *host_val;
|
|
|
|
|
|
|
|
if (!visit_type_uint16(v, name, &value, errp)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
host_val = kvmconfig_get_cfg_addr(cpu, cbomz_cfg);
|
|
|
|
|
|
|
|
if (value != *host_val) {
|
|
|
|
error_report("Unable to set %s to a different value than "
|
|
|
|
"the host (%u)",
|
|
|
|
cbomz_cfg->name, *host_val);
|
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
|
|
|
|
cbomz_cfg->user_set = true;
|
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:36 +03:00
|
|
|
static void kvm_riscv_update_cpu_cfg_isa_ext(RISCVCPU *cpu, CPUState *cs)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
uint64_t id, reg;
|
|
|
|
int i, ret;
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(kvm_multi_ext_cfgs); i++) {
|
|
|
|
KVMCPUConfig *multi_ext_cfg = &kvm_multi_ext_cfgs[i];
|
|
|
|
|
|
|
|
if (!multi_ext_cfg->user_set) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:35 +03:00
|
|
|
id = kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_ISA_EXT,
|
|
|
|
multi_ext_cfg->kvm_reg_id);
|
2023-07-06 13:17:36 +03:00
|
|
|
reg = kvm_cpu_cfg_get(cpu, multi_ext_cfg);
|
|
|
|
ret = kvm_set_one_reg(cs, id, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
error_report("Unable to %s extension %s in KVM, error %d",
|
|
|
|
reg ? "enable" : "disable",
|
|
|
|
multi_ext_cfg->name, ret);
|
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2023-10-18 22:56:33 +03:00
|
|
|
static void cpu_get_cfg_unavailable(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
bool value = false;
|
|
|
|
|
|
|
|
visit_type_bool(v, name, &value, errp);
|
|
|
|
}
|
|
|
|
|
2023-09-25 20:56:58 +03:00
|
|
|
static void cpu_set_cfg_unavailable(Object *obj, Visitor *v,
|
|
|
|
const char *name,
|
|
|
|
void *opaque, Error **errp)
|
|
|
|
{
|
|
|
|
const char *propname = opaque;
|
|
|
|
bool value;
|
|
|
|
|
|
|
|
if (!visit_type_bool(v, name, &value, errp)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (value) {
|
2023-12-18 15:53:17 +03:00
|
|
|
error_setg(errp, "'%s' is not available with KVM",
|
2023-09-25 20:56:58 +03:00
|
|
|
propname);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static void riscv_cpu_add_kvm_unavail_prop(Object *obj, const char *prop_name)
|
|
|
|
{
|
|
|
|
/* Check if KVM created the property already */
|
|
|
|
if (object_property_find(obj, prop_name)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Set the default to disabled for every extension
|
|
|
|
* unknown to KVM and error out if the user attempts
|
|
|
|
* to enable any of them.
|
|
|
|
*/
|
|
|
|
object_property_add(obj, prop_name, "bool",
|
2023-10-18 22:56:33 +03:00
|
|
|
cpu_get_cfg_unavailable,
|
|
|
|
cpu_set_cfg_unavailable,
|
2023-09-25 20:56:58 +03:00
|
|
|
NULL, (void *)prop_name);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void riscv_cpu_add_kvm_unavail_prop_array(Object *obj,
|
|
|
|
const RISCVCPUMultiExtConfig *array)
|
|
|
|
{
|
|
|
|
const RISCVCPUMultiExtConfig *prop;
|
|
|
|
|
|
|
|
g_assert(array);
|
|
|
|
|
|
|
|
for (prop = array; prop && prop->name; prop++) {
|
|
|
|
riscv_cpu_add_kvm_unavail_prop(obj, prop->name);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
static void kvm_riscv_add_cpu_user_properties(Object *cpu_obj)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
|
2023-09-25 20:57:03 +03:00
|
|
|
riscv_add_satp_mode_properties(cpu_obj);
|
|
|
|
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
for (i = 0; i < ARRAY_SIZE(kvm_misa_ext_cfgs); i++) {
|
|
|
|
KVMCPUConfig *misa_cfg = &kvm_misa_ext_cfgs[i];
|
|
|
|
int bit = misa_cfg->offset;
|
|
|
|
|
|
|
|
misa_cfg->name = riscv_get_misa_ext_name(bit);
|
|
|
|
misa_cfg->description = riscv_get_misa_ext_description(bit);
|
|
|
|
|
|
|
|
object_property_add(cpu_obj, misa_cfg->name, "bool",
|
2023-10-18 22:56:33 +03:00
|
|
|
kvm_cpu_get_misa_ext_cfg,
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
kvm_cpu_set_misa_ext_cfg,
|
|
|
|
NULL, misa_cfg);
|
|
|
|
object_property_set_description(cpu_obj, misa_cfg->name,
|
|
|
|
misa_cfg->description);
|
|
|
|
}
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
|
2023-09-25 20:57:03 +03:00
|
|
|
for (i = 0; misa_bits[i] != 0; i++) {
|
|
|
|
const char *ext_name = riscv_get_misa_ext_name(misa_bits[i]);
|
|
|
|
riscv_cpu_add_kvm_unavail_prop(cpu_obj, ext_name);
|
|
|
|
}
|
|
|
|
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
for (i = 0; i < ARRAY_SIZE(kvm_multi_ext_cfgs); i++) {
|
|
|
|
KVMCPUConfig *multi_cfg = &kvm_multi_ext_cfgs[i];
|
|
|
|
|
|
|
|
object_property_add(cpu_obj, multi_cfg->name, "bool",
|
2023-10-18 22:56:33 +03:00
|
|
|
kvm_cpu_get_multi_ext_cfg,
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
kvm_cpu_set_multi_ext_cfg,
|
|
|
|
NULL, multi_cfg);
|
|
|
|
}
|
2023-07-06 13:17:38 +03:00
|
|
|
|
|
|
|
object_property_add(cpu_obj, "cbom_blocksize", "uint16",
|
|
|
|
NULL, kvm_cpu_set_cbomz_blksize,
|
|
|
|
NULL, &kvm_cbom_blocksize);
|
|
|
|
|
|
|
|
object_property_add(cpu_obj, "cboz_blocksize", "uint16",
|
|
|
|
NULL, kvm_cpu_set_cbomz_blksize,
|
|
|
|
NULL, &kvm_cboz_blocksize);
|
2023-09-25 20:57:03 +03:00
|
|
|
|
|
|
|
riscv_cpu_add_kvm_unavail_prop_array(cpu_obj, riscv_cpu_extensions);
|
|
|
|
riscv_cpu_add_kvm_unavail_prop_array(cpu_obj, riscv_cpu_vendor_exts);
|
|
|
|
riscv_cpu_add_kvm_unavail_prop_array(cpu_obj, riscv_cpu_experimental_exts);
|
2023-12-18 15:53:17 +03:00
|
|
|
|
|
|
|
/* We don't have the needed KVM support for profiles */
|
|
|
|
for (i = 0; riscv_profiles[i] != NULL; i++) {
|
|
|
|
riscv_cpu_add_kvm_unavail_prop(cpu_obj, riscv_profiles[i]->name);
|
|
|
|
}
|
target/riscv: add KVM specific MISA properties
Using all TCG user properties in KVM is tricky. First because KVM
supports only a small subset of what TCG provides, so most of the
cpu->cfg flags do nothing for KVM.
Second, and more important, we don't have a way of telling if any given
value is an user input or not. For TCG this has a small impact since we
just validating everything and error out if needed. But for KVM it would
be good to know if a given value was set by the user or if it's a value
already provided by KVM. Otherwise we don't know how to handle failed
kvm_set_one_regs() when writing the configurations back.
These characteristics make it overly complicated to use the same user
facing flags for both KVM and TCG. A simpler approach is to create KVM
specific properties that have specialized logic, forking KVM and TCG use
cases for those cases only. Fully separating KVM/TCG properties is
unneeded at this point - in fact we want the user experience to be as
equal as possible, regardless of the acceleration chosen.
We'll start this fork with the MISA properties, adding the MISA bits
that the KVM driver currently supports. A new KVMCPUConfig type is
introduced. It'll hold general information about an extension. For MISA
extensions we're going to use the newly created getters of
misa_ext_infos[] to populate their name and description. 'offset' holds
the MISA bit (RVA, RVC, ...). We're calling it 'offset' instead of
'misa_bit' because this same KVMCPUConfig struct will be used to
multi-letter extensions later on.
This new type also holds a 'user_set' flag. This flag will be set when
the user set an option that's different than what is already configured
in the host, requiring KVM intervention to write the regs back during
kvm_arch_init_vcpu(). Similar mechanics will be implemented for
multi-letter extensions as well.
There is no need to duplicate more code than necessary, so we're going
to use the existing kvm_riscv_init_user_properties() to add the KVM
specific properties. Any code that is adding a TCG user prop is then
changed slightly to verify first if there's a KVM prop with the same
name already added.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230706101738.460804-13-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:30 +03:00
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:20 +03:00
|
|
|
static int kvm_riscv_get_regs_core(CPUState *cs)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
int i;
|
|
|
|
target_ulong reg;
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
ret = kvm_get_one_reg(cs, RISCV_CORE_REG(env, regs.pc), ®);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
env->pc = reg;
|
|
|
|
|
|
|
|
for (i = 1; i < 32; i++) {
|
2023-12-08 21:38:35 +03:00
|
|
|
uint64_t id = kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_CORE, i);
|
2022-01-12 11:13:20 +03:00
|
|
|
ret = kvm_get_one_reg(cs, id, ®);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
env->gpr[i] = reg;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:21 +03:00
|
|
|
static int kvm_riscv_put_regs_core(CPUState *cs)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
int i;
|
|
|
|
target_ulong reg;
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
reg = env->pc;
|
|
|
|
ret = kvm_set_one_reg(cs, RISCV_CORE_REG(env, regs.pc), ®);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 1; i < 32; i++) {
|
2023-12-08 21:38:35 +03:00
|
|
|
uint64_t id = kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_CORE, i);
|
2022-01-12 11:13:21 +03:00
|
|
|
reg = env->gpr[i];
|
|
|
|
ret = kvm_set_one_reg(cs, id, ®);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:20 +03:00
|
|
|
static int kvm_riscv_get_regs_csr(CPUState *cs)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, sstatus, env->mstatus);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, sie, env->mie);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, stvec, env->stvec);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, sscratch, env->sscratch);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, sepc, env->sepc);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, scause, env->scause);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, stval, env->stval);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, sip, env->mip);
|
|
|
|
KVM_RISCV_GET_CSR(cs, env, satp, env->satp);
|
target/riscv/kvm: fix shadowing in kvm_riscv_(get|put)_regs_csr
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() use an 'int ret' variable
that is used to do an early 'return' if ret > 0. Both are being called
in functions that are also declaring a 'ret' integer, initialized with
'0', and this integer is used as return of the function.
The result is that the compiler is less than pleased and is pointing
shadowing errors:
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_get_regs_csr':
../target/riscv/kvm/kvm-cpu.c:90:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
90 | int ret = kvm_get_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:539:5: note: in expansion of macro 'KVM_RISCV_GET_CSR'
539 | KVM_RISCV_GET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:536:9: note: shadowed declaration is here
536 | int ret = 0;
| ^~~
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_put_regs_csr':
../target/riscv/kvm/kvm-cpu.c:98:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
98 | int ret = kvm_set_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:556:5: note: in expansion of macro 'KVM_RISCV_SET_CSR'
556 | KVM_RISCV_SET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:553:9: note: shadowed declaration is here
553 | int ret = 0;
| ^~~
The macros are doing early returns for non-zero returns and the local
'ret' variable for both functions is used just to do 'return 0', so
remove them from kvm_riscv_get_regs_csr() and kvm_riscv_put_regs_csr()
and do a straight 'return 0' in the end.
For good measure let's also rename the 'ret' variables in
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() to '_ret' to make them more
resilient to these kind of errors.
Fixes: 937f0b4512 ("target/riscv: Implement kvm_arch_get_registers")
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-ID: <20231123101338.1040134-1-dbarboza@ventanamicro.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
2023-11-23 13:13:38 +03:00
|
|
|
|
|
|
|
return 0;
|
2022-01-12 11:13:20 +03:00
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:21 +03:00
|
|
|
static int kvm_riscv_put_regs_csr(CPUState *cs)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, sstatus, env->mstatus);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, sie, env->mie);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, stvec, env->stvec);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, sscratch, env->sscratch);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, sepc, env->sepc);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, scause, env->scause);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, stval, env->stval);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, sip, env->mip);
|
|
|
|
KVM_RISCV_SET_CSR(cs, env, satp, env->satp);
|
|
|
|
|
target/riscv/kvm: fix shadowing in kvm_riscv_(get|put)_regs_csr
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() use an 'int ret' variable
that is used to do an early 'return' if ret > 0. Both are being called
in functions that are also declaring a 'ret' integer, initialized with
'0', and this integer is used as return of the function.
The result is that the compiler is less than pleased and is pointing
shadowing errors:
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_get_regs_csr':
../target/riscv/kvm/kvm-cpu.c:90:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
90 | int ret = kvm_get_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:539:5: note: in expansion of macro 'KVM_RISCV_GET_CSR'
539 | KVM_RISCV_GET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:536:9: note: shadowed declaration is here
536 | int ret = 0;
| ^~~
../target/riscv/kvm/kvm-cpu.c: In function 'kvm_riscv_put_regs_csr':
../target/riscv/kvm/kvm-cpu.c:98:13: error: declaration of 'ret' shadows a previous local [-Werror=shadow=compatible-local]
98 | int ret = kvm_set_one_reg(cs, RISCV_CSR_REG(env, csr), ®); \
| ^~~
../target/riscv/kvm/kvm-cpu.c:556:5: note: in expansion of macro 'KVM_RISCV_SET_CSR'
556 | KVM_RISCV_SET_CSR(cs, env, sstatus, env->mstatus);
| ^~~~~~~~~~~~~~~~~
../target/riscv/kvm/kvm-cpu.c:553:9: note: shadowed declaration is here
553 | int ret = 0;
| ^~~
The macros are doing early returns for non-zero returns and the local
'ret' variable for both functions is used just to do 'return 0', so
remove them from kvm_riscv_get_regs_csr() and kvm_riscv_put_regs_csr()
and do a straight 'return 0' in the end.
For good measure let's also rename the 'ret' variables in
KVM_RISCV_GET_CSR() and KVM_RISCV_SET_CSR() to '_ret' to make them more
resilient to these kind of errors.
Fixes: 937f0b4512 ("target/riscv: Implement kvm_arch_get_registers")
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-ID: <20231123101338.1040134-1-dbarboza@ventanamicro.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
2023-11-23 13:13:38 +03:00
|
|
|
return 0;
|
2022-01-12 11:13:21 +03:00
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:20 +03:00
|
|
|
static int kvm_riscv_get_regs_fp(CPUState *cs)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
int i;
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
if (riscv_has_ext(env, RVD)) {
|
|
|
|
uint64_t reg;
|
|
|
|
for (i = 0; i < 32; i++) {
|
2023-12-08 21:38:32 +03:00
|
|
|
ret = kvm_get_one_reg(cs, RISCV_FP_D_REG(i), ®);
|
2022-01-12 11:13:20 +03:00
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
env->fpr[i] = reg;
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (riscv_has_ext(env, RVF)) {
|
|
|
|
uint32_t reg;
|
|
|
|
for (i = 0; i < 32; i++) {
|
2023-12-08 21:38:31 +03:00
|
|
|
ret = kvm_get_one_reg(cs, RISCV_FP_F_REG(i), ®);
|
2022-01-12 11:13:20 +03:00
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
env->fpr[i] = reg;
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:21 +03:00
|
|
|
static int kvm_riscv_put_regs_fp(CPUState *cs)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
int i;
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
if (riscv_has_ext(env, RVD)) {
|
|
|
|
uint64_t reg;
|
|
|
|
for (i = 0; i < 32; i++) {
|
|
|
|
reg = env->fpr[i];
|
2023-12-08 21:38:32 +03:00
|
|
|
ret = kvm_set_one_reg(cs, RISCV_FP_D_REG(i), ®);
|
2022-01-12 11:13:21 +03:00
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (riscv_has_ext(env, RVF)) {
|
|
|
|
uint32_t reg;
|
|
|
|
for (i = 0; i < 32; i++) {
|
|
|
|
reg = env->fpr[i];
|
2023-12-08 21:38:31 +03:00
|
|
|
ret = kvm_set_one_reg(cs, RISCV_FP_F_REG(i), ®);
|
2022-01-12 11:13:21 +03:00
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:26 +03:00
|
|
|
static void kvm_riscv_get_regs_timer(CPUState *cs)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
if (env->kvm_timer_dirty) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:33 +03:00
|
|
|
KVM_RISCV_GET_TIMER(cs, time, env->kvm_timer_time);
|
|
|
|
KVM_RISCV_GET_TIMER(cs, compare, env->kvm_timer_compare);
|
|
|
|
KVM_RISCV_GET_TIMER(cs, state, env->kvm_timer_state);
|
|
|
|
KVM_RISCV_GET_TIMER(cs, frequency, env->kvm_timer_frequency);
|
2022-01-12 11:13:26 +03:00
|
|
|
|
|
|
|
env->kvm_timer_dirty = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void kvm_riscv_put_regs_timer(CPUState *cs)
|
|
|
|
{
|
|
|
|
uint64_t reg;
|
|
|
|
CPURISCVState *env = &RISCV_CPU(cs)->env;
|
|
|
|
|
|
|
|
if (!env->kvm_timer_dirty) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:33 +03:00
|
|
|
KVM_RISCV_SET_TIMER(cs, time, env->kvm_timer_time);
|
|
|
|
KVM_RISCV_SET_TIMER(cs, compare, env->kvm_timer_compare);
|
2022-01-12 11:13:26 +03:00
|
|
|
|
|
|
|
/*
|
|
|
|
* To set register of RISCV_TIMER_REG(state) will occur a error from KVM
|
|
|
|
* on env->kvm_timer_state == 0, It's better to adapt in KVM, but it
|
|
|
|
* doesn't matter that adaping in QEMU now.
|
|
|
|
* TODO If KVM changes, adapt here.
|
|
|
|
*/
|
|
|
|
if (env->kvm_timer_state) {
|
2023-12-08 21:38:33 +03:00
|
|
|
KVM_RISCV_SET_TIMER(cs, state, env->kvm_timer_state);
|
2022-01-12 11:13:26 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* For now, migration will not work between Hosts with different timer
|
|
|
|
* frequency. Therefore, we should check whether they are the same here
|
|
|
|
* during the migration.
|
|
|
|
*/
|
|
|
|
if (migration_is_running(migrate_get_current()->state)) {
|
2023-12-08 21:38:33 +03:00
|
|
|
KVM_RISCV_GET_TIMER(cs, frequency, reg);
|
2022-01-12 11:13:26 +03:00
|
|
|
if (reg != env->kvm_timer_frequency) {
|
|
|
|
error_report("Dst Hosts timer frequency != Src Hosts");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
env->kvm_timer_dirty = false;
|
|
|
|
}
|
2022-01-12 11:13:21 +03:00
|
|
|
|
2023-07-06 13:17:24 +03:00
|
|
|
typedef struct KVMScratchCPU {
|
|
|
|
int kvmfd;
|
|
|
|
int vmfd;
|
|
|
|
int cpufd;
|
|
|
|
} KVMScratchCPU;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Heavily inspired by kvm_arm_create_scratch_host_vcpu()
|
|
|
|
* from target/arm/kvm.c.
|
|
|
|
*/
|
|
|
|
static bool kvm_riscv_create_scratch_vcpu(KVMScratchCPU *scratch)
|
|
|
|
{
|
|
|
|
int kvmfd = -1, vmfd = -1, cpufd = -1;
|
|
|
|
|
|
|
|
kvmfd = qemu_open_old("/dev/kvm", O_RDWR);
|
|
|
|
if (kvmfd < 0) {
|
|
|
|
goto err;
|
|
|
|
}
|
|
|
|
do {
|
|
|
|
vmfd = ioctl(kvmfd, KVM_CREATE_VM, 0);
|
|
|
|
} while (vmfd == -1 && errno == EINTR);
|
|
|
|
if (vmfd < 0) {
|
|
|
|
goto err;
|
|
|
|
}
|
|
|
|
cpufd = ioctl(vmfd, KVM_CREATE_VCPU, 0);
|
|
|
|
if (cpufd < 0) {
|
|
|
|
goto err;
|
|
|
|
}
|
|
|
|
|
|
|
|
scratch->kvmfd = kvmfd;
|
|
|
|
scratch->vmfd = vmfd;
|
|
|
|
scratch->cpufd = cpufd;
|
|
|
|
|
|
|
|
return true;
|
|
|
|
|
|
|
|
err:
|
|
|
|
if (cpufd >= 0) {
|
|
|
|
close(cpufd);
|
|
|
|
}
|
|
|
|
if (vmfd >= 0) {
|
|
|
|
close(vmfd);
|
|
|
|
}
|
|
|
|
if (kvmfd >= 0) {
|
|
|
|
close(kvmfd);
|
|
|
|
}
|
|
|
|
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void kvm_riscv_destroy_scratch_vcpu(KVMScratchCPU *scratch)
|
|
|
|
{
|
|
|
|
close(scratch->cpufd);
|
|
|
|
close(scratch->vmfd);
|
|
|
|
close(scratch->kvmfd);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void kvm_riscv_init_machine_ids(RISCVCPU *cpu, KVMScratchCPU *kvmcpu)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
struct kvm_one_reg reg;
|
|
|
|
int ret;
|
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
reg.id = RISCV_CONFIG_REG(env, mvendorid);
|
2023-07-06 13:17:24 +03:00
|
|
|
reg.addr = (uint64_t)&cpu->cfg.mvendorid;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
error_report("Unable to retrieve mvendorid from host, error %d", ret);
|
|
|
|
}
|
2023-07-06 13:17:25 +03:00
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
reg.id = RISCV_CONFIG_REG(env, marchid);
|
2023-07-06 13:17:25 +03:00
|
|
|
reg.addr = (uint64_t)&cpu->cfg.marchid;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
error_report("Unable to retrieve marchid from host, error %d", ret);
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
reg.id = RISCV_CONFIG_REG(env, mimpid);
|
2023-07-06 13:17:25 +03:00
|
|
|
reg.addr = (uint64_t)&cpu->cfg.mimpid;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
error_report("Unable to retrieve mimpid from host, error %d", ret);
|
|
|
|
}
|
2023-07-06 13:17:24 +03:00
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:28 +03:00
|
|
|
static void kvm_riscv_init_misa_ext_mask(RISCVCPU *cpu,
|
|
|
|
KVMScratchCPU *kvmcpu)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
struct kvm_one_reg reg;
|
|
|
|
int ret;
|
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
reg.id = RISCV_CONFIG_REG(env, isa);
|
2023-07-06 13:17:28 +03:00
|
|
|
reg.addr = (uint64_t)&env->misa_ext_mask;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
|
|
|
|
if (ret) {
|
|
|
|
error_report("Unable to fetch ISA register from KVM, "
|
|
|
|
"error %d", ret);
|
|
|
|
kvm_riscv_destroy_scratch_vcpu(kvmcpu);
|
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
|
|
|
|
env->misa_ext = env->misa_ext_mask;
|
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:38 +03:00
|
|
|
static void kvm_riscv_read_cbomz_blksize(RISCVCPU *cpu, KVMScratchCPU *kvmcpu,
|
|
|
|
KVMCPUConfig *cbomz_cfg)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
struct kvm_one_reg reg;
|
|
|
|
int ret;
|
|
|
|
|
2023-12-08 21:38:35 +03:00
|
|
|
reg.id = kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_CONFIG,
|
|
|
|
cbomz_cfg->kvm_reg_id);
|
2023-07-06 13:17:38 +03:00
|
|
|
reg.addr = (uint64_t)kvmconfig_get_cfg_addr(cpu, cbomz_cfg);
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
error_report("Unable to read KVM reg %s, error %d",
|
|
|
|
cbomz_cfg->name, ret);
|
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2023-10-03 16:21:48 +03:00
|
|
|
static void kvm_riscv_read_multiext_legacy(RISCVCPU *cpu,
|
|
|
|
KVMScratchCPU *kvmcpu)
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
|
|
|
uint64_t val;
|
|
|
|
int i, ret;
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(kvm_multi_ext_cfgs); i++) {
|
|
|
|
KVMCPUConfig *multi_ext_cfg = &kvm_multi_ext_cfgs[i];
|
|
|
|
struct kvm_one_reg reg;
|
|
|
|
|
2023-12-08 21:38:35 +03:00
|
|
|
reg.id = kvm_riscv_reg_id_ulong(env, KVM_REG_RISCV_ISA_EXT,
|
|
|
|
multi_ext_cfg->kvm_reg_id);
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
reg.addr = (uint64_t)&val;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
if (ret != 0) {
|
|
|
|
if (errno == EINVAL) {
|
|
|
|
/* Silently default to 'false' if KVM does not support it. */
|
|
|
|
multi_ext_cfg->supported = false;
|
|
|
|
val = false;
|
|
|
|
} else {
|
2023-12-18 19:22:44 +03:00
|
|
|
error_report("Unable to read ISA_EXT KVM register %s: %s",
|
|
|
|
multi_ext_cfg->name, strerror(errno));
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
multi_ext_cfg->supported = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
kvm_cpu_cfg_set(cpu, multi_ext_cfg, val);
|
|
|
|
}
|
2023-07-06 13:17:38 +03:00
|
|
|
|
2023-10-12 19:46:03 +03:00
|
|
|
if (cpu->cfg.ext_zicbom) {
|
2023-07-06 13:17:38 +03:00
|
|
|
kvm_riscv_read_cbomz_blksize(cpu, kvmcpu, &kvm_cbom_blocksize);
|
|
|
|
}
|
|
|
|
|
2023-10-12 19:46:04 +03:00
|
|
|
if (cpu->cfg.ext_zicboz) {
|
2023-07-06 13:17:38 +03:00
|
|
|
kvm_riscv_read_cbomz_blksize(cpu, kvmcpu, &kvm_cboz_blocksize);
|
|
|
|
}
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
}
|
|
|
|
|
2023-10-03 16:21:48 +03:00
|
|
|
static int uint64_cmp(const void *a, const void *b)
|
|
|
|
{
|
|
|
|
uint64_t val1 = *(const uint64_t *)a;
|
|
|
|
uint64_t val2 = *(const uint64_t *)b;
|
|
|
|
|
|
|
|
if (val1 < val2) {
|
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (val1 > val2) {
|
|
|
|
return 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void kvm_riscv_init_multiext_cfg(RISCVCPU *cpu, KVMScratchCPU *kvmcpu)
|
|
|
|
{
|
|
|
|
KVMCPUConfig *multi_ext_cfg;
|
|
|
|
struct kvm_one_reg reg;
|
|
|
|
struct kvm_reg_list rl_struct;
|
|
|
|
struct kvm_reg_list *reglist;
|
|
|
|
uint64_t val, reg_id, *reg_search;
|
|
|
|
int i, ret;
|
|
|
|
|
|
|
|
rl_struct.n = 0;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_REG_LIST, &rl_struct);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* If KVM_GET_REG_LIST isn't supported we'll get errno 22
|
|
|
|
* (EINVAL). Use read_legacy() in this case.
|
|
|
|
*/
|
|
|
|
if (errno == EINVAL) {
|
|
|
|
return kvm_riscv_read_multiext_legacy(cpu, kvmcpu);
|
|
|
|
} else if (errno != E2BIG) {
|
|
|
|
/*
|
|
|
|
* E2BIG is an expected error message for the API since we
|
|
|
|
* don't know the number of registers. The right amount will
|
|
|
|
* be written in rl_struct.n.
|
|
|
|
*
|
|
|
|
* Error out if we get any other errno.
|
|
|
|
*/
|
2023-12-18 19:22:44 +03:00
|
|
|
error_report("Error when accessing get-reg-list: %s",
|
|
|
|
strerror(errno));
|
2023-10-03 16:21:48 +03:00
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
|
|
|
|
reglist = g_malloc(sizeof(struct kvm_reg_list) +
|
|
|
|
rl_struct.n * sizeof(uint64_t));
|
|
|
|
reglist->n = rl_struct.n;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_REG_LIST, reglist);
|
|
|
|
if (ret) {
|
2023-12-18 19:22:44 +03:00
|
|
|
error_report("Error when reading KVM_GET_REG_LIST: %s",
|
|
|
|
strerror(errno));
|
2023-10-03 16:21:48 +03:00
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
|
|
|
|
/* sort reglist to use bsearch() */
|
|
|
|
qsort(®list->reg, reglist->n, sizeof(uint64_t), uint64_cmp);
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(kvm_multi_ext_cfgs); i++) {
|
|
|
|
multi_ext_cfg = &kvm_multi_ext_cfgs[i];
|
2023-12-08 21:38:35 +03:00
|
|
|
reg_id = kvm_riscv_reg_id_ulong(&cpu->env, KVM_REG_RISCV_ISA_EXT,
|
|
|
|
multi_ext_cfg->kvm_reg_id);
|
2023-10-03 16:21:48 +03:00
|
|
|
reg_search = bsearch(®_id, reglist->reg, reglist->n,
|
|
|
|
sizeof(uint64_t), uint64_cmp);
|
|
|
|
if (!reg_search) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
reg.id = reg_id;
|
|
|
|
reg.addr = (uint64_t)&val;
|
|
|
|
ret = ioctl(kvmcpu->cpufd, KVM_GET_ONE_REG, ®);
|
|
|
|
if (ret != 0) {
|
2023-12-18 19:22:44 +03:00
|
|
|
error_report("Unable to read ISA_EXT KVM register %s: %s",
|
|
|
|
multi_ext_cfg->name, strerror(errno));
|
2023-10-03 16:21:48 +03:00
|
|
|
exit(EXIT_FAILURE);
|
|
|
|
}
|
|
|
|
|
|
|
|
multi_ext_cfg->supported = true;
|
|
|
|
kvm_cpu_cfg_set(cpu, multi_ext_cfg, val);
|
|
|
|
}
|
|
|
|
|
2023-10-12 19:46:03 +03:00
|
|
|
if (cpu->cfg.ext_zicbom) {
|
2023-10-03 16:21:48 +03:00
|
|
|
kvm_riscv_read_cbomz_blksize(cpu, kvmcpu, &kvm_cbom_blocksize);
|
|
|
|
}
|
|
|
|
|
2023-10-12 19:46:04 +03:00
|
|
|
if (cpu->cfg.ext_zicboz) {
|
2023-10-03 16:21:48 +03:00
|
|
|
kvm_riscv_read_cbomz_blksize(cpu, kvmcpu, &kvm_cboz_blocksize);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2023-09-25 20:57:03 +03:00
|
|
|
static void riscv_init_kvm_registers(Object *cpu_obj)
|
2023-07-06 13:17:24 +03:00
|
|
|
{
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(cpu_obj);
|
|
|
|
KVMScratchCPU kvmcpu;
|
|
|
|
|
|
|
|
if (!kvm_riscv_create_scratch_vcpu(&kvmcpu)) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
kvm_riscv_init_machine_ids(cpu, &kvmcpu);
|
2023-07-06 13:17:28 +03:00
|
|
|
kvm_riscv_init_misa_ext_mask(cpu, &kvmcpu);
|
target/riscv/kvm.c: add multi-letter extension KVM properties
Let's add KVM user properties for the multi-letter extensions that KVM
currently supports: zicbom, zicboz, zihintpause, zbb, ssaia, sstc,
svinval and svpbmt.
As with MISA extensions, we're using the KVMCPUConfig type to hold
information about the state of each extension. However, multi-letter
extensions have more cases to cover than MISA extensions, so we're
adding an extra 'supported' flag as well. This flag will reflect if a
given extension is supported by KVM, i.e. KVM knows how to handle it.
This is determined during KVM extension discovery in
kvm_riscv_init_multiext_cfg(), where we test for EINVAL errors. Any
other error will cause an abort.
The use of the 'user_set' is similar to what we already do with MISA
extensions: the flag set only if the user is changing the extension
state.
The 'supported' flag will be used later on to make an exception for
users that are disabling multi-letter extensions that are unknown to
KVM.
Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Acked-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Andrew Jones <ajones@ventanamicro.com>
Message-Id: <20230706101738.460804-15-dbarboza@ventanamicro.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
2023-07-06 13:17:32 +03:00
|
|
|
kvm_riscv_init_multiext_cfg(cpu, &kvmcpu);
|
2023-07-06 13:17:24 +03:00
|
|
|
|
|
|
|
kvm_riscv_destroy_scratch_vcpu(&kvmcpu);
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
const KVMCapabilityInfo kvm_arch_required_capabilities[] = {
|
|
|
|
KVM_CAP_LAST_INFO
|
|
|
|
};
|
|
|
|
|
|
|
|
int kvm_arch_get_registers(CPUState *cs)
|
|
|
|
{
|
2022-01-12 11:13:20 +03:00
|
|
|
int ret = 0;
|
|
|
|
|
|
|
|
ret = kvm_riscv_get_regs_core(cs);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_riscv_get_regs_csr(cs);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_riscv_get_regs_fp(cs);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
return ret;
|
2022-01-12 11:13:18 +03:00
|
|
|
}
|
|
|
|
|
2023-09-13 12:13:21 +03:00
|
|
|
int kvm_riscv_sync_mpstate_to_kvm(RISCVCPU *cpu, int state)
|
|
|
|
{
|
|
|
|
if (cap_has_mp_state) {
|
|
|
|
struct kvm_mp_state mp_state = {
|
|
|
|
.mp_state = state
|
|
|
|
};
|
|
|
|
|
|
|
|
int ret = kvm_vcpu_ioctl(CPU(cpu), KVM_SET_MP_STATE, &mp_state);
|
|
|
|
if (ret) {
|
|
|
|
fprintf(stderr, "%s: failed to sync MP_STATE %d/%s\n",
|
|
|
|
__func__, ret, strerror(-ret));
|
|
|
|
return -1;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
int kvm_arch_put_registers(CPUState *cs, int level)
|
|
|
|
{
|
2022-01-12 11:13:21 +03:00
|
|
|
int ret = 0;
|
|
|
|
|
|
|
|
ret = kvm_riscv_put_regs_core(cs);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_riscv_put_regs_csr(cs);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_riscv_put_regs_fp(cs);
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2023-09-13 12:13:21 +03:00
|
|
|
if (KVM_PUT_RESET_STATE == level) {
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(cs);
|
|
|
|
if (cs->cpu_index == 0) {
|
|
|
|
ret = kvm_riscv_sync_mpstate_to_kvm(cpu, KVM_MP_STATE_RUNNABLE);
|
|
|
|
} else {
|
|
|
|
ret = kvm_riscv_sync_mpstate_to_kvm(cpu, KVM_MP_STATE_STOPPED);
|
|
|
|
}
|
|
|
|
if (ret) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:21 +03:00
|
|
|
return ret;
|
2022-01-12 11:13:18 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_release_virq_post(int virq)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_fixup_msi_route(struct kvm_irq_routing_entry *route,
|
|
|
|
uint64_t address, uint32_t data, PCIDevice *dev)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_destroy_vcpu(CPUState *cs)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
unsigned long kvm_arch_vcpu_id(CPUState *cpu)
|
|
|
|
{
|
|
|
|
return cpu->cpu_index;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:27 +03:00
|
|
|
static void kvm_riscv_vm_state_change(void *opaque, bool running,
|
|
|
|
RunState state)
|
|
|
|
{
|
|
|
|
CPUState *cs = opaque;
|
|
|
|
|
|
|
|
if (running) {
|
|
|
|
kvm_riscv_put_regs_timer(cs);
|
|
|
|
} else {
|
|
|
|
kvm_riscv_get_regs_timer(cs);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
void kvm_arch_init_irq_routing(KVMState *s)
|
|
|
|
{
|
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:26 +03:00
|
|
|
static int kvm_vcpu_set_machine_ids(RISCVCPU *cpu, CPUState *cs)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
2023-08-02 21:00:58 +03:00
|
|
|
target_ulong reg;
|
2023-07-06 13:17:26 +03:00
|
|
|
uint64_t id;
|
|
|
|
int ret;
|
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
id = RISCV_CONFIG_REG(env, mvendorid);
|
2023-08-02 21:00:58 +03:00
|
|
|
/*
|
|
|
|
* cfg.mvendorid is an uint32 but a target_ulong will
|
|
|
|
* be written. Assign it to a target_ulong var to avoid
|
|
|
|
* writing pieces of other cpu->cfg fields in the reg.
|
|
|
|
*/
|
|
|
|
reg = cpu->cfg.mvendorid;
|
|
|
|
ret = kvm_set_one_reg(cs, id, ®);
|
2023-07-06 13:17:26 +03:00
|
|
|
if (ret != 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
id = RISCV_CONFIG_REG(env, marchid);
|
2023-07-06 13:17:26 +03:00
|
|
|
ret = kvm_set_one_reg(cs, id, &cpu->cfg.marchid);
|
|
|
|
if (ret != 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2023-12-08 21:38:34 +03:00
|
|
|
id = RISCV_CONFIG_REG(env, mimpid);
|
2023-07-06 13:17:26 +03:00
|
|
|
ret = kvm_set_one_reg(cs, id, &cpu->cfg.mimpid);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
int kvm_arch_init_vcpu(CPUState *cs)
|
|
|
|
{
|
2022-01-12 11:13:19 +03:00
|
|
|
int ret = 0;
|
|
|
|
RISCVCPU *cpu = RISCV_CPU(cs);
|
|
|
|
|
2022-01-12 11:13:27 +03:00
|
|
|
qemu_add_vm_change_state_handler(kvm_riscv_vm_state_change, cs);
|
|
|
|
|
2023-07-06 13:17:26 +03:00
|
|
|
if (!object_dynamic_cast(OBJECT(cpu), TYPE_RISCV_CPU_HOST)) {
|
|
|
|
ret = kvm_vcpu_set_machine_ids(cpu, cs);
|
2023-07-06 13:17:31 +03:00
|
|
|
if (ret != 0) {
|
|
|
|
return ret;
|
|
|
|
}
|
2023-07-06 13:17:26 +03:00
|
|
|
}
|
|
|
|
|
2023-07-06 13:17:31 +03:00
|
|
|
kvm_riscv_update_cpu_misa_ext(cpu, cs);
|
2023-07-06 13:17:36 +03:00
|
|
|
kvm_riscv_update_cpu_cfg_isa_ext(cpu, cs);
|
2023-07-06 13:17:31 +03:00
|
|
|
|
2022-01-12 11:13:19 +03:00
|
|
|
return ret;
|
2022-01-12 11:13:18 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_msi_data_to_gsi(uint32_t data)
|
|
|
|
{
|
|
|
|
abort();
|
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_add_msi_route_post(struct kvm_irq_routing_entry *route,
|
|
|
|
int vector, PCIDevice *dev)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2023-08-22 19:31:02 +03:00
|
|
|
int kvm_arch_get_default_type(MachineState *ms)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
int kvm_arch_init(MachineState *ms, KVMState *s)
|
|
|
|
{
|
2023-09-13 12:13:21 +03:00
|
|
|
cap_has_mp_state = kvm_check_extension(s, KVM_CAP_MP_STATE);
|
2022-01-12 11:13:18 +03:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_irqchip_create(KVMState *s)
|
|
|
|
{
|
2023-07-27 13:24:34 +03:00
|
|
|
if (kvm_kernel_irqchip_split()) {
|
|
|
|
error_report("-machine kernel_irqchip=split is not supported on RISC-V.");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* We can create the VAIA using the newer device control API.
|
|
|
|
*/
|
|
|
|
return kvm_check_extension(s, KVM_CAP_DEVICE_CTRL);
|
2022-01-12 11:13:18 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
int kvm_arch_process_async_events(CPUState *cs)
|
|
|
|
{
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
void kvm_arch_pre_run(CPUState *cs, struct kvm_run *run)
|
|
|
|
{
|
|
|
|
}
|
|
|
|
|
|
|
|
MemTxAttrs kvm_arch_post_run(CPUState *cs, struct kvm_run *run)
|
|
|
|
{
|
|
|
|
return MEMTXATTRS_UNSPECIFIED;
|
|
|
|
}
|
|
|
|
|
|
|
|
bool kvm_arch_stop_on_emulation_error(CPUState *cs)
|
|
|
|
{
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:24 +03:00
|
|
|
static int kvm_riscv_handle_sbi(CPUState *cs, struct kvm_run *run)
|
|
|
|
{
|
|
|
|
int ret = 0;
|
|
|
|
unsigned char ch;
|
|
|
|
switch (run->riscv_sbi.extension_id) {
|
|
|
|
case SBI_EXT_0_1_CONSOLE_PUTCHAR:
|
|
|
|
ch = run->riscv_sbi.args[0];
|
|
|
|
qemu_chr_fe_write(serial_hd(0)->be, &ch, sizeof(ch));
|
|
|
|
break;
|
|
|
|
case SBI_EXT_0_1_CONSOLE_GETCHAR:
|
|
|
|
ret = qemu_chr_fe_read_all(serial_hd(0)->be, &ch, sizeof(ch));
|
|
|
|
if (ret == sizeof(ch)) {
|
2023-02-03 16:51:55 +03:00
|
|
|
run->riscv_sbi.ret[0] = ch;
|
2022-01-12 11:13:24 +03:00
|
|
|
} else {
|
2023-02-03 16:51:55 +03:00
|
|
|
run->riscv_sbi.ret[0] = -1;
|
2022-01-12 11:13:24 +03:00
|
|
|
}
|
2023-02-03 16:51:55 +03:00
|
|
|
ret = 0;
|
2022-01-12 11:13:24 +03:00
|
|
|
break;
|
|
|
|
default:
|
|
|
|
qemu_log_mask(LOG_UNIMP,
|
|
|
|
"%s: un-handled SBI EXIT, specific reasons is %lu\n",
|
|
|
|
__func__, run->riscv_sbi.extension_id);
|
|
|
|
ret = -1;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
int kvm_arch_handle_exit(CPUState *cs, struct kvm_run *run)
|
|
|
|
{
|
2022-01-12 11:13:24 +03:00
|
|
|
int ret = 0;
|
|
|
|
switch (run->exit_reason) {
|
|
|
|
case KVM_EXIT_RISCV_SBI:
|
|
|
|
ret = kvm_riscv_handle_sbi(cs, run);
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
qemu_log_mask(LOG_UNIMP, "%s: un-handled exit reason %d\n",
|
|
|
|
__func__, run->exit_reason);
|
|
|
|
ret = -1;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
return ret;
|
2022-01-12 11:13:18 +03:00
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:22 +03:00
|
|
|
void kvm_riscv_reset_vcpu(RISCVCPU *cpu)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &cpu->env;
|
2023-09-13 12:13:21 +03:00
|
|
|
int i;
|
2022-01-12 11:13:22 +03:00
|
|
|
|
|
|
|
if (!kvm_enabled()) {
|
|
|
|
return;
|
|
|
|
}
|
2023-09-13 12:13:21 +03:00
|
|
|
for (i = 0; i < 32; i++) {
|
|
|
|
env->gpr[i] = 0;
|
|
|
|
}
|
2022-01-12 11:13:22 +03:00
|
|
|
env->pc = cpu->env.kernel_addr;
|
|
|
|
env->gpr[10] = kvm_arch_vcpu_id(CPU(cpu)); /* a0 */
|
|
|
|
env->gpr[11] = cpu->env.fdt_addr; /* a1 */
|
|
|
|
env->satp = 0;
|
2023-09-13 12:13:21 +03:00
|
|
|
env->mie = 0;
|
|
|
|
env->stvec = 0;
|
|
|
|
env->sscratch = 0;
|
|
|
|
env->sepc = 0;
|
|
|
|
env->scause = 0;
|
|
|
|
env->stval = 0;
|
|
|
|
env->mip = 0;
|
2022-01-12 11:13:22 +03:00
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:23 +03:00
|
|
|
void kvm_riscv_set_irq(RISCVCPU *cpu, int irq, int level)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
unsigned virq = level ? KVM_INTERRUPT_SET : KVM_INTERRUPT_UNSET;
|
|
|
|
|
|
|
|
if (irq != IRQ_S_EXT) {
|
|
|
|
perror("kvm riscv set irq != IRQ_S_EXT\n");
|
|
|
|
abort();
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_vcpu_ioctl(CPU(cpu), KVM_INTERRUPT, &virq);
|
|
|
|
if (ret < 0) {
|
|
|
|
perror("Set irq failed");
|
|
|
|
abort();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2022-01-12 11:13:18 +03:00
|
|
|
bool kvm_arch_cpu_check_are_resettable(void)
|
|
|
|
{
|
|
|
|
return true;
|
|
|
|
}
|
2022-09-29 10:20:12 +03:00
|
|
|
|
2023-07-27 13:24:35 +03:00
|
|
|
static int aia_mode;
|
|
|
|
|
|
|
|
static const char *kvm_aia_mode_str(uint64_t mode)
|
|
|
|
{
|
|
|
|
switch (mode) {
|
|
|
|
case KVM_DEV_RISCV_AIA_MODE_EMUL:
|
|
|
|
return "emul";
|
|
|
|
case KVM_DEV_RISCV_AIA_MODE_HWACCEL:
|
|
|
|
return "hwaccel";
|
|
|
|
case KVM_DEV_RISCV_AIA_MODE_AUTO:
|
|
|
|
default:
|
|
|
|
return "auto";
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
static char *riscv_get_kvm_aia(Object *obj, Error **errp)
|
|
|
|
{
|
|
|
|
return g_strdup(kvm_aia_mode_str(aia_mode));
|
|
|
|
}
|
|
|
|
|
|
|
|
static void riscv_set_kvm_aia(Object *obj, const char *val, Error **errp)
|
|
|
|
{
|
|
|
|
if (!strcmp(val, "emul")) {
|
|
|
|
aia_mode = KVM_DEV_RISCV_AIA_MODE_EMUL;
|
|
|
|
} else if (!strcmp(val, "hwaccel")) {
|
|
|
|
aia_mode = KVM_DEV_RISCV_AIA_MODE_HWACCEL;
|
|
|
|
} else if (!strcmp(val, "auto")) {
|
|
|
|
aia_mode = KVM_DEV_RISCV_AIA_MODE_AUTO;
|
|
|
|
} else {
|
|
|
|
error_setg(errp, "Invalid KVM AIA mode");
|
|
|
|
error_append_hint(errp, "Valid values are emul, hwaccel, and auto.\n");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2022-09-29 10:20:12 +03:00
|
|
|
void kvm_arch_accel_class_init(ObjectClass *oc)
|
|
|
|
{
|
2023-07-27 13:24:35 +03:00
|
|
|
object_class_property_add_str(oc, "riscv-aia", riscv_get_kvm_aia,
|
|
|
|
riscv_set_kvm_aia);
|
|
|
|
object_class_property_set_description(oc, "riscv-aia",
|
|
|
|
"Set KVM AIA mode. Valid values are "
|
|
|
|
"emul, hwaccel, and auto. Default "
|
|
|
|
"is auto.");
|
|
|
|
object_property_set_default_str(object_class_property_find(oc, "riscv-aia"),
|
|
|
|
"auto");
|
|
|
|
}
|
|
|
|
|
|
|
|
void kvm_riscv_aia_create(MachineState *machine, uint64_t group_shift,
|
|
|
|
uint64_t aia_irq_num, uint64_t aia_msi_num,
|
|
|
|
uint64_t aplic_base, uint64_t imsic_base,
|
|
|
|
uint64_t guest_num)
|
|
|
|
{
|
|
|
|
int ret, i;
|
|
|
|
int aia_fd = -1;
|
|
|
|
uint64_t default_aia_mode;
|
|
|
|
uint64_t socket_count = riscv_socket_count(machine);
|
|
|
|
uint64_t max_hart_per_socket = 0;
|
|
|
|
uint64_t socket, base_hart, hart_count, socket_imsic_base, imsic_addr;
|
|
|
|
uint64_t socket_bits, hart_bits, guest_bits;
|
|
|
|
|
|
|
|
aia_fd = kvm_create_device(kvm_state, KVM_DEV_TYPE_RISCV_AIA, false);
|
|
|
|
|
|
|
|
if (aia_fd < 0) {
|
|
|
|
error_report("Unable to create in-kernel irqchip");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_MODE,
|
|
|
|
&default_aia_mode, false, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to get current KVM AIA mode");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
qemu_log("KVM AIA: default mode is %s\n",
|
|
|
|
kvm_aia_mode_str(default_aia_mode));
|
|
|
|
|
|
|
|
if (default_aia_mode != aia_mode) {
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_MODE,
|
|
|
|
&aia_mode, true, NULL);
|
|
|
|
if (ret < 0)
|
|
|
|
warn_report("KVM AIA: failed to set KVM AIA mode");
|
|
|
|
else
|
|
|
|
qemu_log("KVM AIA: set current mode to %s\n",
|
|
|
|
kvm_aia_mode_str(aia_mode));
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_SRCS,
|
|
|
|
&aia_irq_num, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set number of input irq lines");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_IDS,
|
|
|
|
&aia_msi_num, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set number of msi");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2023-12-18 12:05:41 +03:00
|
|
|
if (socket_count > 1) {
|
|
|
|
socket_bits = find_last_bit(&socket_count, BITS_PER_LONG) + 1;
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_GROUP_BITS,
|
|
|
|
&socket_bits, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set group_bits");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_GROUP_SHIFT,
|
|
|
|
&group_shift, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set group_shift");
|
|
|
|
exit(1);
|
|
|
|
}
|
2023-07-27 13:24:35 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
guest_bits = guest_num == 0 ? 0 :
|
|
|
|
find_last_bit(&guest_num, BITS_PER_LONG) + 1;
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_GUEST_BITS,
|
|
|
|
&guest_bits, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set guest_bits");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_ADDR,
|
|
|
|
KVM_DEV_RISCV_AIA_ADDR_APLIC,
|
|
|
|
&aplic_base, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set the base address of APLIC");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (socket = 0; socket < socket_count; socket++) {
|
|
|
|
socket_imsic_base = imsic_base + socket * (1U << group_shift);
|
|
|
|
hart_count = riscv_socket_hart_count(machine, socket);
|
|
|
|
base_hart = riscv_socket_first_hartid(machine, socket);
|
|
|
|
|
|
|
|
if (max_hart_per_socket < hart_count) {
|
|
|
|
max_hart_per_socket = hart_count;
|
|
|
|
}
|
|
|
|
|
|
|
|
for (i = 0; i < hart_count; i++) {
|
|
|
|
imsic_addr = socket_imsic_base + i * IMSIC_HART_SIZE(guest_bits);
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_ADDR,
|
|
|
|
KVM_DEV_RISCV_AIA_ADDR_IMSIC(i + base_hart),
|
|
|
|
&imsic_addr, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set the IMSIC address for hart %d", i);
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
hart_bits = find_last_bit(&max_hart_per_socket, BITS_PER_LONG) + 1;
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CONFIG,
|
|
|
|
KVM_DEV_RISCV_AIA_CONFIG_HART_BITS,
|
|
|
|
&hart_bits, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: failed to set hart_bits");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
|
|
|
if (kvm_has_gsi_routing()) {
|
|
|
|
for (uint64_t idx = 0; idx < aia_irq_num + 1; ++idx) {
|
|
|
|
/* KVM AIA only has one APLIC instance */
|
|
|
|
kvm_irqchip_add_irq_route(kvm_state, idx, 0, idx);
|
|
|
|
}
|
|
|
|
kvm_gsi_routing_allowed = true;
|
|
|
|
kvm_irqchip_commit_routes(kvm_state);
|
|
|
|
}
|
|
|
|
|
|
|
|
ret = kvm_device_access(aia_fd, KVM_DEV_RISCV_AIA_GRP_CTRL,
|
|
|
|
KVM_DEV_RISCV_AIA_CTRL_INIT,
|
|
|
|
NULL, true, NULL);
|
|
|
|
if (ret < 0) {
|
|
|
|
error_report("KVM AIA: initialized fail");
|
|
|
|
exit(1);
|
|
|
|
}
|
|
|
|
|
2023-10-17 14:34:50 +03:00
|
|
|
kvm_msi_via_irqfd_allowed = true;
|
2022-09-29 10:20:12 +03:00
|
|
|
}
|
2023-09-25 20:56:56 +03:00
|
|
|
|
2023-09-25 20:57:01 +03:00
|
|
|
static void kvm_cpu_instance_init(CPUState *cs)
|
2023-09-25 20:56:58 +03:00
|
|
|
{
|
2023-09-25 20:57:01 +03:00
|
|
|
Object *obj = OBJECT(RISCV_CPU(cs));
|
2023-09-25 20:56:58 +03:00
|
|
|
DeviceState *dev = DEVICE(obj);
|
|
|
|
|
2023-09-25 20:57:03 +03:00
|
|
|
riscv_init_kvm_registers(obj);
|
2023-09-25 20:56:58 +03:00
|
|
|
|
2023-09-25 20:57:03 +03:00
|
|
|
kvm_riscv_add_cpu_user_properties(obj);
|
2023-09-25 20:56:58 +03:00
|
|
|
|
|
|
|
for (Property *prop = riscv_cpu_options; prop && prop->name; prop++) {
|
2023-09-25 20:57:01 +03:00
|
|
|
/* Check if we have a specific KVM handler for the option */
|
2023-09-25 20:56:58 +03:00
|
|
|
if (object_property_find(obj, prop->name)) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
qdev_property_add_static(dev, prop);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2023-09-25 20:57:01 +03:00
|
|
|
static void kvm_cpu_accel_class_init(ObjectClass *oc, void *data)
|
|
|
|
{
|
|
|
|
AccelCPUClass *acc = ACCEL_CPU_CLASS(oc);
|
|
|
|
|
|
|
|
acc->cpu_instance_init = kvm_cpu_instance_init;
|
|
|
|
}
|
|
|
|
|
|
|
|
static const TypeInfo kvm_cpu_accel_type_info = {
|
|
|
|
.name = ACCEL_CPU_NAME("kvm"),
|
|
|
|
|
|
|
|
.parent = TYPE_ACCEL_CPU,
|
|
|
|
.class_init = kvm_cpu_accel_class_init,
|
|
|
|
.abstract = true,
|
|
|
|
};
|
|
|
|
static void kvm_cpu_accel_register_types(void)
|
|
|
|
{
|
|
|
|
type_register_static(&kvm_cpu_accel_type_info);
|
|
|
|
}
|
|
|
|
type_init(kvm_cpu_accel_register_types);
|
|
|
|
|
2023-09-25 20:56:56 +03:00
|
|
|
static void riscv_host_cpu_init(Object *obj)
|
|
|
|
{
|
|
|
|
CPURISCVState *env = &RISCV_CPU(obj)->env;
|
|
|
|
|
|
|
|
#if defined(TARGET_RISCV32)
|
|
|
|
env->misa_mxl_max = env->misa_mxl = MXL_RV32;
|
|
|
|
#elif defined(TARGET_RISCV64)
|
|
|
|
env->misa_mxl_max = env->misa_mxl = MXL_RV64;
|
|
|
|
#endif
|
|
|
|
}
|
|
|
|
|
|
|
|
static const TypeInfo riscv_kvm_cpu_type_infos[] = {
|
|
|
|
{
|
|
|
|
.name = TYPE_RISCV_CPU_HOST,
|
|
|
|
.parent = TYPE_RISCV_CPU,
|
|
|
|
.instance_init = riscv_host_cpu_init,
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
DEFINE_TYPES(riscv_kvm_cpu_type_infos)
|