mirror of https://github.com/xemu-project/xemu.git
target/riscv/cpu: move priv spec functions to tcg-cpu.c
Priv spec validation is TCG specific. Move it to the TCG accel class. Signed-off-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com> Reviewed-by: Andrew Jones <ajones@ventanamicro.com> Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-ID: <20230925175709.35696-20-dbarboza@ventanamicro.com> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
This commit is contained in:
parent
7935e2c49c
commit
31778448f2
|
@ -172,21 +172,6 @@ void isa_ext_update_enabled(RISCVCPU *cpu, uint32_t ext_offset, bool en)
|
|||
*ext_enabled = en;
|
||||
}
|
||||
|
||||
int cpu_cfg_ext_get_min_version(uint32_t ext_offset)
|
||||
{
|
||||
const RISCVIsaExtData *edata;
|
||||
|
||||
for (edata = isa_edata_arr; edata && edata->name; edata++) {
|
||||
if (edata->ext_enable_offset != ext_offset) {
|
||||
continue;
|
||||
}
|
||||
|
||||
return edata->min_version;
|
||||
}
|
||||
|
||||
g_assert_not_reached();
|
||||
}
|
||||
|
||||
const char * const riscv_int_regnames[] = {
|
||||
"x0/zero", "x1/ra", "x2/sp", "x3/gp", "x4/tp", "x5/t0", "x6/t1",
|
||||
"x7/t2", "x8/s0", "x9/s1", "x10/a0", "x11/a1", "x12/a2", "x13/a3",
|
||||
|
@ -925,29 +910,6 @@ static void riscv_cpu_disas_set_info(CPUState *s, disassemble_info *info)
|
|||
}
|
||||
}
|
||||
|
||||
void riscv_cpu_disable_priv_spec_isa_exts(RISCVCPU *cpu)
|
||||
{
|
||||
CPURISCVState *env = &cpu->env;
|
||||
const RISCVIsaExtData *edata;
|
||||
|
||||
/* Force disable extensions if priv spec version does not match */
|
||||
for (edata = isa_edata_arr; edata && edata->name; edata++) {
|
||||
if (isa_ext_is_enabled(cpu, edata->ext_enable_offset) &&
|
||||
(env->priv_ver < edata->min_version)) {
|
||||
isa_ext_update_enabled(cpu, edata->ext_enable_offset, false);
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
warn_report("disabling %s extension for hart 0x" TARGET_FMT_lx
|
||||
" because privilege spec version does not match",
|
||||
edata->name, env->mhartid);
|
||||
#else
|
||||
warn_report("disabling %s extension because "
|
||||
"privilege spec version does not match",
|
||||
edata->name);
|
||||
#endif
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
static void riscv_cpu_satp_mode_finalize(RISCVCPU *cpu, Error **errp)
|
||||
{
|
||||
|
|
|
@ -711,9 +711,7 @@ enum riscv_pmu_event_idx {
|
|||
/* used by tcg/tcg-cpu.c*/
|
||||
void isa_ext_update_enabled(RISCVCPU *cpu, uint32_t ext_offset, bool en);
|
||||
bool isa_ext_is_enabled(RISCVCPU *cpu, uint32_t ext_offset);
|
||||
int cpu_cfg_ext_get_min_version(uint32_t ext_offset);
|
||||
void riscv_cpu_set_misa(CPURISCVState *env, RISCVMXL mxl, uint32_t ext);
|
||||
void riscv_cpu_disable_priv_spec_isa_exts(RISCVCPU *cpu);
|
||||
|
||||
typedef struct RISCVCPUMultiExtConfig {
|
||||
const char *name;
|
||||
|
|
|
@ -99,6 +99,21 @@ static const struct TCGCPUOps riscv_tcg_ops = {
|
|||
#endif /* !CONFIG_USER_ONLY */
|
||||
};
|
||||
|
||||
static int cpu_cfg_ext_get_min_version(uint32_t ext_offset)
|
||||
{
|
||||
const RISCVIsaExtData *edata;
|
||||
|
||||
for (edata = isa_edata_arr; edata && edata->name; edata++) {
|
||||
if (edata->ext_enable_offset != ext_offset) {
|
||||
continue;
|
||||
}
|
||||
|
||||
return edata->min_version;
|
||||
}
|
||||
|
||||
g_assert_not_reached();
|
||||
}
|
||||
|
||||
static void cpu_cfg_ext_auto_update(RISCVCPU *cpu, uint32_t ext_offset,
|
||||
bool value)
|
||||
{
|
||||
|
@ -226,6 +241,29 @@ static void riscv_cpu_validate_v(CPURISCVState *env, RISCVCPUConfig *cfg,
|
|||
}
|
||||
}
|
||||
|
||||
static void riscv_cpu_disable_priv_spec_isa_exts(RISCVCPU *cpu)
|
||||
{
|
||||
CPURISCVState *env = &cpu->env;
|
||||
const RISCVIsaExtData *edata;
|
||||
|
||||
/* Force disable extensions if priv spec version does not match */
|
||||
for (edata = isa_edata_arr; edata && edata->name; edata++) {
|
||||
if (isa_ext_is_enabled(cpu, edata->ext_enable_offset) &&
|
||||
(env->priv_ver < edata->min_version)) {
|
||||
isa_ext_update_enabled(cpu, edata->ext_enable_offset, false);
|
||||
#ifndef CONFIG_USER_ONLY
|
||||
warn_report("disabling %s extension for hart 0x" TARGET_FMT_lx
|
||||
" because privilege spec version does not match",
|
||||
edata->name, env->mhartid);
|
||||
#else
|
||||
warn_report("disabling %s extension because "
|
||||
"privilege spec version does not match",
|
||||
edata->name);
|
||||
#endif
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/*
|
||||
* Check consistency between chosen extensions while setting
|
||||
* cpu->cfg accordingly.
|
||||
|
|
Loading…
Reference in New Issue