Re: [PATCH RFC v2 3/9] target/loongarch: Supplement vcpu env initial when vcpu reset

[Tianrui Zhao]

在 2023年05月02日 19:04, Richard Henderson 写道:
> On 4/27/23 08:26, Tianrui Zhao wrote:
> > Supplement vcpu env initial when vcpu reset, including
> > init vcpu mp_state value to KVM_MP_STATE_RUNNABLE and
> > init vcpu CSR_CPUID,CSR_TID to cpu->cpu_index.
> >
> > Signed-off-by: Tianrui Zhao<zhaotianrui@loongson.cn>
> > ---
> >   target/loongarch/cpu.c | 3 +++
> >   target/loongarch/cpu.h | 2 ++
> >   2 files changed, 5 insertions(+)
>
> Why do you need KVM_MP_STATE_RUNNABLE in loongarch/cpu.c, outside of 
> kvm.c?
> For Arm, we test the architectural power state of the cpu.
>
>
> r~
Thanks for your reviewing, we want to set mp_state to default value when 
vcpu reset, so we add it in cpu.c. When I reference other archs, I think 
I should add a new function named kvm_arch_reset_vcpu in kvm.c which 
will be called by vcpu_reset and move the reset mp_state into it.

Thanks
Tianrui Zhao