> -----Original Message----- > From: Alistair Francis [mailto:alistair23@xxxxxxxxx] > Sent: Wednesday, December 9, 2020 6:22 AM > To: Jiangyifei <jiangyifei@xxxxxxxxxx> > Cc: qemu-devel@xxxxxxxxxx Developers <qemu-devel@xxxxxxxxxx>; open > list:RISC-V <qemu-riscv@xxxxxxxxxx>; Zhangxiaofeng (F) > <victor.zhangxiaofeng@xxxxxxxxxx>; Sagar Karandikar > <sagark@xxxxxxxxxxxxxxxxx>; open list:Overall <kvm@xxxxxxxxxxxxxxx>; > libvir-list@xxxxxxxxxx; Bastian Koppelmann > <kbastian@xxxxxxxxxxxxxxxxxxxxx>; Anup Patel <anup.patel@xxxxxxx>; > yinyipeng <yinyipeng1@xxxxxxxxxx>; Alistair Francis > <Alistair.Francis@xxxxxxx>; kvm-riscv@xxxxxxxxxxxxxxxxxxx; Palmer Dabbelt > <palmer@xxxxxxxxxxx>; dengkai (A) <dengkai1@xxxxxxxxxx>; Wubin (H) > <wu.wubin@xxxxxxxxxx>; Zhanghailiang <zhang.zhanghailiang@xxxxxxxxxx> > Subject: Re: [PATCH RFC v4 09/15] target/riscv: Add host cpu type > > On Thu, Dec 3, 2020 at 4:55 AM Yifei Jiang <jiangyifei@xxxxxxxxxx> wrote: > > > > Currently, host cpu is inherited simply. > > > > Signed-off-by: Yifei Jiang <jiangyifei@xxxxxxxxxx> > > Signed-off-by: Yipeng Yin <yinyipeng1@xxxxxxxxxx> > > --- > > target/riscv/cpu.c | 6 ++++++ > > target/riscv/cpu.h | 1 + > > 2 files changed, 7 insertions(+) > > > > diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c index > > faee98a58c..439dc89ee7 100644 > > --- a/target/riscv/cpu.c > > +++ b/target/riscv/cpu.c > > @@ -186,6 +186,10 @@ static void rv32_imafcu_nommu_cpu_init(Object > > *obj) > > > > #endif > > > > +static void riscv_host_cpu_init(Object *obj) { } > > + > > static ObjectClass *riscv_cpu_class_by_name(const char *cpu_model) { > > ObjectClass *oc; > > @@ -641,10 +645,12 @@ static const TypeInfo riscv_cpu_type_infos[] = { > > DEFINE_CPU(TYPE_RISCV_CPU_SIFIVE_E31, > rvxx_sifive_e_cpu_init), > > DEFINE_CPU(TYPE_RISCV_CPU_SIFIVE_E34, > rv32_imafcu_nommu_cpu_init), > > DEFINE_CPU(TYPE_RISCV_CPU_SIFIVE_U34, > rvxx_sifive_u_cpu_init), > > + DEFINE_CPU(TYPE_RISCV_CPU_HOST, > riscv_host_cpu_init), > > #elif defined(TARGET_RISCV64) > > DEFINE_CPU(TYPE_RISCV_CPU_BASE64, > riscv_base_cpu_init), > > DEFINE_CPU(TYPE_RISCV_CPU_SIFIVE_E51, > rvxx_sifive_e_cpu_init), > > DEFINE_CPU(TYPE_RISCV_CPU_SIFIVE_U54, > rvxx_sifive_u_cpu_init), > > + DEFINE_CPU(TYPE_RISCV_CPU_HOST, > riscv_host_cpu_init), > > Shouldn't this only be included if KVM is configured? Also it should be shared > between RV32 and RV64. > > Alistair > Yes, It should be included by CONFIG_KVM and be shared between RV32 and RV64. Yifei > > #endif > > }; > > > > diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h index > > ad1c90f798..4288898019 100644 > > --- a/target/riscv/cpu.h > > +++ b/target/riscv/cpu.h > > @@ -43,6 +43,7 @@ > > #define TYPE_RISCV_CPU_SIFIVE_E51 > RISCV_CPU_TYPE_NAME("sifive-e51") > > #define TYPE_RISCV_CPU_SIFIVE_U34 > RISCV_CPU_TYPE_NAME("sifive-u34") > > #define TYPE_RISCV_CPU_SIFIVE_U54 > RISCV_CPU_TYPE_NAME("sifive-u54") > > +#define TYPE_RISCV_CPU_HOST > RISCV_CPU_TYPE_NAME("host") > > > > #define RV32 ((target_ulong)1 << (TARGET_LONG_BITS - 2)) #define > > RV64 ((target_ulong)2 << (TARGET_LONG_BITS - 2)) > > -- > > 2.19.1 > > > >
